Hi Dave,
this patch fixes multiple qdiscs to adjust sch->q.qlen when
replacing a non-empty inner qdisc. The only change to the
first patch is in dsmark_graft: this patch directly sets
sch->q.qlen to 0 instead of subtracting the length of the
inner qdisc.
Best regards,
Patrick
# This is a BitKeeper generated patch for the following project:
# Project Name: Linux kernel tree
# This patch format is intended for GNU patch command version 2.5 or higher.
# This patch includes the following deltas:
# ChangeSet 1.1441 -> 1.1442
# net/sched/sch_tbf.c 1.10 -> 1.11
# net/sched/sch_cbq.c 1.13 -> 1.14
# net/sched/sch_dsmark.c 1.10 -> 1.11
# net/sched/sch_prio.c 1.9 -> 1.10
#
# The following is the BitKeeper ChangeSet Log
# --------------------------------------------
# 03/11/18 kaber@xxxxxxxxx 1.1442
# Adjust qlen when grafting in multiple qdiscs
# --------------------------------------------
#
diff -Nru a/net/sched/sch_cbq.c b/net/sched/sch_cbq.c
--- a/net/sched/sch_cbq.c Tue Nov 18 01:18:14 2003
+++ b/net/sched/sch_cbq.c Tue Nov 18 01:18:14 2003
@@ -1670,6 +1670,7 @@
sch_tree_lock(sch);
*old = cl->q;
cl->q = new;
+ sch->q.qlen -= (*old)->q.qlen;
qdisc_reset(*old);
sch_tree_unlock(sch);
diff -Nru a/net/sched/sch_dsmark.c b/net/sched/sch_dsmark.c
--- a/net/sched/sch_dsmark.c Tue Nov 18 01:18:14 2003
+++ b/net/sched/sch_dsmark.c Tue Nov 18 01:18:14 2003
@@ -77,6 +77,7 @@
*old = xchg(&p->q,new);
if (*old)
qdisc_reset(*old);
+ sch->q.qlen = 0;
sch_tree_unlock(sch); /* @@@ move up ? */
return 0;
}
diff -Nru a/net/sched/sch_prio.c b/net/sched/sch_prio.c
--- a/net/sched/sch_prio.c Tue Nov 18 01:18:14 2003
+++ b/net/sched/sch_prio.c Tue Nov 18 01:18:14 2003
@@ -266,6 +266,7 @@
sch_tree_lock(sch);
*old = q->queues[band];
q->queues[band] = new;
+ sch->q.qlen -= (*old)->q.qlen;
qdisc_reset(*old);
sch_tree_unlock(sch);
diff -Nru a/net/sched/sch_tbf.c b/net/sched/sch_tbf.c
--- a/net/sched/sch_tbf.c Tue Nov 18 01:18:14 2003
+++ b/net/sched/sch_tbf.c Tue Nov 18 01:18:14 2003
@@ -455,6 +455,8 @@
sch_tree_lock(sch);
*old = xchg(&q->qdisc, new);
qdisc_reset(*old);
+ sch->q.qlen = 0;
+ sch->stats.backlog = 0;
sch_tree_unlock(sch);
return 0;
|