From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marc Zyngier Subject: Re: [PATCH] arm64: KVM: Add 48bit PA support for EL2 translations Date: Sat, 6 Feb 2016 12:05:24 +0000 Message-ID: <20160206120524.09b9f893@arm.com> References: <1454700496-21221-1-git-send-email-tchalamarla@caviumnetworks.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 8BIT Return-path: In-Reply-To: <1454700496-21221-1-git-send-email-tchalamarla@caviumnetworks.com> Sender: kvm-owner@vger.kernel.org To: tchalamarla@caviumnetworks.com Cc: christoffer.dall@linaro.org, kvmarm@lists.cs.columbia.edu, linux-arm-kernel@lists.infradead.org, kvm@vger.kernel.org List-Id: kvmarm@lists.cs.columbia.edu Tirumalesh, On Fri, 5 Feb 2016 11:28:16 -0800 wrote: > From: Tirumalesh Chalamarla > > Systems where RAM is at higher address and MultiNode systems What influence a "MultiNode" (for whatever definition of node you have) system has on the memory map? > whith higher address RAM, 39 bit max PA for EL2 translations > is not sufficient. I read it as 40 bits, not 39. > > This patch sets max possible PA(48bit) for EL2 translations. >>From the ARM ARM: "If {I}PS is programmed to a value larger than the implemented physical address size, then the PE behaves as if programmed with the implemented physical address size, but software must not rely on this behavior. That is, the output address size is never larger than the implemented physical address size." > Signed-off-by: Tirumalesh Chalamarla > --- > arch/arm64/include/asm/kvm_arm.h | 10 +++++++++- > 1 file changed, 9 insertions(+), 1 deletion(-) > > diff --git a/arch/arm64/include/asm/kvm_arm.h b/arch/arm64/include/asm/kvm_arm.h > index 738a95f..3fc38a4 100644 > --- a/arch/arm64/include/asm/kvm_arm.h > +++ b/arch/arm64/include/asm/kvm_arm.h > @@ -98,7 +98,12 @@ > #define TCR_EL2_RES1 ((1 << 31) | (1 << 23)) > #define TCR_EL2_TBI (1 << 20) > #define TCR_EL2_PS (7 << 16) > +#define TCR_EL2_PS_32B (0 << 16) > +#define TCR_EL2_PS_36B (1 << 16) > #define TCR_EL2_PS_40B (2 << 16) > +#define TCR_EL2_PS_42B (3 << 16) > +#define TCR_EL2_PS_44B (4 << 16) > +#define TCR_EL2_PS_48B (5 << 16) > #define TCR_EL2_TG0 (1 << 14) > #define TCR_EL2_SH0 (3 << 12) > #define TCR_EL2_ORGN0 (3 << 10) > @@ -107,7 +112,10 @@ > #define TCR_EL2_MASK (TCR_EL2_TG0 | TCR_EL2_SH0 | \ > TCR_EL2_ORGN0 | TCR_EL2_IRGN0 | TCR_EL2_T0SZ) > > -#define TCR_EL2_FLAGS (TCR_EL2_RES1 | TCR_EL2_PS_40B) > +/* > + * Assign the highest possible PS. > + */ > +#define TCR_EL2_FLAGS (TCR_EL2_RES1 | TCR_EL2_PS_48B) > > /* VTCR_EL2 Registers bits */ > #define VTCR_EL2_RES1 (1 << 31) So this is a NAK. If you want to address this problem, do it properly (hint: look at how we deal with VTCR_EL2). Thanks, M. -- Jazz is not dead. It just smells funny. From mboxrd@z Thu Jan 1 00:00:00 1970 From: marc.zyngier@arm.com (Marc Zyngier) Date: Sat, 6 Feb 2016 12:05:24 +0000 Subject: [PATCH] arm64: KVM: Add 48bit PA support for EL2 translations In-Reply-To: <1454700496-21221-1-git-send-email-tchalamarla@caviumnetworks.com> References: <1454700496-21221-1-git-send-email-tchalamarla@caviumnetworks.com> Message-ID: <20160206120524.09b9f893@arm.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Tirumalesh, On Fri, 5 Feb 2016 11:28:16 -0800 wrote: > From: Tirumalesh Chalamarla > > Systems where RAM is at higher address and MultiNode systems What influence a "MultiNode" (for whatever definition of node you have) system has on the memory map? > whith higher address RAM, 39 bit max PA for EL2 translations > is not sufficient. I read it as 40 bits, not 39. > > This patch sets max possible PA(48bit) for EL2 translations. >>From the ARM ARM: "If {I}PS is programmed to a value larger than the implemented physical address size, then the PE behaves as if programmed with the implemented physical address size, but software must not rely on this behavior. That is, the output address size is never larger than the implemented physical address size." > Signed-off-by: Tirumalesh Chalamarla > --- > arch/arm64/include/asm/kvm_arm.h | 10 +++++++++- > 1 file changed, 9 insertions(+), 1 deletion(-) > > diff --git a/arch/arm64/include/asm/kvm_arm.h b/arch/arm64/include/asm/kvm_arm.h > index 738a95f..3fc38a4 100644 > --- a/arch/arm64/include/asm/kvm_arm.h > +++ b/arch/arm64/include/asm/kvm_arm.h > @@ -98,7 +98,12 @@ > #define TCR_EL2_RES1 ((1 << 31) | (1 << 23)) > #define TCR_EL2_TBI (1 << 20) > #define TCR_EL2_PS (7 << 16) > +#define TCR_EL2_PS_32B (0 << 16) > +#define TCR_EL2_PS_36B (1 << 16) > #define TCR_EL2_PS_40B (2 << 16) > +#define TCR_EL2_PS_42B (3 << 16) > +#define TCR_EL2_PS_44B (4 << 16) > +#define TCR_EL2_PS_48B (5 << 16) > #define TCR_EL2_TG0 (1 << 14) > #define TCR_EL2_SH0 (3 << 12) > #define TCR_EL2_ORGN0 (3 << 10) > @@ -107,7 +112,10 @@ > #define TCR_EL2_MASK (TCR_EL2_TG0 | TCR_EL2_SH0 | \ > TCR_EL2_ORGN0 | TCR_EL2_IRGN0 | TCR_EL2_T0SZ) > > -#define TCR_EL2_FLAGS (TCR_EL2_RES1 | TCR_EL2_PS_40B) > +/* > + * Assign the highest possible PS. > + */ > +#define TCR_EL2_FLAGS (TCR_EL2_RES1 | TCR_EL2_PS_48B) > > /* VTCR_EL2 Registers bits */ > #define VTCR_EL2_RES1 (1 << 31) So this is a NAK. If you want to address this problem, do it properly (hint: look@how we deal with VTCR_EL2). Thanks, M. -- Jazz is not dead. It just smells funny. From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marc Zyngier Subject: Re: [PATCH] arm64: KVM: Add 48bit PA support for EL2 translations Date: Sat, 6 Feb 2016 12:05:24 +0000 Message-ID: <20160206120524.09b9f893@arm.com> References: <1454700496-21221-1-git-send-email-tchalamarla@caviumnetworks.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 8BIT Cc: , , , To: Return-path: Received: from foss.arm.com ([217.140.101.70]:54413 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751986AbcBFMFe convert rfc822-to-8bit (ORCPT ); Sat, 6 Feb 2016 07:05:34 -0500 In-Reply-To: <1454700496-21221-1-git-send-email-tchalamarla@caviumnetworks.com> Sender: kvm-owner@vger.kernel.org List-ID: Tirumalesh, On Fri, 5 Feb 2016 11:28:16 -0800 wrote: > From: Tirumalesh Chalamarla > > Systems where RAM is at higher address and MultiNode systems What influence a "MultiNode" (for whatever definition of node you have) system has on the memory map? > whith higher address RAM, 39 bit max PA for EL2 translations > is not sufficient. I read it as 40 bits, not 39. > > This patch sets max possible PA(48bit) for EL2 translations. >>From the ARM ARM: "If {I}PS is programmed to a value larger than the implemented physical address size, then the PE behaves as if programmed with the implemented physical address size, but software must not rely on this behavior. That is, the output address size is never larger than the implemented physical address size." > Signed-off-by: Tirumalesh Chalamarla > --- > arch/arm64/include/asm/kvm_arm.h | 10 +++++++++- > 1 file changed, 9 insertions(+), 1 deletion(-) > > diff --git a/arch/arm64/include/asm/kvm_arm.h b/arch/arm64/include/asm/kvm_arm.h > index 738a95f..3fc38a4 100644 > --- a/arch/arm64/include/asm/kvm_arm.h > +++ b/arch/arm64/include/asm/kvm_arm.h > @@ -98,7 +98,12 @@ > #define TCR_EL2_RES1 ((1 << 31) | (1 << 23)) > #define TCR_EL2_TBI (1 << 20) > #define TCR_EL2_PS (7 << 16) > +#define TCR_EL2_PS_32B (0 << 16) > +#define TCR_EL2_PS_36B (1 << 16) > #define TCR_EL2_PS_40B (2 << 16) > +#define TCR_EL2_PS_42B (3 << 16) > +#define TCR_EL2_PS_44B (4 << 16) > +#define TCR_EL2_PS_48B (5 << 16) > #define TCR_EL2_TG0 (1 << 14) > #define TCR_EL2_SH0 (3 << 12) > #define TCR_EL2_ORGN0 (3 << 10) > @@ -107,7 +112,10 @@ > #define TCR_EL2_MASK (TCR_EL2_TG0 | TCR_EL2_SH0 | \ > TCR_EL2_ORGN0 | TCR_EL2_IRGN0 | TCR_EL2_T0SZ) > > -#define TCR_EL2_FLAGS (TCR_EL2_RES1 | TCR_EL2_PS_40B) > +/* > + * Assign the highest possible PS. > + */ > +#define TCR_EL2_FLAGS (TCR_EL2_RES1 | TCR_EL2_PS_48B) > > /* VTCR_EL2 Registers bits */ > #define VTCR_EL2_RES1 (1 << 31) So this is a NAK. If you want to address this problem, do it properly (hint: look at how we deal with VTCR_EL2). Thanks, M. -- Jazz is not dead. It just smells funny.