[Mesa-dev] [PATCH 5/8] main: enable NV_primitive_restart extension by default

[Brian Paul]

On 05/11/2012 04:26 PM, Jordan Justen wrote:
> On Fri, May 11, 2012 at 2:41 PM, Brian Paul<brianp at vmware.com>  wrote:
>> On 05/11/2012 11:35 AM, Marek Olšák wrote:
>>> In that case, I think we should disable it in the state tracker if
>>> there's no driver support. If it's faster for svga, it's very likely
>>> faster for the other DX9-level drivers too.
>>
>> Some Viewperf tests use primitive restart and we wanted to support it with
>> our vmware driver.  So I'd like to at least be able to do "export
>> MESA_EXTENSION_OVERRIDE=GL_NV_primitive_restart" and see the extension
>> supported (as a software/VBO fallback).
>
> Marek requested that all gallium drivers default to enable the
> extension and software primitive restart when required like today, but
> that all non-gallium drivers not use the software primitive restart by
> default, and not advertise support for the extension.
>
> Would that work for you?
>
> With this approach, simply using
> MESA_EXTENSION_OVERRIDE=GL_NV_primitive_restart on a non-gallium
> driver would not work, but it would match what happens today...

I'm not too concerned about non-gallium drivers but I think it should 
always be enabled for the swrast driver because I still occasionally 
test things with that driver.

-Brian