From a16704261fc12259a2e61c00c3cef7be568d71f9 Mon Sep 17 00:00:00 2001 From: Daniele Ceraolo Spurio Date: Wed, 9 Sep 2015 09:46:56 +0100 Subject: [PATCH] tests/gem_ctx_param_basic.c: fix non-root-set-no-zeromap subtest The test expects an ioctl failure when it tries to set CONTEXT_PARAM_NO_ZEROMAP from a non-root process. However, there is no requirement in the kernel for the user to be root to set this parameter, so the test is failing (it never passed as far as I'm aware of). Fix the test by making it expect a successful ioctl completion. Signed-off-by: Daniele Ceraolo Spurio Reviewed-by: Jesse Barnes Signed-off-by: Thomas Wood --- tests/gem_ctx_param_basic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/gem_ctx_param_basic.c b/tests/gem_ctx_param_basic.c index 6a1694d3..45a351b8 100644 --- a/tests/gem_ctx_param_basic.c +++ b/tests/gem_ctx_param_basic.c @@ -127,7 +127,7 @@ igt_main ctx_param.context = ctx; TEST_SUCCESS(LOCAL_IOCTL_I915_GEM_CONTEXT_GETPARAM); ctx_param.value--; - TEST_FAIL(LOCAL_IOCTL_I915_GEM_CONTEXT_SETPARAM, EPERM); + TEST_SUCCESS(LOCAL_IOCTL_I915_GEM_CONTEXT_SETPARAM); } igt_waitchildren();