[PATCH 1/3] Revert bogus GlxPushProvider() in commit a1d41e3

Jon TURNEY jon.turney at dronecode.org.uk
Thu Jul 12 05:30:21 PDT 2012

a1d41e3 "Move extension initialisation prototypes into extinit.h"
also includes a change to GlxExtensionInit to install the swrast GLX

Since b86aa74 "GLX: Insert swrast provider from GlxExtensionInit"
already does this (correctly, by installing the swrast provider
at the end of the chain, rather than at the beginning), and since this
would seem to have the effect of making the swrast provider the most
preferred provider, I'm guessing this wasn't intended.
 glx/glxext.c |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)

diff --git a/glx/glxext.c b/glx/glxext.c
index a1f9d42..70f0df8 100644
--- a/glx/glxext.c
+++ b/glx/glxext.c
@@ -339,9 +339,6 @@ GlxExtensionInit(void)
     if (!__glXContextRes || !__glXDrawableRes)
-    if (serverGeneration == 1)
-        GlxPushProvider(&__glXDRISWRastProvider);
     if (!dixRegisterPrivateKey
         (&glxClientPrivateKeyRec, PRIVATE_CLIENT, sizeof(__GLXclientState)))

More information about the xorg-devel mailing list