Re: [Mesa-dev] [PATCH 9/9] panfrost: Enable sRGB

2019-06-19 Thread Alyssa Rosenzweig
> Actually, I think this is the commit that introduced these regressions:
> 
> panfrost: Figure out job requirements in pan_job.c

...Hmm.


signature.asc
Description: PGP signature
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH 9/9] panfrost: Enable sRGB

2019-06-19 Thread Tomeu Vizoso
On Wed, 19 Jun 2019 at 07:44, Tomeu Vizoso  wrote:
>
> On Tue, 18 Jun 2019 at 17:00, Alyssa Rosenzweig
>  wrote:
> >
> > Now that sRGB formats are supported for both rendering and sampling,
> > advertise support.
> >
> > Signed-off-by: Alyssa Rosenzweig 
>
> Hi there,
>
> this patch seems to have caused the following regressions:
>
> dEQP-GLES2.functional.fbo.render.recreate_colorbuffer.no_rebind_rbo_rgb565
> dEQP-GLES2.functional.fbo.render.resize.tex2d_rgb
>
> For more details, see 
> https://gitlab.freedesktop.org/tomeu/mesa/pipelines/43396
>
> Should we revert it for now?

Actually, I think this is the commit that introduced these regressions:

panfrost: Figure out job requirements in pan_job.c

Cheers,

Tomeu

> Cheers,
>
> Tomeu
>
> > ---
> >  src/gallium/drivers/panfrost/pan_screen.c | 4 
> >  1 file changed, 4 deletions(-)
> >
> > diff --git a/src/gallium/drivers/panfrost/pan_screen.c 
> > b/src/gallium/drivers/panfrost/pan_screen.c
> > index 9cd98cd9051..70bff565930 100644
> > --- a/src/gallium/drivers/panfrost/pan_screen.c
> > +++ b/src/gallium/drivers/panfrost/pan_screen.c
> > @@ -450,10 +450,6 @@ panfrost_is_format_supported( struct pipe_screen 
> > *screen,
> >  if (sample_count > 1)
> >  return FALSE;
> >
> > -   /* sRGB colorspace is not supported (yet?) */
> > -   if (format_desc->colorspace == UTIL_FORMAT_COLORSPACE_SRGB)
> > -   return FALSE;
> > -
> >  /* Format wishlist */
> >  if (format == PIPE_FORMAT_Z24X8_UNORM || format == 
> > PIPE_FORMAT_X8Z24_UNORM)
> >  return FALSE;
> > --
> > 2.20.1
> >
> > ___
> > mesa-dev mailing list
> > mesa-dev@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/mesa-dev
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH 9/9] panfrost: Enable sRGB

2019-06-18 Thread Tomeu Vizoso
On Tue, 18 Jun 2019 at 17:00, Alyssa Rosenzweig
 wrote:
>
> Now that sRGB formats are supported for both rendering and sampling,
> advertise support.
>
> Signed-off-by: Alyssa Rosenzweig 

Hi there,

this patch seems to have caused the following regressions:

dEQP-GLES2.functional.fbo.render.recreate_colorbuffer.no_rebind_rbo_rgb565
dEQP-GLES2.functional.fbo.render.resize.tex2d_rgb

For more details, see https://gitlab.freedesktop.org/tomeu/mesa/pipelines/43396

Should we revert it for now?

Cheers,

Tomeu

> ---
>  src/gallium/drivers/panfrost/pan_screen.c | 4 
>  1 file changed, 4 deletions(-)
>
> diff --git a/src/gallium/drivers/panfrost/pan_screen.c 
> b/src/gallium/drivers/panfrost/pan_screen.c
> index 9cd98cd9051..70bff565930 100644
> --- a/src/gallium/drivers/panfrost/pan_screen.c
> +++ b/src/gallium/drivers/panfrost/pan_screen.c
> @@ -450,10 +450,6 @@ panfrost_is_format_supported( struct pipe_screen *screen,
>  if (sample_count > 1)
>  return FALSE;
>
> -   /* sRGB colorspace is not supported (yet?) */
> -   if (format_desc->colorspace == UTIL_FORMAT_COLORSPACE_SRGB)
> -   return FALSE;
> -
>  /* Format wishlist */
>  if (format == PIPE_FORMAT_Z24X8_UNORM || format == 
> PIPE_FORMAT_X8Z24_UNORM)
>  return FALSE;
> --
> 2.20.1
>
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH 9/9] panfrost: Enable sRGB

2019-06-18 Thread Alyssa Rosenzweig
Now that sRGB formats are supported for both rendering and sampling,
advertise support.

Signed-off-by: Alyssa Rosenzweig 
---
 src/gallium/drivers/panfrost/pan_screen.c | 4 
 1 file changed, 4 deletions(-)

diff --git a/src/gallium/drivers/panfrost/pan_screen.c 
b/src/gallium/drivers/panfrost/pan_screen.c
index 9cd98cd9051..70bff565930 100644
--- a/src/gallium/drivers/panfrost/pan_screen.c
+++ b/src/gallium/drivers/panfrost/pan_screen.c
@@ -450,10 +450,6 @@ panfrost_is_format_supported( struct pipe_screen *screen,
 if (sample_count > 1)
 return FALSE;
 
-   /* sRGB colorspace is not supported (yet?) */
-   if (format_desc->colorspace == UTIL_FORMAT_COLORSPACE_SRGB)
-   return FALSE;
-
 /* Format wishlist */
 if (format == PIPE_FORMAT_Z24X8_UNORM || format == 
PIPE_FORMAT_X8Z24_UNORM)
 return FALSE;
-- 
2.20.1

___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev