bonding: pr_debug instead of pr_warn in bond_arp_send_all

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>

1 files changed, 6 insertions, 8 deletions

diff --git a/drivers/net/bonding/bond_main.c b/drivers/net/bonding/bond_main.c
index b87ad7606b2b..c50679f0a6b6 100644
--- a/drivers/net/bonding/bond_main.c
+++ b/drivers/net/bonding/bond_main.c
@@ -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;