KVM: arm/arm64: Initialise host's MPIDRs by reading the actual register
authorMarc Zyngier <marc.zyngier@arm.com>
Fri, 5 Jul 2019 22:35:56 +0000 (23:35 +0100)
committerMarc Zyngier <marc.zyngier@arm.com>
Mon, 8 Jul 2019 15:29:48 +0000 (16:29 +0100)
commit1e0cf16cdad1ba53e9eeee8746fe57de42f20c97
tree664d34bdb34d0e9e7ee193932c225ed49a4db80e
parentfdec2a9ef853172529baaa192673b4cdb9a44fac
KVM: arm/arm64: Initialise host's MPIDRs by reading the actual register

As part of setting up the host context, we populate its
MPIDR by using cpu_logical_map(). It turns out that contrary
to arm64, cpu_logical_map() on 32bit ARM doesn't return the
*full* MPIDR, but a truncated version.

This leaves the host MPIDR slightly corrupted after the first
run of a VM, since we won't correctly restore the MPIDR on
exit. Oops.

Since we cannot trust cpu_logical_map(), let's adopt a different
strategy. We move the initialization of the host CPU context as
part of the per-CPU initialization (which, in retrospect, makes
a lot of sense), and directly read the MPIDR from the HW. This
is guaranteed to work on both arm and arm64.

Reported-by: Andre Przywara <Andre.Przywara@arm.com>
Tested-by: Andre Przywara <Andre.Przywara@arm.com>
Fixes: 32f139551954 ("arm/arm64: KVM: Statically configure the host's view of MPIDR")
Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
arch/arm/include/asm/kvm_host.h
arch/arm64/include/asm/kvm_host.h
virt/kvm/arm/arm.c