Re: [Xen-devel] [PATCH v2 2/2] VT-d: Fix vt-d flush timeout issue.

2015-12-11 Thread Xu, Quan
rix.com; t...@xen.org; xen-devel@lists.xen.org; Nakajima, > Jun; k...@xen.org > Subject: Re: [Xen-devel] [PATCH v2 2/2] VT-d: Fix vt-d flush timeout issue. > > On 11.12.2015 at 6:05pm, wrote: > > >>> On 11.12.15 at 09:01, wrote: > > > On 11.12.2015 at 3:28pm

Re: [Xen-devel] [PATCH v2 2/2] VT-d: Fix vt-d flush timeout issue.

2015-12-11 Thread Xu, Quan
On 11.12.2015 at 6:05pm, wrote: > >>> On 11.12.15 at 09:01, wrote: > > On 11.12.2015 at 3:28pm, wrote: > >> > From: Xu, Quan > >> > Sent: Thursday, December 10, 2015 5:33 PM > >> > > >> > If IOTLB/Context/IETC flush is timeout, we should think all devices > >> > under this IOMMU cannot function

Re: [Xen-devel] [PATCH v2 2/2] VT-d: Fix vt-d flush timeout issue.

2015-12-11 Thread Jan Beulich
>>> On 11.12.15 at 09:01, wrote: > On 11.12.2015 at 3:28pm, wrote: >> > From: Xu, Quan >> > Sent: Thursday, December 10, 2015 5:33 PM >> > >> > If IOTLB/Context/IETC flush is timeout, we should think all devices >> > under this IOMMU cannot function correctly. >> > So for each device under this I

Re: [Xen-devel] [PATCH v2 2/2] VT-d: Fix vt-d flush timeout issue.

2015-12-11 Thread Xu, Quan
On 11.12.2015 at 4:38pm, wrote: > On 11/12/2015 05:37, Xu, Quan wrote: > > On 11.12.2015 at 3:05pm, Andrew Cooper > wrote: > >> On 10/12/15 09:33, Quan Xu wrote: > >>> diff --git a/xen/include/xen/pci.h b/xen/include/xen/pci.h index > >>> a5aef55..0bf6b1a 100644 > >>> --- a/xen/include/xen/pci.h

Re: [Xen-devel] [PATCH v2 2/2] VT-d: Fix vt-d flush timeout issue.

2015-12-11 Thread Andrew Cooper
On 11/12/2015 05:37, Xu, Quan wrote: > On 11.12.2015 at 3:05pm, Andrew Cooper wrote: >> On 10/12/15 09:33, Quan Xu wrote: >>> diff --git a/xen/include/xen/pci.h b/xen/include/xen/pci.h index >>> a5aef55..0bf6b1a 100644 >>> --- a/xen/include/xen/pci.h >>> +++ b/xen/include/xen/pci.h >>> @@ -41,6 +4

Re: [Xen-devel] [PATCH v2 2/2] VT-d: Fix vt-d flush timeout issue.

2015-12-11 Thread Xu, Quan
On 11.12.2015 at 3:28pm, wrote: > > From: Xu, Quan > > Sent: Thursday, December 10, 2015 5:33 PM > > > > If IOTLB/Context/IETC flush is timeout, we should think all devices > > under this IOMMU cannot function correctly. > > So for each device under this IOMMU we'll mark it as unassignable and > >

Re: [Xen-devel] [PATCH v2 2/2] VT-d: Fix vt-d flush timeout issue.

2015-12-10 Thread Tian, Kevin
> From: Xu, Quan > Sent: Thursday, December 10, 2015 5:33 PM > > If IOTLB/Context/IETC flush is timeout, we should think > all devices under this IOMMU cannot function correctly. > So for each device under this IOMMU we'll mark it as > unassignable and kill the domain owning the device. > > If De

Re: [Xen-devel] [PATCH v2 2/2] VT-d: Fix vt-d flush timeout issue.

2015-12-10 Thread Xu, Quan
On 11.12.2015 at 3:05pm, Andrew Cooper wrote: > On 10/12/15 09:33, Quan Xu wrote: > > diff --git a/xen/include/xen/pci.h b/xen/include/xen/pci.h index > > a5aef55..0bf6b1a 100644 > > --- a/xen/include/xen/pci.h > > +++ b/xen/include/xen/pci.h > > @@ -41,6 +41,7 @@ > > struct pci_dev_info { > >

Re: [Xen-devel] [PATCH v2 2/2] VT-d: Fix vt-d flush timeout issue.

2015-12-10 Thread Andrew Cooper
On 10/12/15 09:33, Quan Xu wrote: > diff --git a/xen/include/xen/pci.h b/xen/include/xen/pci.h > index a5aef55..0bf6b1a 100644 > --- a/xen/include/xen/pci.h > +++ b/xen/include/xen/pci.h > @@ -41,6 +41,7 @@ > struct pci_dev_info { > bool_t is_extfn; > bool_t is_virtfn; > +bool_t is_u

[Xen-devel] [PATCH v2 2/2] VT-d: Fix vt-d flush timeout issue.

2015-12-10 Thread Quan Xu
If IOTLB/Context/IETC flush is timeout, we should think all devices under this IOMMU cannot function correctly. So for each device under this IOMMU we'll mark it as unassignable and kill the domain owning the device. If Device-TLB flush is timeout, we'll mark the target ATS device as unassignable