Re: PATCH: action stats double dip

[jamal <hadi@xxxxxxxxxx>]

On Fri, 2005-03-25 at 16:58, Thomas Graf wrote:
> * jamal <1111786752.1090.678.camel@xxxxxxxxxxxxxxxx> 2005-03-25 16:39

> 
> Agreed, I have to look into this more deeply. As it seems things
> are really messed up anyways. We have the same stats TLV being
> put into TCA_OPTIONS but also into the root array which I think
> lead you to move that TCA_ACT_STATS, both is wrong ;->
> 

The stats for actions seem fine - Andy and to a small extent myself have
verified this. 
What i moved was TCA_ACT_STATS which really belongs to the action
hierachy. The other level still has STATS like before used by qdiscs
etc. I quickly checked those and they looked sane as well. Trust me i
panicked after looking at the code the first time i found the stats were
messed when Andy reported them. Then i realized it was a simple issue of
moving around TCA_ACT_STATS.

cheers,
jamal