[PATCH 2/4] Do not report the GLX_INTEL_swap_event extension for indirect swrast
Jeremy Huddleston Sequoia
jeremyhu at apple.com
Tue Jul 24 11:07:12 PDT 2012
Reviewed-by: Jeremy Huddleston Sequoia <jeremyhu at apple.com>
On Jul 20, 2012, at 05:54, Jon TURNEY <jon.turney at dronecode.org.uk> 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 <jon.turney at dronecode.org.uk>
> ---
> 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
>
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
>
More information about the xorg-devel
mailing list