summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJay Vosburgh <fubar@us.ibm.com>2007-11-06 13:33:29 -0800
committerJeff Garzik <jeff@garzik.org>2007-11-10 04:25:14 -0500
commit3a1521b7e5b6964c293bb8ed6773513f8f503de5 (patch)
treec8beb091e81aa76c5ba7edeb2c4a022270f13097
parent1466a21997212a5fb33d5da9357841972b28b007 (diff)
downloadop-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.c1
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);
OpenPOWER on IntegriCloud