<div dir="ltr"><div><div><div>I'm a bit inclined to drop patch 15 because I think tile mode deserves to remain inline.  I don't care about 17.  I didn't really review it for not breaking things though.  The rest are<br><br></div>Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br><br></div>Don't push until you get Jenkins' ok though!<br><br></div>--Jason<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Nov 29, 2016 at 12:55 PM, Kristian Høgsberg <span dir="ltr"><<a href="mailto:hoegsberg@gmail.com" target="_blank">hoegsberg@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><span class=""><div dir="ltr">On Tue, Nov 29, 2016 at 12:48 PM Kristian H. Kristensen <<a href="mailto:hoegsberg@gmail.com" target="_blank">hoegsberg@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br class="m_-1602583395830110036gmail_msg">
<br class="m_-1602583395830110036gmail_msg">
Here's a few patches to the genxml files that I've been sitting on. The<br class="m_-1602583395830110036gmail_msg">
main part of the series is about emitting C enums for genxml enums, so<br class="m_-1602583395830110036gmail_msg">
it looks nice and pretty in gdb. It also adds support to aubinator so<br class="m_-1602583395830110036gmail_msg">
it knows how to decode enums as well as the inline values Lionel added<br class="m_-1602583395830110036gmail_msg">
support for:<br class="m_-1602583395830110036gmail_msg">
<br class="m_-1602583395830110036gmail_msg">
    Surface Array: true<br class="m_-1602583395830110036gmail_msg">
    Surface Format: 2 (R32G32B32A32_UINT)<br class="m_-1602583395830110036gmail_msg">
    Surface Vertical Alignment: 1 (VALIGN 4)<br class="m_-1602583395830110036gmail_msg">
<br class="m_-1602583395830110036gmail_msg">
Look at that surface format being decoded! Patch 17 is "take it or<br class="m_-1602583395830110036gmail_msg">
leave it". I know nothing in mesa needs unpack functions, but I wouldn't<br class="m_-1602583395830110036gmail_msg">
mind having it upstream so I don't lose it (again).<br class="m_-1602583395830110036gmail_msg"></blockquote><div><br></div></span><div>Oh, and true to form, I forgot the most important point: patch 16 is a legitimate</div><div>fix for the compute shader KSP being split across a innocuous looking KernelStartPointer</div><div>32 bit field and a 16 bit "Kernel Start Pointer High" field. The driver doesn't know this</div><div>and only sets KernelStartPointer. As long as anv uses instruction base offset, it's unlikely to</div><div>cause a problem, but should be fixed.</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>Kristian</div></font></span></div></div>
<br>______________________________<wbr>_________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
<br></blockquote></div><br></div>