diff options
| author | Thomas Gleixner <tglx@linutronix.de> | 2013-02-13 17:13:46 +0100 |
|---|---|---|
| committer | Steven Rostedt <rostedt@goodmis.org> | 2013-04-23 12:25:55 -0400 |
| commit | 7fe852eb40d4b956e753a0b8ce14c90def7d64a5 (patch) | |
| tree | a380e0663966125432ad33d5118da2977c609144 | |
| parent | b18ea0bb87c5e1d120e95a894cf234840149fa0b (diff) | |
| download | linux-rt-7fe852eb40d4b956e753a0b8ce14c90def7d64a5.tar.gz | |
net: Use local_bh_disable in netif_rx_ni()
This code triggers the new WARN in __raise_softirq_irqsoff() though it
actually looks at the softirq pending bit and calls into the softirq
code, but that fits not well with the context related softirq model of
RT. It's correct on mainline though, but going through
local_bh_disable/enable here is not going to hurt badly.
Link: http://lkml.kernel.org/r/1360772031-27186-8-git-send-email-bigeasy@linutronix.de
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
| -rw-r--r-- | net/core/dev.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/net/core/dev.c b/net/core/dev.c index 35fe1a047f6d..1342a4dfa507 100644 --- a/net/core/dev.c +++ b/net/core/dev.c @@ -2964,11 +2964,9 @@ int netif_rx_ni(struct sk_buff *skb) { int err; - migrate_disable(); + local_bh_disable(); err = netif_rx(skb); - if (local_softirq_pending()) - thread_do_softirq(); - migrate_enable(); + local_bh_enable(); return err; } |