On Mon, Oct 17, 2011 at 12:10:24PM -0700, Ethan Jackson wrote:
> An interface's 'link_resets' column represents the number of times
> Open vSwitch has observed its link_state change.

...
>                  if (dev->carrier != change->running) {
>                      dev->carrier = change->running;
> +                    dev->carrier_resets++;
>                  }

That's the reason for the "if".  OK.

> - "version": "6.1.1",
> - "cksum": "282564184 14747",
> + "version": "7.0.0",
> + "cksum": "1329416557 14885",

Adding a column shouldn't necessitate incrementing the schema major
version.

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

Reply via email to