Ben Skeggs wrote:
> Hey,
> 
> Attached is a patch which adds driver hooks to update the status of a
> query object (both to core mesa, and the gallium state tracker).  There
> is currently an issue with applications that spin waiting for
> GL_QUERY_RESULT_AVAILABLE_ARB to be true (eg. arbocclude).  The current
> code will simply check return q->Ready, and doesn't give the driver a
> chance to actually update this field, so the application will end up
> spinning forever.

Ben,

I've actually got more or less a superset of this change queued up to go 
in -- I used FlushQuery() rather than UpdateQuery(), and goes a bit 
further in some respects.

Comparing the two, it seems like you've missed a couple of cases where 
it's necessary to call Flush/UpdateQuery() - basically there are more 
GL_QUERY_RESULT_AVAILABLE_ARB occurances in queryobj.c that need to be 
covered.

I'll push something pretty shortly.

Keith

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Mesa3d-dev mailing list
Mesa3d-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mesa3d-dev

Reply via email to