<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Aug 3, 2017 at 10:22 AM, Jordan Justen <span dir="ltr"><<a href="mailto:jordan.l.justen@intel.com" target="_blank">jordan.l.justen@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">subect: form => from<br>
<br>
Commit message? Why is this being removed?<br><div><div class="h5"></div></div></blockquote><div><br></div><div>See below...<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">
On 2017-08-02 13:35:29, Jason Ekstrand wrote:<br>
> ---<br>
>  src/mesa/drivers/dri/i965/<wbr>intel_mipmap_tree.c | 11 ++---------<br>
>  1 file changed, 2 insertions(+), 9 deletions(-)<br>
><br>
> diff --git a/src/mesa/drivers/dri/i965/<wbr>intel_mipmap_tree.c b/src/mesa/drivers/dri/i965/<wbr>intel_mipmap_tree.c<br>
> index daa49f9..0e0aaa8 100644<br>
> --- a/src/mesa/drivers/dri/i965/<wbr>intel_mipmap_tree.c<br>
> +++ b/src/mesa/drivers/dri/i965/<wbr>intel_mipmap_tree.c<br>
> @@ -59,8 +59,7 @@ intel_miptree_alloc_aux(struct brw_context *brw,<br>
>                          struct intel_mipmap_tree *mt);<br>
><br>
>  static bool<br>
> -is_mcs_supported(const struct brw_context *brw, mesa_format format,<br>
> -                 uint32_t layout_flags)<br>
> +is_mcs_supported(const struct brw_context *brw, mesa_format format)<br>
>  {<br>
>     /* Prior to Gen7, all MSAA surfaces used IMS layout. */<br>
>     if (brw->gen < 7)<br>
> @@ -86,11 +85,6 @@ is_mcs_supported(const struct brw_context *brw, mesa_format format,<br>
>         */<br>
>        if (brw->gen == 7 && _mesa_get_format_datatype(<wbr>format) == GL_INT) {<br>
>           return false;<br>
> -      } else if (layout_flags & MIPTREE_LAYOUT_DISABLE_AUX) {<br>
> -         /* We can't use the CMS layout because it uses an aux buffer, the MCS<br>
> -          * buffer. So fallback to UMS, which is identical to CMS without the<br>
> -          * MCS. */<br>
> -         return false;<br>
<br>
</div></div>I see that intel_miptree_create_for_dri_<wbr>image will set this flag for<br>
'winsys' images, so why is this change safe to make? (Maybe it isn't<br>
quite 'safe' on it's own?)<br>
<br>
I think the answer is in the next patch 'i965/miptree: Refactor<br>
is_mcs_supported' where is_mcs_supported is refactored to<br>
intel_miptree_supports_mcs. I think maybe it move from asking if the<br>
format potentially supports MCS to whether the miptree actually has<br>
MCS enabled. ?<br>
<br>
If so, maybe a small commit message comment would help? Or, maybe<br>
sqaush it with the next patch?<br>
<br>
-Jordan<br>
<span class=""><br>
>        } else {<br>
>           return true;<br>
>        }<br>
> @@ -329,8 +323,7 @@ intel_miptree_choose_aux_<wbr>usage(struct brw_context *brw,<br>
>  {<br>
>     assert(mt->aux_usage == ISL_AUX_USAGE_NONE);<br>
><br>
> -   const unsigned no_flags = 0;<br>
> -   if (mt->surf.samples > 1 && is_mcs_supported(brw, mt->format, no_flags)) {<br></span></blockquote><div><br></div><div>The one caller of this function pases 0 in as the flags parameter.  I can put that in the commit message.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
> +   if (mt->surf.samples > 1 && is_mcs_supported(brw, mt->format)) {<br>
>        assert(mt->surf.msaa_layout == ISL_MSAA_LAYOUT_ARRAY);<br>
>        mt->aux_usage = ISL_AUX_USAGE_MCS;<br>
>     } else if (intel_tiling_supports_ccs(<wbr>brw, mt->surf.tiling) &&<br>
> --<br>
> 2.5.0.400.gff86faf<br>
><br>
</span>> ______________________________<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>
</blockquote></div><br></div></div>