[igt-dev] [PATCH i-g-t] i915/gem_busy: Added description to all subtests

* [igt-dev] [PATCH i-g-t] i915/gem_busy: Added description to all subtests
@ 2021-06-29 12:32 sai.krishna.kuchi
  2021-06-30 14:35 ` [igt-dev] ✗ Fi.CI.BAT: failure for " Patchwork
  0 siblings, 1 reply; 2+ messages in thread
From: sai.krishna.kuchi @ 2021-06-29 12:32 UTC (permalink / raw)
  To: igt-dev

From: "Kuchi, Sai Krishna" <sai.krishna.kuchi@intel.com>

Cc: Melkaveri, Arjun<arjun.melkaveri@intel.com>
Cc: Gandi, Ramadevi <ramadevi.gandi@intel.com>

Signed-off-by: Kuchi, Sai Krishna<sai.krishna.kuchi@intel.com>
---
 tests/i915/gem_busy.c | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/tests/i915/gem_busy.c b/tests/i915/gem_busy.c
index dc481f3c..03cd545c 100644
--- a/tests/i915/gem_busy.c
+++ b/tests/i915/gem_busy.c
@@ -413,18 +413,23 @@ igt_main
 		igt_require_gem(fd);
 	}
 
+	igt_describe_f("Checking busy-ioctl without hang");
 	igt_subtest_group {
 		igt_fixture {
 			igt_fork_hang_detector(fd);
 		}
 
+		igt_describe_f("Basic checking of busy-ioctl for physical engines");
 		igt_subtest_with_dynamic("busy") {
+			igt_describe_f("Basic busy check for all engines by creating seperate"
+			" processes");
 			igt_dynamic("all") {
 				gem_quiescent_gpu(fd);
 				all(fd);
 			}
 
 			__for_each_physical_engine(fd, e) {
+				igt_describe_f("Basic busy check for %s engine", e->name);
 				igt_dynamic_f("%s", e->name) {
 					gem_quiescent_gpu(fd);
 					basic(fd, e, 0);
@@ -432,6 +437,7 @@ igt_main
 			}
 		}
 
+		igt_describe_f("Extended checking of busy-ioctl for phusical engines");
 		igt_subtest_group {
 			igt_fixture {
 				igt_require(has_extended_busy_ioctl(fd));
@@ -439,18 +445,22 @@ igt_main
 			}
 
 			test_each_engine_store("extended", fd, e) {
+				igt_describe_f("Extended busy check for %s engine", e->name);
 				gem_quiescent_gpu(fd);
 				one(fd, e, 0);
 				gem_quiescent_gpu(fd);
 			}
 
 			test_each_engine_store("parallel", fd, e) {
+				igt_describe_f("Extended busy check for %s along with"
+				" all other engines", e->name);
 				gem_quiescent_gpu(fd);
 				one(fd, e, PARALLEL);
 				gem_quiescent_gpu(fd);
 			}
 		}
 
+		igt_describe_f("Checking busy-ioctl with Semaphore");
 		igt_subtest_group {
 			igt_fixture {
 				igt_require(has_extended_busy_ioctl(fd));
@@ -458,12 +468,15 @@ igt_main
 			}
 
 			test_each_engine("semaphore", fd, e) {
+				igt_describe_f("Busy check for %s engine with semaphore", e->name);
 				gem_quiescent_gpu(fd);
 				semaphore(fd, e);
 				gem_quiescent_gpu(fd);
 			}
 		}
 
+		igt_describe_f("Checking busy-ioctl with race b/w two threads, one checking"
+		" busyness and other randomly closing handles");
 		igt_subtest("close-race")
 			close_race(fd);
 
@@ -472,6 +485,7 @@ igt_main
 		}
 	}
 
+	igt_describe_f("Checking busy-ioctl with hang");
 	igt_subtest_group {
 		igt_hang_t hang;
 
@@ -480,11 +494,13 @@ igt_main
 		}
 
 		test_each_engine("hang", fd, e) {
+			igt_describe_f("Basic busy check for %s with hang", e->name);
 			gem_quiescent_gpu(fd);
 			basic(fd, e, HANG);
 			gem_quiescent_gpu(fd);
 		}
 
+		igt_describe_f("Extended checking of busy-ioctl for all physical engines");
 		igt_subtest_group {
 			igt_fixture {
 				igt_require(has_extended_busy_ioctl(fd));
@@ -492,6 +508,7 @@ igt_main
 			}
 
 			test_each_engine_store("hang-extended", fd, e) {
+				igt_describe_f("Extended busy check for %s with hang", e->name);
 				gem_quiescent_gpu(fd);
 				one(fd, e, HANG);
 				gem_quiescent_gpu(fd);
-- 
2.28.0

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

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