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 2C2B1D1F9B4 for ; Thu, 4 Dec 2025 10:48:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Type:MIME-Version: References:In-Reply-To:Subject:Cc:To:From:Message-ID:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=sxMPTWjgR6d2V/IkoxyCBbe+JrmSAFsIpL8uOa4nkMI=; b=JruBqB9N40E+YIrooe5Y6qE1PK VxQbotJ6E00HsNvyTdH/ftNNTVwQx272t/cU9hasvbn21eMCfpXczz5dLPSnz5eoGm1WMGp76SP/g CCgR0S5oZEY2eU4jen8x8gN0yfes8fqEWbFqHDePc5+xYy3KfB0yWTXcDzzoQA+OvjcHplA/viGY+ Xs9OkahO5S9mRq1odagI/n7NBvni/b0qy5bh9qVz/Wz8BhluIOb8Spfs+xcB0vs8QO5CN6S62XHUV vxZyQNSVQVogtg7OgIeqsdC6bLkFP9ASuaWp7ta524XQmMEVBukHr0lHJDd6Oj47JSLnttwcWxwGL QSQOGluQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1vR6tC-00000007rto-3js2; Thu, 04 Dec 2025 10:48:47 +0000 Received: from tor.source.kernel.org ([2600:3c04:e001:324:0:1991:8:25]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1vR6tB-00000007rtc-3Sw1 for linux-arm-kernel@lists.infradead.org; Thu, 04 Dec 2025 10:48:45 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by tor.source.kernel.org (Postfix) with ESMTP id EC7A060192; Thu, 4 Dec 2025 10:48:44 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 959CEC4CEFB; Thu, 4 Dec 2025 10:48:44 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1764845324; bh=5/oknxb9c2y6+JA43OWmQfGTtidST836yMTAIMN8Zu8=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=m5YYNaFRvIOCNKGDHnAF8/Lkp3EUc6SQr8o1TaZqRRMMos2/UEC5xBSuemSTNwWat fJs1ELOTTGx+IxF9nApKew2gF737Nx85S2jQ9m7F628fswmJ+2ZBmlbL6EOPtlvHya KccSRP5TCJBBn+lkvGQaXwPVgxI3As3ZKflSOJUwBAooBvByyCU52Bo5nBcjPiv7lh SpyMIrLxuJ+j2mcudgj9F+XMNN1J5xE1RgqEwvv1bWqSSrH75wpMnfx9d8pqPFvNlT 5w1W5rw9nbzVTeCyGIyheyjwe9PrZphd8cTmqOOunB2p9eCp7+UWp9Fp7c0WIWL6zz YqdSbfURoHEkg== 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.98.2) (envelope-from ) id 1vR6t8-0000000AQD6-2XFp; Thu, 04 Dec 2025 10:48:42 +0000 Date: Thu, 04 Dec 2025 10:48:42 +0000 Message-ID: <86cy4uplo5.wl-maz@kernel.org> From: Marc Zyngier To: Ben Horgan Cc: kvmarm@lists.linux.dev, kvm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Joey Gouly , Suzuki K Poulose , Oliver Upton , Zenghui Yu , Yao Yuan Subject: Re: [PATCH v3 1/9] arm64: Repaint ID_AA64MMFR2_EL1.IDS description In-Reply-To: References: <20251204094806.3846619-1-maz@kernel.org> <20251204094806.3846619-2-maz@kernel.org> 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/30.1 (aarch64-unknown-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=US-ASCII X-SA-Exim-Connect-IP: 185.219.108.64 X-SA-Exim-Rcpt-To: ben.horgan@arm.com, kvmarm@lists.linux.dev, kvm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, joey.gouly@arm.com, suzuki.poulose@arm.com, oupton@kernel.org, yuzenghui@huawei.com, yaoyuan@linux.alibaba.com X-SA-Exim-Mail-From: maz@kernel.org X-SA-Exim-Scanned: No (on disco-boy.misterjones.org); SAEximRunCond expanded to false 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Thu, 04 Dec 2025 10:36:54 +0000, Ben Horgan wrote: > > Hi Marc, > > On 12/4/25 09:47, Marc Zyngier wrote: > > ID_AA64MMFR2_EL1.IDS, as described in the sysreg file, is pretty horrible > > as it diesctly give the ESR value. Repaint it using the usual NI/IMP > > identifiers to describe the absence/presence of FEAT_IDST. > > > > Signed-off-by: Marc Zyngier > > --- > > arch/arm64/kvm/hyp/nvhe/sys_regs.c | 2 +- > > arch/arm64/tools/sysreg | 4 ++-- > > 2 files changed, 3 insertions(+), 3 deletions(-) > > > > diff --git a/arch/arm64/kvm/hyp/nvhe/sys_regs.c b/arch/arm64/kvm/hyp/nvhe/sys_regs.c > > index 82da9b03692d4..107d62921b168 100644 > > --- a/arch/arm64/kvm/hyp/nvhe/sys_regs.c > > +++ b/arch/arm64/kvm/hyp/nvhe/sys_regs.c > > @@ -134,7 +134,7 @@ static const struct pvm_ftr_bits pvmid_aa64mmfr2[] = { > > MAX_FEAT(ID_AA64MMFR2_EL1, UAO, IMP), > > MAX_FEAT(ID_AA64MMFR2_EL1, IESB, IMP), > > MAX_FEAT(ID_AA64MMFR2_EL1, AT, IMP), > > - MAX_FEAT_ENUM(ID_AA64MMFR2_EL1, IDS, 0x18), > > + MAX_FEAT_ENUM(ID_AA64MMFR2_EL1, IDS, IMP), > > MAX_FEAT(ID_AA64MMFR2_EL1, TTL, IMP), > > MAX_FEAT(ID_AA64MMFR2_EL1, BBM, 2), > > MAX_FEAT(ID_AA64MMFR2_EL1, E0PD, IMP), > > diff --git a/arch/arm64/tools/sysreg b/arch/arm64/tools/sysreg > > index 1c6cdf9d54bba..3261e8791ac03 100644 > > --- a/arch/arm64/tools/sysreg > > +++ b/arch/arm64/tools/sysreg > > @@ -2257,8 +2257,8 @@ UnsignedEnum 43:40 FWB > > 0b0001 IMP > > EndEnum > > Enum 39:36 IDS > > Should this also be changed to an UnsignedEnum? I'm not sure this brings much when you only have two values. If IDS was growing a third value, and that there was an actual order in the numbering scheme, then yes, that'd be useful. But at this stage, I'm not confident that this is desirable, let alone necessary. Thanks, M. -- Without deviation from the norm, progress is not possible.