diff options
author | Christian Borntraeger <borntraeger@de.ibm.com> | 2014-09-04 15:13:32 -0400 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2014-09-05 06:01:15 -0400 |
commit | a13f533b2f1d53a7c0baa7490498caeab7bc8ba5 (patch) | |
tree | cc027dcaf928b431acb067975bc9d83618bf0214 /virt | |
parent | 34656113182b704682e23d1363417536addfec97 (diff) |
KVM: remove redundant assigment of return value in kvm_dev_ioctl
The first statement of kvm_dev_ioctl is
long r = -EINVAL;
No need to reassign the same value.
Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'virt')
-rw-r--r-- | virt/kvm/kvm_main.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c index cc7bd286d135..de1ae82ba192 100644 --- a/virt/kvm/kvm_main.c +++ b/virt/kvm/kvm_main.c | |||
@@ -2627,7 +2627,6 @@ static long kvm_dev_ioctl(struct file *filp, | |||
2627 | 2627 | ||
2628 | switch (ioctl) { | 2628 | switch (ioctl) { |
2629 | case KVM_GET_API_VERSION: | 2629 | case KVM_GET_API_VERSION: |
2630 | r = -EINVAL; | ||
2631 | if (arg) | 2630 | if (arg) |
2632 | goto out; | 2631 | goto out; |
2633 | r = KVM_API_VERSION; | 2632 | r = KVM_API_VERSION; |
@@ -2639,7 +2638,6 @@ static long kvm_dev_ioctl(struct file *filp, | |||
2639 | r = kvm_vm_ioctl_check_extension_generic(NULL, arg); | 2638 | r = kvm_vm_ioctl_check_extension_generic(NULL, arg); |
2640 | break; | 2639 | break; |
2641 | case KVM_GET_VCPU_MMAP_SIZE: | 2640 | case KVM_GET_VCPU_MMAP_SIZE: |
2642 | r = -EINVAL; | ||
2643 | if (arg) | 2641 | if (arg) |
2644 | goto out; | 2642 | goto out; |
2645 | r = PAGE_SIZE; /* struct kvm_run */ | 2643 | r = PAGE_SIZE; /* struct kvm_run */ |