Re: [Xen-devel] [PATCH v3 2/2] IOMMU/spinlock: Make the pcidevs_lock a recursive one

2016-03-10 Thread Jan Beulich
>>> On 10.03.16 at 12:27, wrote: > On March 10, 2016 5:53pm, wrote: >> >>> On 09.03.16 at 14:17, wrote: >> > Signed-off-by: Quan Xu >> > Acked-by: Kevin Tian >> >> The patch itself looks mostly fine now (see below for the minor left > issues), but >> the complete lack of a description (which

Re: [Xen-devel] [PATCH v3 2/2] IOMMU/spinlock: Make the pcidevs_lock a recursive one

2016-03-10 Thread Xu, Quan
On March 10, 2016 5:53pm, wrote: > >>> On 09.03.16 at 14:17, wrote: > > Signed-off-by: Quan Xu > > Acked-by: Kevin Tian > > The patch itself looks mostly fine now (see below for the minor left issues), > but > the complete lack of a description (which should state why this change is > being

Re: [Xen-devel] [PATCH v3 2/2] IOMMU/spinlock: Make the pcidevs_lock a recursive one

2016-03-10 Thread Jan Beulich
>>> On 09.03.16 at 14:17, wrote: > Signed-off-by: Quan Xu > Acked-by: Kevin Tian The patch itself looks mostly fine now (see below for the minor left issues), but the complete lack of a description (which should state why this change is being done) makes this not ready to go in anyway. > --- a

Re: [Xen-devel] [PATCH v3 2/2] IOMMU/spinlock: Make the pcidevs_lock a recursive one

2016-03-09 Thread Xu, Quan
On March 10, 2016 1:45am, wrote: > On Wed, 2016-03-09 at 21:17 +0800, Quan Xu wrote: > > Signed-off-by: Quan Xu > > Acked-by: Kevin Tian > > > Reviewed-by: Dario Faggioli > > And I've applied and build tested it, against current staging, and this time, > it > worked. :-) > Dario, thanks!! :

Re: [Xen-devel] [PATCH v3 2/2] IOMMU/spinlock: Make the pcidevs_lock a recursive one

2016-03-09 Thread Dario Faggioli
On Wed, 2016-03-09 at 21:17 +0800, Quan Xu wrote: > Signed-off-by: Quan Xu > Acked-by: Kevin Tian > Reviewed-by: Dario Faggioli And I've applied and build tested it, against current staging, and this time, it worked. :-) Regards, Dario -- <> (Raistlin Majere) -

[Xen-devel] [PATCH v3 2/2] IOMMU/spinlock: Make the pcidevs_lock a recursive one

2016-03-09 Thread Quan Xu
Signed-off-by: Quan Xu Acked-by: Kevin Tian CC: Keir Fraser CC: Jan Beulich CC: Andrew Cooper CC: Suravee Suthikulpanit CC: Feng Wu CC: Kevin Tian CC: Dario Faggioli --- xen/arch/x86/domctl.c | 8 +-- xen/arch/x86/hvm/vmsi.c | 4 +- xen/arch/x86