bonding: complain about missing route only once for A/B ARP probes
authorDavid Decotigny <ddecotig@google.com>
Sat, 4 Sep 2021 06:31:29 +0000 (23:31 -0700)
committerDavid S. Miller <davem@davemloft.net>
Sun, 5 Sep 2021 12:16:51 +0000 (13:16 +0100)
On configs where there is no confirgured direct route to the target of
the ARP probes, these probes are still sent and may be replied to
properly, so no need to repeatedly complain about the missing route.

Signed-off-by: David Decotigny <ddecotig@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/bonding/bond_main.c

index b0966e7339264fc787a1a9d0e70df47171385156..3858da3d3ea7e1d8c756487a7c7a1f8619f43dd2 100644 (file)
@@ -2910,9 +2910,9 @@ static void bond_arp_send_all(struct bonding *bond, struct slave *slave)
                         * probe to generate any traffic (arp_validate=0)
                         */
                        if (bond->params.arp_validate)
-                               net_warn_ratelimited("%s: no route to arp_ip_target %pI4 and arp_validate is set\n",
-                                                    bond->dev->name,
-                                                    &targets[i]);
+                               pr_warn_once("%s: no route to arp_ip_target %pI4 and arp_validate is set\n",
+                                            bond->dev->name,
+                                            &targets[i]);
                        bond_arp_send(slave, ARPOP_REQUEST, targets[i],
                                      0, tags);
                        continue;