638 2012-11-09 11:54:43 +0100 Problem requesting non-hardware-accelerated capabilities on a device capable of hardware acceleration 2012-11-11 09:18:57 +0100 1 1 3 JogAmp Jogl windows 2 All windows RESOLVED FIXED --- normal --- 1 thedering sgothel --- jogl f67d310743cd6dee3d3005b96b61e9adae628899 --- oldest_to_newest 1965 0 381 thedering 2012-11-09 11:54:43 +0100 Created attachment 381 Patch for WindowsWGLGraphicsConfigurationFactory.java I'm creating a GLCanvas with a GLCapabilities object that requests non-hardware-accelerated capabilities and a custom chooser like in this code snippet: GLCapabilities caps = new GLCapabilities(null); caps.setHardwareAccelerated(false); GLCanvas canvas = new GLCanvas(caps, new CustomCapabilitiesChooser(), null, null); The problem is that the provided chooser's chooseCapabilities method is only supplied GLCapabilities that are hardware accelerated. After further investigation I could find the reason for this in the class WindowsWGLGraphicsConfigurationFactory. The method updateGraphicsConfiguration tries to use updateGraphicsConfigurationARB (which only provides hardware-accelerated capabilities) and only if that method fails, it uses updateGraphicsConfigurationGDI (which also provides non-hardware-accelerated capabilities): if( !updateGraphicsConfigurationARB((WindowsWGLDrawableFactory)factory, config, chooser, hdc, extHDC, pfdIDs) ) { updateGraphicsConfigurationGDI(config, chooser, hdc, extHDC, pfdIDs); } I have implemented a simple solution by checking if non-hardware-accelerated capabilities are requested and skipping the ...ARB method in that case, which is attached as a patch. A better solution might be to supply both the capabilities from ARB and GDI to the chooser. 1966 1 sgothel 2012-11-09 17:00:57 +0100 Sounds great. Can you send me a git patch or pull request so we can preserve your authorship ? I guess the non hw-accel use case is sort of limited and we can live w/ this solution, i.e. skipping the ARB caps if software rendering is requested. 1967 2 382 thedering 2012-11-09 17:56:57 +0100 Created attachment 382 GIT patch This is the patch in a format that can be applied using git apply 1971 3 sgothel 2012-11-11 09:18:57 +0100 jogl commit f67d310743cd6dee3d3005b96b61e9adae628899 Thx. 381 2012-11-09 11:54:43 +0100 2012-11-09 17:56:57 +0100 Patch for WindowsWGLGraphicsConfigurationFactory.java updateGraphicsConfiguration_skipARB.patch text/plain 1032 thedering ZGlmZiAtLWdpdCBhL3NyYy9qb2dsL2NsYXNzZXMvam9nYW1wL29wZW5nbC93aW5kb3dzL3dnbC9X aW5kb3dzV0dMR3JhcGhpY3NDb25maWd1cmF0aW9uRmFjdG9yeS5qYXZhIGIvc3JjL2pvZ2wvY2xh c3Nlcy9qb2dhbXAvb3BlbmdsL3dpbmRvd3Mvd2dsL1dpbmRvd3NXR0xHcmFwaGljc0NvbmZpZ3Vy YXRpb25GYWN0b3J5LmphdmEKaW5kZXggN2IzYmMzYS4uOGM1ZWExMiAxMDA2NDQKLS0tIGEvc3Jj L2pvZ2wvY2xhc3Nlcy9qb2dhbXAvb3BlbmdsL3dpbmRvd3Mvd2dsL1dpbmRvd3NXR0xHcmFwaGlj c0NvbmZpZ3VyYXRpb25GYWN0b3J5LmphdmEKKysrIGIvc3JjL2pvZ2wvY2xhc3Nlcy9qb2dhbXAv b3BlbmdsL3dpbmRvd3Mvd2dsL1dpbmRvd3NXR0xHcmFwaGljc0NvbmZpZ3VyYXRpb25GYWN0b3J5 LmphdmEKQEAgLTI4NCw3ICsyODQsOCBAQCBwdWJsaWMgY2xhc3MgV2luZG93c1dHTEdyYXBoaWNz Q29uZmlndXJhdGlvbkZhY3RvcnkgZXh0ZW5kcyBHTEdyYXBoaWNzQ29uZmlndXJhdAogICAgICAg ICAgICAgfQogICAgICAgICB9CiAgICAgICAgIHRyeSB7Ci0gICAgICAgICAgICBpZiggIXVwZGF0 ZUdyYXBoaWNzQ29uZmlndXJhdGlvbkFSQigoV2luZG93c1dHTERyYXdhYmxlRmFjdG9yeSlmYWN0 b3J5LCBjb25maWcsIGNob29zZXIsIGhkYywgZXh0SERDLCBwZmRJRHMpICkgeworICAgICAgICAg ICAgaWYoICEoKEdMQ2FwYWJpbGl0aWVzSW1tdXRhYmxlKWNvbmZpZy5nZXRDaG9zZW5DYXBhYmls aXRpZXMoKSkuZ2V0SGFyZHdhcmVBY2NlbGVyYXRlZCgpCisgICAgICAgICAgICAgICAgICAgIHx8 ICF1cGRhdGVHcmFwaGljc0NvbmZpZ3VyYXRpb25BUkIoKFdpbmRvd3NXR0xEcmF3YWJsZUZhY3Rv cnkpZmFjdG9yeSwgY29uZmlnLCBjaG9vc2VyLCBoZGMsIGV4dEhEQywgcGZkSURzKSApIHsKICAg ICAgICAgICAgICAgICB1cGRhdGVHcmFwaGljc0NvbmZpZ3VyYXRpb25HREkoY29uZmlnLCBjaG9v c2VyLCBoZGMsIGV4dEhEQywgcGZkSURzKTsKICAgICAgICAgICAgIH0KICAgICAgICAgfSBmaW5h bGx5IHsK 382 2012-11-09 17:56:57 +0100 2012-11-09 17:56:57 +0100 GIT patch fix_sw_rendering.patch text/plain 1563 thedering RnJvbSBlMGIyZThlMTliNTA2YzgzMDlkODI3MGQzNWE3ZGJlMjRlODJkMzIzIE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBqdGhlZGVyaW5nIDx0aGVkZXJpbmdAc2ltdXMtc3lzdGVtcy5j b20+CkRhdGU6IEZyaSwgOSBOb3YgMjAxMiAxNzo1MDoyNSArMDEwMApTdWJqZWN0OiBbUEFUQ0hd IEFkZGVkIGEgY29uZGl0aW9uIHRvIHNraXAgdXBkYXRlR3JhcGhpY3NDb25maWd1cmF0aW9uQVJC CiB3aGVuIG5vbi1oYXJkd2FyZS1hY2NlbGVyYXRlZCBjYXBhYmlsaXRpZXMgYXJlIHJlcXVlc3Rl ZCwKIGJlY2F1c2Ugb25seSB1cGRhdGVHcmFwaGljc0NvbmZpZ3VyYXRpb25HREkgcHJvdmlkZXMK IHNvZnR3YXJlIHJlbmRlcmluZyBjYXBhYmlsaXRpZXMuCgotLS0KIC4uLi9vcGVuZ2wvd2luZG93 cy93Z2wvV2luZG93c1dHTEdyYXBoaWNzQ29uZmlndXJhdGlvbkZhY3RvcnkuamF2YSAgICAgfCAz ICsrLQogMSBmaWxlIGNoYW5nZWQsIDIgaW5zZXJ0aW9ucygrKSwgMSBkZWxldGlvbigtKQoKZGlm ZiAtLWdpdCBhL3NyYy9qb2dsL2NsYXNzZXMvam9nYW1wL29wZW5nbC93aW5kb3dzL3dnbC9XaW5k b3dzV0dMR3JhcGhpY3NDb25maWd1cmF0aW9uRmFjdG9yeS5qYXZhIGIvc3JjL2pvZ2wvY2xhc3Nl cy9qb2dhbXAvb3BlbmdsL3dpbmRvd3Mvd2dsL1dpbmRvd3NXR0xHcmFwaGljc0NvbmZpZ3VyYXRp b25GYWN0b3J5LmphdmEKaW5kZXggN2IzYmMzYS4uOGM1ZWExMiAxMDA2NDQKLS0tIGEvc3JjL2pv Z2wvY2xhc3Nlcy9qb2dhbXAvb3BlbmdsL3dpbmRvd3Mvd2dsL1dpbmRvd3NXR0xHcmFwaGljc0Nv bmZpZ3VyYXRpb25GYWN0b3J5LmphdmEKKysrIGIvc3JjL2pvZ2wvY2xhc3Nlcy9qb2dhbXAvb3Bl bmdsL3dpbmRvd3Mvd2dsL1dpbmRvd3NXR0xHcmFwaGljc0NvbmZpZ3VyYXRpb25GYWN0b3J5Lmph dmEKQEAgLTI4NCw3ICsyODQsOCBAQCBwdWJsaWMgY2xhc3MgV2luZG93c1dHTEdyYXBoaWNzQ29u ZmlndXJhdGlvbkZhY3RvcnkgZXh0ZW5kcyBHTEdyYXBoaWNzQ29uZmlndXJhdAogICAgICAgICAg ICAgfQogICAgICAgICB9CiAgICAgICAgIHRyeSB7Ci0gICAgICAgICAgICBpZiggIXVwZGF0ZUdy YXBoaWNzQ29uZmlndXJhdGlvbkFSQigoV2luZG93c1dHTERyYXdhYmxlRmFjdG9yeSlmYWN0b3J5 LCBjb25maWcsIGNob29zZXIsIGhkYywgZXh0SERDLCBwZmRJRHMpICkgeworICAgICAgICAgICAg aWYoICEoKEdMQ2FwYWJpbGl0aWVzSW1tdXRhYmxlKWNvbmZpZy5nZXRDaG9zZW5DYXBhYmlsaXRp ZXMoKSkuZ2V0SGFyZHdhcmVBY2NlbGVyYXRlZCgpCisgICAgICAgICAgICAgICAgICAgIHx8ICF1 cGRhdGVHcmFwaGljc0NvbmZpZ3VyYXRpb25BUkIoKFdpbmRvd3NXR0xEcmF3YWJsZUZhY3Rvcnkp ZmFjdG9yeSwgY29uZmlnLCBjaG9vc2VyLCBoZGMsIGV4dEhEQywgcGZkSURzKSApIHsKICAgICAg ICAgICAgICAgICB1cGRhdGVHcmFwaGljc0NvbmZpZ3VyYXRpb25HREkoY29uZmlnLCBjaG9vc2Vy LCBoZGMsIGV4dEhEQywgcGZkSURzKTsKICAgICAgICAgICAgIH0KICAgICAgICAgfSBmaW5hbGx5 IHsKLS0gCjEuOC4wLm1zeXNnaXQuMAoK