Commit 22f55384 authored Oct 27, 2020 by Qais Yousef Committed by Marc Zyngier Oct 30, 2020

KVM: arm64: Handle Asymmetric AArch32 systems



On a system without uniform support for AArch32 at EL0, it is possible
for the guest to force run AArch32 at EL0 and potentially cause an
illegal exception if running on a core without AArch32. Add an extra
check so that if we catch the guest doing that, then we prevent it from
running again by resetting vcpu->arch.target and return
ARM_EXCEPTION_IL.

We try to catch this misbehaviour as early as possible and not rely on
an illegal exception occuring to signal the problem. Attempting to run a
32bit app in the guest will produce an error from QEMU if the guest
exits while running in AArch32 EL0.

Tested on Juno by instrumenting the host to fake asym aarch32 and
instrumenting KVM to make the asymmetry visible to the guest.

[will: Incorporated feedback from Marc]

Signed-off-by: Qais Yousef <qais.yousef@arm.com>
Signed-off-by: Will Deacon <will@kernel.org>
Signed-off-by: Marc Zyngier <maz@kernel.org>
Cc: James Morse <james.morse@arm.com>
Cc: Marc Zyngier <maz@kernel.org>
Link: https://lore.kernel.org/r/20201021104611.2744565-2-qais.yousef@arm.com
Link: https://lore.kernel.org/r/20201027215118.27003-2-will@kernel.org

parent d86de40d

arch/arm64/kvm/arm.c

+19 −0

Original line number	Diff line number	Diff line
		@@ -808,6 +808,25 @@ int kvm_arch_vcpu_ioctl_run(struct kvm_vcpu *vcpu)

		preempt_enable();

		/*
		* The ARMv8 architecture doesn't give the hypervisor
		* a mechanism to prevent a guest from dropping to AArch32 EL0
		* if implemented by the CPU. If we spot the guest in such
		* state and that we decided it wasn't supposed to do so (like
		* with the asymmetric AArch32 case), return to userspace with
		* a fatal error.
		*/
		if (!system_supports_32bit_el0() && vcpu_mode_is_32bit(vcpu)) {
		/*
		* As we have caught the guest red-handed, decide that
		* it isn't fit for purpose anymore by making the vcpu
		* invalid. The VMM can try and fix it by issuing a
		* KVM_ARM_VCPU_INIT if it really wants to.
		*/
		vcpu->arch.target = -1;
		ret = ARM_EXCEPTION_IL;
		}

		ret = handle_exit(vcpu, ret);
		}

Admin message