On Thu, Apr 14, 2011 at 10:58:35AM -0700, Ethan Jackson wrote:
> Here is an incremental on the previous version. Compile tested only.
Looks good to me.
What motivated this change?
> @@ -1518,9 +1520,13 @@ choose_output_slave(const struct bond *bond, const
> struct flow *flow,
> switch (bond->balance) {
> case BM_AB:
> return bond->active_slave;
> - case BM_STB:
> - return bond->stb_slaves[bond_hash(bond, flow, vlan)
> - % bond->n_stb_slaves];
> + case BM_STABLE:
> + if (bond->n_stb_slaves) {
> + return bond->stb_slaves[bond_hash(bond, flow, vlan)
> + % bond->n_stb_slaves];
> + } else {
> + return bond->active_slave;
> + }
> case BM_SLB:
> case BM_TCP:
> e = lookup_bond_entry(bond, flow, vlan);
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev
