diff options
author | Jiri Bohac <jbohac@suse.cz> | 2012-01-18 12:24:54 +0000 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2012-02-03 09:22:19 -0800 |
commit | 4f59cc372088474573ccc931c02e60c5d1de0508 (patch) | |
tree | 574f9b70d8defa80355ff144f9ebdfa894c67461 /net/l2tp | |
parent | 2703165a0725838607ae66af58dcdd203a519b87 (diff) | |
download | kernel_samsung_smdk4412-4f59cc372088474573ccc931c02e60c5d1de0508.zip kernel_samsung_smdk4412-4f59cc372088474573ccc931c02e60c5d1de0508.tar.gz kernel_samsung_smdk4412-4f59cc372088474573ccc931c02e60c5d1de0508.tar.bz2 |
bonding: fix enslaving in alb mode when link down
[ Upstream commit b924551bed09f61b64f21bffe241afc5526b091a ]
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>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'net/l2tp')
0 files changed, 0 insertions, 0 deletions