<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 13, 2016 at 2:24 AM, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">On 11 June 2016 at 17:02, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
<br>
> The next 20 patches or so are general ISL cleanups and fixes.  If no one is<br>
> too opposed, I'd like to back-port the whole pile to 12.0.  There are two<br>
> reasons for this: First, ISL is new and this is a substantial cleanup;<br>
> back-porting it will make back-porting will keep the initial release of ISL<br>
> cleaner and make back-porting other patches easier in the future.  Second,<br>
> in the middle of the series are a couple of changes that fix some 850<br>
> Vulkan CTS tests on Haswell.<br>
><br>
</span>No serious objections on getting these in 12.0. A bunch of small<br>
suggestions though.<br>
<span class=""><br>
>   i965/gen4: Subtract 1 from buffer sizes<br>
</span>Please polish the commit message a bit.<span class=""><br></span></blockquote><div><br></div><div>I added "The PRM states that the values put in Width, Height, and Depth should be various bits from the value size - 1.  We seem to have done this wrong more-or-less from the start."<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">
>   isl/state: Put surface format setup at the top<br>
>   isl/state: Put all dimension setup together and towards the top<br>
>   isl/state: Put pitch calculations together<br>
</span>Are these solely for cosmetic/ease of read reasons or there's<br>
something more to it ? Can you please mention in the commit message ?<span class=""><br></span></blockquote><div><br></div><div>How about: "This is purely cosmetic, but it makes things look a bit more readable."<em></em><a href="https://www.khronos.org/registry/spir-v/specs/1.1/SPIRV.html#Integer"><em></em></a><a href="https://www.khronos.org/registry/spir-v/specs/1.1/SPIRV.html#Floating"><em><br></em></a></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">
>   genxml: Add enough XML for gens 4, 4.5, and 5 to get SURFACE_STATE<br>
</span>Let me see if I can quickly cook something for the Android build.<br>
<span class=""><br>
<br>
>   isl: Add support for filling out surface states all the way back to<br>
>     gen4<br>
</span>Ditto.<span class=""><br></span></blockquote><div><br></div><div>If you want to give me something, I'm happy to squash it in.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">
<br>
>   isl/formats: Mark RAW as having a block size of 1 byte<br>
</span>Worth mentioning why we want that ?<br></blockquote><div><br></div><div>That patch can actually be dropped.  The original idea was that, for buffer surfaces, we needed to divide by the element size so the size should be non-zero.  Turns out we need to divide by stride so it can be 0 and the patch isn't needed.  I'll defer to Chad on whether it should be 0 or 1.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Thanks<br>
<span class=""><font color="#888888">Emil<br>
</font></span></blockquote></div><br></div></div>