[PATCH i-g-t] tests/chamelium/kms_chamelium_edid: Limit the number of EDIDs tested to prevent CI timeout.

Pranay Samala pranay.samala at intel.com
Fri Dec 13 09:54:32 UTC 2024


This patch reduces the number of non-4K HDMI EDIDs to 25 when
the extended flag is not used, as CI does not use this flag.
DP EDID tests are not reduced due to existing limited EDIDs.
The change ensures that only HDMI non-4K EDIDs are limited,
improving test efficiency and avoiding CI timeouts.

Cc: Mark Yacoub <markyacoub at chromium.org>
Cc: Mark Yacoub <markyacoub at google.com>
Cc: Louis Chauvet <louis.chauvet at bootlin.com>

Signed-off-by: Pranay Samala <pranay.samala at intel.com>
---
 tests/chamelium/kms_chamelium_edid.c | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/tests/chamelium/kms_chamelium_edid.c b/tests/chamelium/kms_chamelium_edid.c
index 5db8204d5..e572a6468 100644
--- a/tests/chamelium/kms_chamelium_edid.c
+++ b/tests/chamelium/kms_chamelium_edid.c
@@ -267,7 +267,8 @@ static const char igt_edid_stress_resolution_desc[] =
 static void edid_stress_resolution(chamelium_data_t *data,
 				   struct chamelium_port *port,
 				   monitor_edid edids_list[],
-				   size_t edids_list_len)
+				   size_t edids_list_len,
+				   bool limit)
 {
 	int i;
 	struct chamelium *chamelium = data->chamelium;
@@ -280,7 +281,7 @@ static void edid_stress_resolution(chamelium_data_t *data,
 	 * if there is environment variable in CI when executing with igt_runner
 	 */
 
-	if (!extended && is_intel_device(data->drm_fd))
+	if (!extended && is_intel_device(data->drm_fd) && limit)
 		edids_list_len = CI_LIMIT;
 
 	for (i = 0; i < edids_list_len; ++i) {
@@ -559,13 +560,13 @@ igt_main_args("e", NULL, help_str, opt_handler, NULL)
 		igt_describe(igt_edid_stress_resolution_desc);
 		connector_subtest("dp-edid-stress-resolution-4k", DisplayPort)
 			edid_stress_resolution(&data, port, DP_EDIDS_4K,
-					       ARRAY_SIZE(DP_EDIDS_4K));
+					       ARRAY_SIZE(DP_EDIDS_4K), false);
 
 		igt_describe(igt_edid_stress_resolution_desc);
 		connector_subtest("dp-edid-stress-resolution-non-4k",
 				  DisplayPort)
 			edid_stress_resolution(&data, port, DP_EDIDS_NON_4K,
-					       ARRAY_SIZE(DP_EDIDS_NON_4K));
+					       ARRAY_SIZE(DP_EDIDS_NON_4K), false);
 
 		igt_describe(igt_edid_resolution_list_desc);
 		connector_subtest("dp-edid-resolution-list", DisplayPort)
@@ -613,12 +614,12 @@ igt_main_args("e", NULL, help_str, opt_handler, NULL)
 		igt_describe(igt_edid_stress_resolution_desc);
 		connector_subtest("hdmi-edid-stress-resolution-4k", HDMIA)
 			edid_stress_resolution(&data, port, HDMI_EDIDS_4K,
-					       ARRAY_SIZE(HDMI_EDIDS_4K));
+					       ARRAY_SIZE(HDMI_EDIDS_4K), false);
 
 		igt_describe(igt_edid_stress_resolution_desc);
 		connector_subtest("hdmi-edid-stress-resolution-non-4k", HDMIA)
 			edid_stress_resolution(&data, port, HDMI_EDIDS_NON_4K,
-					       ARRAY_SIZE(HDMI_EDIDS_NON_4K));
+					       ARRAY_SIZE(HDMI_EDIDS_NON_4K), true);
 
 		igt_describe(test_suspend_resume_edid_change_desc);
 		connector_subtest("hdmi-edid-change-during-suspend", HDMIA)
-- 
2.34.1



More information about the igt-dev mailing list