diff options
| author | Thomas Gleixner <tglx@linutronix.de> | 2013-02-21 12:17:42 -0500 |
|---|---|---|
| committer | Thomas Gleixner <tglx@linutronix.de> | 2013-02-21 14:52:24 -0500 |
| commit | facd8b80c67a3cf64a467c4a2ac5fb31f2e6745b (patch) | |
| tree | 5feb802348c3fe708b9ea562e5e6b3c753ad7fe6 | |
| parent | 74eed0163d0def3fce27228d9ccf3d36e207b286 (diff) | |
irq: Sanitize invoke_softirq
With the irq protection in irq_exit, we can remove the #ifdeffery and
the bh_disable/enable dance in invoke_softirq()
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Cc: Linus Torvalds <torvalds@linuxfoundation.org>
Link: http://lkml.kernel.org/r/alpine.LFD.2.02.1302202155320.22263@ionos
| -rw-r--r-- | kernel/softirq.c | 12 |
1 files changed, 2 insertions, 10 deletions
diff --git a/kernel/softirq.c b/kernel/softirq.c index f2a934673008..24a921bcf04f 100644 --- a/kernel/softirq.c +++ b/kernel/softirq.c | |||
| @@ -322,18 +322,10 @@ void irq_enter(void) | |||
| 322 | 322 | ||
| 323 | static inline void invoke_softirq(void) | 323 | static inline void invoke_softirq(void) |
| 324 | { | 324 | { |
| 325 | if (!force_irqthreads) { | 325 | if (!force_irqthreads) |
| 326 | #ifdef __ARCH_IRQ_EXIT_IRQS_DISABLED | ||
| 327 | __do_softirq(); | 326 | __do_softirq(); |
| 328 | #else | 327 | else |
| 329 | do_softirq(); | ||
| 330 | #endif | ||
| 331 | } else { | ||
| 332 | __local_bh_disable((unsigned long)__builtin_return_address(0), | ||
| 333 | SOFTIRQ_OFFSET); | ||
| 334 | wakeup_softirqd(); | 328 | wakeup_softirqd(); |
| 335 | __local_bh_enable(SOFTIRQ_OFFSET); | ||
| 336 | } | ||
| 337 | } | 329 | } |
| 338 | 330 | ||
| 339 | /* | 331 | /* |
