Reviewed-by: Marek Olšák <[email protected]> Marek
On Fri, Sep 30, 2016 at 8:32 AM, Tapani Pälli <[email protected]> wrote: > This fixes a crash in egl-create-msaa-pbuffer-surface Piglit test > and same crash in many dEQP EGL tests. > > I also found that some Qt example did a workaround because of this > crash: https://bugreports.qt.io/browse/QTBUG-47509 > > v2: Ian pointed out that v1 removed support for all multisample > configs, including window ones. This one removes pbuffer bit > when adding configs, now only pbuffer+msaa gets rejected and > window+msaa continues to work. Fixed also comment (Emil) > > Signed-off-by: Tapani Pälli <[email protected]> > Cc: [email protected] > --- > src/egl/drivers/dri2/egl_dri2.c | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c > index 8e376e3..803627d 100644 > --- a/src/egl/drivers/dri2/egl_dri2.c > +++ b/src/egl/drivers/dri2/egl_dri2.c > @@ -320,6 +320,15 @@ dri2_add_config(_EGLDisplay *disp, const __DRIconfig > *dri_config, int id, > surface_type &= ~EGL_PIXMAP_BIT; > } > > + /* No support for pbuffer + MSAA for now. > + * > + * XXX TODO: pbuffer + MSAA does not work and causes crashes. > + * See QT bugreport: https://bugreports.qt.io/browse/QTBUG-47509 > + */ > + if (base.Samples) { > + surface_type &= ~EGL_PBUFFER_BIT; > + } > + > conf->base.SurfaceType |= surface_type; > > return conf; > -- > 2.7.4 > > _______________________________________________ > mesa-dev mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
