Re: [PATCH i-g-t v5 1/5] intel-ci: Move suspend/resume chamelium tests to end

["Sharma, Swati2" <swati2.sharma@intel.com>]

Hi Sowmiya

Patch LGTM

Reviewed-by: Swati Sharma <swati2.sharma@intel.com>

On 26-03-2026 02:34 pm, Sowmiya S wrote:
> From: Mohammed Bilal <mohammed.bilal@intel.com>
>
> Suspend operations can cause system hangs or connection timeouts,
> so isolating them at the tail ensures stability of main test
> execution.
>
> v3:
> -Sort tests alphabetically (Kamil)
> v4: Add detailed comment (Swati)
>
> Signed-off-by: Sowmiya S <sowmiya.s@intel.com>
> Signed-off-by: Mohammed Bilal <mohammed.bilal@intel.com>
> ---
>   tests/intel-ci/chamelium-only.testlist | 15 +++++++++------
>   1 file changed, 9 insertions(+), 6 deletions(-)
>
> diff --git a/tests/intel-ci/chamelium-only.testlist b/tests/intel-ci/chamelium-only.testlist
> index 9adace28c..0e7c8e301 100644
> --- a/tests/intel-ci/chamelium-only.testlist
> +++ b/tests/intel-ci/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
> @@ -41,9 +39,7 @@ igt@kms_chamelium_frames@hdmi-crc-planes-random
>   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
> @@ -51,7 +47,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
> @@ -61,7 +56,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
> @@ -75,3 +69,12 @@ igt@kms_chamelium_edid@hdmi-edid-change-during-hibernate
>   igt@kms_chamelium_hpd@common-hpd-after-hibernate
>   igt@kms_chamelium_hpd@dp-hpd-after-hibernate
>   igt@kms_chamelium_hpd@hdmi-hpd-after-hibernate
> +# Suspend operations can cause system hangs or connection timeouts,
> +# so isolating them at the tail ensures stability of main test
> +# execution.
> +igt@kms_chamelium_edid@dp-edid-change-during-suspend
> +igt@kms_chamelium_edid@hdmi-edid-change-during-suspend
> +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