<div dir="ltr">On 30 May 2017 at 19:19, Richard Purdie &lt;<a href="mailto:richard.purdie@linuxfoundation.org">richard.purdie@linuxfoundation.org</a>&gt; wrote:<br>&gt;<br>&gt; On Tue, 2017-05-30 at 12:00 -0400, Trevor Woerner wrote:<br>&gt; &gt; On Tue, May 30, 2017 at 11:44 AM, Richard Purdie<br>&gt; &gt; &lt;<a href="mailto:richard.purdie@linuxfoundation.org">richard.purdie@linuxfoundation.org</a>&gt; wrote:<br>&gt; &gt; &gt;<br>&gt; &gt; &gt; &gt;<br>&gt; &gt; &gt; &gt; Maybe this would be a better way forward? Enable it, always, in<br>&gt; &gt; &gt; &gt; ./configure,<br>&gt; &gt; &gt; &gt; add the patch to check for it in the code itself, then you can<br>&gt; &gt; &gt; &gt; build<br>&gt; &gt; &gt; &gt; for all<br>&gt; &gt; &gt; &gt; the gallium targets (including swrast) and know that the patented<br>&gt; &gt; &gt; &gt; code will<br>&gt; &gt; &gt; &gt; only kick in where it&#39;s supported in hardware (and thus patent-<br>&gt; &gt; &gt; &gt; safe)?<br>&gt; &gt; &gt; That sounds like exactly what we need to do!<br>&gt; &gt; Okay. The only caveat being, if someone actually *wanted* to enable<br>&gt; &gt; float for swrast, this would prevent them from going down that path.<br>&gt; &gt;<br>&gt; &gt; ajax on #dri-devel pointed out that some piglit tests go from pass to<br>&gt; &gt; fail instead of being skipped as a result of the above patch since<br>&gt; &gt; the<br>&gt; &gt; piglit tests are done with swrast and run very poorly without float<br>&gt; &gt; enabled.<br>&gt;<br>&gt; If someone really wants to enable it, they can drop the patch. I&#39;m open<br>&gt; to feedback from others like Ross/Jussi but IMO I think we can accept<br>&gt; the degraded piglit results for swrast...<br><div><br></div><div>Sounds like a completely reasonable compromise to me.</div><div><br></div><div>Jussi</div></div>