From 8c361e84ad010552a42593fad4130befc58e9a6a Mon Sep 17 00:00:00 2001 From: =?utf8?q?Marek=20Ol=C5=A1=C3=A1k?= Date: Fri, 3 Jun 2016 11:25:19 +0200 Subject: [PATCH] Revert "egl: Check if API is supported when using eglBindAPI." This reverts commit e8b38ca202fbe8c281aeb81a4b64256983f185e0. It broke Glamor for Gallium at least. --- src/egl/main/eglapi.c | 65 --------------------------------------- src/egl/main/eglcurrent.h | 11 ++++++- src/egl/main/egldisplay.c | 5 --- src/egl/main/egldisplay.h | 1 - 4 files changed, 10 insertions(+), 72 deletions(-) diff --git a/src/egl/main/eglapi.c b/src/egl/main/eglapi.c index 4ef0a2759b5..4700dbe42f6 100644 --- a/src/egl/main/eglapi.c +++ b/src/egl/main/eglapi.c @@ -1196,61 +1196,6 @@ eglGetError(void) } -static bool -_eglDisplaySupportsApi(_EGLDisplay *dpy, EGLenum api) -{ - if (!dpy->Initialized) { - return false; - } - - switch (api) { - case EGL_OPENGL_API: - return !!(dpy->ClientAPIs & EGL_OPENGL_BIT); - case EGL_OPENGL_ES_API: - return dpy->ClientAPIs & EGL_OPENGL_ES_BIT || - dpy->ClientAPIs & EGL_OPENGL_ES2_BIT || - dpy->ClientAPIs & EGL_OPENGL_ES3_BIT_KHR; - case EGL_OPENVG_API: - return !!(dpy->ClientAPIs & EGL_OPENVG_BIT); - } - - return false; -} - - -/** - * Return true if a client API enum is recognized. - */ -static bool -_eglIsApiValid(EGLenum api) -{ - _EGLDisplay *dpy = _eglGlobal.DisplayList; - _EGLThreadInfo *current_thread = _eglGetCurrentThread(); - - if (api != EGL_OPENGL_API && api != EGL_OPENGL_ES_API && - api != EGL_OPENVG_API) { - return false; - } - - while (dpy) { - _EGLThreadInfo *thread = dpy->ThreadList; - - while (thread) { - if (thread == current_thread) { - if (_eglDisplaySupportsApi(dpy, api)) - return true; - } - - thread = thread->Next; - } - - dpy = dpy->Next; - } - - return false; -} - - /** ** EGL 1.2 **/ @@ -1266,16 +1211,6 @@ _eglIsApiValid(EGLenum api) * eglWaitNative() * See section 3.7 "Rendering Context" in the EGL specification for details. */ - - /** - * Section 3.7 (Rendering Contexts) of the EGL 1.5 spec says: - * - * "api must specify one of the supported client APIs, either - * EGL_OPENGL_API, EGL_OPENGL_ES_API, or EGL_OPENVG_API... If api - * is not one of the values specified above, or if the client API - * specified by api is not supported by the implementation, an - * EGL_BAD_PARAMETER error is generated." - */ EGLBoolean EGLAPIENTRY eglBindAPI(EGLenum api) { diff --git a/src/egl/main/eglcurrent.h b/src/egl/main/eglcurrent.h index 6c203be0729..1e386acdafb 100644 --- a/src/egl/main/eglcurrent.h +++ b/src/egl/main/eglcurrent.h @@ -56,7 +56,6 @@ extern "C" { */ struct _egl_thread_info { - _EGLThreadInfo *Next; /* used to link threads */ EGLint LastError; _EGLContext *CurrentContexts[_EGL_API_NUM_APIS]; /* use index for fast access to current context */ @@ -64,6 +63,16 @@ struct _egl_thread_info }; +/** + * Return true if a client API enum is recognized. + */ +static inline EGLBoolean +_eglIsApiValid(EGLenum api) +{ + return (api >= _EGL_API_FIRST_API && api <= _EGL_API_LAST_API); +} + + /** * Convert a client API enum to an index, for use by thread info. * The client API enum is assumed to be valid. diff --git a/src/egl/main/egldisplay.c b/src/egl/main/egldisplay.c index 907a60747d0..f6db03ab50c 100644 --- a/src/egl/main/egldisplay.c +++ b/src/egl/main/egldisplay.c @@ -240,7 +240,6 @@ _EGLDisplay * _eglFindDisplay(_EGLPlatformType plat, void *plat_dpy) { _EGLDisplay *dpy; - _EGLThreadInfo *thread = _eglGetCurrentThread(); if (plat == _EGL_INVALID_PLATFORM) return NULL; @@ -266,13 +265,9 @@ _eglFindDisplay(_EGLPlatformType plat, void *plat_dpy) /* add to the display list */ dpy->Next = _eglGlobal.DisplayList; _eglGlobal.DisplayList = dpy; - dpy->ThreadList = NULL; } } - thread->Next = dpy->ThreadList; - dpy->ThreadList = thread; - mtx_unlock(_eglGlobal.Mutex); return dpy; diff --git a/src/egl/main/egldisplay.h b/src/egl/main/egldisplay.h index 8a730ed2d9c..6bfc8589a42 100644 --- a/src/egl/main/egldisplay.h +++ b/src/egl/main/egldisplay.h @@ -140,7 +140,6 @@ struct _egl_display _EGLPlatformType Platform; /**< The type of the platform display */ void *PlatformDisplay; /**< A pointer to the platform display */ - _EGLThreadInfo *ThreadList;/**< A pointer to the thread the display was created form */ _EGLDriver *Driver; /**< Matched driver of the display */ EGLBoolean Initialized; /**< True if the display is initialized */ -- 2.30.2