summaryrefslogtreecommitdiff
path: root/target-arm
diff options
context:
space:
mode:
authorPeter Maydell <peter.maydell@linaro.org>2013-05-03 18:47:22 +0100
committerPeter Maydell <peter.maydell@linaro.org>2013-05-03 18:47:22 +0100
commit536f25e4c77592b936e50728c83894c23f4f61c8 (patch)
tree12d0f528b01a2b97c93cd252df9e159a2b8dcfc9 /target-arm
parent743bddb4b35ceaaf6f95aea581a4130dcae6205a (diff)
downloadqemu-536f25e4c77592b936e50728c83894c23f4f61c8.tar.gz
qemu-536f25e4c77592b936e50728c83894c23f4f61c8.tar.bz2
qemu-536f25e4c77592b936e50728c83894c23f4f61c8.zip
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>
Diffstat (limited to 'target-arm')
-rw-r--r--target-arm/kvm.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/target-arm/kvm.c b/target-arm/kvm.c
index d8acace9b7..b7bdc034fd 100644
--- a/target-arm/kvm.c
+++ b/target-arm/kvm.c
@@ -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;
}