diff options
author | Jay Vosburgh <fubar@us.ibm.com> | 2007-11-06 13:33:29 -0800 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2007-11-10 04:25:14 -0500 |
commit | 3a1521b7e5b6964c293bb8ed6773513f8f503de5 (patch) | |
tree | c8beb091e81aa76c5ba7edeb2c4a022270f13097 | |
parent | 1466a21997212a5fb33d5da9357841972b28b007 (diff) | |
download | op-kernel-dev-3a1521b7e5b6964c293bb8ed6773513f8f503de5.zip op-kernel-dev-3a1521b7e5b6964c293bb8ed6773513f8f503de5.tar.gz |
bonding: don't validate address at device open
The standard validate_addr handler refuses to accept the all zeroes address
as valid. However, it's common historical practice for the bonding
master to be configured up prior to having any slaves, at which time the
master will have a MAC address of all zeroes.
Resolved by setting the dev->validate_addr to NULL. The master still can't
end up with an invalid address, as the set_mac_address function tests
for validity.
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
-rw-r--r-- | drivers/net/bonding/bond_main.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/net/bonding/bond_main.c b/drivers/net/bonding/bond_main.c index 6937ef0..a198404 100644 --- a/drivers/net/bonding/bond_main.c +++ b/drivers/net/bonding/bond_main.c @@ -4405,6 +4405,7 @@ static int bond_init(struct net_device *bond_dev, struct bond_params *params) bond_dev->set_multicast_list = bond_set_multicast_list; bond_dev->change_mtu = bond_change_mtu; bond_dev->set_mac_address = bond_set_mac_address; + bond_dev->validate_addr = NULL; bond_set_mode_ops(bond, bond->params.mode); |