Re: [Xen-devel] [PATCH v8 09/11] vt-d: fix the IOMMU flush issue

2016-06-14 Thread Tian, Kevin
> From: Xu, Quan > Sent: Tuesday, June 14, 2016 5:04 PM > > On June 14, 2016 4:27 PM, Jan Beulich wrote: > > >>> On 14.06.16 at 10:10, wrote: > > > On June 13, 2016 11:52 PM, Jan Beulich wrote: > > >> >>> "Xu, Quan" 06/13/16 5:22 PM >>> > > >> >From: Quan Xu > > >> >@@ -546,17 +550,37 @@ stat

Re: [Xen-devel] [PATCH v8 09/11] vt-d: fix the IOMMU flush issue

2016-06-14 Thread Xu, Quan
On June 14, 2016 4:27 PM, Jan Beulich wrote: > >>> On 14.06.16 at 10:10, wrote: > > On June 13, 2016 11:52 PM, Jan Beulich wrote: > >> >>> "Xu, Quan" 06/13/16 5:22 PM >>> > >> >From: Quan Xu > >> >@@ -546,17 +550,37 @@ static int __must_check iommu_flush_all(void) > >> >struct acpi_drhd_unit

Re: [Xen-devel] [PATCH v8 09/11] vt-d: fix the IOMMU flush issue

2016-06-14 Thread Jan Beulich
>>> On 14.06.16 at 10:10, wrote: > On June 13, 2016 11:52 PM, Jan Beulich wrote: >> >>> "Xu, Quan" 06/13/16 5:22 PM >>> >> >From: Quan Xu >> >@@ -546,17 +550,37 @@ static int __must_check iommu_flush_all(void) >> >struct acpi_drhd_unit *drhd; struct iommu *iommu; int flush_dev_iotlb; >> >+i

Re: [Xen-devel] [PATCH v8 09/11] vt-d: fix the IOMMU flush issue

2016-06-14 Thread Xu, Quan
On June 13, 2016 11:52 PM, Jan Beulich wrote: > >>> "Xu, Quan" 06/13/16 5:22 PM >>> > >From: Quan Xu > >@@ -546,17 +550,37 @@ static int __must_check iommu_flush_all(void) > >struct acpi_drhd_unit *drhd; struct iommu *iommu; int flush_dev_iotlb; > >+int rc = 0; > > > >flush_all_cache(); > >

Re: [Xen-devel] [PATCH v8 09/11] vt-d: fix the IOMMU flush issue

2016-06-13 Thread Jan Beulich
>>> "Xu, Quan" 06/13/16 5:22 PM >>> >From: Quan Xu > >The propagation value from IOMMU flush interfaces may be positive, which >indicates callers need to flush cache, not one of faliures. > >when the propagation value is positive, this patch fixes this flush issue >as follows: >- call iommu_flush

[Xen-devel] [PATCH v8 09/11] vt-d: fix the IOMMU flush issue

2016-06-13 Thread Xu, Quan
From: Quan Xu The propagation value from IOMMU flush interfaces may be positive, which indicates callers need to flush cache, not one of faliures. when the propagation value is positive, this patch fixes this flush issue as follows: - call iommu_flush_write_buffer() to flush cache. - return