From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by gabe.freedesktop.org (Postfix) with ESMTPS id CF06010E2C8 for ; Mon, 29 May 2023 14:40:48 +0000 (UTC) Message-ID: Date: Mon, 29 May 2023 20:10:45 +0530 MIME-Version: 1.0 Content-Language: en-US To: juhapekka.heikkila@gmail.com, igt-dev@lists.freedesktop.org References: <20230526064520.310041-1-swati2.sharma@intel.com> <6dc23a8c-c694-4577-55dd-2f21ef11595f@gmail.com> From: "Sharma, Swati2" In-Reply-To: <6dc23a8c-c694-4577-55dd-2f21ef11595f@gmail.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Subject: Re: [igt-dev] [PATCH i-g-t 1/3] tests/kms_cursor_crc: limit test to one pipe for sim List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: Thanks for the review. Pushed! On 29-May-23 6:05 PM, Juha-Pekka Heikkila wrote: > Reviewed-by: Juha-Pekka Heikkila > > On 26.5.2023 9.45, Swati Sharma wrote: >> To reduce execution time in simulation, lets limit test to >> one pipe only. >> >> Signed-off-by: Swati Sharma >> --- >>   tests/kms_cursor_crc.c | 4 ++++ >>   1 file changed, 4 insertions(+) >> >> diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c >> index 3a353a03..f837dbc4 100644 >> --- a/tests/kms_cursor_crc.c >> +++ b/tests/kms_cursor_crc.c >> @@ -728,6 +728,10 @@ static bool execution_constraint(enum pipe pipe) >>           pipe != active_pipes[last_pipe]) >>           return true; >> +    if (!extended && igt_run_in_simulation() && >> +        pipe != active_pipes[0]) >> +        return true; >> + >>       return false; >>   } >