diff options
author | Nicholas Krause <xerofoify@gmail.com> | 2015-08-06 13:05:54 -0400 |
---|---|---|
committer | Christian Borntraeger <borntraeger@de.ibm.com> | 2015-08-07 06:15:23 -0400 |
commit | 152b28392a8d9dd08e789b48b602eb75eef436fa (patch) | |
tree | b554a0a76e747de082ebbde52989929044388404 /arch/s390/kvm | |
parent | fdf036507f1fc036d5a06753e9e8b13f46de73e8 (diff) |
KVM: s390: Fix assumption that kvm_set_irq_routing is always run successfully
This fixes the assumption that kvm_set_irq_routing is always run
successfully by instead making it equal to the variable r which
we use for returning in the function kvm_arch_vm_ioctl instead
of making r equal to zero when calling this particular function
and incorrectly making the caller of kvm_arch_vm_ioctl think
the function has run successfully.
Signed-off-by: Nicholas Krause <xerofoify@gmail.com>
Message-Id: <1438880754-27149-1-git-send-email-xerofoify@gmail.com>
Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
Diffstat (limited to 'arch/s390/kvm')
-rw-r--r-- | arch/s390/kvm/kvm-s390.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c index 4bdb860b5c49..397b88dc1b2c 100644 --- a/arch/s390/kvm/kvm-s390.c +++ b/arch/s390/kvm/kvm-s390.c | |||
@@ -962,8 +962,7 @@ long kvm_arch_vm_ioctl(struct file *filp, | |||
962 | if (kvm->arch.use_irqchip) { | 962 | if (kvm->arch.use_irqchip) { |
963 | /* Set up dummy routing. */ | 963 | /* Set up dummy routing. */ |
964 | memset(&routing, 0, sizeof(routing)); | 964 | memset(&routing, 0, sizeof(routing)); |
965 | kvm_set_irq_routing(kvm, &routing, 0, 0); | 965 | r = kvm_set_irq_routing(kvm, &routing, 0, 0); |
966 | r = 0; | ||
967 | } | 966 | } |
968 | break; | 967 | break; |
969 | } | 968 | } |