diff options
author | Roman Gushchin <[email protected]> | 2024-07-26 20:31:10 +0000 |
---|---|---|
committer | Andrew Morton <[email protected]> | 2024-09-01 20:25:51 -0700 |
commit | 57979fabff554bf4d1edeeee69251b22ca9bf55e (patch) | |
tree | 8a46efc7e788296316151d1142fde163471ae750 | |
parent | 941ce635234162c420c9185816c59f7d5dd1ad07 (diff) |
mm: page_counters: initialize usage using ATOMIC_LONG_INIT() macro
When a page_counter structure is initialized, there is no need to use an
atomic set operation to initialize the usage counter because at this point
the structure is not visible to anybody else. ATOMIC_LONG_INIT() is what
should be used in such cases.
Link: https://lkml.kernel.org/r/[email protected]
Signed-off-by: Roman Gushchin <[email protected]>
Acked-by: Shakeel Butt <[email protected]>
Acked-by: Johannes Weiner <[email protected]>
Cc: Michal Hocko <[email protected]>
Cc: Muchun Song <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
-rw-r--r-- | include/linux/page_counter.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/page_counter.h b/include/linux/page_counter.h index 1b3e92c09b80..66ebf9a73158 100644 --- a/include/linux/page_counter.h +++ b/include/linux/page_counter.h @@ -52,7 +52,7 @@ static inline void page_counter_init(struct page_counter *counter, struct page_counter *parent, bool protection_support) { - atomic_long_set(&counter->usage, 0); + counter->usage = (atomic_long_t)ATOMIC_LONG_INIT(0); counter->max = PAGE_COUNTER_MAX; counter->parent = parent; counter->protection_support = protection_support; |