diff options
author | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2014-05-05 17:18:30 +0200 |
---|---|---|
committer | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2014-07-09 18:14:33 +0200 |
commit | c41247e1d4864c863ee25e029dd53acdb2abc000 (patch) | |
tree | 86011181ada393aadad02a79085678bda508d2cc /init | |
parent | rcu: Handle obsolete references to TINY_PREEMPT_RCU (diff) | |
download | linux-c41247e1d4864c863ee25e029dd53acdb2abc000.tar.xz linux-c41247e1d4864c863ee25e029dd53acdb2abc000.zip |
signal: Explain local_irq_save() call
The explicit local_irq_save() in __lock_task_sighand() is needed to avoid
a potential deadlock condition, as noted in a841796f11c90d53 (signal:
align __lock_task_sighand() irq disabling and RCU). However, someone
reading the code might be forgiven for concluding that this separate
local_irq_save() was completely unnecessary. This commit therefore adds
a comment referencing the shiny new block comment on rcu_read_unlock().
Reported-by: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Acked-by: Oleg Nesterov <oleg@redhat.com>
Reviewed-by: Lai Jiangshan <laijs@cn.fujitsu.com>
Diffstat (limited to 'init')
0 files changed, 0 insertions, 0 deletions