diff options
| author |   Nathan Skrzypczak <nathan.skrzypczak@gmail.com> | 2021-09-17 17:29:14 +0200 |
|---|---|---|
| committer |   Neale Ranns <neale@graphiant.com> | 2021-10-11 12:04:03 +0000 |
| commit | 275bd796346c3b1618170f4eda36b9a41ade9c87 (patch) | |
| tree | 197df1102b0a3981c6a875100f521806a170f45c /src/vnet/mfib | |
| parent | bd23b405fbe99034d75a46b060567e5adf62c04e (diff) | |
ip: fix fib and mfib locks
This patches fixes an issue that could cause
fib locks to underflow: if an API user deletes
a fib and quickly recreates it, the fib may not
have been actually deleted. As a result, the
lock would not be incremented on the create call
leading to the fib potentially disappearing
afterwards - or to the lock to underflow when
the fib is deleted again.
In order to keep the existing API semantics,
we use the locks with API and CLI source as flags.
This means we need to use a different counter
for the interface-related locks.
This also prevents an issue where an interface being
bound to a vrf via API and released via CLI could
mess up the lock counter.
Finally, this will help with cleaning up the
interface-related locks on interface deletion
in a later patch.
Type: fix
Change-Id: I93030a7660646d6dd179ddf27fe4e708aa11b90e
Signed-off-by: Nathan Skrzypczak <nathan.skrzypczak@gmail.com>
Signed-off-by: Aloys Augustin <aloaugus@cisco.com>
Diffstat (limited to 'src/vnet/mfib')
| -rw-r--r-- | src/vnet/mfib/mfib_types.h | 34 |
1 files changed, 19 insertions, 15 deletions
diff --git a/src/vnet/mfib/mfib_types.h b/src/vnet/mfib/mfib_types.h index edc25fe5b99..34e8f6b928d 100644 --- a/src/vnet/mfib/mfib_types.h +++ b/src/vnet/mfib/mfib_types.h @@ -160,23 +160,25 @@ typedef enum mfib_itf_flags_t_ */ typedef enum mfib_source_t_ { - MFIB_SOURCE_SPECIAL, - MFIB_SOURCE_6RD, - MFIB_SOURCE_API, - MFIB_SOURCE_CLI, - MFIB_SOURCE_VXLAN, - MFIB_SOURCE_DHCP, - MFIB_SOURCE_SRv6, - MFIB_SOURCE_GTPU, - MFIB_SOURCE_VXLAN_GPE, - MFIB_SOURCE_GENEVE, - MFIB_SOURCE_IGMP, - MFIB_SOURCE_VXLAN_GBP, - MFIB_SOURCE_PLUGIN_LOW, - MFIB_SOURCE_RR, - MFIB_SOURCE_DEFAULT_ROUTE, + MFIB_SOURCE_SPECIAL, + MFIB_SOURCE_6RD, + MFIB_SOURCE_API, + MFIB_SOURCE_CLI, + MFIB_SOURCE_VXLAN, + MFIB_SOURCE_DHCP, + MFIB_SOURCE_SRv6, + MFIB_SOURCE_GTPU, + MFIB_SOURCE_VXLAN_GPE, + MFIB_SOURCE_GENEVE, + MFIB_SOURCE_IGMP, + MFIB_SOURCE_VXLAN_GBP, + MFIB_SOURCE_PLUGIN_LOW, + MFIB_SOURCE_RR, + MFIB_SOURCE_INTERFACE, /* used exclusively for mfib locks */ + MFIB_SOURCE_DEFAULT_ROUTE, } mfib_source_t; +/* clang-format off */ #define MFIB_SOURCE_NAMES { \ [MFIB_SOURCE_SPECIAL] = "Special", \ [MFIB_SOURCE_6RD] = "6RD", \ @@ -192,8 +194,10 @@ typedef enum mfib_source_t_ [MFIB_SOURCE_VXLAN_GBP] = "VXLAN-GBP", \ [MFIB_SOURCE_PLUGIN_LOW] = "plugin-low", \ [MFIB_SOURCE_RR] = "Recursive-resolution", \ + [MFIB_SOURCE_INTERFACE] = "Interface", \ [MFIB_SOURCE_DEFAULT_ROUTE] = "Default Route", \ } +/* clang-format on */ #define FOREACH_MFIB_SOURCE(_ms) \ for (_ms = MFIB_SOURCE_SPECIAL; \ |