diff options
author | Nicolas de Pesloüan <nicolas.2p.debian@free.fr> | 2009-10-07 23:11:00 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-10-07 23:11:00 +0200 |
commit | 49b4ad92d1a5bb9909deb3216ffec6f0febc7b71 (patch) | |
tree | 0059b4df75efc6a8b1a44751912ed6a425d9ed4f /drivers/net/bonding | |
parent | bonding: fix a parameter name in error message (diff) | |
download | linux-49b4ad92d1a5bb9909deb3216ffec6f0febc7b71.tar.xz linux-49b4ad92d1a5bb9909deb3216ffec6f0febc7b71.zip |
bonding: remove useless assignment
The variable old_active is first set to bond->curr_active_slave.
Then, it is unconditionally set to new_active, without being used in between.
The first assignment, having no side effect, is useless.
Signed-off-by: Nicolas de Pesloüan <nicolas.2p.debian@free.fr>
Reviewed-by: Jiri Pirko <jpirko@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/bonding')
-rw-r--r-- | drivers/net/bonding/bond_main.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/bonding/bond_main.c b/drivers/net/bonding/bond_main.c index 05877cb182e7..ef6af1cb7d39 100644 --- a/drivers/net/bonding/bond_main.c +++ b/drivers/net/bonding/bond_main.c @@ -1119,7 +1119,7 @@ static struct slave *bond_find_best_slave(struct bonding *bond) int mintime = bond->params.updelay; int i; - new_active = old_active = bond->curr_active_slave; + new_active = bond->curr_active_slave; if (!new_active) { /* there were no active slaves left */ if (bond->slave_cnt > 0) /* found one slave */ |