[igt-dev] [PATCH i-g-t 1/2] i915/gem_exec_create: Added test description for test case.

sai.gowtham.ch at intel.com sai.gowtham.ch at intel.com
Mon May 23 12:19:15 UTC 2022


From: Sai Gowtham Ch <sai.gowtham.ch at intel.com>

Added test description for all the subtests that are available.

Cc: Kamil Konieczny <kamil.konieczny at linux.intel.com>
Signed-off-by: Sai Gowtham Ch <sai.gowtham.ch at intel.com>
---
 tests/i915/gem_exec_create.c | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/tests/i915/gem_exec_create.c b/tests/i915/gem_exec_create.c
index 78d5ffb3..fd09ef02 100644
--- a/tests/i915/gem_exec_create.c
+++ b/tests/i915/gem_exec_create.c
@@ -47,6 +47,10 @@
 #include "i915_drm.h"
 #include "i915/intel_memory_region.h"
 
+IGT_TEST_DESCRIPTION("\n This test overloads the driver with transient active "
+		"objects and checks if we don't kill the system under the "
+		"mempressure. Some of the symptoms this test look for include "
+		"mysterious hangs.");
 #define ENGINE_FLAGS  (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
 
 static double elapsed(const struct timespec *start, const struct timespec *end)
@@ -166,6 +170,7 @@ igt_main
 					    I915_DEVICE_MEMORY);
 	}
 
+	igt_describe("Checks if we kill the systems by overloading it with active objects");
 	igt_subtest_with_dynamic("legacy")
 		for_each_combination(regions, 1, set) {
 			char *sub_name = memregion_dynamic_subtest_name(regions);
@@ -177,6 +182,7 @@ igt_main
 			free(sub_name);
 		}
 
+	igt_describe("Quick sanitycheck of the above test setup iterating over engines");
 	igt_subtest_with_dynamic("basic")
 		for_each_combination(regions, 1, set) {
 			char *sub_name = memregion_dynamic_subtest_name(regions);
@@ -188,6 +194,7 @@ igt_main
 			free(sub_name);
 		}
 
+	igt_describe("This test does a concurrent execution");
 	igt_subtest_with_dynamic("forked")
 		for_each_combination(regions, 1, set) {
 			char *sub_name = memregion_dynamic_subtest_name(regions);
@@ -199,6 +206,9 @@ igt_main
 			free(sub_name);
 		}
 
+
+	igt_describe("This test does a forced reclaim, behaving like a bad application"
+		     " leaking its bo cache");
 	igt_subtest_with_dynamic("madvise")
 		for_each_combination(regions, 1, set) {
 			char *sub_name = memregion_dynamic_subtest_name(regions);
-- 
2.35.1



More information about the igt-dev mailing list