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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (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 4520ECDB47E for ; Wed, 18 Oct 2023 12:41:07 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Subject:Cc:To:From:Message-ID:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=3fDvJcEJ4OfWybdVJCYhmNA6rVX2kaSGdr8mOhorAxU=; b=oCF0l6Mi8/Vqal L6s9QP9s/YpzPLkyHNAO9VqWmdWHb7izHa5RpPCcHWI8G1T2ooie1LYds8cGE6/YTc3DSbrmfNdFX 0XXtnT1anGL+BjyEkw91Rp2xQ2XTpUfrR/wWM0McdgnzCr4Y2Hibh+OyyEGKI7J7+GUWj65q3XCDa yRjn4QDyXCihc9UZS2wOj4qU2faMhqSeMFWe1BEaiSZZdu150e5KrmyAH6y97KnpjNgCE0LLp1lVo PTD8/rMDP9lVYL7NcJXTU/yoBkVt7/cesqUVsofarx+Wg2V2LY5Ead9WmxqU6WGzMU1l+LVG2022x 9YcjYdlSbOVwpAyjinWw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qt5qv-00Eghs-1A; Wed, 18 Oct 2023 12:40:45 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qt5qs-00EggT-1m for linux-arm-kernel@lists.infradead.org; Wed, 18 Oct 2023 12:40:44 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id D2DDF61775; Wed, 18 Oct 2023 12:40:41 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id BD5D8C433CA; Wed, 18 Oct 2023 12:40:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1697632840; bh=ANCj0Q2OJ1qzkkQO+Sp4XFTRcw4ECnG/L7DzjhbHY0o=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=PnhgzYZkM/eyLI9CA2rHMDopAQgkqfFT1BRiLbSCJoEyPGzAsGy7SKqrTfX89CTVB pE1qCMR/gLq0c7TJ/JQbahHI+QBWuRfY8qbsL8/u6wI4yYrP9eWrQ8nQyizWPxEdR/ d9GOWpiXjJrCe0mV4YeAYuzCtUbxyb1nqjxSo5yBtCEysiG4Ucjwalxw8kxwO6wqJF U3GIIf0c4YHHMmfKdrVigxcM7U+4QOKPuNK4wvPRh1rnDIU1OaKs2u2oE8tiN7y3tn HtHsjD69uI1pKgYtuVdEdJPQ4SUMvEpJUQhWr4zBmwz+6NE/0qzxjOUl5u6QII5smk hLXYiIrqfAPVw== Received: from sofa.misterjones.org ([185.219.108.64] helo=goblin-girl.misterjones.org) by disco-boy.misterjones.org with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.95) (envelope-from ) id 1qt5qo-005LoC-6Y; Wed, 18 Oct 2023 13:40:38 +0100 Date: Wed, 18 Oct 2023 13:40:37 +0100 Message-ID: <86r0lsm7cq.wl-maz@kernel.org> From: Marc Zyngier To: Anshuman Khandual Cc: linux-arm-kernel@lists.infradead.org, Catalin Marinas , Will Deacon , Oliver Upton , linux-kernel@vger.kernel.org Subject: Re: [PATCH] arm64: Independently update HDFGRTR_EL2 and HDFGWTR_EL2 In-Reply-To: <20231018030007.1968317-1-anshuman.khandual@arm.com> References: <20231018030007.1968317-1-anshuman.khandual@arm.com> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI-EPG/1.14.7 (Harue) FLIM-LB/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL-LB/10.8 EasyPG/1.0.0 Emacs/29.1 (aarch64-unknown-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") X-SA-Exim-Connect-IP: 185.219.108.64 X-SA-Exim-Rcpt-To: anshuman.khandual@arm.com, linux-arm-kernel@lists.infradead.org, catalin.marinas@arm.com, will@kernel.org, oliver.upton@linux.dev, linux-kernel@vger.kernel.org X-SA-Exim-Mail-From: maz@kernel.org X-SA-Exim-Scanned: No (on disco-boy.misterjones.org); SAEximRunCond expanded to false X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231018_054042_675267_4947F18F X-CRM114-Status: GOOD ( 26.96 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Wed, 18 Oct 2023 04:00:07 +0100, Anshuman Khandual wrote: > > Currently PMSNEVFR_EL1 system register read, and write access EL2 traps are > disabled, via setting the same bit (i.e 62) in HDFGRTR_EL2, and HDFGWTR_EL2 > respectively. Although very similar, bit fields are not exact same in these > two EL2 trap configure registers particularly when it comes to read-only or > write-only accesses such as ready-only 'HDFGRTR_EL2.nBRBIDR' which needs to > be set while enabling BRBE on NVHE platforms. Using the exact same bit mask > fields for both these trap register risk writing into their RESERVED areas, > which is undesirable. Sorry, I don't understand at all what you are describing. You seem to imply that the read and write effects of the FGT doesn't apply the same way. But my reading of the ARM ARM is that behave completely symmetrically. Also, what is nBRBIDR doing here? It is still set to 0. What 'RESERVED' state are you talking about? > > Cc: Catalin Marinas > Cc: Will Deacon > Cc: Oliver Upton > Cc: Marc Zyngier > Cc: linux-arm-kernel@lists.infradead.org > Cc: linux-kernel@vger.kernel.org > Signed-off-by: Anshuman Khandual > --- > This applies on v6.6-rc6. > > I guess it should be okay to use 'x2' as it is in the clobbered register > list for init_el2_state() function. But please do let me know otherwise. > > arch/arm64/include/asm/el2_setup.h | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/arch/arm64/include/asm/el2_setup.h b/arch/arm64/include/asm/el2_setup.h > index 899b5c10f84c..c534afb1a30d 100644 > --- a/arch/arm64/include/asm/el2_setup.h > +++ b/arch/arm64/include/asm/el2_setup.h > @@ -206,16 +206,19 @@ > cbz x1, .Lskip_fgt_\@ > > mov x0, xzr > + mov x2, xzr > mrs x1, id_aa64dfr0_el1 > ubfx x1, x1, #ID_AA64DFR0_EL1_PMSVer_SHIFT, #4 > cmp x1, #3 > b.lt .Lset_debug_fgt_\@ > + > /* Disable PMSNEVFR_EL1 read and write traps */ > - orr x0, x0, #(1 << 62) > + orr x0, x0, #HDFGRTR_EL2_nPMSNEVFR_EL1_MASK > + orr x2, x2, #HDFGWTR_EL2_nPMSNEVFR_EL1_MASK > > .Lset_debug_fgt_\@: > msr_s SYS_HDFGRTR_EL2, x0 > - msr_s SYS_HDFGWTR_EL2, x0 > + msr_s SYS_HDFGWTR_EL2, x2 So what has changed here, aside from clobbering an extra register? The masks are the same, the initial values are the same... Is it in preparation for some other work? /me puzzled. M. -- Without deviation from the norm, progress is not possible. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel