[ovs-dev] [PATCH] ofproto: Resubmit statistics improperly account during failover.
Ethan Jackson
ethan at nicira.com
Tue May 3 20:40:04 UTC 2011
> ofproto tracks facet stats that have already been accounted into
> OpenFlow rule using dp_packet_count and dp_byte_count. When we
> query the database, only packets or bytes in excess of those
> values should be considered. The 'stat's parameter to
> facet_update_stats() should only be passed those values in excess.
> Some of the callers, however, were not doing this:
> facet_revalidate() failed to subtract already-accounted packets
> and bytes, and so did facet_uninstall().
Yes.
>
> If I'm right about that, then your patch is correct but insufficient:
> we should make a similar change to facet_uninstall(), perhaps
> factoring out the common code into a function.
Good catch. It would be extremely edge case-ish for this to actually
be a noticeable bug, but it's a bug nonetheless.
I'll revise the patch and resend it.
Ethan
More information about the dev
mailing list