<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Jun 28, 2017 at 2:15 PM, Chad Versace <span dir="ltr"><<a href="mailto:chadversary@chromium.org" target="_blank">chadversary@chromium.org</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 16 Jun 2017, Jason Ekstrand wrote:<br>
> Any form of CCS on gen9+ only works on Y-tiled images.  The only caller<br>
> of create_for_bo which uses Y-tiled BOs is create_for_dri_image.<br>
<br>
</span>If I understand ARC++ correctly, then intel_update_image_buffer() also<br>
calls intel_miptree_create_for_bo() for Android Y-tiled winsys buffers.<br>
(I've confirmed it with code inspection, but not with actual debug<br>
logging). That should be noted in the commit message.<br>
<br>
This patch shouldn't degrade ARC++ performance, though, because ARC++ is<br>
still using an old Mesa that never allocated CCS for Android winsys<br>
buffers.<br></blockquote><div><br></div><div>No, it shouldn't degrade ARC++ performance because patch 15 (the previous one) makes intel_update_image_buffer() call intel_miptree_create_for_dri_image().  :-)<br><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">
At the end of the patch series, will Android's Y-tiled winsys buffers<br>
get the benefit of a private CCS?<br>
<span class=""><br>
> +   /* Since CCS_E can compress more than just clear color, we create the<br>
> +    * CCS for it up-front.  For CCS_D which only compresses clears, we<br>
> +    * create the CCS on-demand when a clear occurs that wants one.<br>
> +    */<br>
> +   if (mt->aux_usage == ISL_AUX_USAGE_CCS_E) {<br>
> +      if (!intel_miptree_alloc_ccs(brw, mt)) {<br>
> +         intel_miptree_release(&mt);<br>
> +         return NULL;<br>
> +      }<br>
> +   }<br>
> +<br>
<br>
</span>The above hunk is a duplicate. The same 'if' tree appears immediately<br>
above it.<br></blockquote><div><br></div><div>It wasn't duplicated so much as rebased into the wrong hunk.  It should have ended up in create_for_dri_image.  I've moved it.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
With the hunk de-duplicated, this patch is<br>
Reviewed-by: Chad Versace <<a href="mailto:chadversary@chromium.org">chadversary@chromium.org</a>><br></blockquote><div><br></div><div>Thanks!<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
>     return mt;<br>
>  }<br>
<span class="HOEnZb"><font color="#888888">><br>
> --<br>
> 2.5.0.400.gff86faf<br>
><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>
</font></span></blockquote></div><br></div></div>