Re: [Xen-devel] [PATCH for-4.13 v2] passthrough: simplify locking and logging

2019-11-07 Thread Jan Beulich
On 01.11.2019 19:28, Igor Druzhinin wrote: > This patch instead acquires the lock once for assignment (or test assign) > operations directly in iommu_do_pci_domctl() and thus can remove the > duplicate domain ownership check in assign_device(). Whilst in the > neighbourhood, the patch also removes

Re: [Xen-devel] [PATCH for-4.13 v2] passthrough: simplify locking and logging

2019-11-04 Thread Durrant, Paul
.org > > > Cc: Igor Druzhinin ; jgr...@suse.com; > > > jbeul...@suse.com > > > Subject: Re: [Xen-devel] [PATCH for-4.13 v2] passthrough: simplify > locking > > > and logging > > > > > > On 04/11/2019 08:31, Durrant, Paul wrote

Re: [Xen-devel] [PATCH for-4.13 v2] passthrough: simplify locking and logging

2019-11-04 Thread Anthony PERARD
suse.com > > Subject: Re: [Xen-devel] [PATCH for-4.13 v2] passthrough: simplify locking > > and logging > > > > On 04/11/2019 08:31, Durrant, Paul wrote: > > >> -Original Message- > > >> From: Igor Druzhinin > > >> Sent: 0

Re: [Xen-devel] [PATCH for-4.13 v2] passthrough: simplify locking and logging

2019-11-04 Thread Durrant, Paul
> -Original Message- > From: Andrew Cooper > Sent: 04 November 2019 11:06 > To: Durrant, Paul ; xen-devel@lists.xenproject.org > Cc: Igor Druzhinin ; jgr...@suse.com; > jbeul...@suse.com > Subject: Re: [Xen-devel] [PATCH for-4.13 v2] passthrough: simplify locking >

Re: [Xen-devel] [PATCH for-4.13 v2] passthrough: simplify locking and logging

2019-11-04 Thread Andrew Cooper
On 04/11/2019 08:31, Durrant, Paul wrote: >> -Original Message- >> From: Igor Druzhinin >> Sent: 01 November 2019 19:28 >> To: xen-devel@lists.xenproject.org >> Cc: Durrant, Paul ; jbeul...@suse.com; >> jgr...@suse.com >> Subject: [PATCH for-4.13 v2] passthrough: simplify locking and

Re: [Xen-devel] [PATCH for-4.13 v2] passthrough: simplify locking and logging

2019-11-04 Thread Durrant, Paul
> -Original Message- > From: Igor Druzhinin > Sent: 01 November 2019 19:28 > To: xen-devel@lists.xenproject.org > Cc: Durrant, Paul ; jbeul...@suse.com; > jgr...@suse.com > Subject: [PATCH for-4.13 v2] passthrough: simplify locking and logging > > From: Paul Durrant > > Dropping the

[Xen-devel] [PATCH for-4.13 v2] passthrough: simplify locking and logging

2019-11-01 Thread Igor Druzhinin
From: Paul Durrant Dropping the pcidevs lock between calling device_assigned() and assign_device() means that the latter has to do the same check as the former for no obvious gain. Also, since long running operations under pcidevs lock already drop the lock and return -ERESTART periodically