netdev: Remove warning from __netif_schedule().
authorDavid S. Miller <davem@davemloft.net>
Wed, 23 Jul 2008 21:01:29 +0000 (14:01 -0700)
committerDavid S. Miller <davem@davemloft.net>
Wed, 23 Jul 2008 21:01:29 +0000 (14:01 -0700)
It isn't helping anything and we aren't going to be able to change all
the drivers that do queue wakeups in strange situations.

Just letting a noop_qdisc get scheduled will work because when
qdisc_run() executes via net_tx_work() it will simply find no packets
pending when it makes the ->dequeue() call in qdisc_restart.

Signed-off-by: David S. Miller <davem@davemloft.net>
net/core/dev.c

index 6bf217da9d8f8e90c188b52bd1d6e1aaa0f3dc1b..ccf97f9f37ebeea2b55fa12c2c48cceb912ac772 100644 (file)
@@ -1341,9 +1341,6 @@ static void dev_queue_xmit_nit(struct sk_buff *skb, struct net_device *dev)
 
 void __netif_schedule(struct Qdisc *q)
 {
-       if (WARN_ON_ONCE(q == &noop_qdisc))
-               return;
-
        if (!test_and_set_bit(__QDISC_STATE_SCHED, &q->state)) {
                struct softnet_data *sd;
                unsigned long flags;