[igt-dev] [PATCH i-g-t] intel-ci: Reorder system suspend to last

[Petri Latvala]

On Fri, Apr 22, 2022 at 12:59:50PM +0530, Sharma, Swati2 wrote:
> Please find one nitpick below. Rest looks good to me.
> With that fixed.
> 
> Reviewed-by:
> Swati Sharma <swati2.sharma at intel.com>

And FWIW CI's pov:

Acked-by: Petri Latvala <petri.latvala at intel.com>

> 
> On 22-Apr-22 11:29 AM, Anshuman Gupta wrote:
> > Reorder the system wide igt tests to last of fast-feedback.testlist,
> > so we don't loose the other tests coverage on the platforms which
> > are largely getting hit by Suspend/Resume related failures.
> > 
> > Cc: Petri Latvala <petri.latvala at intel.com>
> > Cc: Swati Sharma <swati2.sharma at intel.com>
> > Signed-off-by: Anshuman Gupta <anshuman.gupta at intel.com>
> > ---
> >   tests/intel-ci/fast-feedback.testlist | 12 +++++++-----
> >   1 file changed, 7 insertions(+), 5 deletions(-)
> > 
> > diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist
> > index f6329708b..c9613e609 100644
> > --- a/tests/intel-ci/fast-feedback.testlist
> > +++ b/tests/intel-ci/fast-feedback.testlist
> > @@ -25,8 +25,6 @@ igt at gem_exec_fence@nb-await
> >   igt at gem_exec_gttfill@basic
> >   igt at gem_exec_parallel@engines
> >   igt at gem_exec_store@basic
> > -igt at gem_exec_suspend@basic-s0
> > -igt at gem_exec_suspend@basic-s3
> >   igt at gem_flink_basic@bad-flink
> >   igt at gem_flink_basic@bad-open
> >   igt at gem_flink_basic@basic
> > @@ -98,7 +96,6 @@ igt at kms_chamelium@hdmi-edid-read
> >   igt at kms_chamelium@hdmi-crc-fast
> >   igt at kms_chamelium@vga-hpd-fast
> >   igt at kms_chamelium@vga-edid-read
> > -igt at kms_chamelium@common-hpd-after-suspend
> >   igt at kms_prop_blob@basic
> >   igt at kms_cursor_legacy@basic-busy-flip-before-cursor-atomic
> >   igt at kms_cursor_legacy@basic-busy-flip-before-cursor-legacy
> > @@ -128,7 +125,6 @@ igt at kms_pipe_crc_basic@read-crc-pipe-a
> >   igt at kms_pipe_crc_basic@read-crc-pipe-b
> >   igt at kms_pipe_crc_basic@read-crc-pipe-c
> >   igt at kms_pipe_crc_basic@read-crc-pipe-a-frame-sequence
> > -igt at kms_pipe_crc_basic@suspend-read-crc-pipe-a
> >   igt at kms_psr@primary_page_flip
> >   igt at kms_psr@cursor_plane_move
> >   igt at kms_psr@sprite_plane_onoff
> > @@ -174,8 +170,14 @@ igt at gem_lmem_swapping@parallel-random-engines
> >   igt at gem_lmem_swapping@random-engines
> >   igt at gem_lmem_swapping@verify-random
> >   igt at i915_pm_rpm@module-reload
> > -igt at i915_suspend@system-suspend-without-i915
> >   # Kernel selftests
> >   igt at i915_selftest@live
> >   igt at dmabuf@all
> > +
> > +# System wide Suspend Test
> 
> nitpick: System wide suspend tests
> 
> > +igt at i915_suspend@system-suspend-without-i915
> > +igt at gem_exec_suspend@basic-s0
> > +igt at gem_exec_suspend@basic-s3
> > +igt at kms_chamelium@common-hpd-after-suspend
> > +igt at kms_pipe_crc_basic@suspend-read-crc-pipe-a
> 
> -- 
> ~Swati Sharma