summaryrefslogtreecommitdiffstats
path: root/include/net/pkt_sched.h
diff options
context:
space:
mode:
authorJarek Poplawski <jarkao2@gmail.com>2008-09-23 01:05:56 -0700
committerDavid S. Miller <davem@davemloft.net>2008-09-23 01:05:56 -0700
commitf4ab543201992fe499bef5c406e09f23aa97b4d5 (patch)
treee5e52782cdc408bd27717d82e87535a24fd516d6 /include/net/pkt_sched.h
parentcd07a8ea0dd4b204919b4c9ced8d9efdd9924495 (diff)
downloadop-kernel-dev-f4ab543201992fe499bef5c406e09f23aa97b4d5.zip
op-kernel-dev-f4ab543201992fe499bef5c406e09f23aa97b4d5.tar.gz
pkt_sched: Remove the tx queue state check in qdisc_run()
The current check wrongly uses the state of one (currently the first) tx queue for all tx queues in case of non-default qdiscs. This check mainly prevented requeuing loop with __netif_schedule(), but now it's controlled inside __qdisc_run(), while dequeuing. The wrongness of this check was first noticed by Herbert Xu. Signed-off-by: Jarek Poplawski <jarkao2@gmail.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/net/pkt_sched.h')
-rw-r--r--include/net/pkt_sched.h5
1 files changed, 1 insertions, 4 deletions
diff --git a/include/net/pkt_sched.h b/include/net/pkt_sched.h
index b786a5b..4082f39 100644
--- a/include/net/pkt_sched.h
+++ b/include/net/pkt_sched.h
@@ -90,10 +90,7 @@ extern void __qdisc_run(struct Qdisc *q);
static inline void qdisc_run(struct Qdisc *q)
{
- struct netdev_queue *txq = q->dev_queue;
-
- if (!netif_tx_queue_stopped(txq) &&
- !test_and_set_bit(__QDISC_STATE_RUNNING, &q->state))
+ if (!test_and_set_bit(__QDISC_STATE_RUNNING, &q->state))
__qdisc_run(q);
}
OpenPOWER on IntegriCloud