[Mesa-dev] [PATCH] Revert "i965/fs: Use align1 mode on ternary instructions on Gen10+"

[Anuj Phogat]

On Mon, Dec 25, 2017 at 3:05 AM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> On 22 December 2017 at 21:54, Anuj Phogat <anuj.phogat at gmail.com> wrote:
>> This reverts commit 9cd60fce9c22737000a8f8dc711141f8a523fe75.
>> Above commit caused 2000+ piglit tests to assert fail. Disabling
>> the align1 mode on gen10 for now to avoid failures.
>>
>> Cc: Matt Turner <mattst88 at gmail.com>
>> Cc: Rafael Antognolli <rafael.antognolli at intel.com>
>> Signed-off-by: Anuj Phogat <anuj.phogat at gmail.com>
>
> Ouch, any ideas why this did not flag up earlier? The commit itself
> landed back in Oct.
> Should we pick this for stable, since 17.3.x has the offending commit?
>
We still don't have a Cannonlake system plugged in to our CI system.
That's the reason we couldn't catch these regressions.

I posted this patch as a temporary solution to avoid assertion. I
would wait to hear from Matt before picking it to stable.


> Thanks
> Emil