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

Reply via email to