public inbox for igt-dev@lists.freedesktop.org
 help / color / mirror / Atom feed
* [PATCH i-g-t] tests/kms_hdmi_inject: Retry ELD check to avoid transient audio races
@ 2026-04-27 10:58 Jeevan B
  2026-04-27 19:38 ` ✓ i915.CI.BAT: success for tests/kms_hdmi_inject: Retry ELD check to avoid transient audio races (rev2) Patchwork
                   ` (3 more replies)
  0 siblings, 4 replies; 6+ messages in thread
From: Jeevan B @ 2026-04-27 10:58 UTC (permalink / raw)
  To: igt-dev; +Cc: chaitanya.kumar.borah, Jeevan B

ELD population can lag HDMI audio enable on some platforms.
Retry once to avoid spurious test failures.

Closes: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/14867
Signed-off-by: Jeevan B <jeevan.b@intel.com>
---
 tests/kms_hdmi_inject.c | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/tests/kms_hdmi_inject.c b/tests/kms_hdmi_inject.c
index 00eb5e4a4..f388a84c3 100644
--- a/tests/kms_hdmi_inject.c
+++ b/tests/kms_hdmi_inject.c
@@ -200,7 +200,13 @@ hdmi_inject_audio(int drm_fd, drmModeConnector *connector)
 	 * Test if we have /proc/asound/HDMI/eld#0.0 and is its contents are
 	 * valid.
 	 */
-	igt_assert(eld_has_igt());
+	if (!eld_has_igt()) {
+		igt_info("HDMI audio enabled but ELD not visible yet; retrying\n");
+		usleep(200 * 1000);
+
+		igt_assert_f(eld_has_igt(),
+			     "HDMI ELD missing after retry\n");
+	}
 
 	igt_remove_fb(drm_fd, &fb);
 
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 6+ messages in thread
* [PATCH i-g-t] tests/kms_hdmi_inject: Retry ELD check to avoid transient audio races
@ 2026-04-27 10:56 Jeevan B
  0 siblings, 0 replies; 6+ messages in thread
From: Jeevan B @ 2026-04-27 10:56 UTC (permalink / raw)
  To: igt-dev; +Cc: chaitanya.kumar.borah, Jeevan B

ELD population can lag HDMI audio enable on some platforms.
Retry once to avoid spurious test failures.

Closes: https://gitlab.freedesktop.org/drm/xe/kernel/-/issues/14867
Signed-off-by: Jeevan B <jeevan.b@intel.com>
---
 tests/kms_hdmi_inject.c | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/tests/kms_hdmi_inject.c b/tests/kms_hdmi_inject.c
index 00eb5e4a4..f388a84c3 100644
--- a/tests/kms_hdmi_inject.c
+++ b/tests/kms_hdmi_inject.c
@@ -200,7 +200,13 @@ hdmi_inject_audio(int drm_fd, drmModeConnector *connector)
 	 * Test if we have /proc/asound/HDMI/eld#0.0 and is its contents are
 	 * valid.
 	 */
-	igt_assert(eld_has_igt());
+	if (!eld_has_igt()) {
+		igt_info("HDMI audio enabled but ELD not visible yet; retrying\n");
+		usleep(200 * 1000);
+
+		igt_assert_f(eld_has_igt(),
+			     "HDMI ELD missing after retry\n");
+	}
 
 	igt_remove_fb(drm_fd, &fb);
 
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 6+ messages in thread
end of thread, other threads:[~2026-04-28  1:15 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-04-27 10:58 [PATCH i-g-t] tests/kms_hdmi_inject: Retry ELD check to avoid transient audio races Jeevan B
2026-04-27 19:38 ` ✓ i915.CI.BAT: success for tests/kms_hdmi_inject: Retry ELD check to avoid transient audio races (rev2) Patchwork
2026-04-27 20:29 ` ✓ Xe.CI.BAT: " Patchwork
2026-04-27 23:36 ` ✗ i915.CI.Full: failure " Patchwork
2026-04-28  1:15 ` ✓ Xe.CI.FULL: success " Patchwork
  -- strict thread matches above, loose matches on Subject: below --
2026-04-27 10:56 [PATCH i-g-t] tests/kms_hdmi_inject: Retry ELD check to avoid transient audio races Jeevan B

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox