]> git.baikalelectronics.ru Git - kernel.git/commit
signal: Remove no longer required irqsave/restore
authorAnna-Maria Gleixner <anna-maria@linutronix.de>
Fri, 25 May 2018 09:05:07 +0000 (11:05 +0200)
committerThomas Gleixner <tglx@linutronix.de>
Sun, 10 Jun 2018 04:14:01 +0000 (06:14 +0200)
commitadbe27b4f1cdfb38a2db4cff8b7619f60a574160
treee4a46207c2932db28237039791f11790f266a1d2
parent499db6bf97fab32d325289927702f3488b17018e
signal: Remove no longer required irqsave/restore

Commit 520408f4fc1e ("signal: align __lock_task_sighand() irq disabling and
RCU") introduced a rcu read side critical section with interrupts
disabled. The changelog suggested that a better long-term fix would be "to
make rt_mutex_unlock() disable irqs when acquiring the rt_mutex structure's
->wait_lock".

This long-term fix has been made in commit beb9b5554a1e ("rtmutex: Make
wait_lock irq safe") for a different reason.

Therefore revert commit 520408f4fc1e ("signal: align >
__lock_task_sighand() irq disabling and RCU") as the interrupt disable
dance is not longer required.

The change was tested on the base of beb9b5554a1e ("rtmutex: Make wait_lock
irq safe") with a four hour run of rcutorture scenario TREE03 with lockdep
enabled as suggested by Paul McKenney.

Signed-off-by: Anna-Maria Gleixner <anna-maria@linutronix.de>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Acked-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Acked-by: "Eric W. Biederman" <ebiederm@xmission.com>
Cc: bigeasy@linutronix.de
Link: https://lkml.kernel.org/r/20180525090507.22248-3-anna-maria@linutronix.de
kernel/signal.c