[Mesa-dev] [PATCH 0/2] Extra check for GenQueriesARB and DeleteQueriesARB

Thu Apr 7 05:29:47 UTC 2016

On 04/06/2016 07:47 PM, Kenneth Graunke wrote:
> On Tuesday, April 5, 2016 2:26:59 PM PDT kevin.rogovin at intel.com wrote:
>> From: Kevin Rogovin <kevin.rogovin at intel.com>
>>
>> This patch sequence enforces an extra check
>> for GenQueriesARB and DeleteQueriesARB that
>> is not preset in GenQueries and DeleteQueries.
>>
>> Kevin Rogovin (2):
>>   Differentiate between GenQueries and GenQueriesARB
>>   Differentiate between DeleteQueries and DeleteQueriesARB
>>
>>  src/mapi/glapi/gen/gl_API.xml |  4 ++--
>>  src/mesa/main/context.c       |  1 +
>>  src/mesa/main/queryobj.c      | 51 +++++++++++++++++++++++++++++++++++++
> +-----
>>  src/mesa/main/queryobj.h      |  4 ++++
>>  4 files changed, 52 insertions(+), 8 deletions(-)
> 
> NAK for the series.
> 
> There is no reason to reject GenQueries during an active query - all it
> does is create a name.  It's completely harmless.
> 
> You are correct that the ARB_occlusion_query specification does say that
> an error should be generated.  However, I believe the fact that no OpenGL
> specification includes this error is because it was removed intentionally,
> and should be treated as a clarification / bug fix.
> 
> Khronos bug #7612 calls this a "bug fix", and mentions that the
> specification was updated to match shipping implementations:
> https://cvs.khronos.org/bugzilla/show_bug.cgi?id=7612
> 
> Also, making the ARB and non-ARB versions behave differently over an
> error that nobody wants seems pretty asinine, IMO.
> 
> For reference, we removed this error in commit
> 29996e219912fb1fdf35a6595d8f6261384a18dc.

As far as I can tell neither NVIDIA nor AMD generate this error either.
 In fact, NVIDIA uses the same GLX protocol for the ARB and core
functions making it impossible to have different behavior for the ARB
and core functions in indirect contexts.