]> www.infradead.org Git - users/jedix/linux-maple.git/commit
bonding: add ns target multicast address to slave device
authorHangbin Liu <liuhangbin@gmail.com>
Mon, 11 Nov 2024 10:16:49 +0000 (10:16 +0000)
committerPaolo Abeni <pabeni@redhat.com>
Thu, 14 Nov 2024 10:16:28 +0000 (11:16 +0100)
commit8eb36164d1a6769a20ed43033510067ff3dab9ee
tree104794543b0d05ed65577f233c60476cd799d9cc
parentdc065076ee7768377d7c16af7d1b0767782d8c98
bonding: add ns target multicast address to slave device

Commit 4598380f9c54 ("bonding: fix ns validation on backup slaves")
tried to resolve the issue where backup slaves couldn't be brought up when
receiving IPv6 Neighbor Solicitation (NS) messages. However, this fix only
worked for drivers that receive all multicast messages, such as the veth
interface.

For standard drivers, the NS multicast message is silently dropped because
the slave device is not a member of the NS target multicast group.

To address this, we need to make the slave device join the NS target
multicast group, ensuring it can receive these IPv6 NS messages to validate
the slave’s status properly.

There are three policies before joining the multicast group:
1. All settings must be under active-backup mode (alb and tlb do not support
   arp_validate), with backup slaves and slaves supporting multicast.
2. We can add or remove multicast groups when arp_validate changes.
3. Other operations, such as enslaving, releasing, or setting NS targets,
   need to be guarded by arp_validate.

Fixes: 4e24be018eb9 ("bonding: add new parameter ns_targets")
Signed-off-by: Hangbin Liu <liuhangbin@gmail.com>
Reviewed-by: Nikolay Aleksandrov <razor@blackwall.org>
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
drivers/net/bonding/bond_main.c
drivers/net/bonding/bond_options.c
include/net/bond_options.h