Thanks, I pushed this.
On Mon, Oct 17, 2011 at 11:50:10AM -0700, Ethan Jackson wrote: > Looks good. > > Ethan > > On Mon, Oct 17, 2011 at 11:43, Ben Pfaff <b...@nicira.com> 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 <pfazz...@nicira.com> > > --- > > ?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 > > dev@openvswitch.org > > http://openvswitch.org/mailman/listinfo/dev > > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
