diff mbox series

[v1,1/1] bonding: complain about missing route only once for A/B ARP probes

Message ID 20210904063129.3969050-1-decot+git@google.com
State New
Headers show
Series [v1,1/1] bonding: complain about missing route only once for A/B ARP probes | expand

Commit Message

David Decotigny Sept. 4, 2021, 6:31 a.m. UTC 
From: David Decotigny <ddecotig@google.com>

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>
---
 drivers/net/bonding/bond_main.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Comments

patchwork-bot+netdevbpf@kernel.org Sept. 5, 2021, 12:20 p.m. UTC | #1 
Hello:

This patch was applied to netdev/net.git (refs/heads/master):

On Fri,  3 Sep 2021 23:31:29 -0700 you wrote:
> From: David Decotigny <ddecotig@google.com>

> 

> 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.

> 

> 

> [...]


Here is the summary with links:
  - [v1,1/1] bonding: complain about missing route only once for A/B ARP probes
    https://git.kernel.org/netdev/net/c/0a4fd8df07dd

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
diff mbox series

Patch

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