Sounds reasonable to me ..
-phil.
On 6/25/2013 9:00 AM, Vadim Pakhnushev wrote:
Hi,
Please review the fix for 8016254:
http://bugs.sun.com/view_bug.do?bug_id=8016254
https://jbs.oracle.com/bugs/browse/JDK-8016254
http://cr.openjdk.java.net/~vadim/8016254/webrev.00/
The problem is that Intel 865G chipset (and some other ancient video
cards) doesn't support GL_ARB_depth_texture extension, although does
support GL_EXT_framebuffer_object.
So the OGLContext_IsFBObjectExtensionAvailable function calls
OGLSD_InitFBObject which tries to create a depth renderbuffer.
Ideally glRenderbufferStorageEXT should return GL_INVALID_ENUM to
indicate that the format is not supported, but instead it passes
incorrect values further to the driver and MESA driver crashes.
The proposed fix is to check GL_ARB_depth_texture extension and return
false from OGLContext_IsFBObjectExtensionAvailable if it's not supported.
Thanks,
Vadim