KVM: s390: floating irqs: fix user triggerable endless loop
authorDavid Hildenbrand <dahi@linux.vnet.ibm.com>
Thu, 15 Jan 2015 16:56:18 +0000 (17:56 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 6 Mar 2015 22:53:02 +0000 (14:53 -0800)
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>
arch/s390/kvm/interrupt.c

index b73b5db1ab42ac583e34a5e7ad6909eded718722..57ed057cba65f77297e2739b97668752541c57a0 100644 (file)
@@ -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 {