diff options
author | Baoquan He <bhe@redhat.com> | 2016-09-15 04:50:50 -0400 |
---|---|---|
committer | Joerg Roedel <jroedel@suse.de> | 2016-09-19 09:49:01 -0400 |
commit | 134414fffc7fdad81e228d232f8ae12a0456127a (patch) | |
tree | 62e06f833b35b06c524cfeac4f3ca5d2924ae78a | |
parent | 368d06cd536ecace3a904ae2e6e42771636d52aa (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.c | 3 |
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 | } |