Re: [PATCH v5 4/6] intel-ci: Move i915 suspend/resume tests to end

Kamil Konieczny
2026-03-12T15:49:18Z

Hi Mohammed,
On 2026-03-05 at 14:09:04 +0530, Mohammed Bilal wrote:
> Reorganize FULL chamelium testlist by moving suspend and resume tests to
> the end. Suspend operations can cause system hangs or connection timeouts,
> so isolating them at the tail ensures stability of main test execution.
>
> v2:
> - Move suspend resume tests to the end of the testlist (Kunal)
>
> v3:
> -Use move-fix-add approach (Kamil)
>
> v4:
> -Restructure move-fix-add approach (Kamil)
>
> v5:
> -Update commit message (Kamil)
> -Split commit (Swati)
>
> Signed-off-by: Mohammed Bilal <mohammed.bilal@intel.com>
> ---
> tests/intel-ci/i915.chamelium-only.testlist | 13 +++++++------
> 1 file changed, 7 insertions(+), 6 deletions(-)
>
> diff --git a/tests/intel-ci/i915.chamelium-only.testlist b/tests/intel-ci/i915.chamelium-only.testlist
> index 6ab9029a4..0db71f7ef 100644
> --- a/tests/intel-ci/i915.chamelium-only.testlist
> +++ b/tests/intel-ci/i915.chamelium-only.testlist
> @@ -12,13 +12,11 @@ igt@kms_chamelium_color@ctm-negative
> igt@kms_chamelium_color@ctm-red-to-blue
> igt@kms_chamelium_color@degamma
> igt@kms_chamelium_color@gamma
> -igt@kms_chamelium_edid@dp-edid-change-during-suspend
> igt@kms_chamelium_edid@dp-edid-read
> igt@kms_chamelium_edid@dp-edid-resolution-list
> igt@kms_chamelium_edid@dp-edid-stress-resolution-4k
> igt@kms_chamelium_edid@dp-edid-stress-resolution-non-4k
> igt@kms_chamelium_edid@dp-mode-timings
> -igt@kms_chamelium_edid@hdmi-edid-change-during-suspend
> igt@kms_chamelium_edid@hdmi-edid-read
> # Chamelium v1 board HDMI receiver supports max 4k@30 resolution
> # removed test as its having a mode 4k@60 resulting
> @@ -43,9 +41,7 @@ igt@kms_chamelium_frames@hdmi-crc-nonplanar-formats
> igt@kms_chamelium_frames@hdmi-crc-single
> igt@kms_chamelium_frames@hdmi-frame-dump
> igt@kms_chamelium_frames@vga-frame-dump
> -igt@kms_chamelium_hpd@common-hpd-after-suspend
> igt@kms_chamelium_hpd@dp-hpd
> -igt@kms_chamelium_hpd@dp-hpd-after-suspend
> igt@kms_chamelium_hpd@dp-hpd-enable-disable-mode
> igt@kms_chamelium_hpd@dp-hpd-fast
> igt@kms_chamelium_hpd@dp-hpd-for-each-pipe
> @@ -53,7 +49,6 @@ igt@kms_chamelium_hpd@dp-hpd-storm
> igt@kms_chamelium_hpd@dp-hpd-storm-disable
> igt@kms_chamelium_hpd@dp-hpd-with-enabled-mode
> igt@kms_chamelium_hpd@hdmi-hpd
> -igt@kms_chamelium_hpd@hdmi-hpd-after-suspend
> igt@kms_chamelium_hpd@hdmi-hpd-enable-disable-mode
> igt@kms_chamelium_hpd@hdmi-hpd-fast
> igt@kms_chamelium_hpd@hdmi-hpd-for-each-pipe
> @@ -63,7 +58,6 @@ igt@kms_chamelium_hpd@hdmi-hpd-with-enabled-mode
> igt@kms_chamelium_audio@hdmi-audio
> igt@kms_chamelium_audio@hdmi-audio-edid
> igt@kms_chamelium_hpd@vga-hpd
> -igt@kms_chamelium_hpd@vga-hpd-after-suspend
> igt@kms_chamelium_hpd@vga-hpd-enable-disable-mode
> igt@kms_chamelium_hpd@vga-hpd-fast
> igt@kms_chamelium_hpd@vga-hpd-for-each-pipe
> @@ -71,6 +65,13 @@ igt@kms_chamelium_hpd@vga-hpd-with-enabled-mode
> igt@kms_chamelium_hpd@vga-hpd-without-ddc
> igt@kms_chamelium_sharpness_filter@filter-basic
> igt@kms_tiled_display@basic-test-pattern-with-chamelium
> +#Place suspend tests at end
> +igt@kms_chamelium_hpd@common-hpd-after-suspend
> +igt@kms_chamelium_hpd@dp-hpd-after-suspend
> +igt@kms_chamelium_hpd@hdmi-hpd-after-suspend
> +igt@kms_chamelium_hpd@vga-hpd-after-suspend
> +igt@kms_chamelium_edid@dp-edid-change-during-suspend
> +igt@kms_chamelium_edid@hdmi-edid-change-during-suspend

Any reason why is it not in alphabetical order?
imho chamelium_edid should be before chamelium_hpd.

Regards,
Kamil

> #Place hibernate tests at end
> igt@kms_chamelium_edid@dp-edid-change-during-hibernate
> igt@kms_chamelium_edid@hdmi-edid-change-during-hibernate
> --
> 2.48.1
>