On Sat, 2004-10-09 at 12:26, Thomas Graf wrote:
> > 1-4 look good.
> >
> > Caveat to note Dave: both old TC_STAT as well as new TC_STAT2 will be
> > generated. Newer software can ignore TC_STAT and older s/ware can ignore
> > TC_STAT2.
>
> Correct, I tested all these cases and it works perfectly fine.
>
> I also completed porting TCA_XSTATS to the new gnet_stats by
> introducing dump_stats to Qdisc_ops. I ported your requeues patch
> and added a missing update in SFQ qdisc. I will send those changes
> once these have been approved.
Much appreciated.
> I will do the same for classes
> and classifiers, although we will have to talk about it a little
> as soon as you feel better again. I guess the classifier part will
> involve changing the action code.
The action code change is trivial since the dump_stats exists. Located
in cls_api.c in call to copy_stats.
My recomendation is to to not even bother using old API since this is
new code. Go ahead and rip it out and just send TC_STAT2 _only_
Sorry, dont have the energy to do it right now.
>
> > patch 5 is a fix thats independent to 1-4 and should be applied
> > regardless of the status of 1-4
>
> Right, same for the cbq fix I sent a bit later which I have no clue how it is
> possible that such a major bug can live for so long, I've been triggering
> this bug for over 3 years and didn't notice anything until now :->
Youve heard of the TheLinuxWAY(tm) I hope ;-> Otherwise know as
cutnpaste. One has bug others follow ;->
BTW, you would be submitting the iproute2 patches as well, correct ?
i.e you are not just using your tool to do the testing?
cheers,
jamal
|