Re: [PATCH i-g-t 1/2] lib/igt_kms: Prevent bigjoiner assignment if next pipe is already in use

[Karthik B S <karthik.b.s@intel.com>]

On 4/3/2025 4:11 PM, Jeevan B wrote:
> Added a check to prevent a bigjoiner pipe from using the next pipe
> if it's already assigned, avoiding conflicts and ensuring pipe
> allocation goes smoothly.
>
> Signed-off-by: Jeevan B <jeevan.b@intel.com>
Reviewed-by: Karthik B S <karthik.b.s@intel.com>
> ---
>   lib/igt_kms.c | 8 ++++++++
>   1 file changed, 8 insertions(+)
>
> diff --git a/lib/igt_kms.c b/lib/igt_kms.c
> index 99c8707c7..f3bc481f2 100644
> --- a/lib/igt_kms.c
> +++ b/lib/igt_kms.c
> @@ -6777,6 +6777,14 @@ bool igt_check_bigjoiner_support(igt_display_t *display)
>   				return false;
>   			}
>   
> +			for (int j = 0; j < pipes_in_use; j++) {
> +				if (pipes[j].idx == pipes[i].idx + 1) {
> +					igt_info("pipe-%s: Next pipe is already assigned to another output.\n",
> +						 kmstest_pipe_name(pipes[j].idx));
> +					return false;
> +				}
> +			}
> +
>   			if (!display->pipes[pipes[i].idx + 1].enabled) {
>   				igt_info("Consecutive pipe-%s: Fused-off, couldn't be used as a Bigjoiner Secondary.\n",
>   					 kmstest_pipe_name(display->pipes[pipes[i].idx + 1].pipe));