[PATCH xserver] glamor: Store the actual EGL/GLX context pointer in lastGLContext

Michel Dänzer michel at daenzer.net
Tue Jun 6 07:53:49 UTC 2017


On 01/06/17 11:49 PM, Keith Packard wrote:
> Michel Dänzer <michel at daenzer.net> writes:
> 
>>> Thanks for debugging -- this sounds like a pain.
>>
>> Indeed, it was. :}
> 
> I agree, nice work.
> 
>> Setting lastGLContext = NULL from CloseScreen would also prevent the
>> problem; I considered doing that instead of or in addition to the fix in
>> glamor_make_current, but finally decided against it because it would be
>> more of a workaround than a fix for the actual problem (and could
>> unnecessarily force a MakeCurrent call after a server reset in some cases).
> 
> And it looks like FreeScreen is already doing the right stuff.
> 
> Might be nice to have the glx side also clean up at some point, but I'm
> not sure where we'd stick that as we just use glXGetCurrentContext() for
> that, and have no control over lifetime.

I don't think the GLX code is affected by the same issue.


> I think we're done here; this looks like a simple "thinko" to me.
> 
> Reviewed-by: Keith Packard <keithp at keithp.com>

Thanks Keith and Eric for the review! Fix pushed.


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer



More information about the xorg-devel mailing list