[igt-dev] [PATCH i-g-t v1 1/1] tests/kms_flip: add possibility to ignore long HPDs

Vinod Govindapillai vinod.govindapillai at intel.com
Tue Nov 14 08:45:04 UTC 2023


In some CI environments, spurious long HPDs are impacting the
kms_flip test results. The provision to set ignore long HPDs was
not being used by the kms_flip tests. Call igt_display_require
for kms_flip as well which will track the usage of environment
variable IGT_KMS_IGNORE_HPD

Signed-off-by: Vinod Govindapillai <vinod.govindapillai at intel.com>
---
 tests/kms_flip.c | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/tests/kms_flip.c b/tests/kms_flip.c
index fc7cee050..c32b19412 100755
--- a/tests/kms_flip.c
+++ b/tests/kms_flip.c
@@ -2116,12 +2116,13 @@ igt_main_args("e", NULL, help_str, opt_handler, NULL)
 		{ 0, TEST_BO_TOOBIG | TEST_NO_2X_OUTPUT, "bo-too-big" },
 		{ 10, TEST_FLIP | TEST_SUSPEND, "flip-vs-suspend" },
 	};
+	igt_display_t display;
 	int i;
 
 	igt_fixture {
 		drm_fd = drm_open_driver_master(DRIVER_ANY);
 
-		igt_enable_connectors(drm_fd);
+		igt_display_require(&display, drm_fd);
 
 		kmstest_set_vt_graphics_mode();
 		igt_install_exit_handler(kms_flip_exit_handler);
@@ -2192,6 +2193,8 @@ igt_main_args("e", NULL, help_str, opt_handler, NULL)
 	}
 	igt_stop_signal_helper();
 
-	igt_fixture
+	igt_fixture {
+		igt_display_fini(&display);
 		drm_close_driver(drm_fd);
+	}
 }
-- 
2.34.1



More information about the igt-dev mailing list