<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Tue, Nov 29, 2016 at 12:48 PM Kristian H. Kristensen <<a href="mailto:hoegsberg@gmail.com">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="gmail_msg">
<br class="gmail_msg">
Here's a few patches to the genxml files that I've been sitting on. The<br class="gmail_msg">
main part of the series is about emitting C enums for genxml enums, so<br class="gmail_msg">
it looks nice and pretty in gdb. It also adds support to aubinator so<br class="gmail_msg">
it knows how to decode enums as well as the inline values Lionel added<br class="gmail_msg">
support for:<br class="gmail_msg">
<br class="gmail_msg">
    Surface Array: true<br class="gmail_msg">
    Surface Format: 2 (R32G32B32A32_UINT)<br class="gmail_msg">
    Surface Vertical Alignment: 1 (VALIGN 4)<br class="gmail_msg">
<br class="gmail_msg">
Look at that surface format being decoded! Patch 17 is "take it or<br class="gmail_msg">
leave it". I know nothing in mesa needs unpack functions, but I wouldn't<br class="gmail_msg">
mind having it upstream so I don't lose it (again).<br class="gmail_msg"></blockquote><div><br></div><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><div><br></div><div>Kristian</div></div></div>