From mboxrd@z Thu Jan 1 00:00:00 1970 From: Steve Rutherford Subject: Re: [PATCH v3 01/24] KVM: x86: race between KVM_SET_GSI_ROUTING and KVM_CREATE_IRQCHIP Date: Mon, 24 Apr 2017 15:39:53 -0700 Message-ID: References: <20170407085041.5257-1-david@redhat.com> <20170407085041.5257-2-david@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Cc: KVM list , Paolo Bonzini , =?UTF-8?B?UmFkaW0gS3LEjW3DocWZ?= , Peter Xu To: David Hildenbrand Return-path: Received: from mail-yw0-f174.google.com ([209.85.161.174]:35380 "EHLO mail-yw0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S966825AbdDXWkf (ORCPT ); Mon, 24 Apr 2017 18:40:35 -0400 Received: by mail-yw0-f174.google.com with SMTP id u70so88379433ywe.2 for ; Mon, 24 Apr 2017 15:40:34 -0700 (PDT) In-Reply-To: <20170407085041.5257-2-david@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: Sorry to resurrect this thread. There's an srcu_synchronize_expedited call in kvm_set_irq_routing, so this lock might get held for a fairly long while. Wouldn't it be a better idea to check the irqchip mode? On Fri, Apr 7, 2017 at 1:50 AM, David Hildenbrand wrote: > Avoid races between KVM_SET_GSI_ROUTING and KVM_CREATE_IRQCHIP by taking > the kvm->lock when setting up routes. > > If KVM_CREATE_IRQCHIP fails, KVM_SET_GSI_ROUTING could have already set > up routes pointing at pic/ioapic, being silently removed already. > > Also, as a side effect, this patch makes sure that KVM_SET_GSI_ROUTING > and KVM_CAP_SPLIT_IRQCHIP cannot run in parallel. > > Signed-off-by: David Hildenbrand > --- > virt/kvm/kvm_main.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c > index 88257b3..a80e400 100644 > --- a/virt/kvm/kvm_main.c > +++ b/virt/kvm/kvm_main.c > @@ -3082,8 +3082,11 @@ static long kvm_vm_ioctl(struct file *filp, > routing.nr * sizeof(*entries))) > goto out_free_irq_routing; > } > + /* avoid races with KVM_CREATE_IRQCHIP on x86 */ > + mutex_lock(&kvm->lock); > r = kvm_set_irq_routing(kvm, entries, routing.nr, > routing.flags); > + mutex_unlock(&kvm->lock); > out_free_irq_routing: > vfree(entries); > break; > -- > 2.9.3 >