diff options
author | Alexander Duyck <alexander.h.duyck@intel.com> | 2008-09-21 01:07:34 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-09-21 01:07:34 -0400 |
commit | a574420ff46cff0245e6b0fce2e961aa2717743d (patch) | |
tree | 5baf2134ac902803152c92733f8ed580dd9c7b75 /net/sched/sch_multiq.c | |
parent | 618d9f25548ba6fc3a9cd2ce5cd56f4f015b0635 (diff) |
multiq: requeue should rewind the current_band
Currently dequeueing a packet and requeueing the same packet will cause a
different packet to be pulled on the next dequeue. This change forces
requeue to rewind the current_band.
Signed-off-by: Alexander Duyck <alexander.h.duyck@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sched/sch_multiq.c')
-rw-r--r-- | net/sched/sch_multiq.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/net/sched/sch_multiq.c b/net/sched/sch_multiq.c index 5d9cd68e91d..915f3149dde 100644 --- a/net/sched/sch_multiq.c +++ b/net/sched/sch_multiq.c | |||
@@ -97,6 +97,7 @@ static int | |||
97 | multiq_requeue(struct sk_buff *skb, struct Qdisc *sch) | 97 | multiq_requeue(struct sk_buff *skb, struct Qdisc *sch) |
98 | { | 98 | { |
99 | struct Qdisc *qdisc; | 99 | struct Qdisc *qdisc; |
100 | struct multiq_sched_data *q = qdisc_priv(sch); | ||
100 | int ret; | 101 | int ret; |
101 | 102 | ||
102 | qdisc = multiq_classify(skb, sch, &ret); | 103 | qdisc = multiq_classify(skb, sch, &ret); |
@@ -113,6 +114,10 @@ multiq_requeue(struct sk_buff *skb, struct Qdisc *sch) | |||
113 | if (ret == NET_XMIT_SUCCESS) { | 114 | if (ret == NET_XMIT_SUCCESS) { |
114 | sch->q.qlen++; | 115 | sch->q.qlen++; |
115 | sch->qstats.requeues++; | 116 | sch->qstats.requeues++; |
117 | if (q->curband) | ||
118 | q->curband--; | ||
119 | else | ||
120 | q->curband = q->bands - 1; | ||
116 | return NET_XMIT_SUCCESS; | 121 | return NET_XMIT_SUCCESS; |
117 | } | 122 | } |
118 | if (net_xmit_drop_count(ret)) | 123 | if (net_xmit_drop_count(ret)) |