On Mon, 2005-03-28 at 02:18, Meelis Roos wrote:
> > For now just turn on actions under qos menu.
>
> The help tells
> You MUST NOT turn this on if you dont have an update iproute2.
> So I did not turn it on. I have iproute 20041019-3 in Debian unstable.
> Will something break? I'm not using QoS actively, sonly try some setups
> sometimes.
Well, it is NOT supposed to break - but lets find out shall we? ;->
Thomas: I have been doing some thinking.
If we are going to make u32 an exception then this code should be
surrounded by #ifdef ..ACTION..
The rest of the classifiers can also follow the same scheme.
Or the rest of the classifiers can have their code in the generic stats
(and removed from within the classifiers).
Which means that the other classifiers may have to use a slightly
different function name unless you want to pass a variable to
distinguish the two cases. My time is about to become scarce, so if you
have time please create the patch.
cheers,
jamal
|