From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mgamail.intel.com (mgamail.intel.com [192.55.52.88]) by gabe.freedesktop.org (Postfix) with ESMTPS id A63DF10E28C for ; Thu, 14 Sep 2023 13:44:00 +0000 (UTC) Message-ID: <344efd54-9002-dce5-2ad2-b60706a6763e@intel.com> Date: Thu, 14 Sep 2023 19:13:39 +0530 Content-Language: en-US To: Stanislav Lisovskiy , References: <20230914080329.16648-1-stanislav.lisovskiy@intel.com> From: "Modem, Bhanuprakash" In-Reply-To: <20230914080329.16648-1-stanislav.lisovskiy@intel.com> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Subject: Re: [igt-dev] [PATCH] tests/intel/i915_pipe_stress: Use only first pipe if Bigjoiner is used List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: juha-pekka.heikkila@intel.com Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: Hi Stan, On Thu-14-09-2023 01:33 pm, Stanislav Lisovskiy wrote: > We currently are having test failures for Bigjoiner, because that > test tries to use all pipes simultaneously, not realizing that > if we use a Bigjoiner mode on pipe A, the adjacent pipe won't be > available. > For now just fix it by using only one pipe, if resolution > 3840 > is detected. > > Signed-off-by: Stanislav Lisovskiy > --- > tests/intel/i915_pipe_stress.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/tests/intel/i915_pipe_stress.c b/tests/intel/i915_pipe_stress.c > index 3cd84d027f..a050fa0117 100644 > --- a/tests/intel/i915_pipe_stress.c > +++ b/tests/intel/i915_pipe_stress.c > @@ -590,6 +590,12 @@ static void stress_pipes(struct data *data, struct timespec *start, > igt_pipe_crc_stop(data->pipe_crc[pipe]); > igt_assert_crc_equal(&crc, &crc2); > > + /* > + * Do not try to use other pipes, if Bigjoiner is used > + */ > + if (data->highest_mode[pipe]->hdisplay > 3840) > + break; I think, you can use "igt_check_bigjoiner_support()" to know the selected pipe can drive the Bigjoiner or not. - Bhanu > + > ++pipe; > } > }