Commit 950ddcb1 authored by Mahesh Bandewar's avatar Mahesh Bandewar Committed by David S. Miller

bonding: simple code refactor

Remove duplicate code.
Signed-off-by: default avatarMahesh Bandewar <maheshb@google.com>
Signed-off-by: default avatarAndy Gospodarek <gospo@cumulusnetworks.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent a50dad35
...@@ -2900,6 +2900,8 @@ static int bond_slave_netdev_event(unsigned long event, ...@@ -2900,6 +2900,8 @@ static int bond_slave_netdev_event(unsigned long event,
if (old_duplex != slave->duplex) if (old_duplex != slave->duplex)
bond_3ad_adapter_duplex_changed(slave); bond_3ad_adapter_duplex_changed(slave);
} }
/* Fallthrough */
case NETDEV_DOWN:
/* Refresh slave-array if applicable! /* Refresh slave-array if applicable!
* If the setup does not use miimon or arpmon (mode-specific!), * If the setup does not use miimon or arpmon (mode-specific!),
* then these events will not cause the slave-array to be * then these events will not cause the slave-array to be
...@@ -2911,10 +2913,6 @@ static int bond_slave_netdev_event(unsigned long event, ...@@ -2911,10 +2913,6 @@ static int bond_slave_netdev_event(unsigned long event,
if (bond_mode_uses_xmit_hash(bond)) if (bond_mode_uses_xmit_hash(bond))
bond_update_slave_arr(bond, NULL); bond_update_slave_arr(bond, NULL);
break; break;
case NETDEV_DOWN:
if (bond_mode_uses_xmit_hash(bond))
bond_update_slave_arr(bond, NULL);
break;
case NETDEV_CHANGEMTU: case NETDEV_CHANGEMTU:
/* TODO: Should slaves be allowed to /* TODO: Should slaves be allowed to
* independently alter their MTU? For * independently alter their MTU? For
......
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