[Mesa-dev] [PATCH] dri2: Insert a synchronisation point for glXWaitX

[Adam Jackson]

On Wed, 2016-08-17 at 08:17 -0700, Eric Anholt wrote:
 
> All I'm saying is that it's the thing that was intentionally used for
> this purpose in DRI1 and early DRI2, and maybe if it works we shouldn't
> just accidentally drop it in a refactor.

I don't see how the patch in this thread does that. I confess I've been
looped in on this somewhat late, so if there's another thread I should be
looking at for context...

> I wholeheartedly endorse fixing glXWaitX to DTRT and using it.  Having
> written about how it's the right way a bunch, are you planning on
> writing a piglit test and the patch?  Or should it go somewhere on my
> list (it's not near the top)?

It's somewhat difficult to write a test for it that fails reliably for
even one driver lets alone multiple, but I'm sure I can provoke DRI2 into
doing something stupid. I'll see what I can come up with.

- ajax