Skip to content

Commit 19833ae

Browse files
committed
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/[email protected]/ Signed-off-by: Jakub Kicinski <[email protected]>
2 parents 4b6c606 + d0c0064 commit 19833ae

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

kernel/jump_label.c

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -115,8 +115,6 @@ EXPORT_SYMBOL_GPL(static_key_count);
115115

116116
void static_key_slow_inc_cpuslocked(struct static_key *key)
117117
{
118-
int v, v1;
119-
120118
STATIC_KEY_CHECK_USE(key);
121119
lockdep_assert_cpus_held();
122120

@@ -132,11 +130,9 @@ void static_key_slow_inc_cpuslocked(struct static_key *key)
132130
* so it counts as "enabled" in jump_label_update(). Note that
133131
* atomic_inc_unless_negative() checks >= 0, so roll our own.
134132
*/
135-
for (v = atomic_read(&key->enabled); v > 0; v = v1) {
136-
v1 = atomic_cmpxchg(&key->enabled, v, v + 1);
137-
if (likely(v1 == v))
133+
for (int v = atomic_read(&key->enabled); v > 0; )
134+
if (likely(atomic_try_cmpxchg(&key->enabled, &v, v + 1)))
138135
return;
139-
}
140136

141137
jump_label_lock();
142138
if (atomic_read(&key->enabled) == 0) {

0 commit comments

Comments
 (0)