[igt-dev] [PATCH 1/2] tests/i915/gem_exec_parallel : Add subtests description

janga.rahul.kumar at intel.com janga.rahul.kumar at intel.com
Mon Jun 27 06:43:46 UTC 2022


From: Janga Rahul Kumar <janga.rahul.kumar at intel.com>

Add test description to all the available subtests.

Cc: Kamil Konieczny <kamil.konieczny at linux.intel.com>
Signed-off-by: Janga Rahul Kumar <janga.rahul.kumar at intel.com>
---
 tests/i915/gem_exec_parallel.c | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/tests/i915/gem_exec_parallel.c b/tests/i915/gem_exec_parallel.c
index 1dfbaa24..687ca70c 100644
--- a/tests/i915/gem_exec_parallel.c
+++ b/tests/i915/gem_exec_parallel.c
@@ -34,6 +34,8 @@
 #include "igt.h"
 #include "igt_gt.h"
 
+IGT_TEST_DESCRIPTION("Exercise to fill buffers using many clients working in parallel.");
+
 #define ENGINE_MASK  (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
 
 #define VERIFY 0
@@ -309,11 +311,12 @@ igt_main
 	const struct mode {
 		const char *name;
 		unsigned flags;
+		const char *describe;
 	} modes[] = {
-		{ "basic", 0 },
-		{ "contexts", CONTEXTS },
-		{ "fds", FDS },
-		{ "userptr", USERPTR },
+		{ "basic", 0, "Check basic functionality per engine." },
+		{ "contexts", CONTEXTS, "Check with many contexts." },
+		{ "fds", FDS, "Check with many fds." },
+		{ "userptr", USERPTR, "Check basic userptr thrashing." },
 		{ NULL }
 	};
 	const intel_ctx_t *ctx;
@@ -327,6 +330,7 @@ igt_main
 		igt_fork_hang_detector(fd);
 	}
 
+	igt_describe("Check with engines working in parallel.");
 	igt_subtest_with_dynamic("engines") {
 		for (const struct mode *m = modes; m->name; m++)
 			igt_dynamic(m->name)
@@ -335,6 +339,7 @@ igt_main
 	}
 
 	for (const struct mode *m = modes; m->name; m++) {
+		igt_describe(m->describe);
 		igt_subtest_with_dynamic(m->name) {
 			for_each_ctx_engine(fd, ctx, e) {
 				if (gem_class_can_store_dword(fd, e->class))
-- 
2.25.1



More information about the igt-dev mailing list