passthrough: drop break statement following c/s cd7dedad820
authorIgor Druzhinin <igor.druzhinin@citrix.com>
Thu, 5 Dec 2019 12:31:03 +0000 (13:31 +0100)
committerJan Beulich <jbeulich@suse.com>
Thu, 5 Dec 2019 12:31:03 +0000 (13:31 +0100)
The locking responsibilities have changed and a premature break in
this section now causes the following assertion:

Assertion '!preempt_count()' failed at preempt.c:36

Reported-by: Sander Eikelenboom <linux@eikelenboom.it>
Suggested-by: Jan Beulich <jbeulich@suse.com>
Signed-off-by: Igor Druzhinin <igor.druzhinin@citrix.com>
Reviewed-by: Paul Durrant <pdurrant@amazon.com>
Reviewed-by: Jan Beulich <jbeulich@suse.com>
xen/drivers/passthrough/pci.c

index ced0c28e4f52c6d4f790ca7e961eb976c386d768..c07a63981ac13f53ab36280ea689a482bbad23f3 100644 (file)
@@ -1705,7 +1705,6 @@ int iommu_do_pci_domctl(
                        seg, bus, PCI_SLOT(devfn), PCI_FUNC(devfn));
                 ret = -EINVAL;
             }
-            break;
         }
         else if ( !ret )
             ret = assign_device(d, seg, bus, devfn, flags);