diff options
| author |   Jakub Kicinski <kuba@kernel.org> | 2022-12-01 15:36:27 -0800 |
|---|---|---|
| committer | Jakub Kicinski <kuba@kernel.org> | 2022-12-01 15:36:27 -0800 |
| commit | 19833ae2703d4b5ca184db50a11351cfa611da01 (patch) | |
| tree | 891b142600f0367a9a003ceb64059bbcbaf42aa5 | |
| parent | r8169: use tp_to_dev instead of open code (diff) | |
| parent | jump_label: Use atomic_try_cmpxchg() in static_key_slow_inc_cpuslocked() (diff) | |
| download | wireguard-linux-19833ae2703d4b5ca184db50a11351cfa611da01.tar.xz wireguard-linux-19833ae2703d4b5ca184db50a11351cfa611da01.zip | |
Merge branch 'locking/core' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip
Pull in locking/core from tip (just a single patch) to avoid a conflict
with a jump_label change needed by a TCP cleanup.
Link: https://lore.kernel.org/all/Y4B17nBArWS1Iywo@hirez.programming.kicks-ass.net/
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
| -rw-r--r-- | kernel/jump_label.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/kernel/jump_label.c b/kernel/jump_label.c index 714ac4c3b556..4d6c6f5f60db 100644 --- a/kernel/jump_label.c +++ b/kernel/jump_label.c @@ -115,8 +115,6 @@ EXPORT_SYMBOL_GPL(static_key_count); void static_key_slow_inc_cpuslocked(struct static_key *key) { - int v, v1; - STATIC_KEY_CHECK_USE(key); lockdep_assert_cpus_held(); @@ -132,11 +130,9 @@ void static_key_slow_inc_cpuslocked(struct static_key *key) * so it counts as "enabled" in jump_label_update(). Note that * atomic_inc_unless_negative() checks >= 0, so roll our own. */ - for (v = atomic_read(&key->enabled); v > 0; v = v1) { - v1 = atomic_cmpxchg(&key->enabled, v, v + 1); - if (likely(v1 == v)) + for (int v = atomic_read(&key->enabled); v > 0; ) + if (likely(atomic_try_cmpxchg(&key->enabled, &v, v + 1))) return; - } jump_label_lock(); if (atomic_read(&key->enabled) == 0) { |