<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Nov 10, 2017 at 4:12 PM, Andres Gomez <span dir="ltr"><<a href="mailto:agomez@igalia.com" target="_blank">agomez@igalia.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Jason, having this into account, I'll leave this patch out of 17.2 so<br>
far we don't have another one that fixes this regression (?)<br></blockquote><div><br></div><div>This patch doesn't regress anything, it just isn't sufficient to fix the bug on little-core.</div><div><br></div><div>--Jason<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I noticed that the patch bisected by Mark is a different one so I'm not<br>
sure I'm understanding the status, though.<br>
<br>
Let me know what you think.<br>
<div><div class="h5"><br>
On Thu, 2017-11-09 at 17:01 -0800, Jason Ekstrand wrote:<br>
> On Thu, Nov 9, 2017 at 2:23 PM, Matt Turner <<a href="mailto:mattst88@gmail.com">mattst88@gmail.com</a>> wrote:<br>
> > On Thu, Nov 2, 2017 at 3:54 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> > > Register strides higher than 4 are uncommon but they can happen.  For<br>
> > > instance, if you have a 64-bit extract_u8 operation, we turn that into<br>
> > > UB -> UQ MOV with a source stride of 8.  Our previous calculation would<br>
> > > try to generate a stride of <32;8,8>:ub which is invalid because the<br>
> > > maximum horizontal stride is 4.  To solve this problem, we instead use a<br>
> > > stride of <8;1,0>.  As noted in the comment, this does not work as a<br>
> > > destination but that's ok as very few things actually generate that<br>
> > > stride.<br>
> ><br>
> > Please put the tests you fixed in the commit message. It's not okay to<br>
> > leave that out for all the reasons that I'm sure you know.<br>
><br>
> I didn't because the test passes before and after the patch.  I guess I could have included that information though.<br>
><br>
> > Looks like this doesn't work on CHV, BXT, GLK :(<br>
> ><br>
> > KHR-GL46.shader_ballot_tests.<wbr>ShaderBallotBitmasks now fails on CHV,<br>
> > BXT, GLK with:<br>
> ><br>
> > mov(8)          g21<1>UQ        g19<8,1,0>UB                    { align1 1Q };<br>
> >         ERROR: Source and destination horizontal stride must equal and<br>
> > a multiple of a qword when the execution type is 64-bit<br>
> >         ERROR: Vstride must be Width * Hstride when the execution type is 64-bit<br>
> ><br>
> > Modulo the typo in the first error, I think both of these are correct.<br>
> > I don't think we can extract_u8 to a 64-bit type on Atom :(<br>
><br>
> That's unfortunate...  Quickly racking my brain, I don't see a slick way to implement that opcode.  How would you feel about some late opt_algebraic lowering?<br>
><br>
> > This is filed as <a href="https://bugs.freedesktop.org/show_bug.cgi?id=103628" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/<wbr>show_bug.cgi?id=103628</a><br>
><br>
</div></div>> ______________________________<wbr>_________________<br>
> mesa-stable mailing list<br>
> <a href="mailto:mesa-stable@lists.freedesktop.org">mesa-stable@lists.freedesktop.<wbr>org</a><br>
> <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-stable" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-stable</a><br>
<span class="HOEnZb"><font color="#888888">--<br>
Br,<br>
<br>
Andres<br>
</font></span></blockquote></div><br></div></div>