diff options
| author |   Steven <sluong@cisco.com> | 2018-08-17 09:44:08 -0700 |
|---|---|---|
| committer | steven luong <sluong@cisco.com> | 2018-08-31 05:16:42 +0000 |
| commit | fe47e29fc6ddcb664f76f4ebbfd2593cb0282e43 (patch) | |
| tree | d7ba0d609a8a4d34c1fc0a15264a48a3edb1f804 /src/plugins | |
| parent | ead8eb34e305b7bb145f7a113107010f28cdc954 (diff) | |
lacp: issue with slave interface admin up after it is added to the bond group [VPP-1409]
When the slave interface is admin up after it is added to the bond group,
lacp does not send lacp pdu's to the interface because its periodic timer
is not running.
The issue was the slave interface's variables got reset after the state
machines were initialized.
Change-Id: I2942556ce29a4acc97db3be40293e69bed7b6679
Signed-off-by: Steven <sluong@cisco.com>
(cherry picked from commit bf21dc85053931bc597e0fab1112061aa60e986b)
Diffstat (limited to 'src/plugins')
| -rw-r--r-- | src/plugins/lacp/lacp.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/plugins/lacp/lacp.c b/src/plugins/lacp/lacp.c index dfe11b18ee8..7bd6e374dab 100644 --- a/src/plugins/lacp/lacp.c +++ b/src/plugins/lacp/lacp.c @@ -363,10 +363,10 @@ lacp_sw_interface_up_down (vnet_main_t * vnm, u32 sw_if_index, u32 flags) { if (sif->lacp_enabled) { - lacp_init_state_machines (vm, sif); lacp_init_neighbor (sif, sif->actor_admin.system, ntohs (sif->actor_admin.port_number), ntohs (sif->actor_admin.key)); + lacp_init_state_machines (vm, sif); } } } @@ -392,10 +392,10 @@ lacp_hw_interface_up_down (vnet_main_t * vnm, u32 hw_if_index, u32 flags) { if (sif->lacp_enabled) { - lacp_init_state_machines (vm, sif); lacp_init_neighbor (sif, sif->actor_admin.system, ntohs (sif->actor_admin.port_number), ntohs (sif->actor_admin.key)); + lacp_init_state_machines (vm, sif); } } } |