gem_write_read_ring_switch: Copy the missing igt_require
authorChris Wilson <chris@chris-wilson.co.uk>
Wed, 14 Aug 2013 07:17:22 +0000 (08:17 +0100)
committerChris Wilson <chris@chris-wilson.co.uk>
Wed, 14 Aug 2013 07:17:22 +0000 (08:17 +0100)
An oversight in the mass conversion to the new framework as the test was
called from two locations.

Ideally, the checks could be moved back to the caller and the framework
still work. This is just the patch of least resistence.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=68091

tests/gem_write_read_ring_switch.c

index b9db8f2..9da393d 100644 (file)
@@ -204,8 +204,13 @@ int main(int argc, char **argv)
        for (i = 0; i < ARRAY_SIZE(tests); i++) {
                char name[180];
                snprintf(name, sizeof(name), "%s-interruptible", tests[i].name);
-               igt_subtest(name)
+               igt_subtest(name) {
+                       gem_require_ring(fd, tests[i].ring);
+                       /* Testing render only makes sense with separate blt. */
+                       if (tests[i].ring == I915_EXEC_RENDER)
+                               gem_require_ring(fd, I915_EXEC_BLT);
                        run_test(tests[i].ring, name);
+               }
        }
        igt_stop_signal_helper();