On Mon, Feb 13, 2017 at 10:52 PM, nickcooper-zhangtonghao <n...@opencloud.tech> wrote: > The balance-tcp mode requires the upstream switch to support 802.3ad > with successful LACP negotiation. When bond ports are configured to > balance-tcp mode without lacp or lacp is disabled, drop the traffic. > > Signed-off-by: nickcooper-zhangtonghao <n...@opencloud.tech>
I think the following change is equivalent and may be easier to follow, what do you think? diff --git a/ofproto/bond.c b/ofproto/bond.c index 2703bc9..4c2781d 100644 --- a/ofproto/bond.c +++ b/ofproto/bond.c @@ -787,6 +787,9 @@ bond_check_admissibility(struct bond *bond, const void *slave_, goto out; } case LACP_DISABLED: + if (bond->balance == BM_TCP) { + goto out; + } break; } _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev