[ovs-dev] [PATCH 06/14] in-band: Fix inconsistency in in-band code.

[Justin Pettit]

Looks good.

--Justin


On Apr 8, 2010, at 5:07 PM, Ben Pfaff wrote:

> The IBR_TO_LOCAL_ARP and IBR_FROM_LOCAL_ARP flows are dropped if there is
> no local MAC.  I don't see why IBR_FROM_LOCAL_DHCP should be different, so
> this commit adds it too.
> ---
> ofproto/in-band.c |    1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/ofproto/in-band.c b/ofproto/in-band.c
> index 857618f..767f3e2 100644
> --- a/ofproto/in-band.c
> +++ b/ofproto/in-band.c
> @@ -504,6 +504,7 @@ in_band_run(struct in_band *in_band)
>                     (OFPFW_DL_TYPE | OFPFW_DL_SRC | OFPFW_NW_PROTO),
>                     OFPP_NORMAL);
>     } else {
> +        drop_flow(in_band, IBR_FROM_LOCAL_DHCP);
>         drop_flow(in_band, IBR_TO_LOCAL_ARP);
>         drop_flow(in_band, IBR_FROM_LOCAL_ARP);
>     }
> -- 
> 1.6.6.1
> 
> 
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev_openvswitch.org