<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Apr 6, 2018 at 8:22 AM, Rafael Antognolli <span dir="ltr"><<a href="mailto:rafael.antognolli@intel.com" target="_blank">rafael.antognolli@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Fri, Apr 06, 2018 at 06:07:52PM +0300, Topi Pohjolainen wrote:<br>
> Otherwise even the clear color gets initialised to 0xFF. This<br>
> allows enabling of color fast clears on ICL without regressing<br>
> multisampling tests.<br>
><br>
> CC: Rafael Antognolli <<a href="mailto:rafael.antognolli@intel.com">rafael.antognolli@intel.com</a>><br>
> CC: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br>
> CC: Nanley Chery <<a href="mailto:nanley.g.chery@intel.com">nanley.g.chery@intel.com</a>><br>
> Signed-off-by: Topi Pohjolainen <<a href="mailto:topi.pohjolainen@intel.com">topi.pohjolainen@intel.com</a>><br>
> ---<br>
>  src/mesa/drivers/dri/i965/<wbr>intel_mipmap_tree.c | 7 ++++++-<br>
>  1 file changed, 6 insertions(+), 1 deletion(-)<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 89074a6..25f901d 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>
> @@ -1680,7 +1680,12 @@ intel_miptree_init_mcs(struct brw_context *brw,<br>
>        return;<br>
>     }<br>
>     void *data = map;<br>
> -   memset(data, init_value, mt->mcs_buf->size);<br>
> +<br>
> +   /* Only initialize until clear color (if present). */<br>
> +   const unsigned aux_size = mt->mcs_buf->clear_color_<wbr>offset ?<br>
> +                                mt->mcs_buf->clear_color_<wbr>offset :<br>
> +                                mt->mcs_buf->size;<br>
> +   memset(data, init_value, aux_size);<br></div></div></blockquote><div><br></div><div>Why not just use mt->mcs_buf->aux_surf.size?<br></div><div><br></div><div>Also, I think we probably want to memset the clear color to 0 in case we get a recycled BO with unknown garbage in the clear value.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
</div></div>Hmm... that's a good catch, and I think we definitely should not<br>
overwrite the clear color here.<br>
<br>
However, the initial value of the clear color shouldn't matter, right? I<br>
think there might still be a bug hidden somewhere...<br>
<br>
Regardless of that, this patch is<br>
<br>
Reviewed-by: Rafael Antognolli <<a href="mailto:rafael.antognolli@intel.com">rafael.antognolli@intel.com</a>><br>
<div class="HOEnZb"><div class="h5"><br>
>     brw_bo_unmap(mt->mcs_buf->bo);<br>
>  }<br>
><br>
> --<br>
> 2.7.4<br>
><br>
</div></div></blockquote></div><br></div></div>