[PATCH i-g-t] tests/intel/kms_pipe_stress: Skip invalid pipe-output combinations

Karthik B S karthik.b.s at intel.com
Fri Jul 25 04:35:04 UTC 2025


Hi Jeevan,

On 7/11/2025 2:50 PM, Jeevan B wrote:
> Add intel_pipe_output_combo_valid check to skip invalid pipe-output
> combinations which are not valid.
>
> Signed-off-by: Jeevan B <jeevan.b at intel.com>
> ---
>   tests/intel/kms_pipe_stress.c | 5 +++++
>   1 file changed, 5 insertions(+)
>
> diff --git a/tests/intel/kms_pipe_stress.c b/tests/intel/kms_pipe_stress.c
> index 296c93b58..a896c830b 100644
> --- a/tests/intel/kms_pipe_stress.c
> +++ b/tests/intel/kms_pipe_stress.c
> @@ -578,6 +578,11 @@ static void stress_pipes(struct data *data, struct timespec *start,
>   		if (!data->highest_mode[pipe])
>   			continue;
>   
> +		if (!intel_pipe_output_combo_valid(&data->display)) {
> +			igt_output_set_pipe(output, PIPE_NONE);
> +			continue;
> +		}
> +

Since this is a stress test we wouldn't want to skip one output here 
incase we don't find a valid combo. Can we add the logic to try on the 
next pipe here so that we don't skip the output completely?

Regards,
Karthik.B.S
>   		igt_assert_f(data->display.pipes[pipe].n_planes < MAX_PLANES,
>   			    "Currently we don't support more than %d planes!",
>   			     MAX_PLANES);


More information about the igt-dev mailing list