bonding: pr_debug instead of pr_warn in bond_arp_send_all
authorVeaceslav Falico <vfalico@redhat.com>
Wed, 28 Aug 2013 21:25:16 +0000 (23:25 +0200)
committerDavid S. Miller <davem@davemloft.net>
Thu, 29 Aug 2013 20:19:43 +0000 (16:19 -0400)
They're simply annoying and will spam dmesg constantly if we hit them, so
convert to pr_debug so that we still can access them in case of debugging.

CC: Jay Vosburgh <fubar@us.ibm.com>
CC: Andy Gospodarek <andy@greyhouse.net>
Signed-off-by: Veaceslav Falico <vfalico@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/bonding/bond_main.c

index b87ad76..c50679f 100644 (file)
@@ -2336,10 +2336,8 @@ static void bond_arp_send_all(struct bonding *bond, struct slave *slave)
                rt = ip_route_output(dev_net(bond->dev), targets[i], 0,
                                     RTO_ONLINK, 0);
                if (IS_ERR(rt)) {
-                       if (net_ratelimit()) {
-                               pr_warning("%s: no route to arp_ip_target %pI4\n",
-                                          bond->dev->name, &targets[i]);
-                       }
+                       pr_debug("%s: no route to arp_ip_target %pI4\n",
+                                bond->dev->name, &targets[i]);
                        continue;
                }
 
@@ -2386,10 +2384,10 @@ static void bond_arp_send_all(struct bonding *bond, struct slave *slave)
                rcu_read_unlock();
 
                /* Not our device - skip */
-               if (net_ratelimit())
-                       pr_warning("%s: no path to arp_ip_target %pI4 via rt.dev %s\n",
-                                  bond->dev->name, &targets[i],
-                                  rt->dst.dev ? rt->dst.dev->name : "NULL");
+               pr_debug("%s: no path to arp_ip_target %pI4 via rt.dev %s\n",
+                        bond->dev->name, &targets[i],
+                        rt->dst.dev ? rt->dst.dev->name : "NULL");
+
                ip_rt_put(rt);
                continue;