[igt-dev] [PATCH i-g-t] i915/gem_wait: Add description for test and subtests

apoorva1.singh at intel.com apoorva1.singh at intel.com
Mon Nov 16 16:59:54 UTC 2020


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

Signed-off-by: Apoorva Singh <apoorva1.singh at intel.com>
Cc: Melkaveri, Arjun <arjun.melkaveri at intel.com>
---
 tests/i915/gem_wait.c | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/tests/i915/gem_wait.c b/tests/i915/gem_wait.c
index 7b2b1c2b..1cf40009 100644
--- a/tests/i915/gem_wait.c
+++ b/tests/i915/gem_wait.c
@@ -31,6 +31,8 @@
 #include "igt.h"
 #include "igt_vgem.h"
 
+IGT_TEST_DESCRIPTION("Tests the GEM_WAIT ioctl");
+
 static int __gem_wait(int fd, struct drm_i915_gem_wait *w)
 {
 	int err;
@@ -176,9 +178,11 @@ igt_main
 		igt_require_gem(fd);
 	}
 
+	igt_describe("Tests that GEM_WAIT ioctl returns relevant error with invalid flag");
 	igt_subtest("invalid-flags")
 		invalid_flags(fd);
 
+	igt_describe("Tests that GEM_WAIT ioctl returns relevant error with invalid buffer object");
 	igt_subtest("invalid-buf")
 		invalid_buf(fd);
 
@@ -200,8 +204,11 @@ igt_main
 			igt_fork_signal_helper();
 		}
 
-		for (const typeof(*tests) *t = tests; t->name; t++)
+		for (const typeof(*tests) *t = tests; t->name; t++) {
+			igt_describe_f("For all engines, test the GEM_WAIT ioctl waiting on "
+			       "buffer object in %s mode", t->name);
 			test_all_engines(t->name, fd, t->flags);
+		}
 
 		igt_fixture {
 			igt_stop_signal_helper();
@@ -227,8 +234,11 @@ igt_main
 			igt_fork_signal_helper();
 		}
 
-		for (const typeof(*tests) *t = tests; t->name; t++)
+		for (const typeof(*tests) *t = tests; t->name; t++) {
+			igt_describe_f("For all engines, test the GEM_WAIT ioctl waiting on "
+			       "buffer object in %s mode when hang is allowed", (t->name+5));
 			test_all_engines(t->name, fd, t->flags);
+		}
 
 		igt_fixture {
 			igt_stop_signal_helper();
-- 
2.25.1



More information about the igt-dev mailing list