Skip to content

Commit

Permalink
mm/frontswap.c: use non-atomic '__set_bit()' when possible
Browse files Browse the repository at this point in the history
The 'a' and 'b' bitmaps are local to this function, so no concurrent
access can occur.  So the non-atomic '__set_bit()' can be used to save a
few cycles.

Link: https://lkml.kernel.org/r/e52476da5cee57151745c5c3c934a69798dc6fa4.1638132190.git.christophe.jaillet@wanadoo.fr
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
  • Loading branch information
Christophe JAILLET authored and Linus Torvalds committed Jan 15, 2022
1 parent 62c9827 commit 3795f46
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions mm/frontswap.c
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,7 @@ void frontswap_register_ops(struct frontswap_ops *ops)
spin_lock(&swap_lock);
plist_for_each_entry(si, &swap_active_head, list) {
if (!WARN_ON(!si->frontswap_map))
set_bit(si->type, a);
__set_bit(si->type, a);
}
spin_unlock(&swap_lock);

Expand All @@ -149,7 +149,7 @@ void frontswap_register_ops(struct frontswap_ops *ops)
spin_lock(&swap_lock);
plist_for_each_entry(si, &swap_active_head, list) {
if (si->frontswap_map)
set_bit(si->type, b);
__set_bit(si->type, b);
}
spin_unlock(&swap_lock);

Expand Down

0 comments on commit 3795f46

Please sign in to comment.