On 08/22/2017 03:12 PM, Andre Przywara wrote:
Hi,
Hi Andre,
On 14/08/17 15:24, Julien Grall wrote:
Aliasing FAR_EL2 to HIFAR makes the code confusing because on ARMv8
FAR_EL2[31:0] is architecturally mapped to HDFAR and FAR_EL2[63:32] to
FAR_EL2.
^^^
I guess you mean HIFAR here.
Othe
Hi,
On 14/08/17 15:24, Julien Grall wrote:
> Aliasing FAR_EL2 to HIFAR makes the code confusing because on ARMv8
> FAR_EL2[31:0] is architecturally mapped to HDFAR and FAR_EL2[63:32] to
> FAR_EL2.
^^^
I guess you mean HIFAR here.
Otherwise the patch makes sense.
> See D7.2.30 in ARM DDI 048
Aliasing FAR_EL2 to HIFAR makes the code confusing because on ARMv8
FAR_EL2[31:0] is architecturally mapped to HDFAR and FAR_EL2[63:32] to
FAR_EL2. See D7.2.30 in ARM DDI 0487B.a. Open-code the alias instead.
Signed-off-by: Julien Grall
---
xen/arch/arm/traps.c | 8 +++-
xen/include/
