[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