raspi/linux
1
0
Fork 1
mirror of https://github.com/raspberrypi/linux.git synced 2025-12-06 01:49:46 +00:00
Code Issues Projects Releases Wiki Activity

bonding: fix ad_actor_system option setting to default

Browse Source 
[ Upstream commit 1c15b05bae ]

When 802.3ad bond mode is configured the ad_actor_system option is set to
"00:00:00:00:00:00". But when trying to set the all-zeroes MAC as actors'
system address it was failing with EINVAL.

An all-zeroes ethernet address is valid, only multicast addresses are not
valid values.

Fixes: 171a42c38c ("bonding: add netlink support for sys prio, actor sys mac, and port key")
Signed-off-by: Fernando Fernandez Mancera <ffmancera@riseup.net>
Acked-by: Jay Vosburgh <jay.vosburgh@canonical.com>
Link: https://lore.kernel.org/r/20211221111345.2462-1-ffmancera@riseup.net
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
Fernando Fernandez Mancera
2021-12-21 12:13:45 +01:00
committed by Greg Kroah-Hartman
parent 6809da5185
commit c6d2754006
2 changed files with 7 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
Documentation/networking/bonding.rst
View File

@@ -196,11 +196,12 @@ ad_actor_sys_prio
ad_actor_system ad_actor_system
In an AD system, this specifies the mac-address for the actor in In an AD system, this specifies the mac-address for the actor in
protocol packet exchanges (LACPDUs). The value cannot be NULL or protocol packet exchanges (LACPDUs). The value cannot be a multicast
multicast. It is preferred to have the local-admin bit set for this address. If the all-zeroes MAC is specified, bonding will internally
mac but driver does not enforce it. If the value is not given then use the MAC of the bond itself. It is preferred to have the
system defaults to using the masters' mac address as actors' system local-admin bit set for this mac but driver does not enforce it. If
address. the value is not given then system defaults to using the masters'
mac address as actors' system address.
This parameter has effect only in 802.3ad mode and is available through This parameter has effect only in 802.3ad mode and is available through
SysFs interface. SysFs interface.

2
drivers/net/bonding/bond_options.c
View File

@@ -1466,7 +1466,7 @@ static int bond_option_ad_actor_system_set(struct bonding *bond,
mac = (u8 *)&newval->value; mac = (u8 *)&newval->value;
} }
if (!is_valid_ether_addr(mac)) if (is_multicast_ether_addr(mac))
goto err; goto err;
netdev_dbg(bond->dev, "Setting ad_actor_system to %pM\n", mac); netdev_dbg(bond->dev, "Setting ad_actor_system to %pM\n", mac);