Re: [ovs-discuss] Inaccurate statistics in flow_removed struct using OpenvSwitch

2011-05-18 Thread Ben Pfaff
It's busted again due to the merge from next. I have a patch out to fix it again, though. (I haven't tested the accuracy, though.) On Wed, May 18, 2011 at 03:22:51PM -0700, kk yap wrote: > Hi, > > Thanks for the very fast response. TY tested the code and it worked. > > Regards > KK > > On 18

Re: [ovs-discuss] Inaccurate statistics in flow_removed struct using OpenvSwitch

2011-05-18 Thread kk yap
Hi, Thanks for the very fast response. TY tested the code and it worked. Regards KK On 18 May 2011 12:55, Ethan Jackson wrote: >> Looking at the code, ovs-openflowd returns facet + stat as packet >> count for the flow removed.  The number recorded by facet is correct, >> but the sum of the val

Re: [ovs-discuss] Inaccurate statistics in flow_removed struct using OpenvSwitch

2011-05-18 Thread Ethan Jackson
> Looking at the code, ovs-openflowd returns facet + stat as packet > count for the flow removed.  The number recorded by facet is correct, > but the sum of the value exceed the real value.  This occurs at the > facet_update_stats function.  What is the purpose of the addition? > Can we expect accu

Re: [ovs-discuss] Inaccurate statistics in flow_removed struct using OpenvSwitch

2011-05-18 Thread Ethan Jackson
Thanks for reporting this issue. I've CCd you on a fix which should be reviewed and merged shortly. Ethan On Tue, May 17, 2011 at 17:55, kk yap wrote: > Hi, > > We are using OVS as a software switch, with last commit > aa77020c361ffba011d7ec9603caa55aaa01cd1f. > > When a flow of ten pings is se