[igt-dev] [PATCH i-g-t v2] Follow kernel's resource streamer removal

Lucas De Marchi lucas.demarchi at intel.com
Thu Jul 19 19:16:14 UTC 2018


Resource streamer is being removed from all GENs, so just test it
returns EINVAL for all GENs when a batch is submitted with
I915_EXEC_RESOURCE_STREAMER keeping one test per ring.

v2: let one test per ring rather than just one test

Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
---

This requires the patch for the kernel removing RS from all gens.

 tests/gem_exec_params.c | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/tests/gem_exec_params.c b/tests/gem_exec_params.c
index 04c21c05..f822297f 100644
--- a/tests/gem_exec_params.c
+++ b/tests/gem_exec_params.c
@@ -375,25 +375,21 @@ igt_main
 	}
 
 	igt_subtest("rs-invalid-on-bsd-ring") {
-		igt_require(IS_HASWELL(devid) || intel_gen(devid) >= 8);
 		execbuf.flags = I915_EXEC_BSD | LOCAL_I915_EXEC_RESOURCE_STREAMER;
 		RUN_FAIL(EINVAL);
 	}
 
 	igt_subtest("rs-invalid-on-blt-ring") {
-		igt_require(IS_HASWELL(devid) || intel_gen(devid) >= 8);
 		execbuf.flags = I915_EXEC_BLT | LOCAL_I915_EXEC_RESOURCE_STREAMER;
 		RUN_FAIL(EINVAL);
 	}
 
 	igt_subtest("rs-invalid-on-vebox-ring") {
-		igt_require(IS_HASWELL(devid) || intel_gen(devid) >= 8);
 		execbuf.flags = I915_EXEC_VEBOX | LOCAL_I915_EXEC_RESOURCE_STREAMER;
 		RUN_FAIL(EINVAL);
 	}
 
-	igt_subtest("rs-invalid-gen") {
-		igt_require(!IS_HASWELL(devid) && intel_gen(devid) < 8);
+	igt_subtest("rs-invalid-on-render-ring") {
 		execbuf.flags = I915_EXEC_RENDER | LOCAL_I915_EXEC_RESOURCE_STREAMER;
 		RUN_FAIL(EINVAL);
 	}
-- 
2.17.1



More information about the igt-dev mailing list