aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBaoquan He <bhe@redhat.com>2016-09-15 04:50:50 -0400
committerJoerg Roedel <jroedel@suse.de>2016-09-19 09:49:01 -0400
commit134414fffc7fdad81e228d232f8ae12a0456127a (patch)
tree62e06f833b35b06c524cfeac4f3ca5d2924ae78a
parent368d06cd536ecace3a904ae2e6e42771636d52aa (diff)
iommu/amd: Clean up the cmpxchg64 invocation
Change it as it's designed for and keep it consistent with other places. Signed-off-by: Baoquan He <bhe@redhat.com> Signed-off-by: Joerg Roedel <jroedel@suse.de>
-rw-r--r--drivers/iommu/amd_iommu.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/iommu/amd_iommu.c b/drivers/iommu/amd_iommu.c
index bebe728019af..897940c79a54 100644
--- a/drivers/iommu/amd_iommu.c
+++ b/drivers/iommu/amd_iommu.c
@@ -1342,7 +1342,8 @@ static u64 *alloc_pte(struct protection_domain *domain,
1342 1342
1343 __npte = PM_LEVEL_PDE(level, virt_to_phys(page)); 1343 __npte = PM_LEVEL_PDE(level, virt_to_phys(page));
1344 1344
1345 if (cmpxchg64(pte, __pte, __npte)) { 1345 /* pte could have been changed somewhere. */
1346 if (cmpxchg64(pte, __pte, __npte) != __pte) {
1346 free_page((unsigned long)page); 1347 free_page((unsigned long)page);
1347 continue; 1348 continue;
1348 } 1349 }