diff options
| author |   Vladimir Isaev <visaev@netgate.com> | 2020-02-04 11:54:27 +0300 |
|---|---|---|
| committer |   Ole Trøan <otroan@employees.org> | 2020-02-05 15:34:49 +0000 |
| commit | 72e31bc2d9b910147c09e1c329713fccc873a018 (patch) | |
| tree | 7242c77805ea5541be6416f3b600f1ed1374b775 /src/plugins/oddbuf/node.c | |
| parent | a488f37559493657b59181c1212e57b3b9cc3689 (diff) | |
stats: fix state counter removal
Avoid using vec_del1() for directory vector to keep indexes valid all
the time.
There are state counters for each slave in LACP bond mode which can be
dynamically created and removed. Vector index is used to access these
counters. But also vec_del1() is used to remove counter from vector.
This function changes the index of the last element, so after this we
are unable to access ex-last element using old index.
As a result it is not possible to add-del-add two interfaces to the LACP
bond:
DBGvpp# create bond mode lacp
BondEthernet0
DBGvpp# create packet-generator interface pg1
DBGvpp# create packet-generator interface pg2
DBGvpp# bond add BondEthernet0 pg1
DBGvpp# bond add BondEthernet0 pg2
DBGvpp# bond del pg1
DBGvpp# bond del pg2
DBGvpp# bond add BondEthernet0 pg1
DBGvpp# bond add BondEthernet0 pg2
bond add: /if/lacp/1/3/partner-state is already register
Type: fix
Signed-off-by: Vladimir Isaev <visaev@netgate.com>
Change-Id: I2c86e13905eefdef6233369cd4ab5c1b53d123bd
Diffstat (limited to 'src/plugins/oddbuf/node.c')
0 files changed, 0 insertions, 0 deletions