[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