Commit 278339a4 authored by Jay Vosburgh's avatar Jay Vosburgh Committed by David S. Miller

bonding: propogate vlan_features to bonding master

Propogate the vlan_features of the slave devices to the bonding
master device, using the same logic as for regular features.

	Tested by Or Gerlitz <ogerlitz@voltaire.com>, who also removed
the debug logic from the original test patch.
Signed-off-by: default avatarOr Gerlitz <ogerlitz@voltaire.com>
Signed-off-by: default avatarJay Vosburgh <fubar@us.ibm.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent c06f51ea
...@@ -1331,6 +1331,7 @@ static int bond_compute_features(struct bonding *bond) ...@@ -1331,6 +1331,7 @@ static int bond_compute_features(struct bonding *bond)
struct slave *slave; struct slave *slave;
struct net_device *bond_dev = bond->dev; struct net_device *bond_dev = bond->dev;
unsigned long features = bond_dev->features; unsigned long features = bond_dev->features;
unsigned long vlan_features = 0;
unsigned short max_hard_header_len = max((u16)ETH_HLEN, unsigned short max_hard_header_len = max((u16)ETH_HLEN,
bond_dev->hard_header_len); bond_dev->hard_header_len);
int i; int i;
...@@ -1343,10 +1344,14 @@ static int bond_compute_features(struct bonding *bond) ...@@ -1343,10 +1344,14 @@ static int bond_compute_features(struct bonding *bond)
features &= ~NETIF_F_ONE_FOR_ALL; features &= ~NETIF_F_ONE_FOR_ALL;
vlan_features = bond->first_slave->dev->vlan_features;
bond_for_each_slave(bond, slave, i) { bond_for_each_slave(bond, slave, i) {
features = netdev_increment_features(features, features = netdev_increment_features(features,
slave->dev->features, slave->dev->features,
NETIF_F_ONE_FOR_ALL); NETIF_F_ONE_FOR_ALL);
vlan_features = netdev_increment_features(vlan_features,
slave->dev->vlan_features,
NETIF_F_ONE_FOR_ALL);
if (slave->dev->hard_header_len > max_hard_header_len) if (slave->dev->hard_header_len > max_hard_header_len)
max_hard_header_len = slave->dev->hard_header_len; max_hard_header_len = slave->dev->hard_header_len;
} }
...@@ -1354,6 +1359,7 @@ static int bond_compute_features(struct bonding *bond) ...@@ -1354,6 +1359,7 @@ static int bond_compute_features(struct bonding *bond)
done: done:
features |= (bond_dev->features & BOND_VLAN_FEATURES); features |= (bond_dev->features & BOND_VLAN_FEATURES);
bond_dev->features = netdev_fix_features(features, NULL); bond_dev->features = netdev_fix_features(features, NULL);
bond_dev->vlan_features = netdev_fix_features(vlan_features, NULL);
bond_dev->hard_header_len = max_hard_header_len; bond_dev->hard_header_len = max_hard_header_len;
return 0; return 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment