<div dir="ltr">On 20 February 2013 12:26, Carl Worth <span dir="ltr"><<a href="mailto:cworth@cworth.org" target="_blank">cworth@cworth.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
I recently ran into a bunch of piglit tests not getting run, (returning<br>
"skip"), when they used to work fine. The tests say things like:<br>
<br>
        piglit: info: Failed to create GL 2.0 compatibility context<br>
        piglit: info: Failed to create any GL context<br>
<br>
I tracked the failure down via gdb to the following line of code in<br>
waffle (glx_config.c):<br>
<br>
            if (attrs->context_full_version != 10 && !dpy->ARB_create_context) {<br>
                wcore_errorf(WAFFLE_ERROR_UNSUPPORTED_ON_PLATFORM,<br>
                             "GLX_ARB_create_context is required in order to "<br>
                             "request a GL version not equal to the default "<br>
                             "value 1.0");<br>
                return false;<br>
            }<br>
<br>
In my case, attrs->context_full_version is 20.<br>
<br>
When I asked about this in IRC, Eric helped me out a bit:<br>
<br>
        <anholt> cworth: your x server is old, and piglit/waffle don't<br>
        fall back to just making an old context and seeing if it's good<br>
        enough.<br>
<br>
I'll go ahead and upgrade my X server so I can get things running again,<br>
but I'm left with the following questions:<br>
<br>
  1. I'm curious what's the new thing required in the X server here?<br>
<br>
     I'm obviously running tests that worked previously, so I'm not<br>
     really needing any new behavior. So this looks like a regression in<br>
     piglit/waffle.<br></blockquote><div><br></div><div>What's changed was my Piglit patch "shader_runner and glslparsertest: choose GL version more accurately", which caused Piglit to request a GL 2.1 context when running certin GLSL 1.20 tests where it previously requested a 1.0 context.  We're currently discussing what to do about that in the email thread of the same name (see <a href="http://lists.freedesktop.org/archives/piglit/2013-February/004949.html">http://lists.freedesktop.org/archives/piglit/2013-February/004949.html</a> for where the discussion started).<br>
<br></div><div>Eric has argued that we shouldn't require a GL 2.1 context when running GLSL 1.20 tests, because sometimes Mesa supports a version of GLSL higher than is required by the supported GL version.  I was initially skeptical about this but I may be coming around.<br>
<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
  2. Assuming this is a bug, should the missing fallback code be added<br>
     to piglit or to waffle?<br></blockquote><div><br>Personally, I think so, regardless of what we decide to do about 
"shader_runner and glslparsertest: choose GL version more accurately".  Piglit tests that require GL 2.1 should be able to run on any GL 2.1-supporting platform, regardless of whether that platform supports ARB_create_context.  Especially since a lot of people are still running Fedora 17, which doesn't have ARB_create_context X server support yet.<br>
<br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
  3. If waffle, Chad do you have a bug tracker set up for waffle? If<br>
     not, shall we add a component to <a href="http://bugs.freedesktop.org" target="_blank">bugs.freedesktop.org</a>?<br>
<br>
  4. If we don't add automatic fallback code, is there some logical<br>
     place that we could put a better error message in place? It was not<br>
     at all obvious to me from "failed to create context" that my X<br>
     server had suddenly gotten too old, (when it worked previously).<br>
<span class=""><font color="#888888"><br>
-Carl<br></font></span></blockquote><div><br></div><div>FYI, Michel Dänzer discovered this too, and we've been discussing what to do about it in the e-mail thread "shader_runner and glslparsertest: choose GL version more accurately."<br>
</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class=""><font color="#888888">
<br>
--<br>
<a href="mailto:carl.d.worth@intel.com">carl.d.worth@intel.com</a><br>
</font></span><br>_______________________________________________<br>
Piglit mailing list<br>
<a href="mailto:Piglit@lists.freedesktop.org">Piglit@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/piglit" target="_blank">http://lists.freedesktop.org/mailman/listinfo/piglit</a><br>
<br></blockquote></div><br></div></div>