[igt-dev] [PATCH i-g-t 1/1] test/kms_plane_scaling: Don't run scaler clip/clamp with all formats

[Kahola, Mika]

> -----Original Message-----
> From: Khajapasha, Mohammed <mohammed.khajapasha at intel.com>
> Sent: Tuesday, March 23, 2021 2:30 PM
> To: igt-dev at lists.freedesktop.org
> Cc: Kahola, Mika <mika.kahola at intel.com>; juhapekka.heikkila at gmail.com
> Subject: [PATCH i-g-t 1/1] test/kms_plane_scaling: Don't run scaler clip/clamp
> with all formats
> 
> Don't run plane scaler tests with all formats for pipe C & D.
> We probably don't need to test plane scaler tests with each format for pipe C
> & D.
> 
> $ time ./build/tests/kms_plane_scaling
> - real	6m53,143s
> + real	4m39,406s
> 
> Cc: juhapekka.heikkila at gmail.com
> Cc: Mika Kahola <mika.kahola at intel.com>
> Signed-off-by: Mohammed Khajapasha
> <mohammed.khajapasha at intel.com>

Reviewed-by: Mika Kahola <mika.kahola at intel.com>

> ---
>  tests/kms_plane_scaling.c | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)
> 
> diff --git a/tests/kms_plane_scaling.c b/tests/kms_plane_scaling.c index
> c63eb428..31f14275 100644
> --- a/tests/kms_plane_scaling.c
> +++ b/tests/kms_plane_scaling.c
> @@ -230,6 +230,18 @@ static bool test_format(data_t *data,
>  	return true;
>  }
> 
> +static bool test_pipe_iteration(data_t *data, enum pipe pipe, int
> +iteration) {
> +	if (!is_i915_device(data->drm_fd) ||
> +	    data->extended)
> +		return true;
> +
> +	if ((pipe > PIPE_B) && (iteration >= 2))
> +		return false;
> +
> +	return true;
> +}
> +
>  static void test_scaler_with_rotation_pipe(data_t *d, enum pipe pipe,
>  					   igt_output_t *output)
>  {
> @@ -532,6 +544,9 @@ test_scaler_with_clipping_clamping_scenario(data_t
> *d, enum pipe pipe, igt_outpu
>  		unsigned f1 = d->plane1->drm_plane->formats[i];
>  		struct igt_vec tested_formats2;
> 
> +		if (!test_pipe_iteration(d, pipe, i))
> +			continue;
> +
>  		if (!test_format(d, &tested_formats1, f1) ||
>  		    !can_scale(d, f1))
>  			continue;
> --
> 2.29.2.154.g7f7ebe054a