<div dir="auto"><div><div class="gmail_extra"><div class="gmail_quote">On Mar 2, 2017 3:38 AM, "Emil Velikov" <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@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 1 March 2017 at 20:06, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> On Wed, Mar 1, 2017 at 10:35 AM, Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>><br>
> wrote:<br>
>><br>
>> Hello list,<br>
>><br>
>> The candidate for the Mesa 17.0.1 is now available. Currently we have:<br>
>>  - 60 queued<br>
>>  - 9 nominated (outstanding)<br>
>>  - and 0 rejected patch(es)<br>
>><br>
>><br>
>> The current queue consists of:<br>
>><br>
>> On the GLX/EGL front we have a GLVND fix for "The Binding of Isaac:<br>
>> Rebirth"<br>
>> and other games, eglQuerySurface now returns correct geometry when running<br>
>> under X11/DRI3.<br>
>><br>
>> There's a number of crash fixes affecting all Gallium drivers. An old<br>
>> regression fix for r300 on BE hardware been fixed. The radeonsi driver has<br>
>> fixes for Tessellation shaders on Carrizo and Stoney hardware.<br>
>><br>
>> While on the nouveau side, compute shader have been improved on some<br>
>> nvc0 devices.<br>
>><br>
>> The vc4 and etnaviv drivers have also seen a couple of small fixe.<br>
>><br>
>> For the Intel drivers (both GL and Vulkan) we have a diverse bunch of<br>
>> patches -<br>
>> from CTS fixes for Sandy Bridge, to improved swizzle clears and improved<br>
>> handling of GPUs without (Last Level Cache) LLC.<br>
>><br>
>> On integration side - we had some Android build fixes and a new script to<br>
>> parse and look for bug fixes.<br>
>><br>
>><br>
>> And for those of you wondering - the delay was caused by a buggy<br>
>> optimisation<br>
>> pass, that has since been removed.<br>
><br>
><br>
> I don't see the remove patch on the queue below.<br>
><br>
</div>As always, fixes are squashed with the regressing commit.<br>
<br>
In this case "i965/fs: Remove the inline pack_double_2x32<br>
optimization" is folded with "i965/fs: Fix the inline<br>
nir_op_pack_double optimization".<br>
Admittedly I should have dropped the regressing commit in the first place.<br></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">No, it was a bugfix it just didn't fix enough of it.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
How does the following [kind of] note look ? I don't mind adding it<br>
for future instances.<br>
<br>
...<br>
<div class="quoted-text">       i965/fs: Fix the inline nir_op_pack_double optimization<br>
</div>* Squashed with:<br>
       i965/fs: Remove the inline pack_double_2x32 optimization<br>
...</blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">In this particular case, I wouldn't even mention the "fix" commit.  The correct fix was to just drop the optimization altogether.</div><div dir="auto"></div></div>