[Piglit] [PATCH 06/14] egl_khr_fence_sync: fix eglCreateSyncKHR() parameter

Marek Olšák maraeo at gmail.com
Mon Apr 13 11:28:06 PDT 2015


From: Daniel Kurtz <djkurtz at chromium.org>

EGL_SYNC_TYPE_KHR is not a valid fence type; the correct error code for
this is actually EGL_BAD_ATTRIBUTE, not EGL_BAD_MATCH.

However, the test is trying to test the 'no context' condition, which
means it should use a valid fence type: EGL_SYNC_FENCE_KHR

Signed-off-by: Daniel Kurtz <djkurtz at chromium.org>
Reviewed-by: Marek Olšák <marek.olsak at amd.com>
---
 tests/egl/spec/egl_khr_fence_sync/egl_khr_fence_sync.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/egl/spec/egl_khr_fence_sync/egl_khr_fence_sync.c b/tests/egl/spec/egl_khr_fence_sync/egl_khr_fence_sync.c
index 04f8dfe..0f443c6 100644
--- a/tests/egl/spec/egl_khr_fence_sync/egl_khr_fence_sync.c
+++ b/tests/egl/spec/egl_khr_fence_sync/egl_khr_fence_sync.c
@@ -540,7 +540,7 @@ test_eglCreateSyncKHR_no_current_context(void *test_data)
 	}
 	eglMakeCurrent(g_dpy, EGL_NO_SURFACE, EGL_NO_SURFACE, EGL_NO_CONTEXT);
 
-	sync = peglCreateSyncKHR(g_dpy, EGL_SYNC_TYPE_KHR, NULL);
+	sync = peglCreateSyncKHR(g_dpy, EGL_SYNC_FENCE_KHR, NULL);
 	if (sync != EGL_NO_SYNC_KHR) {
 		piglit_loge("eglCreateSyncKHR() succeeded when no context was "
 			  "current");
-- 
2.1.0



More information about the Piglit mailing list