Reviewed-by: Jeremy Huddleston Sequoia <[email protected]> On Jul 20, 2012, at 05:54, Jon TURNEY <[email protected]> wrote:
> Commit 84956ca4 bogusly adds GLX_INTEL_swap_event to the extensions reported > by swrast. > > "DRI2 supports this now - and already enables it explicitly - but drisw does > not and should not. Otherwise toolkits like clutter will only ever SwapBuffers > once and wait forever for an event that's not coming." > > (A similar bug for direct swrast is already fixed in mesa commit 25620eb1) > > (Note that this may be papering over the cracks somewhat, as if we do report > GLX_INTEL_swap_event, some clutter apps fail with GLXBadDrawable calling > GLXChangeDrawableAttributes to change the setting of > GLX_BUFFER_SWAP_COMPLETE_INTEL_MASK > in the GLX_EVENT_MASK, apparently after the drawable is destroyed, which > suggests > a bug with GLXDrawable lifetimes) > > Signed-off-by: Jon TURNEY <[email protected]> > --- > glx/glxscreens.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/glx/glxscreens.c b/glx/glxscreens.c > index 386987e..037b037 100644 > --- a/glx/glxscreens.c > +++ b/glx/glxscreens.c > @@ -174,7 +174,7 @@ static char GLXServerExtensions[] = > "GLX_SGIS_multisample " > #endif > "GLX_SGIX_fbconfig " > - "GLX_SGIX_pbuffer " "GLX_MESA_copy_sub_buffer " "GLX_INTEL_swap_event"; > + "GLX_SGIX_pbuffer " "GLX_MESA_copy_sub_buffer "; > > static Bool > glxCloseScreen(ScreenPtr pScreen) > -- > 1.7.9 > > _______________________________________________ > [email protected]: X.Org development > Archives: http://lists.x.org/archives/xorg-devel > Info: http://lists.x.org/mailman/listinfo/xorg-devel > _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
