diff options
author | Steven Luong <sluong@cisco.com> | 2019-10-09 10:29:47 -0700 |
---|---|---|
committer | Andrew Yourtchenko <ayourtch@gmail.com> | 2019-10-22 11:21:05 +0000 |
commit | eb0cea6919c1be60abc063dc7289818baee6b0e3 (patch) | |
tree | 363f27d9b08641ecddd0529a63a001c293a8e1ad | |
parent | e36e5cbac4a5e7948f8a9bab9d22d6a9c31bc99c (diff) |
bonding: feature arc may not be enabled for the slave interface
vnet_feature_enable_disable takes sw_if_index, not hw_if_index. If there
is a subinterface created prior to the slave interface is created,
sw_if_index and hw_if_index start to diverge and the problem will happen.
Type: fix
Signed-off-by: Steven Luong <sluong@cisco.com>
Change-Id: I11e1f099378832f83b748526c6cbeb56960fad3c
(cherry picked from commit 1a41a35b27da6921d6d86a9f1ad5f1b46e1185f7)
-rw-r--r-- | src/vnet/bonding/cli.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/vnet/bonding/cli.c b/src/vnet/bonding/cli.c index 014edd28237..ff458daaa0e 100644 --- a/src/vnet/bonding/cli.c +++ b/src/vnet/bonding/cli.c @@ -280,7 +280,7 @@ bond_delete_neighbor (vlib_main_t * vm, bond_if_t * bif, slave_if_t * sif) bond_disable_collecting_distributing (vm, sif); vnet_feature_enable_disable ("device-input", "bond-input", - sif_hw->hw_if_index, 0, 0, 0); + sif->sw_if_index, 0, 0, 0); /* Put back the old mac */ vnet_hw_interface_change_mac_address (vnm, sif_hw->hw_if_index, @@ -696,7 +696,7 @@ bond_enslave (vlib_main_t * vm, bond_enslave_args_t * args) } args->rv = vnet_feature_enable_disable ("device-input", "bond-input", - sif_hw->hw_if_index, 1, 0, 0); + sif->sw_if_index, 1, 0, 0); if (args->rv) { |