diff options
author | Christoffer Dall <christoffer.dall@linaro.org> | 2014-09-28 10:04:26 -0400 |
---|---|---|
committer | Christoffer Dall <christoffer.dall@linaro.org> | 2014-10-16 04:57:41 -0400 |
commit | 2df36a5dd6792870bef48f63bfca42055ea5b79c (patch) | |
tree | ef822d51fb8cd4adb030eab48c7dab131b632461 /include/kvm | |
parent | 3d08c629244257473450a8ba17cb8184b91e68f8 (diff) |
arm/arm64: KVM: Fix BE accesses to GICv2 EISR and ELRSR regs
The EIRSR and ELRSR registers are 32-bit registers on GICv2, and we
store these as an array of two such registers on the vgic vcpu struct.
However, we access them as a single 64-bit value or as a bitmap pointer
in the generic vgic code, which breaks BE support.
Instead, store them as u64 values on the vgic structure and do the
word-swapping in the assembly code, which already handles the byte order
for BE systems.
Tested-by: Victor Kamensky <victor.kamensky@linaro.org>
Acked-by: Marc Zyngier <marc.zyngier@arm.com>
Signed-off-by: Christoffer Dall <christoffer.dall@linaro.org>
Diffstat (limited to 'include/kvm')
-rw-r--r-- | include/kvm/arm_vgic.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/kvm/arm_vgic.h b/include/kvm/arm_vgic.h index ec559d3264cc..206dcc3b3f7a 100644 --- a/include/kvm/arm_vgic.h +++ b/include/kvm/arm_vgic.h | |||
@@ -219,8 +219,8 @@ struct vgic_v2_cpu_if { | |||
219 | u32 vgic_hcr; | 219 | u32 vgic_hcr; |
220 | u32 vgic_vmcr; | 220 | u32 vgic_vmcr; |
221 | u32 vgic_misr; /* Saved only */ | 221 | u32 vgic_misr; /* Saved only */ |
222 | u32 vgic_eisr[2]; /* Saved only */ | 222 | u64 vgic_eisr; /* Saved only */ |
223 | u32 vgic_elrsr[2]; /* Saved only */ | 223 | u64 vgic_elrsr; /* Saved only */ |
224 | u32 vgic_apr; | 224 | u32 vgic_apr; |
225 | u32 vgic_lr[VGIC_V2_MAX_LRS]; | 225 | u32 vgic_lr[VGIC_V2_MAX_LRS]; |
226 | }; | 226 | }; |