Re: [Xen-devel] [PATCH 10/27] xen/arm: arm32: Don't define FAR_EL1
On 08/22/2017 03:37 PM, Andre Przywara wrote: Hi, Hi Andre, On 14/08/17 15:24, Julien Grall wrote: Aliasing FAR_EL1 to IFAR is wrong because on ARMv8 FAR_EL1[31:0] is architecturally mapped to DFAR and FAR_EL1[63:32] to DFAR. Should be IFAR, I guess? Yep. I will fix it. Please put a quid into the copy-and-paste piggy bank ;-) 2 quite with the copy-paste in patch #9 :). Cheers, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH 10/27] xen/arm: arm32: Don't define FAR_EL1
Hi, On 14/08/17 15:24, Julien Grall wrote: > Aliasing FAR_EL1 to IFAR is wrong because on ARMv8 FAR_EL1[31:0] is > architecturally mapped to DFAR and FAR_EL1[63:32] to DFAR. Should be IFAR, I guess? Please put a quid into the copy-and-paste piggy bank ;-) Otherwise it's fine. > As FAR_EL1 is not currently used in ARM32 code, remove it. > > Signed-off-by: Julien Grall Reviewed-by: Andre Przywara Cheers, Andre. > --- > xen/include/asm-arm/cpregs.h | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/xen/include/asm-arm/cpregs.h b/xen/include/asm-arm/cpregs.h > index 1889d7cbfb..9e138489f0 100644 > --- a/xen/include/asm-arm/cpregs.h > +++ b/xen/include/asm-arm/cpregs.h > @@ -306,7 +306,6 @@ > #define DACR32_EL2 DACR > #define ESR_EL1 DFSR > #define ESR_EL2 HSR > -#define FAR_EL1 HIFAR > #define HCR_EL2 HCR > #define HPFAR_EL2 HPFAR > #define HSTR_EL2HSTR > ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel
[Xen-devel] [PATCH 10/27] xen/arm: arm32: Don't define FAR_EL1
Aliasing FAR_EL1 to IFAR is wrong because on ARMv8 FAR_EL1[31:0] is architecturally mapped to DFAR and FAR_EL1[63:32] to DFAR. As FAR_EL1 is not currently used in ARM32 code, remove it. Signed-off-by: Julien Grall --- xen/include/asm-arm/cpregs.h | 1 - 1 file changed, 1 deletion(-) diff --git a/xen/include/asm-arm/cpregs.h b/xen/include/asm-arm/cpregs.h index 1889d7cbfb..9e138489f0 100644 --- a/xen/include/asm-arm/cpregs.h +++ b/xen/include/asm-arm/cpregs.h @@ -306,7 +306,6 @@ #define DACR32_EL2 DACR #define ESR_EL1 DFSR #define ESR_EL2 HSR -#define FAR_EL1 HIFAR #define HCR_EL2 HCR #define HPFAR_EL2 HPFAR #define HSTR_EL2HSTR -- 2.11.0 ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel