diff options
author | David Hildenbrand <dahi@linux.vnet.ibm.com> | 2015-01-15 17:56:18 +0100 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2015-03-06 14:53:02 -0800 |
commit | 6d6cdcaf12560db583b098baca23e3bdab78bcaa (patch) | |
tree | 55fac7a8e662fc5f70dbd9ff7c832fd2ef20683f /arch/s390 | |
parent | 6d351caba56c7fc13eda9a8adb2c41621ac62b26 (diff) | |
download | linux-stable-6d6cdcaf12560db583b098baca23e3bdab78bcaa.tar.gz linux-stable-6d6cdcaf12560db583b098baca23e3bdab78bcaa.tar.bz2 linux-stable-6d6cdcaf12560db583b098baca23e3bdab78bcaa.zip |
KVM: s390: floating irqs: fix user triggerable endless loop
commit 8e2207cdd087ebb031e9118d1fd0902c6533a5e5 upstream.
If a vm with no VCPUs is created, the injection of a floating irq
leads to an endless loop in the kernel.
Let's skip the search for a destination VCPU for a floating irq if no
VCPUs were created.
Reviewed-by: Dominik Dingel <dingel@linux.vnet.ibm.com>
Reviewed-by: Cornelia Huck <cornelia.huck@de.ibm.com>
Signed-off-by: David Hildenbrand <dahi@linux.vnet.ibm.com>
Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'arch/s390')
-rw-r--r-- | arch/s390/kvm/interrupt.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/arch/s390/kvm/interrupt.c b/arch/s390/kvm/interrupt.c index b73b5db1ab42..57ed057cba65 100644 --- a/arch/s390/kvm/interrupt.c +++ b/arch/s390/kvm/interrupt.c @@ -850,6 +850,8 @@ static int __inject_vm(struct kvm *kvm, struct kvm_s390_interrupt_info *inti) list_add_tail(&inti->list, &iter->list); } atomic_set(&fi->active, 1); + if (atomic_read(&kvm->online_vcpus) == 0) + goto unlock_fi; sigcpu = find_first_bit(fi->idle_mask, KVM_MAX_VCPUS); if (sigcpu == KVM_MAX_VCPUS) { do { |