Re: [Xen-devel] [PATCH v9 3/3] VT-d: Fix vt-d Device-TLB flush timeout issue

2016-04-11 Thread Jan Beulich
>>> On 10.04.16 at 16:28, wrote: > On April 05, 2016 5:48pm, Jan Beulich wrote: >> >>> On 01.04.16 at 16:47, wrote: >> > +{ >> > +ASSERT(pdev->domain); >> > +list_del(&pdev->domain_list); >> > +pdev->domain = NULL; >> > +pci_hide_existing_d

Re: [Xen-devel] [PATCH v9 3/3] VT-d: Fix vt-d Device-TLB flush timeout issue

2016-04-10 Thread Xu, Quan
On April 05, 2016 5:48pm, Jan Beulich wrote: > >>> On 01.04.16 at 16:47, wrote: > > +static void dev_invalidate_iotlb_timeout(struct iommu *iommu, u16 did, > > + u16 seg, u8 bus, u8 devfn) > { > > +struct domain *d = NULL; > > +struct pci_dev *pdev;

Re: [Xen-devel] [PATCH v9 3/3] VT-d: Fix vt-d Device-TLB flush timeout issue

2016-04-05 Thread Jan Beulich
>>> On 01.04.16 at 16:47, wrote: > If Device-TLB flush timed out, we would hide the target ATS Please re-consider the use of the word "would" in all your patch descriptions. > --- a/xen/drivers/passthrough/vtd/qinval.c > +++ b/xen/drivers/passthrough/vtd/qinval.c > @@ -217,6 +217,58 @@ static in