[igt-dev] [PATCH i-g-t] intel-ci: Sanitycheck CRC readback on the first pipe only

[Martin Peres]

On 11/04/2019 13:52, Chris Wilson wrote:
> For fast-feedback, we need to sanitycheck that CRC reads work, otherwise
> the display validation tests are flawed. However, we don't need to
> extensively test each pipe as the HW is fundamentally the same. Instead,
> do the full suite of tests on one pipe, and just verify that the interface
> works for the other pipes. This should speed BAT up by reducing
> NUM_PIPES suspend and resume cycles to just 1.
> 
> The shards should cover the extended test suite on all pipes for
> complete validation.

I agree with the reasoning:

Acked-by: Martin Peres <martin.peres at linux.intel.com>


> 
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Petri Latvala <petri.latvala at intel.com>
> Cc: Martin Peres <martin.peres at free.fr>
> Cc: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
> ---
>  tests/intel-ci/fast-feedback.testlist | 13 +------------
>  1 file changed, 1 insertion(+), 12 deletions(-)
> 
> diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist
> index 8ae5e8a28..a3b917fd0 100644
> --- a/tests/intel-ci/fast-feedback.testlist
> +++ b/tests/intel-ci/fast-feedback.testlist
> @@ -205,25 +205,14 @@ igt at kms_force_connector_basic@force-edid
>  igt at kms_force_connector_basic@force-load-detect
>  igt at kms_force_connector_basic@prune-stale-modes
>  igt at kms_frontbuffer_tracking@basic
> -igt at kms_pipe_crc_basic@bad-source
>  igt at kms_pipe_crc_basic@hang-read-crc-pipe-a
> -igt at kms_pipe_crc_basic@hang-read-crc-pipe-b
> -igt at kms_pipe_crc_basic@hang-read-crc-pipe-c
>  igt at kms_pipe_crc_basic@nonblocking-crc-pipe-a
>  igt at kms_pipe_crc_basic@nonblocking-crc-pipe-a-frame-sequence
> -igt at kms_pipe_crc_basic@nonblocking-crc-pipe-b
> -igt at kms_pipe_crc_basic@nonblocking-crc-pipe-b-frame-sequence
> -igt at kms_pipe_crc_basic@nonblocking-crc-pipe-c
> -igt at kms_pipe_crc_basic@nonblocking-crc-pipe-c-frame-sequence
>  igt at kms_pipe_crc_basic@read-crc-pipe-a
> -igt at kms_pipe_crc_basic@read-crc-pipe-a-frame-sequence
>  igt at kms_pipe_crc_basic@read-crc-pipe-b
> -igt at kms_pipe_crc_basic@read-crc-pipe-b-frame-sequence
>  igt at kms_pipe_crc_basic@read-crc-pipe-c
> -igt at kms_pipe_crc_basic@read-crc-pipe-c-frame-sequence
> +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_pipe_crc_basic@suspend-read-crc-pipe-b
> -igt at kms_pipe_crc_basic@suspend-read-crc-pipe-c
>  igt at kms_psr@primary_page_flip
>  igt at kms_psr@cursor_plane_move
>  igt at kms_psr@sprite_plane_onoff
>