Re: [Xen-devel] [PATCH v2] passthrough: drop break statement following c/s cd7dedad820
On 05.12.2019 13:20, Durrant, Paul wrote: >> -Original Message- >> From: Igor Druzhinin >> Sent: 05 December 2019 12:14 >> To: xen-devel@lists.xenproject.org >> Cc: jbeul...@suse.com; li...@eikelenboom.it; Durrant, Paul >> ; Igor Druzhinin >> Subject: [PATCH v2] passthrough: drop break statement following c/s >> cd7dedad820 >> >> 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 >> >> Suggested-by: Paul Durrant > > Actually, it was suggested by Jan, but you can put my R-b on the patch. And mine: Reviewed-by: Jan Beulich I'll get this committed soon. Jan ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH v2] passthrough: drop break statement following c/s cd7dedad820
On 05/12/2019 12:20, Durrant, Paul wrote: >> -Original Message- >> From: Igor Druzhinin >> Sent: 05 December 2019 12:14 >> To: xen-devel@lists.xenproject.org >> Cc: jbeul...@suse.com; li...@eikelenboom.it; Durrant, Paul >> ; Igor Druzhinin >> Subject: [PATCH v2] passthrough: drop break statement following c/s >> cd7dedad820 >> >> 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 >> >> Suggested-by: Paul Durrant > > Actually, it was suggested by Jan, but you can put my R-b on the patch. > Oh, indeed :) Please fix up while committing as you wish. Igor ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH v2] passthrough: drop break statement following c/s cd7dedad820
> -Original Message- > From: Igor Druzhinin > Sent: 05 December 2019 12:14 > To: xen-devel@lists.xenproject.org > Cc: jbeul...@suse.com; li...@eikelenboom.it; Durrant, Paul > ; Igor Druzhinin > Subject: [PATCH v2] passthrough: drop break statement following c/s > cd7dedad820 > > 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 > > Suggested-by: Paul Durrant Actually, it was suggested by Jan, but you can put my R-b on the patch. Paul > Reported-by: Sander Eikelenboom > Signed-off-by: Igor Druzhinin > --- > xen/drivers/passthrough/pci.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/xen/drivers/passthrough/pci.c b/xen/drivers/passthrough/pci.c > index ced0c28..c07a639 100644 > --- a/xen/drivers/passthrough/pci.c > +++ b/xen/drivers/passthrough/pci.c > @@ -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); > -- > 2.7.4 ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
[Xen-devel] [PATCH v2] passthrough: drop break statement following c/s cd7dedad820
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 Suggested-by: Paul Durrant Reported-by: Sander Eikelenboom Signed-off-by: Igor Druzhinin --- xen/drivers/passthrough/pci.c | 1 - 1 file changed, 1 deletion(-) diff --git a/xen/drivers/passthrough/pci.c b/xen/drivers/passthrough/pci.c index ced0c28..c07a639 100644 --- a/xen/drivers/passthrough/pci.c +++ b/xen/drivers/passthrough/pci.c @@ -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); -- 2.7.4 ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel