gbp: Ownership of dynamically created vxlan-gbp tunnels managed via gbp_itf 43/20543/7
authorNeale Ranns <nranns@cisco.com>
Fri, 5 Jul 2019 07:53:45 +0000 (00:53 -0700)
committerNeale Ranns <nranns@cisco.com>
Tue, 9 Jul 2019 15:45:52 +0000 (15:45 +0000)
commite28c87cd00644205e9bebca054029a8e655ed015
tree8e6e0bac3b6d269c00b569a3ebf60338237c8a29
parent777d2aee8054e77e6f265879e1dfe3776d90d758
gbp: Ownership of dynamically created vxlan-gbp tunnels managed via gbp_itf

Type: fix

This solves the ownership of vxlan-gbp tunnels. When the last reference of these goes away they need to be deleted. Currently there are two owners; gbp_itf via gef_itf and the lock held by the gbp_endpoint_location_t. The problem is that the
loc removes its reference whilst the fwd still holds the gbp_itf, and things go wrong.
This change moves the lifecycle management of the vxlan-gbp tunnel to the gbp_itf. When the last lock of the gbp_itf goes, so does the tunnel. now both the EP's loc and fwd can hold a lock on the gbp_itf and it's only removed when required.
The other change is the management of the 'user' of the gbp_itf. Since each user can enable and disable different features, it's the job of the gbp_itf to apply the combined set. determining a unique 'uesr' from the caller was near impossible, so I moved that to the gbp_itf, and return the allocated user, hence the 'handle' that encodes both user and interface.

The hash table maps from sw_if_index to pool index.

Change-Id: I4c7bf4c0e5dcf33d1c545f262365e69151febcf4
Signed-off-by: Neale Ranns <nranns@cisco.com>
21 files changed:
src/plugins/gbp/gbp_api.c
src/plugins/gbp/gbp_bridge_domain.c
src/plugins/gbp/gbp_bridge_domain.h
src/plugins/gbp/gbp_contract.c
src/plugins/gbp/gbp_endpoint.c
src/plugins/gbp/gbp_endpoint.h
src/plugins/gbp/gbp_endpoint_group.c
src/plugins/gbp/gbp_endpoint_group.h
src/plugins/gbp/gbp_ext_itf.c
src/plugins/gbp/gbp_ext_itf.h
src/plugins/gbp/gbp_itf.c
src/plugins/gbp/gbp_itf.h
src/plugins/gbp/gbp_learn.c
src/plugins/gbp/gbp_learn.h
src/plugins/gbp/gbp_learn_node.c
src/plugins/gbp/gbp_recirc.c
src/plugins/gbp/gbp_recirc.h
src/plugins/gbp/gbp_route_domain.h
src/plugins/gbp/gbp_vxlan.c
src/plugins/gbp/gbp_vxlan.h
test/test_gbp.py