On Thu, Dec 08, 2011 at 12:53:33PM -0800, Pravin B Shelar wrote:
> Fixed according to comments from Ben.
> v1-v2:
>         - updated comments
>         - added UINT64_MAX check before updating stats
>         - removed memset from ofproto_port_get_stats()

Looking again, I think you also need to zero the new 'stats' member in
construct(), I don't see anything else that ensures that this is
initialized.

Otherwise, thanks, this looks good.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to