<div dir="auto"><div><div class="gmail_extra"><div class="gmail_quote">On Jan 11, 2017 7:34 AM, "Erik Faye-Lund" <<a href="mailto:kusmabite@gmail.com">kusmabite@gmail.com</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="elided-text">On Wed, Jan 11, 2017 at 4:14 PM, Nicolai Hähnle <<a href="mailto:nhaehnle@gmail.com">nhaehnle@gmail.com</a>> wrote:<br>
> On 11.01.2017 13:17, Marek Olšák wrote:<br>
>><br>
>> On Tue, Jan 10, 2017 at 6:48 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br>
>> wrote:<br>
>>><br>
>>> I'll be honest, I'm not a fan... Given that D3D10 has one defined<br>
>>> behavior,<br>
>>> D3D9 has another, and GL doesn't specify, I don't really think we should<br>
>>> be<br>
>>> making a global change to all drivers to do the D3D9 behavior just to fix<br>
>>> one app.  Sure, other apps probably have the same bug, but are we going<br>
>>> to<br>
>>> have apps that expect the D3D10 behavior that we've now explicitly made<br>
>>> not<br>
>>> work?<br>
>>><br>
>>> If we're going to hack around an app bug, I would really rather see it<br>
>>> behind a driconf option rather than a global change to driver behavior.<br>
>>> Even better, it'd be cool if we could see the app get fixed. (Yes, I know<br>
>>> that's not likely).<br>
>><br>
>><br>
>> I think we are not in a position to refuse this workaround, or put<br>
>> more precisely, to have a different behavior from everybody else. By<br>
>> "we", I mean i965, radeonsi, svga. All closed drivers use abs. Many<br>
>> Mesa drivers also use abs internally (r300, r600, nv30, nv50/nvc0).<br>
>> This is not really a workaround for a specific application, even<br>
>> though it's strongly motivated by that. It's a fix to align the few<br>
>> remaining drivers with all others.<br>
>><br>
>> We talked with the publisher about this a very long time ago. While I<br>
>> don't remember the details (Nicolai?), I think they refused to fix it<br>
>> because radeonsi appeared to be the only driver not doing abs.<br>
><br>
><br>
> If I remember correctly, it wasn't so much a refusal as a lack of<br>
> follow-through. They even had an option in their framework to add the<br>
> abs(...) when translating shaders, but somehow didn't turn it on<br>
> unconditionally for some reason...<br>
<br>
</div>VP even says so here:<br>
<a href="https://github.com/virtual-programming/specops-linux/issues/20" rel="noreferrer" target="_blank">https://github.com/virtual-<wbr>programming/specops-linux/<wbr>issues/20</a><br>
<br>
They recommend against patching mesa to do abs, though.<br>
</blockquote></div><br></div></div><div class="gmail_extra" dir="auto">Wow... That was a lame reply.</div></div>