[Mesa-dev] [PATCH] egl/gbm: Fix config validation

Eric Engestrom eric at engestrom.ch
Tue Oct 1 15:00:51 UTC 2019


On Tuesday, 2019-10-01 14:01:09 +0100, Robin Murphy wrote:
> [ + some CCs that I probably should have figured out first time around... ]
> 
> On 23/09/2019 16:40, Adam Jackson wrote:
> > On Sat, 2019-09-21 at 18:07 +0100, Robin Murphy wrote:
> > > In converting to shift/size-based validation, we lost a condition from
> > > the ARGB/XRGB equivalence check, which left it working one way round
> > > but not the other, and broke applications like glmark2-es2-drm on some
> > > platforms. Restore the equivalent check that *both* configs actually
> > > have an alpha channel before considering a mismatch.
> > > 
> > > Fixes: 7b4ed2b513ef ("egl: Convert configs to use shifts and sizes instead of masks")
> > > Signed-off-by: Robin Murphy <robin.murphy at arm.com>
> > 
> > Makes sense to me.
> > 
> > Reviewed-by: Adam Jackson <ajax at redhat.com>
> 
> Thanks - assuming nobody else disagrees, is there a maintainer who could
> pick this up, or would I need to go off and do Gitlab shenanigans myself?

Just pushed it, with my review-by; thanks :)

> 
> Cheers,
> Robin.


More information about the mesa-dev mailing list