In test egl_khr_create_context/default-minor-version-gl, context creation
is not required to succeed. When it fails, the test skips.

However, for tests default-{major,minor}-version-gles, context creation
*is*  required to succeed. This patch adds comments explaining why,

CC: Matt Turner <[email protected]>
Signed-off-by: Chad Versace <[email protected]>
---
 tests/egl/spec/egl_khr_create_context/default-major-version-gles.c | 3 +++
 tests/egl/spec/egl_khr_create_context/default-minor-version-gles.c | 4 ++++
 2 files changed, 7 insertions(+)

diff --git a/tests/egl/spec/egl_khr_create_context/default-major-version-gles.c 
b/tests/egl/spec/egl_khr_create_context/default-major-version-gles.c
index cf61e01..e7da845 100644
--- a/tests/egl/spec/egl_khr_create_context/default-major-version-gles.c
+++ b/tests/egl/spec/egl_khr_create_context/default-major-version-gles.c
@@ -45,6 +45,9 @@ int main(int argc, char **argv)
         * Request an OpenGL ES 1.x context by explicitly setting the minor
         * version to 0 and leaving the major version at the default value of
         * 1.
+        *
+        * The EGL_OPENGL_ES_BIT is set in EGLConfig's EGL_RENDERABLE_TYPE,
+        * so the driver is required to succeed in creating an ES1 context.
         */
        ctx = eglCreateContext(egl_dpy, cfg, EGL_NO_CONTEXT, attribs);
        if (ctx == EGL_NO_CONTEXT) {
diff --git a/tests/egl/spec/egl_khr_create_context/default-minor-version-gles.c 
b/tests/egl/spec/egl_khr_create_context/default-minor-version-gles.c
index 2331832..5fbb8d4 100644
--- a/tests/egl/spec/egl_khr_create_context/default-minor-version-gles.c
+++ b/tests/egl/spec/egl_khr_create_context/default-minor-version-gles.c
@@ -49,6 +49,10 @@ int main(int argc, char **argv)
         * Request an OpenGL ES 1.x or 2.0 context by explicitly setting the
         * major version and leaving the minor version at the default value of
         * 0.
+        *
+        * The EGLConfig's EGL_RENDERABLE_TYPE and the attribute list's
+        * EGL_CONTEXT_MAJOR_VERSION_KHR have been chosen so that the driver
+        * is required to succeed at context creation.
         */
        ctx = eglCreateContext(egl_dpy, cfg, EGL_NO_CONTEXT, attribs);
        if (ctx == EGL_NO_CONTEXT) {
-- 
1.8.1.1

_______________________________________________
Piglit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to