[ovs-dev] [PATCH] ofproto: Resubmit statistics improperly account during failover.
Ethan Jackson
ethan at nicira.com
Fri May 6 23:57:56 UTC 2011
Wow, completely forgot to merge this. Just did so.
Thanks for the review,
Ethan
On Tue, May 3, 2011 at 18:42, Ben Pfaff <blp at nicira.com> wrote:
> On Tue, May 03, 2011 at 05:53:22PM -0700, Ethan Jackson wrote:
>> > The only potential issue I see here is with logging on error paths.
>> > If dpif_flow_put() or dpif_flow_del() returns an error then they will
>> > zero their stats arguments, which will then almost certainly cause an
>> > error to be logged in facet_reset_dp_stats(). That's going to be a
>> > bit confusing, although it won't cause any actual problems. If you
>> > see a nice way to avoid it, though, I'd think about it.
>>
>> I think I will just drop the log message. I can't really imagine it
>> ever be helpful in finding a problem given how vague it is, and how
>> many ways it could be triggered.
>
> OK, fair enough.
>
More information about the dev
mailing list