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

[apoorva1.singh at intel.com]

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

Add description for test and all the subtests

v2: Rebase

Signed-off-by: Apoorva Singh <apoorva1.singh at intel.com>
Cc: Kamil Konieczny <kamil.konieczny at linux.intel.com>
Cc: Arjun Melkaveri <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 b17927b6..9f84ef33 100644
--- a/tests/i915/gem_wait.c
+++ b/tests/i915/gem_wait.c
@@ -32,6 +32,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;
@@ -185,9 +187,11 @@ igt_main
 		ctx = intel_ctx_create_all_physical(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);
 
@@ -209,8 +213,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, ctx, t->flags);
+		}
 
 		igt_fixture {
 			igt_stop_signal_helper();
@@ -236,8 +243,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, ctx, t->flags);
+		}
 
 		igt_fixture {
 			igt_stop_signal_helper();
-- 
2.25.1