Re: [PATCH v8 2/2] iommu/amd: Add basic debugfs infrastructure for AMD IOMMU

2018-06-04 Thread Randy Dunlap
On 05/29/2018 11:39 AM, Greg KH wrote: > On Tue, May 29, 2018 at 01:23:23PM -0500, Gary R Hook wrote: >> Implement a skeleton framework for debugfs support in the >> AMD IOMMU. Add a hidden boolean to Kconfig that is defined >> for the AMD IOMMU when general IOMMY DebugFS support is >> enabled. >>

Re: [PATCH 5/7] iommu/dma: add support for non-strict mode

2018-06-04 Thread Leizhen (ThunderTown)
On 2018/6/2 1:51, kbuild test robot wrote: > Hi Zhen, > > Thank you for the patch! Perhaps something to improve: > > [auto build test WARNING on linus/master] > [also build test WARNING on v4.17-rc7 next-20180601] > [cannot apply to iommu/next] > [if your patch is applied to the wrong git

Re: [PATCH 5/7] iommu/dma: add support for non-strict mode

2018-06-04 Thread Leizhen (ThunderTown)
On 2018/5/31 21:04, Robin Murphy wrote: > On 31/05/18 08:42, Zhen Lei wrote: >> 1. Save the related domain pointer in struct iommu_dma_cookie, make iovad >> capable call domain->ops->flush_iotlb_all to flush TLB. >> 2. Define a new iommu capable: IOMMU_CAP_NON_STRICT, which used to indicate

Re: [PATCH 4/7] iommu/amd: make sure TLB to be flushed before IOVA freed

2018-06-04 Thread Leizhen (ThunderTown)
On 2018/5/31 21:04, Robin Murphy wrote: > On 31/05/18 08:42, Zhen Lei wrote: >> Although the mapping has already been removed in the page table, it maybe >> still exist in TLB. Suppose the freed IOVAs is reused by others before the >> flush operation completed, the new user can not correctly

Re: [PATCH 3/7] iommu: prepare for the non-strict mode support

2018-06-04 Thread Leizhen (ThunderTown)
On 2018/5/31 21:04, Robin Murphy wrote: > On 31/05/18 08:42, Zhen Lei wrote: >> In common, a IOMMU unmap operation follow the below steps: >> 1. remove the mapping in page table of the specified iova range >> 2. execute tlbi command to invalid the mapping which is cached in TLB >> 3. wait for

Re: [PATCH 1/7] iommu/dma: fix trival coding style mistake

2018-06-04 Thread Leizhen (ThunderTown)
On 2018/5/31 21:03, Robin Murphy wrote: > On 31/05/18 08:42, Zhen Lei wrote: >> The static function iova_reserve_iommu_regions is only called by function >> iommu_dma_init_domain, and the 'if (!dev)' check in iommu_dma_init_domain >> affect it only, so we can safely move the check into it. I

[GIT PULL] dma-mapping updates for Linux 4.18

2018-06-04 Thread Christoph Hellwig
Hi Linus, please pull dma dma-mapping update below. Note that this includes a lot of changes to the architecture Kconfig files, which have created quite a few trivial conflicts in linux-next. In all the cases there is no actual interaction, just separate additions/removal that are right next to