Looks good.

Ethan

On Mon, Oct 17, 2011 at 11:43, Ben Pfaff <[email protected]> wrote:
> This log message comes up for packets that are flooded through the network.
> If the upstream switch doesn't realize that an active-backup bond is in
> use, and there is significant packet flooding in the network, then we will
> get a lot of these messages.
>
> (This message doesn't get logged for multicast or broadcast packets since
> they get dropped earlier in the function.)
>
> Reported-by: Eivind Bulie Haanaes
> Bug-report: http://forums.citrix.com/thread.jspa?messageID=1589125
> CC: Paul Fazzone <[email protected]>
> ---
>  lib/bond.c |    6 +++---
>  1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/lib/bond.c b/lib/bond.c
> index 1f2b641..73ee814 100644
> --- a/lib/bond.c
> +++ b/lib/bond.c
> @@ -585,9 +585,9 @@ bond_check_admissibility(struct bond *bond, const void 
> *slave_,
>         if (bond->active_slave != slave) {
>             static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(1, 5);
>
> -            VLOG_WARN_RL(&rl, "active-backup bond received packet on backup"
> -                         " slave (%s) destined for " ETH_ADDR_FMT,
> -                         slave->name, ETH_ADDR_ARGS(eth_dst));
> +            VLOG_DBG_RL(&rl, "active-backup bond received packet on backup"
> +                        " slave (%s) destined for " ETH_ADDR_FMT,
> +                        slave->name, ETH_ADDR_ARGS(eth_dst));
>             return BV_DROP;
>         }
>     }
> --
> 1.7.4.4
>
> _______________________________________________
> dev mailing list
> [email protected]
> http://openvswitch.org/mailman/listinfo/dev
>
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to