[igt-dev] [PATCH i-g-t] gem_storedw_loop: Remove unused variable

* [igt-dev] [PATCH i-g-t] gem_storedw_loop: Remove unused variable
@ 2018-08-22 22:21 Antonio Argenziano
  2018-08-22 22:39 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Antonio Argenziano @ 2018-08-22 22:21 UTC (permalink / raw)
  To: igt-dev

'long-*' subtests didn't make any sense now so removed those as well.

Signed-off-by: Antonio Argenziano <antonio.argenziano@intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
---
 tests/gem_storedw_loop.c | 31 +++++++++++++------------------
 1 file changed, 13 insertions(+), 18 deletions(-)

diff --git a/tests/gem_storedw_loop.c b/tests/gem_storedw_loop.c
index 8e57abe2..b00555e0 100644
--- a/tests/gem_storedw_loop.c
+++ b/tests/gem_storedw_loop.c
@@ -65,7 +65,7 @@ mmap_coherent(int fd, uint32_t handle, int size)
 }
 
 static void
-store_dword_loop(int fd, int ring, int count, int divider)
+store_dword_loop(int fd, int ring, int divider)
 {
 	int i, val = 0;
 	struct drm_i915_gem_execbuffer2 execbuf;
@@ -143,19 +143,19 @@ store_dword_loop(int fd, int ring, int count, int divider)
 }
 
 static void
-store_test(int fd, int ring, int count)
+store_test(int fd, int ring)
 {
 	gem_require_ring(fd, ring);
-	store_dword_loop(fd, ring, count, 1);
-	store_dword_loop(fd, ring, count, 2);
+	store_dword_loop(fd, ring, 1);
+	store_dword_loop(fd, ring, 2);
 	if (!igt_run_in_simulation()) {
-		store_dword_loop(fd, ring, count, 3);
-		store_dword_loop(fd, ring, count, 5);
-		store_dword_loop(fd, ring, count, 7);
-		store_dword_loop(fd, ring, count, 11);
-		store_dword_loop(fd, ring, count, 13);
-		store_dword_loop(fd, ring, count, 17);
-		store_dword_loop(fd, ring, count, 19);
+		store_dword_loop(fd, ring, 3);
+		store_dword_loop(fd, ring, 5);
+		store_dword_loop(fd, ring, 7);
+		store_dword_loop(fd, ring, 11);
+		store_dword_loop(fd, ring, 13);
+		store_dword_loop(fd, ring, 17);
+		store_dword_loop(fd, ring, 19);
 	}
 }
 
@@ -184,14 +184,9 @@ igt_main
 	}
 
 	for (e = intel_execution_engines; e->name; e++) {
-		igt_subtest_f("short-%s", e->name) {
+		igt_subtest_f("store-%s", e->name) {
 			check_test_requirements(fd, e->exec_id);
-			store_test(fd, e->exec_id | e->flags, 4*1024);
-		}
-
-		igt_subtest_f("long-%s", e->name) {
-			check_test_requirements(fd, e->exec_id);
-			store_test(fd, e->exec_id | e->flags, 4*1024*1024);
+			store_test(fd, e->exec_id | e->flags);
 		}
 	}
 
-- 
2.16.2

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply related	[flat|nested] 5+ messages in thread