[igt-dev] [PATCH] [PATCH i-g-t] i915/gem_reg_read: Added description for test case

priyanka.dandamudi at intel.com priyanka.dandamudi at intel.com
Wed Jul 1 07:55:12 UTC 2020


From: Priyanka Dandamudi <priyanka.dandamudi at intel.com>

Added test description for test and to all 3 subtests.

v2:Added a detailed description for test and modified subtest
description.

Cc: Arjun Melkaveri  <Arjun.Melkaveri at intel.com>
Signed-off-by: Priyanka Dandamudi <priyanka.dandamudi at intel.com>
---
 tests/i915/gem_reg_read.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/tests/i915/gem_reg_read.c b/tests/i915/gem_reg_read.c
index 79facc1f..137c4323 100644
--- a/tests/i915/gem_reg_read.c
+++ b/tests/i915/gem_reg_read.c
@@ -32,6 +32,9 @@
 #include <sys/utsname.h>
 #include <time.h>
 
+IGT_TEST_DESCRIPTION("Test the register read ioctl "
+		     "and register used in this test "
+		     "is render ring timestamp.");
 
 static bool is_x86_64;
 static bool has_proper_timestamp;
@@ -156,14 +159,19 @@ igt_main
 		has_proper_timestamp = check_timestamp(fd);
 	}
 
+	igt_describe("Verifies reading bad register result in error or not.");
 	igt_subtest("bad-register")
 		igt_assert_eq(read_register(fd, 0x12345678, &val), -EINVAL);
 
+	igt_describe("Test checks if the timestamp counter is moving and increasing."
+                     "It also uses (reg | 1) flag  to get a proper 36b timestamp. ");
 	igt_subtest("timestamp-moving") {
 		igt_skip_on(timer_query(fd, &val) != 0);
 		test_timestamp_moving(fd);
 	}
 
+	igt_describe("Test checks if the timestamp counter is monotonic"
+		     " by checking the counter value continously for 5sec.");
 	igt_subtest("timestamp-monotonic") {
 		igt_skip_on(timer_query(fd, &val) != 0);
 		test_timestamp_monotonic(fd);
-- 
2.26.2



More information about the igt-dev mailing list