Commit 536f25e4 authored May 03, 2013 by Peter Maydell

target-arm: Fix incorrect check of kvm_vcpu_ioctl return value



kvm_vcpu_ioctl() returns -ETHING on error, not ETHING -- correct
an incorrect check in kvm_arch_init_vcpu(). This would not have
had any significant ill-effects -- we would just have propagated
the less useful ENOENT up to the caller rather than the more
accurate EINVAL in the unlikely case that the kernel didn't
have VFP-D32 support.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

parent 743bddb4

target-arm/kvm.c

+2 −2

Original line number	Diff line number	Diff line
		@@ -62,8 +62,8 @@ int kvm_arch_init_vcpu(CPUState *cs)
		r.id = KVM_REG_ARM \| KVM_REG_SIZE_U64 \| KVM_REG_ARM_VFP \| 31;
		r.addr = (uintptr_t)(&v);
		ret = kvm_vcpu_ioctl(cs, KVM_GET_ONE_REG, &r);
		if (ret == ENOENT) {
		return EINVAL;
		if (ret == -ENOENT) {
		return -EINVAL;
		}
		return ret;
		}