]> asedeno.scripts.mit.edu Git - linux.git/commit
locking/static_key: Don't take sleeping locks in __static_key_slow_dec_deferred()
authorJakub Kicinski <jakub.kicinski@netronome.com>
Sat, 30 Mar 2019 00:08:54 +0000 (17:08 -0700)
committerIngo Molnar <mingo@kernel.org>
Mon, 29 Apr 2019 06:29:21 +0000 (08:29 +0200)
commit94b5f312cfb4a66055d9b688dc9ab6b297eb9dcc
tree7a4dc14eb16fa750262bb6cc3252d0e106fea8d4
parentb92e793bbe4a1c49dbf78d8d526561e7a7dd568a
locking/static_key: Don't take sleeping locks in __static_key_slow_dec_deferred()

Changing jump_label state is protected by jump_label_lock().
Rate limited static_key_slow_dec(), however, will never
directly call jump_label_update(), it will schedule a delayed
work instead.  Therefore it's unnecessary to take both the
cpus_read_lock() and jump_label_lock().

This allows static_key_slow_dec_deferred() to be called
from atomic contexts, like socket destructing in net/tls,
without the need for another indirection.

Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Reviewed-by: Simon Horman <simon.horman@netronome.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Will Deacon <will.deacon@arm.com>
Cc: alexei.starovoitov@gmail.com
Cc: ard.biesheuvel@linaro.org
Cc: oss-drivers@netronome.com
Cc: yamada.masahiro@socionext.com
Link: https://lkml.kernel.org/r/20190330000854.30142-4-jakub.kicinski@netronome.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
kernel/jump_label.c