On Wed, 2006-15-02 at 09:54 +0100, Patrick McHardy wrote:

[..]
>      [PKT_SCHED]: Convert sch_red to a classful qdisc

This is the only one i have issues with (i.e ACK all the other ones)[1].

I can understand for a RED algorithm to have a set of parameters for
a physical queue (even in the case of multiple virtual queues within
a physical queue); 
What you have introduced is a single set of RED parameters for possibly 
many physical queues/classes. I am trying to make sense of what that means.
 
In addition i am not sure if the idea of mucking around with "limit" as 
a signal to the kernel.
My suggestion is you perhaps make this a new qdisc (and ensure multiple
RED parameter sets - one per class)

cheers,
jamal

[1]I have to put a disclaimer: I am always a stickler as far as RED
is concerned (Thomas can testify to that) because it is very delicate
(not the code rather the algorithm) and i have suffered a lot in the
past getting those parameters tuned. There was a gent who was interested
in putting together regression tests for RED but he keeps disappearing on us.



-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to