Hi Fabian, On Wed, Mar 12, 2014 at 2:24 PM, Fabian Bieler <[email protected]> wrote: > On 2014-03-12 19:51, Felipe Tonello wrote: >> >> Hi Fabian, >> >> On Tue, Mar 11, 2014 at 5:24 PM, Fabian Bieler <[email protected]> >> wrote: >>> >>> Hello! >>> >>> Sync objects, vertex arrays, queries, transform feedback objects and >>> sampler >>> objects are not supported by gles2. >> >> >> Ok. But then how would this extension work with GLES2? I don't see >> anywhere in the spec saying what's the minimum requirement for OpenGL >> ES. It only states that requires OpenGL 1.1. >> >> For exemple, glObjectPtrLabel requires a sync object. So only this >> type of call will not work with OpenGL ES 2.0? Or the whole extension? > > I'm pretty sure the extension is supposed to work with ES 2, too (according > to the spec the OpenGL ES WG approved it in June 2012 which actually > precedes the public release of the GLES 3 spec by a month). Also, I think > there are still lots of ES 2 apps in development. Denying them access to > this extension just because the usual "If the GL is OpenGL ES 2.0, all > references to [...] are deleted" phrase was omitted from the spec seems > silly to me.
I see your point. > > glObjectPtrLabel could be dropped instead of being included as a useless > entry-point in that case, although given that in Mesa ES 2 and ES 3 share an > api implementation this may not matter to us. (I don't know if there are > drivers in Mesa that support ES 2 but not ES 3.) Sure. > >> >>> ProgramPipeline objects and display lists are only supported by desktop >>> gl. >>> #ifdef-hell, here we go again ;) >>> >>> Sorry for not picking up on these problems in my first review. >> >> >> No problem. :) >> >>> >>> Be sure to test sync objects in gles3 regardless of the presence of the >>> GL_ARB_sync extension. >> >> >> Ok. Also based on the documentation glFenceSync() is from OpenGL 3.2 >> even without the extension. So I can probably send this small patch >> for now. > > I'm afraid I'm not following. What small patch are you talking about? I didn't send yet. But it's basically testing the version of OpenGL as well, instead of only the GL_ARB_sync extension. Felipe _______________________________________________ Piglit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/piglit
