Re: [Xen-devel] [PATCH v5 7/7] VT-d: Fix vt-d Device-TLB flush timeout issue.

2016-02-18 Thread Xu, Quan
> On February 18, 2016 5:21pm, wrote: > >>> On 18.02.16 at 09:47, wrote: > >> On February 18, 2016 4:36pm, wrote: > >> >>> On 18.02.16 at 08:36, wrote: > >> >> On February 17, 2016 10:41pm, wrote: > >> >> >>> On 05.02.16 at 11:18, wrote: > >> >> > --- a/xen/drivers/passthrough/vtd/qinval.

Re: [Xen-devel] [PATCH v5 7/7] VT-d: Fix vt-d Device-TLB flush timeout issue.

2016-02-18 Thread Jan Beulich
>>> On 18.02.16 at 09:47, wrote: >> On February 18, 2016 4:36pm, wrote: >> >>> On 18.02.16 at 08:36, wrote: >> >> On February 17, 2016 10:41pm, wrote: >> >> >>> On 05.02.16 at 11:18, wrote: >> >> > --- a/xen/drivers/passthrough/vtd/qinval.c >> >> > +++ b/xen/drivers/passthrough/vtd/qinval.c

Re: [Xen-devel] [PATCH v5 7/7] VT-d: Fix vt-d Device-TLB flush timeout issue.

2016-02-18 Thread Xu, Quan
> On February 18, 2016 4:36pm, wrote: > >>> On 18.02.16 at 08:36, wrote: > >> On February 17, 2016 10:41pm, wrote: > >> >>> On 05.02.16 at 11:18, wrote: > >> > --- a/xen/drivers/passthrough/vtd/qinval.c > >> > +++ b/xen/drivers/passthrough/vtd/qinval.c > >> > +if ( pci_hide_device(

Re: [Xen-devel] [PATCH v5 7/7] VT-d: Fix vt-d Device-TLB flush timeout issue.

2016-02-18 Thread Jan Beulich
>>> On 18.02.16 at 08:36, wrote: >> On February 17, 2016 10:41pm, wrote: >> >>> On 05.02.16 at 11:18, wrote: >> > --- a/xen/drivers/passthrough/vtd/qinval.c >> > +++ b/xen/drivers/passthrough/vtd/qinval.c >> > +if ( pci_hide_device(bus, devfn) ) >> >> But now I'm _really_ puzzled:

Re: [Xen-devel] [PATCH v5 7/7] VT-d: Fix vt-d Device-TLB flush timeout issue.

2016-02-17 Thread Xu, Quan
> On February 17, 2016 10:41pm, wrote: > >>> On 05.02.16 at 11:18, wrote: > > --- a/xen/drivers/passthrough/vtd/qinval.c > > +++ b/xen/drivers/passthrough/vtd/qinval.c > > +if ( pci_hide_device(bus, devfn) ) > > But now I'm _really_ puzzled: You acquire the exact lock that > pci_hide

Re: [Xen-devel] [PATCH v5 7/7] VT-d: Fix vt-d Device-TLB flush timeout issue.

2016-02-17 Thread Jan Beulich
>>> On 05.02.16 at 11:18, wrote: > --- a/xen/drivers/passthrough/vtd/qinval.c > +++ b/xen/drivers/passthrough/vtd/qinval.c > @@ -229,6 +229,69 @@ int qinval_device_iotlb(struct iommu *iommu, > return 0; > } > > +static void dev_invalidate_iotlb_timeout(struct iommu *iommu, u16 did, > +