tests/gem_exec_params: One more invalid ring tests
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Mon, 28 Apr 2014 13:26:15 +0000 (15:26 +0200)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Mon, 28 Apr 2014 13:26:15 +0000 (15:26 +0200)
With the vebox 2 patches the number of internal rings don't match the
number of exposed rings. So add another subtest with an invalid ring
which should be invalid both internally and externally. The bug this
will catch is using the ring structure before validation, which the
old "invalide-ring" wont be able to due to the internal vebox2 ring.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
tests/gem_exec_params.c

index 306039c..769969d 100644 (file)
@@ -117,6 +117,11 @@ igt_main
                RUN_FAIL(EINVAL);
        }
        igt_subtest("invalid-ring") {
+               execbuf.flags = I915_EXEC_RING_MASK;
+               RUN_FAIL(EINVAL);
+       }
+
+       igt_subtest("invalid-ring2") {
                execbuf.flags = LOCAL_I915_EXEC_VEBOX+1;
                RUN_FAIL(EINVAL);
        }