<p dir="ltr"></p>
<p dir="ltr">On Sep 7, 2016 12:27 PM, "Nanley Chery" <<a href="mailto:nanleychery@gmail.com">nanleychery@gmail.com</a>> wrote:<br>
><br>
> On Fri, Sep 02, 2016 at 03:50:41PM -0700, Jason Ekstrand wrote:<br>
> > This series does a few cleanups to ISL so that it can actually handle<br>
> > multisampled HiZ correctly.  When Chad and I first discussed HiZ, we<br>
> > thought it would all work out easy as ISL already had all the required<br>
> > code.  We were correct that it had all the required code but wrong about<br>
> > the easy part.  It turns out that there were some troublesome asserts and<br>
> > the choose_tiling and choose_msaa_layout paths weren't 100% HiZ friendly at<br>
> > least not when mutlisampling gets involved.<br>
> ><br>
> > This little series cleans things up a bit and gets multisampled HiZ<br>
> > working.  In the end, I'm very happy with these changes as I think they're<br>
> > almost all good changes even without HiZ.<br>
> ><br>
> > This series can replace patch 02/12 of Nanley's recent HiZ series.<br>
><br>
> Doesn't this also need to add support for 3D textures in<br>
> isl_surf_get_hiz_surf() to replace that patch?</p>
<p dir="ltr">Yes, we will need a patch that fixes up the asserts.  Surprisingly, this series plus a revert of your "Update get_hiz_surf" patch passes the CTS...</p>
<p dir="ltr">> ><br>
> > Jason Ekstrand (6):<br>
> >   intel/isl: Add a format_supports_multisampling helper<br>
> >   intel/isl: Fix up asserts in calc_phys_level0_extent_sa<br>
> >   intel/isl: Allow multisampling with ISL_FORMAT_HiZ<br>
> >   intel/isl: Handle HiZ and CCS tiling more directly<br>
> >   intel/isl: Remove tiling checks from choose_msaa_layout<br>
> >   intel/isl: Add a detailed comment about multisampling with HiZ<br>
> ><br>
> >  src/intel/isl/isl.c        | 83 ++++++++++++++++++++++++++++++++++++++++++----<br>
> >  src/intel/isl/isl.h        |  2 ++<br>
> >  src/intel/isl/isl_format.c | 35 +++++++++++++++++++<br>
> >  src/intel/isl/isl_gen6.c   | 19 +----------<br>
> >  src/intel/isl/isl_gen7.c   | 40 +++++-----------------<br>
> >  src/intel/isl/isl_gen8.c   | 15 +--------<br>
> >  6 files changed, 123 insertions(+), 71 deletions(-)<br>
> ><br>
> > --<br>
> > 2.5.0.400.gff86faf<br>
> ><br>
> > _______________________________________________<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">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br></p>