<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 20, 2015 at 8:49 AM, Brian Paul <span dir="ltr"><<a href="mailto:brian.e.paul@gmail.com" target="_blank">brian.e.paul@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div class="h5">On Thu, Nov 19, 2015 at 7:04 PM, Ian Romanick <span dir="ltr"><<a href="mailto:idr@freedesktop.org" target="_blank">idr@freedesktop.org</a>></span> wrote:<br></div></div><div class="gmail_extra"><div class="gmail_quote"><div><div class="h5"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On 11/19/2015 05:40 PM, Nanley Chery wrote:<br>
><br>
> On Thu, Nov 19, 2015 at 12:13 PM, Brian Paul <<a href="mailto:brianp@vmware.com" target="_blank">brianp@vmware.com</a><br>
</span><span>> <mailto:<a href="mailto:brianp@vmware.com" target="_blank">brianp@vmware.com</a>>> wrote:<br>
><br>
>     Hi Nanley,<br>
><br>
><br>
> Hi Brian,<br>
><br>
><br>
>     Maybe you can fix an issue I have with the new extension code.<br>
><br>
>     Previously, I could do something like export<br>
>     MESA_EXTENSION_OVERRIDE="-ARB_clear_buffer_object" and I would no<br>
>     longer see it in the GL_EXTENSIONS string, even if it was an "always<br>
>     on" extension.<br>
><br>
>     Now when I try that I get:<br>
><br>
>     Mesa 11.1.0-devel implementation error: Trying to disable<br>
>     permanently enabled extensions: GL_ARB_get_texture_sub_image<br>
>     Please report at <a href="https://bugs.freedesktop.org/enter_bug.cgi?product=Mesa" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/enter_bug.cgi?product=Mesa</a><br>
><br>
>     The whole point of the "-GL_EXT_foobar" syntax was to hide an<br>
>     extension from the application when it queries the driver's extensions.<br>
><br>
>     Can you please fix this so it works as before?<br>
><br>
><br>
> I have two branches that provide the ability to disable permanently<br>
> enabled extensions:<br>
> 1. The first only modifies the extension strings and is located here:<br>
> <a href="http://cgit.freedesktop.org/~nchery/mesa/commit/?h=mod_always_on" rel="noreferrer" target="_blank">http://cgit.freedesktop.org/~nchery/mesa/commit/?h=mod_always_on</a><br>
</span>> <<a href="http://cgit.freedesktop.org/%7Enchery/mesa/commit/?h=mod_always_on" rel="noreferrer" target="_blank">http://cgit.freedesktop.org/%7Enchery/mesa/commit/?h=mod_always_on</a>><br>
<span>> 2. The second modifies the extension strings and disables the extension<br>
> within the driver (assuming appropriate the helper function is used). It<br>
> also provides some performance benefits. :<br>
> <a href="http://cgit.freedesktop.org/~nchery/mesa/commit/?h=init_ext_vals" rel="noreferrer" target="_blank">http://cgit.freedesktop.org/~nchery/mesa/commit/?h=init_ext_vals</a><br>
</span>> <<a href="http://cgit.freedesktop.org/%7Enchery/mesa/commit/?h=init_ext_vals" rel="noreferrer" target="_blank">http://cgit.freedesktop.org/%7Enchery/mesa/commit/?h=init_ext_vals</a>><br>
<span>><br>
> I'd appreciate any feedback on the two approaches as I work to get the<br>
> feature upstreamed.<br>
<br>
</span>I think #2 might be better, but there's a lot of churn.  I don't know<br>
that we want that much churn right around the time of the release branch<br>
point, and I think it would be good to have this resolved in 11.1.  I<br>
also have a few bits of feedback in #2, so it might take a couple<br>
iterations before that could land.<br><br></blockquote></div></div><div><br>Since 11.1 is coming up, can we go with the simpler #1 for now, then go to #2 after 11.1 branches?<span class="HOEnZb"><font color="#888888"><br><br></font></span></div></div></div></div></blockquote><div><br></div><div>I don't mind going this route. Unfortunately, I'm running into presently unexplainable linker errors in the process of making the gtest for this feature.<br><br></div><div>- Nanley<br></div></div></div></div>