<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Aug 11, 2016 at 12:55 PM, Nanley Chery <span dir="ltr"><<a href="mailto:nanleychery@gmail.com" target="_blank">nanleychery@gmail.com</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 Tue, Jul 26, 2016 at 10:04:21PM -0700, Jason Ekstrand wrote:<br>
> The real objective of this series is patch 5 which prevents us from<br>
> accidentally creating a surface state with a format unsupported by the<br>
> hardware.  This turns some of the new Vulkan CTS tests from a hang into an<br>
> informative crash.  In order to get there, however, we needed to update the<br>
> format table in isl with some of the new formats added on Haswell and later<br>
> generations.  In order to do that, we had to fix up the dri driver, and own<br>
> the rabbit hole we go!<br>
><br>
<br>
</span>Contrary to your introduction, I cannot see how patches 4 and 5 are dependent<br>
on patches 1-3. Could you please explain this further?<br></blockquote><div><br></div><div>4 doesn't depend on 1-3 because it's only render and 1-3 are only for sampling.  I'm not sure about 5.  It may not depend on 2-3 because if those formats aren't returning sampling support, applications shouldn't be creating surfaces with them.  I'm not 100% sure though. :/<br><br></div><div>--Jason<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
- Nanley<br>
<span class=""><br>
> At the end of the series, the hangs in the latest CTS are gone (they came<br>
> from trying to clear an unsupported image format).<br>
><br>
> Jason Ekstrand (5):<br>
>   i965/surface_formats: Don't advertise 8 or 16-bit RGB formats<br>
>   isl/formats: Report ETC as being samplable on Bay Trail<br>
>   isl/formats: Update the table with more samplable formats<br>
>   anv/image: Don't create invalid render target surfaces<br>
>   isl/state: Add some asserts about format capabilities<br>
><br>
>  src/intel/isl/isl_format.c                      | 48 +++++++++++++++++--------<br>
>  src/intel/isl/isl_surface_<wbr>state.c               |  5 +++<br>
>  src/intel/vulkan/anv_image.c                    | 13 ++++++-<br>
>  src/mesa/drivers/dri/i965/brw_<wbr>surface_formats.c | 10 ++++++<br>
>  4 files changed, 60 insertions(+), 16 deletions(-)<br>
><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>