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 Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 0BCE4C7EE22 for ; Wed, 17 May 2023 20:09:17 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 77CB010E46C; Wed, 17 May 2023 20:09:16 +0000 (UTC) Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by gabe.freedesktop.org (Postfix) with ESMTPS id 3EB5610E46C for ; Wed, 17 May 2023 20:09:14 +0000 (UTC) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id F26D363A18; Wed, 17 May 2023 20:09:12 +0000 (UTC) Received: from rdvivi-mobl4 (fmdmzpr02-ext.fm.intel.com [192.55.54.37]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.kernel.org (Postfix) with ESMTPSA id 73C88C4339B; Wed, 17 May 2023 20:09:10 +0000 (UTC) Date: Wed, 17 May 2023 16:09:06 -0400 From: Rodrigo Vivi To: "Das, Nirmoy" Message-ID: References: <20230517-mtl_disable_render_pg-v1-1-6495eebbfb24@intel.com> <2b4a54d2-e323-9a10-1ba4-3d2e96197bb9@linux.intel.com> <300d9f0c-f74f-55f9-40b4-13b65c3b4584@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <300d9f0c-f74f-55f9-40b4-13b65c3b4584@linux.intel.com> Subject: Re: [Intel-gfx] [PATCH] drm/i915/mtl: do not enable render power-gating on MTL X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: "intel-gfx@lists.freedesktop.org" , "Hajda, Andrzej" , "Vivi, Rodrigo" , "Das, Nirmoy" Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" On Wed, May 17, 2023 at 05:36:33PM +0200, Das, Nirmoy wrote: > > On 5/17/2023 5:25 PM, Vivi, Rodrigo wrote: > > On Wed, 2023-05-17 at 17:12 +0200, Das, Nirmoy wrote: > > > On 5/17/2023 3:59 PM, Andrzej Hajda wrote: > > > > Multiple CI tests fails with forcewake ack timeouts > > > > if render power gating is enabled. > > > > BSpec 52698 clearly states it should be 0 for MTL. > > > > > > > > Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/4983 > > > > Signed-off-by: Andrzej Hajda > > > > --- > > > >   drivers/gpu/drm/i915/gt/intel_rc6.c | 5 +++-- > > > >   1 file changed, 3 insertions(+), 2 deletions(-) > > > > > > > > diff --git a/drivers/gpu/drm/i915/gt/intel_rc6.c > > > > b/drivers/gpu/drm/i915/gt/intel_rc6.c > > > > index 908a3d0f2343f4..ebb2373dd73640 100644 > > > > --- a/drivers/gpu/drm/i915/gt/intel_rc6.c > > > > +++ b/drivers/gpu/drm/i915/gt/intel_rc6.c > > > > @@ -117,8 +117,9 @@ static void gen11_rc6_enable(struct intel_rc6 > > > > *rc6) > > > >                         GEN6_RC_CTL_RC6_ENABLE | > > > >                         GEN6_RC_CTL_EI_MODE(1); > > > > -       /* Wa_16011777198 - Render powergating must remain disabled > > > > */ > > > > -       if (IS_DG2_GRAPHICS_STEP(gt->i915, G10, STEP_A0, STEP_C0) > > > > || > > > > +       /* Wa_16011777198 and BSpec 52698 - Render powergating must > > > > be off */ > > > Nice catch! > > Indeed! What a mess in the workaround database. > > It is telling us that no_impact on MTL SKUs while we clearly needs > > that. I tried to reopen that and get that fixed in the hsds. > > > > > > > instead of bspec you could add Wa_14012655556. > > not actually. > > 16011777198 is the right lineage number for 14012655556. > > Besides, 14012655556 is for DG2 anyway. > > > > Let's keep the way Adrzej put with the BSPec reference besides the > > lineage. > > Makes sense, didn't realize 14012655556  is much older. > > Thanks! > > > > > > > > > > +       if (IS_METEORLAKE(gt->i915) || > > > > +           IS_DG2_GRAPHICS_STEP(gt->i915, G10, STEP_A0, STEP_C0) > > > > || > > > >             IS_DG2_GRAPHICS_STEP(gt->i915, G11, STEP_A0, STEP_B0)) > > > >                 pg_enable = > > > >                         GEN9_MEDIA_PG_ENABLE | > > > > > > > > --- > > > > base-commit: 01d3dd92d1b71421f6ee85e1bea829e0a917d979 > > > > change-id: 20230517-mtl_disable_render_pg-b9f9f1567f9e > > > ^ unwanted artifacts ?   Otherwise this looks good to me. > > > > > > Reviewed-by: Nirmoy Das > > with the artifacts removed: > > Reviewed-by: Rodrigo Vivi Folks, please do not merge this patch. At least not as it is right now... We need to root cause this. The hw bug related to this workaround was really fixed and this workaround should not be needed in MTL. We need to find the root cause instead of masking it here. Or at least merge as temporary FIXME/XXX and then work to get the root cause... The BSPec will get updated to remove the MTL mention there. Sorry about that. > > > > > > > > Best regards,