diff options
author | Ian Campbell <ian.campbell@citrix.com> | 2013-07-15 09:24:05 +0100 |
---|---|---|
committer | Ian Campbell <ian.campbell@citrix.com> | 2013-07-17 10:48:54 +0100 |
commit | 8348707ce1c76e2fecbe9d413d5d8f2085bf57cb (patch) | |
tree | ea5212c891fb924bb73e01aaa387402a8c702dcc | |
parent | 26b48cf251c6ce39c07462b833d33195c172466b (diff) | |
download | xen-8348707ce1c76e2fecbe9d413d5d8f2085bf57cb.tar.gz xen-8348707ce1c76e2fecbe9d413d5d8f2085bf57cb.tar.bz2 xen-8348707ce1c76e2fecbe9d413d5d8f2085bf57cb.zip |
xen: arm: correctly configure NSACR.
Previously we were setting it up twice, the second time neglecting to set the
NS_SMP bit.
NSACR.NS_SMP is a processor specific bit which on Cortex-A7 and -A15 regulates
access to the (also processor specific) ACTLR.SMP bit. Not setting NSACR.NS_SMP
meant that Xen's attempts to set ACTLR.SMP was silently ignored. Setting this
bit is required in order to cause the processor to take part in cache and TLB
coherency protocols. Failure to set this bit leads to random memory corruption
in guests (although nothing like as catastrophic as you might expect!).
An alternative fix would have been to set ACTLR.SMP when in Secure World,
however Linux expects to set ACTLR.SMP itself in NS mode, so it's a good bet
that bootloaders will set NSACR.NS_SMP instead.
While here switch to a read-modify-write of NSACR to preserve any existing
bits -- seems safer.
Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
Acked-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Acked-by: Julien Grall <julien.grall@linaro.org>
-rw-r--r-- | xen/arch/arm/arm32/mode_switch.S | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/xen/arch/arm/arm32/mode_switch.S b/xen/arch/arm/arm32/mode_switch.S index c92a1cf9f0..3500eb06c4 100644 --- a/xen/arch/arm/arm32/mode_switch.S +++ b/xen/arch/arm/arm32/mode_switch.S @@ -104,7 +104,11 @@ enter_hyp_mode: * memory-mapped control registers live, we can't find out the * right frequency. */ mcr CP32(r0, CNTFRQ) - ldr r0, =0x40c00 /* SMP, c11, c10 in non-secure mode */ + + mrc CP32(r0,NSACR) + ldr r4, =0x3fff /* Allow access to all co-processors in NS mode */ + orr r0, r0, r4 + orr r0, r0, #(1<<18) /* CA7/CA15: Allow access to ACTLR.SMP in NS mode */ mcr CP32(r0, NSACR) add r0, r1, #GIC_DR_OFFSET @@ -143,9 +147,6 @@ skip_spis: mov r0, #0 mcr CP32(r0, FCSEIDR) mcr CP32(r0, CONTEXTIDR) - /* Allow non-secure access to coprocessors, FIQs, VFP and NEON */ - ldr r1, =0x3fff /* 14 CP bits set, all others clear */ - mcr CP32(r1, NSACR) mrs r0, cpsr /* Copy the CPSR */ add r0, r0, #0x4 /* 0x16 (Monitor) -> 0x1a (Hyp) */ |