<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Jun 6, 2017 at 11:26 PM, Chad Versace <span dir="ltr"><<a href="mailto:chad@kiwitree.net" target="_blank">chad@kiwitree.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri 26 May 2017, Jason Ekstrand wrote:<br>
> ---<br>
>  src/mesa/drivers/dri/i965/brw_<wbr>clear.c         | 12 ++++++------<br>
>  src/mesa/drivers/dri/i965/brw_<wbr>context.c       |  7 ++++---<br>
>  src/mesa/drivers/dri/i965/brw_<wbr>draw.c          | 17 +++++++++--------<br>
>  src/mesa/drivers/dri/i965/<wbr>intel_mipmap_tree.c | 23 ++++++++++++++++++++++-<br>
>  src/mesa/drivers/dri/i965/<wbr>intel_mipmap_tree.h |  9 +++++++++<br>
>  5 files changed, 50 insertions(+), 18 deletions(-)<br>
<br>
</span>Yes! For me, in this patch, the series finally clicked for me.<br></blockquote><div><br></div><div>I don't know why this patch was the one that convinced you... The best is yet to come!<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Up until this patch, I've been cheerfully reviewing nice cleanups, nice<br>
refactors, nice simplifications. All those changes, however, felt like<br>
incremental improvements, each localized in concept.<br>
<br>
In this patch, I finally see how big of a global difference the<br>
series makes. Compared to the existing code (which was hairy<br>
spaghetti), reasoning about aux state and writing correct code to handle<br>
aux state transitions is now relatively effortless.<br>
<br>
Patch 23 is<br>
Reviewed-by: Chad Versace <<a href="mailto:chadversary@chromium.org">chadversary@chromium.org</a>><br>
</blockquote></div><br></div></div>