Thread (17 messages) 17 messages, 4 authors, 2005-03-31

Re: cls_u32 compile failure in current 2.6.12-rc1+BK tree

From: jamal <hidden>
Date: 2005-03-28 12:24:18

On Mon, 2005-03-28 at 02:18, Meelis Roos wrote:
quoted
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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help