diff options
| author | Jiri Bohac <jbohac@suse.cz> | 2012-01-18 12:24:54 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2012-01-18 20:59:53 -0500 |
| commit | b924551bed09f61b64f21bffe241afc5526b091a (patch) | |
| tree | cafe1bd4b49cf92ef799c86f8d28c9eb02677c0b /drivers/net/ethernet/faraday/ftmac100.c | |
| parent | 2b53d07891630dead46d65c8f896955fd3ae0302 (diff) | |
| download | linux-b924551bed09f61b64f21bffe241afc5526b091a.tar.gz linux-b924551bed09f61b64f21bffe241afc5526b091a.tar.xz | |
bonding: fix enslaving in alb mode when link down
bond_alb_init_slave() is called from bond_enslave() and sets the slave's MAC
address. This is done differently for TLB and ALB modes.
bond->alb_info.rlb_enabled is used to discriminate between the two modes but
this flag may be uninitialized if the slave is being enslaved prior to calling
bond_open() -> bond_alb_initialize() on the master.
It turns out all the callers of alb_set_slave_mac_addr() pass
bond->alb_info.rlb_enabled as the hw parameter.
This patch cleans up the unnecessary parameter of alb_set_slave_mac_addr() and
makes the function decide based on the bonding mode instead, which fixes the
above problem.
Reported-by: Narendra K <Narendra_K@Dell.com>
Signed-off-by: Jiri Bohac <jbohac@suse.cz>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/faraday/ftmac100.c')
0 files changed, 0 insertions, 0 deletions