Re: [PATCH v3 0/6] ATS capability support for Intel IOMMU

2009-03-22 Thread Yu Zhao
On Fri, Mar 20, 2009 at 07:15:51PM +0800, David Woodhouse wrote: On Fri, 2009-03-20 at 10:47 +0800, Zhao, Yu wrote: If it's possible, I'd like it go through the PCI tree because the ATS depends on the SR-IOV. This dependency is not reflected in this v3 series since the SR-IOV is not

Re: [PATCH v3 0/6] ATS capability support for Intel IOMMU

2009-03-19 Thread Jesse Barnes
On Thu, 12 Feb 2009 20:50:32 +0800 Yu Zhao yu.z...@intel.com wrote: This patch series implements Address Translation Service support for the Intel IOMMU. ATS makes the PCI Endpoint be able to request the DMA address translation from the IOMMU and cache the translation in the Endpoint, thus

Re: [PATCH v3 0/6] ATS capability support for Intel IOMMU

2009-03-19 Thread Jesse Barnes
On Thu, 12 Feb 2009 20:50:32 +0800 Yu Zhao yu.z...@intel.com wrote: This patch series implements Address Translation Service support for the Intel IOMMU. ATS makes the PCI Endpoint be able to request the DMA address translation from the IOMMU and cache the translation in the Endpoint, thus

Re: [PATCH v3 0/6] ATS capability support for Intel IOMMU

2009-03-19 Thread Zhao, Yu
Jesse Barnes wrote: On Thu, 12 Feb 2009 20:50:32 +0800 Yu Zhao yu.z...@intel.com wrote: This patch series implements Address Translation Service support for the Intel IOMMU. ATS makes the PCI Endpoint be able to request the DMA address translation from the IOMMU and cache the translation in

Re: [PATCH v3 0/6] ATS capability support for Intel IOMMU

2009-02-26 Thread Grant Grundler
On Thu, Feb 26, 2009 at 10:50:35AM +0800, Yu Zhao wrote: ... Yu, Can you please add something to Documentation/PCI/pci.txt? New API I'm seeing are: +extern int pci_enable_ats(struct pci_dev *dev, int ps); +extern void pci_disable_ats(struct pci_dev *dev); +extern int

Re: [PATCH v3 0/6] ATS capability support for Intel IOMMU

2009-02-25 Thread Yu Zhao
On Sun, Feb 15, 2009 at 06:59:10AM +0800, Grant Grundler wrote: On Thu, Feb 12, 2009 at 08:50:32PM +0800, Yu Zhao wrote: This patch series implements Address Translation Service support for the Intel IOMMU. ATS makes the PCI Endpoint be able to request the DMA address translation from the

Re: [PATCH v3 0/6] ATS capability support for Intel IOMMU

2009-02-25 Thread Greg KH
On Thu, Feb 26, 2009 at 10:50:35AM +0800, Yu Zhao wrote: On Sun, Feb 15, 2009 at 06:59:10AM +0800, Grant Grundler wrote: On Thu, Feb 12, 2009 at 08:50:32PM +0800, Yu Zhao wrote: This patch series implements Address Translation Service support for the Intel IOMMU. ATS makes the PCI

Re: [PATCH v3 0/6] ATS capability support for Intel IOMMU

2009-02-14 Thread Grant Grundler
On Thu, Feb 12, 2009 at 08:50:32PM +0800, Yu Zhao wrote: This patch series implements Address Translation Service support for the Intel IOMMU. ATS makes the PCI Endpoint be able to request the DMA address translation from the IOMMU and cache the translation in the Endpoint, thus alleviate

[PATCH v3 0/6] ATS capability support for Intel IOMMU

2009-02-12 Thread Yu Zhao
This patch series implements Address Translation Service support for the Intel IOMMU. ATS makes the PCI Endpoint be able to request the DMA address translation from the IOMMU and cache the translation in the Endpoint, thus alleviate IOMMU pressure and improve the hardware performance in the I/O

Re: [PATCH v3 0/6] ATS capability support for Intel IOMMU

2009-02-12 Thread Matthew Wilcox
On Thu, Feb 12, 2009 at 08:50:32PM +0800, Yu Zhao wrote: 2, avoid using pci_find_ext_capability every time when reading ATS Invalidate Queue Depth (Matthew Wilcox) er ... I didn't say it was a problem. I said I couldn't tell if it was a problem. There's no point in taking up an extra 4

Re: [PATCH v3 0/6] ATS capability support for Intel IOMMU

2009-02-12 Thread Zhao, Yu
Matthew Wilcox wrote: On Thu, Feb 12, 2009 at 08:50:32PM +0800, Yu Zhao wrote: 2, avoid using pci_find_ext_capability every time when reading ATS Invalidate Queue Depth (Matthew Wilcox) er ... I didn't say it was a problem. I said I couldn't tell if it was a problem. There's no point