[Mesa-dev] [PATCH] egl/dri2: dri2_make_current: Set EGL error if bindContext fails

[Nicolas Boichat]

Gentle ping, any concern about the patch below?

On Tue, Jun 7, 2016 at 6:14 PM, Nicolas Boichat <drinkcat at chromium.org> wrote:
> Without this, if a configuration is, say, available only on GLES2/3, but
> not on GLES1, eglMakeCurrent fails with error "EGL_SUCCESS".
>
> In this patch, we set error to EGL_BAD_MATCH, which is what CTS/dEQP
> dEQP-EGL.functional.surfaceless_context expect.
>
> Signed-off-by: Nicolas Boichat <drinkcat at google.com>
> ---
>  src/egl/drivers/dri2/egl_dri2.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c
> index bfde640..1a38421 100644
> --- a/src/egl/drivers/dri2/egl_dri2.c
> +++ b/src/egl/drivers/dri2/egl_dri2.c
> @@ -1231,6 +1231,7 @@ dri2_make_current(_EGLDriver *drv, _EGLDisplay *disp, _EGLSurface *dsurf,
>        _eglPutSurface(old_rsurf);
>        _eglPutContext(old_ctx);
>
> +      _eglError(EGL_BAD_MATCH, "bindContext error");
>        return EGL_FALSE;
>     }
>  }
> --
> 2.8.0.rc3.226.g39d4020
>