<div dir="auto"><div><br><div class="gmail_extra"><br><div class="gmail_quote">On Mar 23, 2017 11:00 PM, "Samuel Iglesias Gonsálvez" <<a href="mailto:siglesias@igalia.com" target="_blank">siglesias@igalia.com</a>> wrote:<br type="attribution"><blockquote class="m_7659869580131551747quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="m_7659869580131551747elided-text">On Thu, 2017-03-23 at 11:09 -0700, Nanley Chery wrote:<br>
> On Thu, Mar 23, 2017 at 12:25:28PM +0100, Samuel Iglesias Gonsálvez<br>
> wrote:<br>
> > A resolve is not needed on Skylake in this case. We were forcing<br>
> > a resolve because we set the input_aux_usage to ISL_AUX_USAGE_NONE.<br>
> ><br>
> > Signed-off-by: Samuel Iglesias Gonsálvez <<a href="mailto:siglesias@igalia.com" target="_blank">siglesias@igalia.com</a>><br>
> > ---<br>
> ><br>
> > This doesn't fix the problem with BDW but I found it while<br>
> > reviewing<br>
> > the code.<br>
> ><br>
> >  src/intel/vulkan/genX_cmd_buf<wbr>fer.c | 4 ++--<br>
> >  1 file changed, 2 insertions(+), 2 deletions(-)<br>
> ><br>
> > diff --git a/src/intel/vulkan/genX_cmd_bu<wbr>ffer.c<br>
> > b/src/intel/vulkan/genX_cmd_bu<wbr>ffer.c<br>
> > index e2364dbfd52..39856b9af7c 100644<br>
> > --- a/src/intel/vulkan/genX_cmd_bu<wbr>ffer.c<br>
> > +++ b/src/intel/vulkan/genX_cmd_bu<wbr>ffer.c<br>
> > @@ -305,8 +305,8 @@ color_attachment_compute_aux_u<wbr>sage(struct<br>
> > anv_device *device,<br>
> >            * doesn't also support color compression.<br>
> >            */<br>
> >           att_state->input_aux<wbr>_usage = ISL_AUX_USAGE_NONE;<br>
> > -      } else if (GEN_GEN == 8) {<br>
> > -         /* Broadwell can sample from fast-cleared images */<br>
> > +      } else if (GEN_GEN >= 8) {<br>
> > +         /* Broadwell/Skylake can sample from fast-cleared images<br>
> > */<br>
> >           att_state->input_aux<wbr>_usage = ISL_AUX_USAGE_CCS_D;<br>
><br>
> This doesn't work in all cases. SKL can only sample from CCS_D if the<br>
> format is supported for CCS_E.<br>
><br>
<br>
</div>It is already covered. The 'if' condition before this is triggered only<br>
if gen9+ and format doesn't support CCS_E, so if you end up here in<br>
gen9+ is because the format supports CCS_E.<br>
<div class="m_7659869580131551747quoted-text"></div></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">You're right. Sorry about that. Let's go with your​ patch. </div><div dir="auto">Reviewed-by: Nanley Chery <<a href="mailto:nanley.g.chery@intel.com" target="_blank">nanley.g.chery@intel.com</a>></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="m_7659869580131551747quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="m_7659869580131551747quoted-text"><br>
> By the way, I actually have a work-in-progress branch that has a<br>
> patch to do this:<br>
> <a href="https://cgit.freedesktop.org/~nchery/mesa/commit/?h=ccs-layouts/1-ccs" rel="noreferrer" target="_blank">https://cgit.freedesktop.org/~<wbr>nchery/mesa/commit/?h=ccs-layo<wbr>uts/1-ccs</a><br>
> d-layout&id=e428f44e1835cb00f5<wbr>2848d09350957ef6f73495<br>
><br>
<br>
</div>This is a similar solution to this bug. If you prefer to stick with<br>
yours, please add my R-b to it:<br>
<br>
Reviewed-by: Samuel Iglesias Gonsálvez <<a href="mailto:siglesias@igalia.com" target="_blank">siglesias@igalia.com</a>><br>
<br>
Sam</blockquote></div><br></div></div></div>