Commit 77140d29 authored by Veaceslav Falico's avatar Veaceslav Falico Committed by David S. Miller
Browse files

bonding: rework bond_find_best_slave() to use bond_for_each_slave()



bond_find_best_slave() does not have to be balanced - i.e. return the slave
that is *after* some other slave, but rather return the best slave that
suits, except of bond->primary_slave - in which case we just return it if
it's suitable.

After that we just look through all the slaves and return either first up
slave or the slave whose link came back earliest.

We also don't care about curr_active_slave lock cause we use it in
bond_should_change_active() only and there we take it right away - i.e. it
won't go away.

CC: Jay Vosburgh <fubar@us.ibm.com>
CC: Andy Gospodarek <andy@greyhouse.net>
Signed-off-by: default avatarVeaceslav Falico <vfalico@redhat.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 6475ae4c
...@@ -785,43 +785,24 @@ static bool bond_should_change_active(struct bonding *bond) ...@@ -785,43 +785,24 @@ static bool bond_should_change_active(struct bonding *bond)
/** /**
* find_best_interface - select the best available slave to be the active one * find_best_interface - select the best available slave to be the active one
* @bond: our bonding struct * @bond: our bonding struct
*
* Warning: Caller must hold curr_slave_lock for writing.
*/ */
static struct slave *bond_find_best_slave(struct bonding *bond) static struct slave *bond_find_best_slave(struct bonding *bond)
{ {
struct slave *new_active, *old_active; struct slave *slave, *bestslave = NULL;
struct slave *bestslave = NULL; struct list_head *iter;
int mintime = bond->params.updelay; int mintime = bond->params.updelay;
int i;
new_active = bond->curr_active_slave; if (bond->primary_slave && bond->primary_slave->link == BOND_LINK_UP &&
bond_should_change_active(bond))
if (!new_active) { /* there were no active slaves left */ return bond->primary_slave;
new_active = bond_first_slave(bond);
if (!new_active)
return NULL; /* still no slave, return NULL */
}
if ((bond->primary_slave) && bond_for_each_slave(bond, slave, iter) {
bond->primary_slave->link == BOND_LINK_UP && if (slave->link == BOND_LINK_UP)
bond_should_change_active(bond)) { return slave;
new_active = bond->primary_slave; if (slave->link == BOND_LINK_BACK && IS_UP(slave->dev) &&
} slave->delay < mintime) {
mintime = slave->delay;
/* remember where to stop iterating over the slaves */ bestslave = slave;
old_active = new_active;
bond_for_each_slave_from(bond, new_active, i, old_active) {
if (new_active->link == BOND_LINK_UP) {
return new_active;
} else if (new_active->link == BOND_LINK_BACK &&
IS_UP(new_active->dev)) {
/* link up, but waiting for stabilization */
if (new_active->delay < mintime) {
mintime = new_active->delay;
bestslave = new_active;
}
} }
} }
......
Supports Markdown
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