[Piglit] [PATCH] egl_ext_device_drm: don't fail the test if open("/dev/dri/cardX") errors

Emil Velikov emil.l.velikov at gmail.com
Wed Oct 3 12:21:56 UTC 2018


From: Emil Velikov <emil.velikov at collabora.com>

As pointed out by Mathias opening the card node can fail in some cases.
In boils down to a) the node is owned by root:video b) by default, the
user is not part of the video group c) logind dynamically allows any
user to open the node, once a user has logged-in.

Thus if we use a remote machine accessible only over ssh, the test will
fail. A fairly common setup that one could use for their CI.

Demote the failure to a warning.

Cc: Mathias Fröhlich <mathias.froehlich at web.de>
Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
---
 tests/egl/spec/egl_ext_device_drm/egl_ext_device_drm.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/tests/egl/spec/egl_ext_device_drm/egl_ext_device_drm.c b/tests/egl/spec/egl_ext_device_drm/egl_ext_device_drm.c
index 537c8b60f..2236783c8 100644
--- a/tests/egl/spec/egl_ext_device_drm/egl_ext_device_drm.c
+++ b/tests/egl/spec/egl_ext_device_drm/egl_ext_device_drm.c
@@ -162,7 +162,9 @@ main(void)
 		if (fd < 0) {
 			printf("Failed to open drm device file %s: %s\n",
 				devstring, strerror(errno));
-			piglit_report_result(PIGLIT_FAIL);
+			printf("Make sure you have permissions to open %s\n");
+			result = PIGLIT_WARN;
+			continue;
 		}
 #ifndef EGL_DRM_MASTER_FD_EXT
 #define EGL_DRM_MASTER_FD_EXT                   0x333C
-- 
2.19.0



More information about the Piglit mailing list