2013-06-17 05:53:09

by Li, ZhenHua

[permalink] [raw]
Subject: [PATCH 1/1] x86/iommu: remove useless code

From: "Li, Zhen-Hua" <[email protected]>

In intel-iommu.c, the function dma_pte_addr does not change the
value of its parameter (pte). So it is useless if it nobody catches
its return value.

Signed-off-by: Li, Zhen-Hua <[email protected]>
---
drivers/iommu/intel-iommu.c | 1 -
1 file changed, 1 deletion(-)

diff --git a/drivers/iommu/intel-iommu.c b/drivers/iommu/intel-iommu.c
index b4f0e28..d9d378b 100644
--- a/drivers/iommu/intel-iommu.c
+++ b/drivers/iommu/intel-iommu.c
@@ -809,7 +809,6 @@ static struct dma_pte *pfn_to_dma_pte(struct dmar_domain *domain,
/* Someone else set it while we were thinking; use theirs. */
free_pgtable_page(tmp_page);
} else {
- dma_pte_addr(pte);
domain_flush_cache(domain, pte, sizeof(*pte));
}
}
--
1.7.10.4