From 97a18f548548a6ee1b9be14c6fc72090b8839875 Mon Sep 17 00:00:00 2001 From: Joerg Roedel Date: Tue, 7 May 2019 09:39:32 +0200 Subject: [PATCH] Revert "iommu/amd: Flush not present cache in iommu_map_page" This reverts commit 1a1079011da32db87e19fcb39e70d082f89da921. This commit caused a NULL-ptr deference bug and must be reverted for now. Signed-off-by: Joerg Roedel --- drivers/iommu/amd_iommu.c | 19 ++++--------------- 1 file changed, 4 insertions(+), 15 deletions(-) diff --git a/drivers/iommu/amd_iommu.c b/drivers/iommu/amd_iommu.c index 23c1a7eebb06..fde16c5b0a70 100644 --- a/drivers/iommu/amd_iommu.c +++ b/drivers/iommu/amd_iommu.c @@ -1307,16 +1307,6 @@ static void domain_flush_complete(struct protection_domain *domain) } } -/* Flush the not present cache if it exists */ -static void domain_flush_np_cache(struct protection_domain *domain, - dma_addr_t iova, size_t size) -{ - if (unlikely(amd_iommu_np_cache)) { - domain_flush_pages(domain, iova, size); - domain_flush_complete(domain); - } -} - /* * This function flushes the DTEs for all devices in domain @@ -2399,7 +2389,10 @@ static dma_addr_t __map_single(struct device *dev, } address += offset; - domain_flush_np_cache(&dma_dom->domain, address, size); + if (unlikely(amd_iommu_np_cache)) { + domain_flush_pages(&dma_dom->domain, address, size); + domain_flush_complete(&dma_dom->domain); + } out: return address; @@ -2578,8 +2571,6 @@ static int map_sg(struct device *dev, struct scatterlist *sglist, s->dma_length = s->length; } - domain_flush_np_cache(domain, s->dma_address, s->dma_length); - return nelems; out_unmap: @@ -3060,8 +3051,6 @@ static int amd_iommu_map(struct iommu_domain *dom, unsigned long iova, ret = iommu_map_page(domain, iova, paddr, page_size, prot, GFP_KERNEL); mutex_unlock(&domain->api_lock); - domain_flush_np_cache(domain, iova, page_size); - return ret; } -- 2.45.2