[ovs-dev] [bridge 02/15] bridge: Don't bother revalidating in combination with bridge_flush().

Ethan Jackson ethan at nicira.com
Mon Mar 21 19:50:57 UTC 2011


Looks Fine.

On Mon, Mar 21, 2011 at 10:59 AM, Ben Pfaff <blp at nicira.com> wrote:
> iface_destroy() always calls bridge_flush(), which causes every flow to be
> revalidated, so there is no point in revalidating individual tags also.
> ---
>  vswitchd/bridge.c |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)
>
> diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c
> index 1bf4148..949a63e 100644
> --- a/vswitchd/bridge.c
> +++ b/vswitchd/bridge.c
> @@ -4368,7 +4368,6 @@ iface_destroy(struct iface *iface)
>         netdev_close(iface->netdev);
>
>         if (del_active) {
> -            ofproto_revalidate(port->bridge->ofproto, port->active_iface_tag);
>             bond_choose_active_iface(port);
>             bond_send_learning_packets(port);
>         }
> --
> 1.7.1
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>



More information about the dev mailing list