> > Yes. This is the tc command I used to configure the qdisc (with
> > q_rr.c attached from my patches iproute2 package):
> >
> > # tc qdisc add dev eth2 root handle 1: rr bands 8 RTNETLINK
> answers:
> > No such file or directory
>
> Again, I bet you don't have CONFIG_NET_SCH_RR enabled:
Ch
Waskiewicz Jr, Peter P wrote:
The code looks correct. Are you sure you had the config
option enabled during your test?
Yes. This is the tc command I used to configure the qdisc (with q_rr.c
attached from my patches iproute2 package):
# tc qdisc add dev eth2 root handle 1: rr bands 8
RTNETLI
> Waskiewicz Jr, Peter P wrote:
> >> Please post the code.
> >>
> >>
> >
> > Code is attached. Please forgive the attachment and any whitespace
> > damage...currently using Doubtlook to send this (cringe).
> >
>
> The code looks correct. Are you sure you had the config
> option enabled
Waskiewicz Jr, Peter P wrote:
Please post the code.
Code is attached. Please forgive the attachment and any whitespace
damage...currently using Doubtlook to send this (cringe).
The code looks correct. Are you sure you had the config option
enabled during your test?
-
To unsubscribe
> Please post the code.
>
Code is attached. Please forgive the attachment and any whitespace
damage...currently using Doubtlook to send this (cringe).
Thanks,
-PJ Waskiewicz
sch_prio.c
Description: sch_prio.c
Waskiewicz Jr, Peter P wrote:
BTw, couldn't you just merge sch_rr with prio? AFAICT you
only need a new dequeue function, a new struct Qdisc_ops and
a MODULE_ALIAS.
Ok, I have this somewhat working, but need to poll for some help from
the community. I used MODULE_ALIAS("sch_rr") in sch_prio.c
> BTw, couldn't you just merge sch_rr with prio? AFAICT you
> only need a new dequeue function, a new struct Qdisc_ops and
> a MODULE_ALIAS.
Ok, I have this somewhat working, but need to poll for some help from
the community. I used MODULE_ALIAS("sch_rr") in sch_prio.c, and
modprobe is happily
Waskiewicz Jr, Peter P wrote:
Nested netlink attributes, like most qdisc use, instead of
struct tc_rr_qopt (or additionally). The way you've done it
makes it hard to add further attributes later.
I'm going to need to think about this more, since I'm not immediately
getting what you're ref
> > Hmm. This is the sch_prio from the first 2.6.23-dev tree. I'll
> > resync and make sure it's the correct one.
>
> Current 2.6.22-rc and net-2.6.23 have
>
> if (band >= q->bands)
I just pulled 2.6.23 down, and see that is true. I must have had that
left over. I'll fix that.
> >
Waskiewicz Jr, Peter P wrote:
>>>+band = TC_H_MIN(band) - 1;
>>>+if (band > q->bands) {
>>
>>You copied an off-by-one from an old sch_prio version here.
>
>
> Hmm. This is the sch_prio from the first 2.6.23-dev tree. I'll resync
> and make sure it's the correct one.
Current 2.6.22-rc a
> PJ Waskiewicz wrote:
> >
> > diff --git a/net/sched/sch_prio.c b/net/sched/sch_prio.c index
> > 6d7542c..44ecdc6 100644
> > --- a/net/sched/sch_prio.c
> > +++ b/net/sched/sch_prio.c
> > }
> > +#ifdef CONFIG_NET_SCH_PRIO_MQ
> > + /* setup queue to band mapping */
> > + if (q->bands < sch-
PJ Waskiewicz wrote:
diff --git a/net/sched/sch_prio.c b/net/sched/sch_prio.c
index 6d7542c..44ecdc6 100644
--- a/net/sched/sch_prio.c
+++ b/net/sched/sch_prio.c
}
+#ifdef CONFIG_NET_SCH_PRIO_MQ
+ /* setup queue to band mapping */
+ if (q->bands < sch->dev->egress_subqueue_co
Add the new sch_rr qdisc for multiqueue network device support.
Allow sch_prio to be compiled with or without multiqueue hardware
support.
Signed-off-by: Peter P Waskiewicz Jr <[EMAIL PROTECTED]>
---
include/linux/pkt_sched.h | 11 +
net/sched/Kconfig | 22 ++
net/sched/Makefile
13 matches
Mail list logo
