From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from out3-smtp.messagingengine.com (out3-smtp.messagingengine.com [66.111.4.27]) by gabe.freedesktop.org (Postfix) with ESMTPS id B76CA10ED7C for ; Thu, 16 Mar 2023 17:39:44 +0000 (UTC) Message-ID: Date: Thu, 16 Mar 2023 10:39:39 -0700 MIME-Version: 1.0 Content-Language: en-US To: Zack Rusin , igt-dev@lists.freedesktop.org References: <20230316022659.73202-1-zack@kde.org> <20230316022659.73202-9-zack@kde.org> From: "Maaz Mombasawala (VMware)" In-Reply-To: <20230316022659.73202-9-zack@kde.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [igt-dev] [PATCH i-g-t v2 8/8] lib/igt_kms: vmwgfx returns 8 crtc List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: krastevm@vmware.com, mombasawalam@vmware.com, petri.latvala@intel.com, banackm@vmware.com Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: LGTM! Reviewed-by: Maaz Mombasawala -- Maaz Mombasawala (VMware) On 3/15/23 19:26, Zack Rusin wrote: > From: Zack Rusin > > To avoid crashes in multiple tests we need to extend the number of > available pipe's to match the numbers returned by the virtualized drivers. > > Signed-off-by: Zack Rusin > --- > lib/igt_kms.c | 2 +- > lib/igt_kms.h | 2 ++ > 2 files changed, 3 insertions(+), 1 deletion(-) > > diff --git a/lib/igt_kms.c b/lib/igt_kms.c > index 8c7dd85b..d2e9eccc 100644 > --- a/lib/igt_kms.c > +++ b/lib/igt_kms.c > @@ -839,7 +839,7 @@ static igt_plane_t *igt_get_assigned_primary(igt_output_t *output, igt_pipe_t *p > */ > const char *kmstest_pipe_name(enum pipe pipe) > { > - static const char str[] = "A\0B\0C\0D\0E\0F"; > + static const char str[] = "A\0B\0C\0D\0E\0F\0G\0H"; > > _Static_assert(sizeof(str) == IGT_MAX_PIPES * 2, > "Missing pipe name"); > diff --git a/lib/igt_kms.h b/lib/igt_kms.h > index be5482e0..64807a7f 100644 > --- a/lib/igt_kms.h > +++ b/lib/igt_kms.h > @@ -62,6 +62,8 @@ enum pipe { > PIPE_D, > PIPE_E, > PIPE_F, > + PIPE_G, > + PIPE_H, > IGT_MAX_PIPES > }; > const char *kmstest_pipe_name(enum pipe pipe);