As they say: Only thing worse than no comment is an incorrect one.
Looks good, thanks.
Ethan
On Tue, Nov 15, 2011 at 17:17, Ben Pfaff <b...@nicira.com> wrote:
> ---
> vswitchd/bridge.c | 7 +++----
> 1 files changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c
> index 6a25b95..8736962 100644
> --- a/vswitchd/bridge.c
> +++ b/vswitchd/bridge.c
> @@ -373,8 +373,8 @@ bridge_reconfigure(const struct ovsrec_open_vswitch
> *ovs_cfg)
> * iface"s according to 'ovs_cfg', with only very minimal configuration
> * otherwise.
> *
> - * This is purely an update to bridge data structures. Nothing is pushed
> - * down to ofproto or lower layers. */
> + * This is mostly an update to bridge data structures. Very little is
> + * pushed down to ofproto or lower layers. */
> add_del_bridges(ovs_cfg);
> HMAP_FOR_EACH (br, node, &all_bridges) {
> bridge_add_del_ports(br);
> @@ -2187,8 +2187,7 @@ bridge_add_del_ports(struct bridge *br)
> }
>
> /* Get rid of deleted ports.
> - * Get rid of deleted interfaces on ports that still exist.
> - * Update 'cfg' of ports that still exist. */
> + * Get rid of deleted interfaces on ports that still exist. */
> HMAP_FOR_EACH_SAFE (port, next, hmap_node, &br->ports) {
> port->cfg = shash_find_data(&new_ports, port->name);
> if (!port->cfg) {
> --
> 1.7.4.4
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
