Re: [PATCH net] bonding: 3ad: don't set BOND_LINK_FAIL on a slave that is not up
From: patchwork-bot+netdevbpf
Date: Thu Nov 05 2026 - 05:40:02 EST
Hello:
This patch was applied to netdev/net.git (main)
by Paolo Abeni <pabeni@xxxxxxxxxx>:
On Sun, 1 Nov 2026 16:56:53 +0200 you wrote:
> In 802.3ad mode, when bond_update_speed_duplex() fails on NETDEV_UP or
> NETDEV_CHANGE, bond_slave_netdev_event() sets the slave link to FAIL if
> the slave has ever been up (last_link_up != 0), and to DOWN otherwise,
> so that miimon can set it right once speed/duplex are valid.
>
> last_link_up is never cleared, so a slave that is currently DOWN but
> was up before is set to FAIL as well. miimon then treats it as a slave
> that is still in use and takes it through the full down path again:
> the link failure count is incremented, "link status definitely down"
> is logged and the 802.3ad port is disabled a second time, although the
> slave has not been up since it went down.
>
> [...]
Here is the summary with links:
- [net] bonding: 3ad: don't set BOND_LINK_FAIL on a slave that is not up
https://git.kernel.org/netdev/net/c/9a65b9c2a975
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html