[PATCH i-g-t v2] tests/kms_rmfb: Fix subtest description

Lucas De Marchi lucas.demarchi at intel.com
Thu Feb 1 00:44:48 UTC 2024


Kernel driver is responsible to do those actions. kms_rmfb is then
checking if that was true in 2 cases: a) DRM_IOCTL_MODE_RMFB was called
or b) the file descriptor was closed.

v2: Fix swapped description (Swati)

Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
---
 tests/kms_rmfb.c | 23 ++++++++++++-----------
 1 file changed, 12 insertions(+), 11 deletions(-)

diff --git a/tests/kms_rmfb.c b/tests/kms_rmfb.c
index 39e8bbd55..0ea689d60 100644
--- a/tests/kms_rmfb.c
+++ b/tests/kms_rmfb.c
@@ -24,7 +24,7 @@
 /**
  * TEST: kms rmfb
  * Category: Display
- * Description: This tests rmfb and close-fd behavior. In these casesthe
+ * Description: This tests rmfb and close-fd behavior. In these cases the
  *              framebuffers should be removed from the crtc.
  * Driver requirement: i915, xe
  * Functionality: kms_gem_interop
@@ -42,12 +42,13 @@
 
 /**
  * SUBTEST: close-fd
- * Description: RMFB is supposed to free the framebuffers from any and all planes
- *              so test this and make sure it works with fd close and reopen.
+ * Description: Kernel driver is supposed to free the framebuffers from any and all planes
+ *		when the fd is closed. Ensure that is the case by closing and re-opening
+ *		it.
  *
  * SUBTEST: rmfb-ioctl
- * Description: RMFB is supposed to free the framebuffers from any and all planes
- *              so test this and make sure it works.
+ * Description: Kernel driver is supposed to free the framebuffers from any and all planes
+ *              when DRM_IOCTL_MODE_RMFB ioctl is called. Ensure that is the case.
  */
 
 #ifndef DRM_CAP_CURSOR_WIDTH
@@ -57,7 +58,7 @@
 #define DRM_CAP_CURSOR_HEIGHT 0x9
 #endif
 
-IGT_TEST_DESCRIPTION("This tests rmfb and close-fd behavior. In these cases"
+IGT_TEST_DESCRIPTION("This tests rmfb and close-fd behavior. In these cases "
 		     "the framebuffers should be removed from the crtc.");
 
 struct rmfb_data {
@@ -198,11 +199,11 @@ igt_main
 		const char *name;
 		const char *description;
 	} tests[] = {
-		{ false, "rmfb-ioctl", "RMFB is supposed to free the framebuffers from any and all "
-				       "planes so test this and make sure it works."},
-		{ true, "close-fd", "RMFB is supposed to free the framebuffers from any and all "
-				    "planes so test this and make sure it works with fd close "
-				    "and reopen."},
+		{ false, "rmfb-ioctl", "Kernel driver is supposed to free the framebuffers from any and all planes "
+				       "when DRM_IOCTL_MODE_RMFB ioctl is called. Ensure that is the case." },
+		{ true, "close-fd", "Kernel driver is supposed to free the framebuffers from any and all planes "
+				    "when the fd is closed. Ensure that is the case by closing and re-opening "
+				    "it" },
 	};
 	struct rmfb_data data = {};
 	int i;
-- 
2.43.0



More information about the igt-dev mailing list