<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Jan 27, 2017 at 2:31 PM, Matt Turner <span dir="ltr"><<a href="mailto:mattst88@gmail.com" target="_blank">mattst88@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 Fri, Jan 27, 2017 at 2:18 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> This causes hangs on Broadwell if you try to render to it.  I have no<br>
> idea how we managed to not hit this earlier.<br>
><br>
> Cc: "13.0 17.0" <<a href="mailto:mesa-stable@lists.freedesktop.org">mesa-stable@lists.<wbr>freedesktop.org</a>><br>
> ---<br>
>  src/intel/isl/isl_format.c | 5 +++--<br>
>  1 file changed, 3 insertions(+), 2 deletions(-)<br>
><br>
> diff --git a/src/intel/isl/isl_format.c b/src/intel/isl/isl_format.c<br>
> index c8daece..bc157d5 100644<br>
> --- a/src/intel/isl/isl_format.c<br>
> +++ b/src/intel/isl/isl_format.c<br>
> @@ -218,9 +218,10 @@ static const struct surface_format_info format_info[] = {<br>
>     SF(50, 50,  x,  x,  x,  x,  x,  x,  x,    x,   P8A8_UNORM_PALETTE1)<br>
>     SF( x,  x,  x,  x,  x,  x,  x,  x,  x,    x,   A1B5G5R5_UNORM)<br>
>     /* According to the PRM, A4B4G4R4_UNORM isn't supported until Sky Lake<br>
> -    * but empirical testing indicates that it works just fine on Broadwell.<br>
> +    * but empirical testing indicates that at least sampling works just fine<br>
> +    * on Broadwell.<br>
<br>
</span>How long until we determine that sampling is also somehow broken?<br></blockquote><div><br></div><div>Good question.  There are hundreds of Vulkan CTS tests that say it's working just fine.  The difference is that it hasn't been until recently that tests have been added which test it for rendering.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Maybe we should just believe the documentation?</blockquote><div><br></div><div>We really need this to work for conformance on Broadwell. <br></div></div></div></div>