diff options
author | Peter Zijlstra <peterz@infradead.org> | 2022-10-21 14:47:29 +0300 |
---|---|---|
committer | Dave Hansen <dave.hansen@linux.intel.com> | 2022-12-15 21:37:27 +0300 |
commit | eb780dcae02d5a71e6979aa7b8c708dea8597adf (patch) | |
tree | 78eb2b5232da53c6f6e2acb27042cc2544f89dd2 | |
parent | d4a72e7fe61a1ea9ad4accf3532411ca685eaead (diff) | |
download | linux-eb780dcae02d5a71e6979aa7b8c708dea8597adf.tar.xz |
mm: Remove pointless barrier() after pmdp_get_lockless()
pmdp_get_lockless() should itself imply any ordering required.
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Link: https://lkml.kernel.org/r/20221022114425.298833095%40infradead.org
-rw-r--r-- | mm/hmm.c | 1 | ||||
-rw-r--r-- | mm/vmscan.c | 3 |
2 files changed, 0 insertions, 4 deletions
@@ -362,7 +362,6 @@ again: * values. */ pmd = pmdp_get_lockless(pmdp); - barrier(); if (!pmd_devmap(pmd) && !pmd_trans_huge(pmd)) goto again; diff --git a/mm/vmscan.c b/mm/vmscan.c index 88ef873b2d83..4936a88bb26a 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c @@ -4041,9 +4041,6 @@ restart: for (i = pmd_index(start), addr = start; addr != end; i++, addr = next) { pmd_t val = pmdp_get_lockless(pmd + i); - /* for pmdp_get_lockless() */ - barrier(); - next = pmd_addr_end(addr, end); if (!pmd_present(val) || is_huge_zero_pmd(val)) { |