linux-stable/virt/kvm/arm/hyp
Marc Zyngier b85437d007 kvm: arm/arm64: vgic-v3: Tighten synchronization for guests using v2 on v3
commit 27e91ad1e7 upstream.

On guest exit, and when using GICv2 on GICv3, we use a dsb(st) to
force synchronization between the memory-mapped guest view and
the system-register view that the hypervisor uses.

This is incorrect, as the spec calls out the need for "a DSB whose
required access type is both loads and stores with any Shareability
attribute", while we're only synchronizing stores.

We also lack an isb after the dsb to ensure that the latter has
actually been executed before we start reading stuff from the sysregs.

The fix is pretty easy: turn dsb(st) into dsb(sy), and slap an isb()
just after.

Cc: stable@vger.kernel.org
Fixes: f68d2b1b73 ("arm64: KVM: Implement vgic-v3 save/restore")
Acked-by: Christoffer Dall <cdall@kernel.org>
Reviewed-by: Andre Przywara <andre.przywara@arm.com>
Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2018-03-21 12:06:43 +01:00
..
timer-sr.c KVM: arm/arm64: Move cntvoff to each timer context 2017-02-08 15:13:33 +00:00
vgic-v2-sr.c KVM: arm/arm64: Fix broken GICH_ELRSR big endian conversion 2017-12-14 09:53:01 +01:00
vgic-v3-sr.c kvm: arm/arm64: vgic-v3: Tighten synchronization for guests using v2 on v3 2018-03-21 12:06:43 +01:00