On 2020-04-08 17:19, Chris Wilson wrote:
> S4-devices is a debug only path, and while it can prove informative as
> to the nature of suspend issues, being a debug only path it is not so
> relevant towards the driver health. Relegate it to the shards and idle
> runs, so we can shave 20s off BAT.
> 
> gem_exec_suspend@basic itself is just a test suspend, a sanitycheck to
> ensure the test is functional, for manual debugging of the tests.
> 
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Martin Peres <martin.peres@linux.intel.com>
> Cc: Petri Latvala <petri.latvala@intel.com>

Acked-by: Martin Peres <martin.peres@linux.intel.com>
> ---
>  tests/intel-ci/fast-feedback.testlist | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist
> index 620d5f3a7..2ccad4386 100644
> --- a/tests/intel-ci/fast-feedback.testlist
> +++ b/tests/intel-ci/fast-feedback.testlist
> @@ -23,10 +23,8 @@ igt@gem_exec_fence@nb-await
>  igt@gem_exec_gttfill@basic
>  igt@gem_exec_parallel@engines
>  igt@gem_exec_store@basic
> -igt@gem_exec_suspend@basic
>  igt@gem_exec_suspend@basic-s0
>  igt@gem_exec_suspend@basic-s3
> -igt@gem_exec_suspend@basic-s4-devices
>  igt@gem_flink_basic@bad-flink
>  igt@gem_flink_basic@bad-open
>  igt@gem_flink_basic@basic
>