<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Mar 16, 2017 at 4:17 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 Wed 15 Mar 2017, Jason Ekstrand wrote:<br>
> On Mon, Mar 13, 2017 at 3:28 PM, Chad Versace <<a href="mailto:chadversary@chromium.org">chadversary@chromium.org</a>><br>
> wrote:<br>
><br>
> > ---<br>
> >  src/intel/isl/isl.h | 9 +++++++++<br>
> >  1 file changed, 9 insertions(+)<br>
> ><br>
> > diff --git a/src/intel/isl/isl.h b/src/intel/isl/isl.h<br>
> > index 9d92906ca71..b79793b0c93 100644<br>
> > --- a/src/intel/isl/isl.h<br>
> > +++ b/src/intel/isl/isl.h<br>
> > @@ -473,6 +473,9 @@ typedef uint32_t isl_tiling_flags_t;<br>
> >  /** The Skylake BSpec refers to Yf and Ys as "standard tiling formats". */<br>
> >  #define ISL_TILING_STD_Y_MASK             (ISL_TILING_Yf_BIT | \<br>
> >                                             ISL_TILING_Ys_BIT)<br>
> > +<br>
> > +#define ISL_TILING_AUX_MASK               (ISL_TILING_HIZ_BIT | \<br>
> > +                                           ISL_TILING_CCS_BIT)<br>
> ><br>
><br>
> What about MCS?<br>
<br>
</span>Right. This is bad code.<br>
<br>
How about I test against ISL_SURF_USAGE_{HIZ,MCS,CCS_D,<wbr>CCS_E} instead?<br>
In the next patch, where it matters?<br>
</blockquote></div><br></div><div class="gmail_extra">Sure.  Either works so long as you  include all the AUX bits.<br><br></div><div class="gmail_extra">--Jason<br></div></div>