[PATCH] tests/gem_ctx_param_basic: fix up non-root-set-no-zeromap subtest

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This subtest is trying to set the no-zeromap flag on the context without
root privs.  Rather than expecting an EPERM on what's presumably a
nonzero value, we should expect success on a set call w/o root privs.
This looks like a copy & paste error from when the subtest was added,
since setting the ban period has different expected behavior.

Cc: David Weinehall <david.weinehall@xxxxxxxxxxxxxxx>
Signed-off-by: Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx>
---
 tests/gem_ctx_param_basic.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/gem_ctx_param_basic.c b/tests/gem_ctx_param_basic.c
index 6a1694d..f7d9592 100644
--- a/tests/gem_ctx_param_basic.c
+++ b/tests/gem_ctx_param_basic.c
@@ -126,8 +126,8 @@ 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);
+			ctx_param.value = 0;
+			TEST_SUCCESS(LOCAL_IOCTL_I915_GEM_CONTEXT_SETPARAM);
 		}
 
 		igt_waitchildren();
-- 
1.9.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux