diff options
| author | Sebastian Andrzej Siewior <bigeasy@linutronix.de> | 2016-02-09 18:17:18 +0100 |
|---|---|---|
| committer | Steven Rostedt <rostedt@goodmis.org> | 2016-11-16 14:23:17 -0500 |
| commit | f610d65cafe21adda6826f7f3257dc499660cb16 (patch) | |
| tree | 8e819236b2100e0ebdf2da4fa46b78aeac42aa2a | |
| parent | 834cd01b4d8c3ace26c9dfa9cc556d4f7fd59f2e (diff) | |
| download | linux-rt-f610d65cafe21adda6826f7f3257dc499660cb16.tar.gz | |
kernel: softirq: unlock with irqs on
We unlock the lock while the interrupts are off. This isn't a problem
now but will get because the migrate_disable() + enable are not
symmetrical in regard to the status of interrupts.
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
| -rw-r--r-- | kernel/softirq.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/softirq.c b/kernel/softirq.c index ecf0ab0f7b4d..d2c870a826da 100644 --- a/kernel/softirq.c +++ b/kernel/softirq.c @@ -561,10 +561,10 @@ static int __thread_do_softirq(int cpu) */ if (local_softirq_pending()) __do_softirq_common(cpu >= 0); - local_unlock(local_softirq_lock); unpin_current_cpu(); - preempt_disable(); local_irq_enable(); + local_unlock(local_softirq_lock); + preempt_disable(); return 0; } |