RE: [PATCH i-g-t v1 13/16] tests/kms_sharpness_filter: limit #crtcs tested during suspend

> -----Original Message-----
> From: igt-dev <[email protected]> On Behalf Of Michał
> Grzelak
> Sent: Monday, July 13, 2026 2:25 PM
> To: [email protected]
> Cc: Thasleem, Mohammed <[email protected]>; Grzelak,
> Michal <[email protected]>
> Subject: [PATCH i-g-t v1 13/16] tests/kms_sharpness_filter: limit #crtcs tested
> during suspend
> 
> We are iterating through each crtc while testing suspend. This results in
> extended execution time. In turn this leads to hitting CI enforced timeout.
> 
> Limit number of tested crtcs to first and last. Test each crtc when passing
> commandline parameter.
> 
> v1->v2
> - remove unused vars (Jani)
> 
> Signed-off-by: Michał Grzelak <[email protected]>
> ---
>  tests/intel/kms_sharpness_filter.c | 13 +++++++++++--
>  1 file changed, 11 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/intel/kms_sharpness_filter.c
> b/tests/intel/kms_sharpness_filter.c
> index adec4a60c5..a8578cf4fc 100644
> --- a/tests/intel/kms_sharpness_filter.c
> +++ b/tests/intel/kms_sharpness_filter.c
> @@ -233,6 +233,7 @@ static const struct subtest_entry {  typedef struct {
>  	int drm_fd;
>  	bool limited;
> +	bool all_crtcs;
>  	bool all_outputs;
>  	struct igt_fb fb[4];
>  	igt_crtc_t *crtc;
> @@ -635,6 +636,10 @@ run_sharpness_filter_test(data_t *data, enum
> test_type type)
> 
>  			igt_output_set_crtc(data->output, data->crtc);
> 
> +			if (type == TEST_FILTER_SUSPEND && !data->all_crtcs
> && crtc->crtc_index != 0 &&
> +						crtc->crtc_index != display-
> >n_crtcs-1)
> +					continue;
> +
>  			/*
>  			 * FIXME: Joiner + CASF currently unsupported.
>  			 * Remove this check once support is implemented.
> @@ -776,6 +781,9 @@ static int opt_handler(int opt, int opt_index, void
> *_data)
>  	case 'l':
>  		data->limited = true;
>  		break;
> +	case 'c':
> +		data->all_crtcs = true;
> +		break;
>  	case 'o':
>  		data->all_outputs = true;
>  		break;
> @@ -788,11 +796,12 @@ static int opt_handler(int opt, int opt_index, void
> *_data)
> 
>  static const char help_str[] =
>  	"  --limited|-l\t\tLimit execution to 1 valid pipe-output combo\n"
> -	"  --all-outputs|-o\t\tExtend suspend tests for all outputs\n";
> +	"  --all-outputs|-o\t\tExtend suspend tests for all outputs\n"
> +	"  --all-crtcs|-c\t\tExtend suspend tests for all crtcs\n";
> 
>  data_t data = {};
> 
> -int igt_main_args("lo", NULL, help_str, opt_handler, &data)
> +int igt_main_args("loc", NULL, help_str, opt_handler, &data)
>  {
>  	igt_fixture() {
>  		data.drm_fd = drm_open_driver_master(DRIVER_ANY);
> --
LGTM,
Reviewed-by: Nemesa Garg <[email protected]>

> 2.45.2