[Mesa-dev] [PATCH 2/2] isl/formats: Only advertise sampling for A4B4G4R4 on Broadwell

Jason Ekstrand jason at jlekstrand.net
Fri Jan 27 22:51:51 UTC 2017


On Fri, Jan 27, 2017 at 2:31 PM, Matt Turner <mattst88 at gmail.com> wrote:

> On Fri, Jan 27, 2017 at 2:18 PM, Jason Ekstrand <jason at jlekstrand.net>
> wrote:
> > This causes hangs on Broadwell if you try to render to it.  I have no
> > idea how we managed to not hit this earlier.
> >
> > Cc: "13.0 17.0" <mesa-stable at lists.freedesktop.org>
> > ---
> >  src/intel/isl/isl_format.c | 5 +++--
> >  1 file changed, 3 insertions(+), 2 deletions(-)
> >
> > diff --git a/src/intel/isl/isl_format.c b/src/intel/isl/isl_format.c
> > index c8daece..bc157d5 100644
> > --- a/src/intel/isl/isl_format.c
> > +++ b/src/intel/isl/isl_format.c
> > @@ -218,9 +218,10 @@ static const struct surface_format_info
> format_info[] = {
> >     SF(50, 50,  x,  x,  x,  x,  x,  x,  x,    x,   P8A8_UNORM_PALETTE1)
> >     SF( x,  x,  x,  x,  x,  x,  x,  x,  x,    x,   A1B5G5R5_UNORM)
> >     /* According to the PRM, A4B4G4R4_UNORM isn't supported until Sky
> Lake
> > -    * but empirical testing indicates that it works just fine on
> Broadwell.
> > +    * but empirical testing indicates that at least sampling works just
> fine
> > +    * on Broadwell.
>
> How long until we determine that sampling is also somehow broken?
>

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.


> Maybe we should just believe the documentation?


We really need this to work for conformance on Broadwell.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170127/be82cceb/attachment-0001.html>


More information about the mesa-dev mailing list