bond: Demote active-backup WARN to DBG.

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 <pfazzone@nicira.com>

diff --git a/AUTHORS b/AUTHORS
index 8822092b0b2a03daa1e1ea7a6aa5b4d5b24c7623..3229f3466077370f2feccd274437c53424e0fa09 100644 (file)
--- a/AUTHORS
+++ b/AUTHORS
@@ -63,6 +63,7 @@ Dave Walker             DaveWalker@ubuntu.com
 Derek Cormier           derek.cormier@lab.ntt.co.jp
 Duffie Cooley           dcooley@nicira.com
 DK Moon                 dkmoon@nicira.com
+Eivind Bulie Haanaes
 Gaetano Catalli         gaetano.catalli@gmail.com
 George Shuklin          amarao@desunote.ru
 Ghanem Bahri            bahri.ghanem@gmail.com

diff --git a/lib/bond.c b/lib/bond.c
index 1f2b641a11ca2ee72344f351e71b2b46f7f838c4..73ee814cd5f92a62a9f15236edc77702c4158c17 100644 (file)
--- 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;
         }
     }