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

2016-03-29 Thread Xu, Quan
On March 29, 2016 10:21pm, Konrad Rzeszutek Wilk wrote: > On Tue, Mar 29, 2016 at 01:32:02AM +, Xu, Quan wrote: > > On March 28, 2016 10:11pm, Konrad Rzeszutek Wilk > wrote: > > > > > > > > > > +list_del(&pdev->domain_list); > > > > > > +pdev->domain = NULL; > > > > >

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

2016-03-29 Thread Konrad Rzeszutek Wilk
On Tue, Mar 29, 2016 at 01:32:02AM +, Xu, Quan wrote: > On March 28, 2016 10:11pm, Konrad Rzeszutek Wilk > wrote: > > > > > > > > +list_del(&pdev->domain_list); > > > > > +pdev->domain = NULL; > > > > > +pci_hide_existing_device(pdev); > > > > > +

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

2016-03-28 Thread Xu, Quan
On March 28, 2016 10:11pm, Konrad Rzeszutek Wilk wrote: > > > > > > +list_del(&pdev->domain_list); > > > > +pdev->domain = NULL; > > > > +pci_hide_existing_device(pdev); > > > > +break; > > > > +} > > > > +} > > > > + > > > > +pcidevs

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

2016-03-28 Thread Konrad Rzeszutek Wilk
> > > > +list_del(&pdev->domain_list); > > > +pdev->domain = NULL; > > > +pci_hide_existing_device(pdev); > > > +break; > > > +} > > > +} > > > + > > > +pcidevs_unlock(); > > > + > > > +if ( !is_hardware_domain(d) ) > > > +

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

2016-03-28 Thread Xu, Quan
+cc Konrad Rzeszutek Wilk , who is also reviewing this patch. On March 24, 2016 11:38pm, Dario Faggioli wrote: > On Thu, 2016-03-24 at 13:57 +0800, Quan Xu wrote: > > If Device-TLB flush timed out, we would hide the target ATS device and > > crash the domain owning this ATS device. If impacted d

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

2016-03-27 Thread Xu, Quan
On March 26, 2016 4:32am, Konrad Rzeszutek Wilk wrote: > On Thu, Mar 24, 2016 at 01:57:59PM +0800, Quan Xu wrote: > > If Device-TLB flush timed out, we would hide the target ATS device and > > crash the domain owning this ATS device. If impacted domain is > > hardware domain, just throw out a warn

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

2016-03-27 Thread Xu, Quan
On March 26, 2016 4:40am, Konrad Rzeszutek Wilk wrote: > On Thu, Mar 24, 2016 at 04:38:05PM +0100, Dario Faggioli wrote: > > On Thu, 2016-03-24 at 13:57 +0800, Quan Xu wrote: > > > If Device-TLB flush timed out, we would hide the target ATS device > > > and crash the domain owning this ATS device.

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

2016-03-25 Thread Konrad Rzeszutek Wilk
On Thu, Mar 24, 2016 at 04:38:05PM +0100, Dario Faggioli wrote: > On Thu, 2016-03-24 at 13:57 +0800, Quan Xu wrote: > > If Device-TLB flush timed out, we would hide the target ATS > > device and crash the domain owning this ATS device. If impacted > > domain is hardware domain, just throw out a war

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

2016-03-25 Thread Konrad Rzeszutek Wilk
On Thu, Mar 24, 2016 at 01:57:59PM +0800, Quan Xu wrote: > If Device-TLB flush timed out, we would hide the target ATS > device and crash the domain owning this ATS device. If impacted > domain is hardware domain, just throw out a warning. > > The hidden device should be disallowed to be further a

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

2016-03-24 Thread Xu, Quan
On March 24, 2016 11:38pm, wrote: > On Thu, 2016-03-24 at 13:57 +0800, Quan Xu wrote: > > If Device-TLB flush timed out, we would hide the target ATS device and > > crash the domain owning this ATS device. If impacted domain is > > hardware domain, just throw out a warning. > > > > The hidden devi

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

2016-03-24 Thread Dario Faggioli
On Thu, 2016-03-24 at 13:57 +0800, Quan Xu wrote: > If Device-TLB flush timed out, we would hide the target ATS > device and crash the domain owning this ATS device. If impacted > domain is hardware domain, just throw out a warning. > > The hidden device should be disallowed to be further assigned

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

2016-03-23 Thread Quan Xu
If Device-TLB flush timed out, we would hide the target ATS device and crash the domain owning this ATS device. If impacted domain is hardware domain, just throw out a warning. The hidden device should be disallowed to be further assigned to any domain. Signed-off-by: Quan Xu --- xen/drivers/pa