From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.7]) by gabe.freedesktop.org (Postfix) with ESMTPS id EE18810E07C for ; Thu, 21 Dec 2023 05:01:06 +0000 (UTC) Message-ID: <580ad5c1-40d3-4e31-b6b9-5ca3a28a053e@intel.com> Date: Thu, 21 Dec 2023 10:30:47 +0530 Subject: Re: [PATCH i-g-t 2/3] lib/igt_fb: With Intel blitter path ignore legacy rules on Xe Content-Language: en-US To: Juha-Pekka Heikkila , References: <20231220195940.3778662-1-juhapekka.heikkila@gmail.com> <20231220195940.3778662-2-juhapekka.heikkila@gmail.com> From: "Modem, Bhanuprakash" In-Reply-To: <20231220195940.3778662-2-juhapekka.heikkila@gmail.com> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit MIME-Version: 1.0 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: On 21-12-2023 01:29 am, Juha-Pekka Heikkila wrote: > If running with Xe device there's no need to care about legacy > size limitations, just say blitter is ok before checking for > legacy blitter size limits. > > Signed-off-by: Juha-Pekka Heikkila Reviewed-by: Bhanuprakash Modem > --- > lib/igt_fb.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/lib/igt_fb.c b/lib/igt_fb.c > index 683eb176b..06374efa0 100644 > --- a/lib/igt_fb.c > +++ b/lib/igt_fb.c > @@ -2513,6 +2513,9 @@ static bool blitter_ok(const struct igt_fb *fb) > is_gen12_mc_ccs_modifier(fb->modifier)) > return false; > > + if (is_xe_device(fb->fd)) > + return true; > + > for (int i = 0; i < fb->num_planes; i++) { > int width = fb->plane_width[i]; > From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Message-ID: <580ad5c1-40d3-4e31-b6b9-5ca3a28a053e@intel.com> Date: Thu, 21 Dec 2023 10:30:47 +0530 Subject: Re: [PATCH i-g-t 2/3] lib/igt_fb: With Intel blitter path ignore legacy rules on Xe Content-Language: en-US References: <20231220195940.3778662-1-juhapekka.heikkila@gmail.com> <20231220195940.3778662-2-juhapekka.heikkila@gmail.com> From: "Modem, Bhanuprakash" In-Reply-To: <20231220195940.3778662-2-juhapekka.heikkila@gmail.com> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit MIME-Version: 1.0 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: To: Juha-Pekka Heikkila , igt-dev@lists.freedesktop.org Message-ID: <20231221050047.nEZk4XvvKzTIgrKbrIVuESO_JTalTRdgmZz1X6BfLZY@z> On 21-12-2023 01:29 am, Juha-Pekka Heikkila wrote: > If running with Xe device there's no need to care about legacy > size limitations, just say blitter is ok before checking for > legacy blitter size limits. > > Signed-off-by: Juha-Pekka Heikkila Reviewed-by: Bhanuprakash Modem > --- > lib/igt_fb.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/lib/igt_fb.c b/lib/igt_fb.c > index 683eb176b..06374efa0 100644 > --- a/lib/igt_fb.c > +++ b/lib/igt_fb.c > @@ -2513,6 +2513,9 @@ static bool blitter_ok(const struct igt_fb *fb) > is_gen12_mc_ccs_modifier(fb->modifier)) > return false; > > + if (is_xe_device(fb->fd)) > + return true; > + > for (int i = 0; i < fb->num_planes; i++) { > int width = fb->plane_width[i]; >