[igt-dev] [PATCH i-g-t, v4] i915/gem_exec_store: Add test description

apoorva1.singh at intel.com apoorva1.singh at intel.com
Wed Mar 2 09:15:17 UTC 2022


From: Apoorva Singh <apoorva1.singh at intel.com>

Add description for test and all subtests

v2: Rebase

v3: Incorporated changes in description based
    on feedback

v4: Minor alignment adjustments

Signed-off-by: Apoorva Singh <apoorva1.singh at intel.com>
Cc: Arjun Melkaveri <arjun.melkaveri at intel.com>
Cc: Kamil Konieczny <kamil.konieczny at linux.intel.com>
Reviewed-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>
---
 tests/i915/gem_exec_store.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/tests/i915/gem_exec_store.c b/tests/i915/gem_exec_store.c
index cb99f951..efb9907e 100644
--- a/tests/i915/gem_exec_store.c
+++ b/tests/i915/gem_exec_store.c
@@ -35,6 +35,8 @@
 #include "igt_device.h"
 #include "igt_gt.h"
 
+IGT_TEST_DESCRIPTION("Exercise store dword functionality using execbuf-ioctl");
+
 #define ENGINE_MASK  (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
 
 /* Without alignment detection we assume the worst-case scenario. */
@@ -432,15 +434,21 @@ igt_main
 		igt_fork_hang_detector(fd);
 	}
 
+	igt_describe("Verify that all capable engines can store dwords to a common buffer object");
 	igt_subtest("basic")
 		store_all(fd, ctx);
 
+	igt_describe("Verify that each capable engine can store a dword to a buffer object");
 	test_each_engine("dword", fd, ctx, e)
 		store_dword(fd, ctx, e);
 
+	igt_describe("Verify that each capable engine can store a dword to different cachelines "
+		     "of a buffer object");
 	test_each_engine("cachelines", fd, ctx, e)
 		store_cachelines(fd, ctx, e, 0);
 
+	igt_describe("Verify that each capable engine can store a dword to various page-sized "
+		     "buffer objects");
 	test_each_engine("pages", fd, ctx, e)
 		store_cachelines(fd, ctx, e, PAGES);
 
-- 
2.25.1



More information about the igt-dev mailing list