mir/ovs
2
0
Fork 0
mirror of https://github.com/openvswitch/ovs synced 2025-08-31 14:25:26 +00:00
Code Issues Releases Wiki Activity

bond: Honor updelay and downdelay when LACP is in use.

Browse Source 
Since OVS added LACP support back in 2011, bonds have ignored the updelay
and downdelay values for bonds with configured LACP.  The reason is not
clear, but at least one user needs support in this case, so this commit
enables it.

Reported-at: https://mail.openvswitch.org/pipermail/ovs-discuss/2018-October/047490.html
Reported-by: Daniel Leaberry <dleaberry@purestorage.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
Acked-by: Justin Pettit <jpettit@ovn.org>
This commit is contained in:
Ben Pfaff
2018-10-10 13:35:00 -07:00
parent 72713c651d
commit f94178077d
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
NEWS
View File

@@ -12,6 +12,7 @@ Post-v2.10.0
* Add option for simple round-robin based Rxq to PMD assignment. * Add option for simple round-robin based Rxq to PMD assignment.
It can be set with pmd-rxq-assign. It can be set with pmd-rxq-assign.
- Add 'symmetric_l3' hash function. - Add 'symmetric_l3' hash function.
- OVS now honors 'updelay' and 'downdelay' for bonds with LACP configured.
v2.10.0 - 18 Aug 2018 v2.10.0 - 18 Aug 2018
--------------------- ---------------------

3
ofproto/bond.c
View File

@@ -1717,8 +1717,7 @@ bond_link_status_update(struct bond_slave *slave)
VLOG_INFO_RL(&rl, "interface %s: will not be %s", VLOG_INFO_RL(&rl, "interface %s: will not be %s",
slave->name, up ? "disabled" : "enabled"); slave->name, up ? "disabled" : "enabled");
} else { } else {
int delay = (bond->lacp_status != LACP_DISABLED ? 0 int delay = up ? bond->updelay : bond->downdelay;
: up ? bond->updelay : bond->downdelay);
slave->delay_expires = time_msec() + delay; slave->delay_expires = time_msec() + delay;
if (delay) { if (delay) {
VLOG_INFO_RL(&rl, "interface %s: will be %s if it stays %s " VLOG_INFO_RL(&rl, "interface %s: will be %s if it stays %s "