mirror of
https://github.com/openvswitch/ovs
synced 2025-10-13 14:07:02 +00:00
208ff13939
When adding a route to a bridge, by executing "$appctl ovs/route/add $IP/$MASK $BR", a reference to the existing netdev is taken and stored in an instantiated ip_dev struct which is then stored in an addr_list list in tnl-ports.c. When OvS is signaled to exit, as a result of a "$appctl $OVS_PID exit --cleanup", for example, the bridge takes care of destroying its allocated port and iface structs. While destroying and freeing an iface, the netdev associated with it is also destroyed. However, for this to happen its ref_cnt must be 0. Otherwise the destructor of the netdev (specific to each datapath) won't be called. On the userspace datapath this means a system interface, such as "br0", wouldn't get deleted upon exit of OvS (when a route happens to be assocaited). This was first observed in the "ptap - triangle bridge setup with L2 and L3 GRE tunnels" test, which runs as part of the system userspace testsuite and uses the netdev datapath (as opoosed to several tests which use the dummy datapath, where this issue isn't seen). The test would pass every other time and fail the rest of the times because the needed system interfaces (br-p1, br-p2 and br-p3) were already present (from the previous successfull run which didn't clean up properly), leading to a failure. To fix the leak and clean up the interfaces upon exit, on its final stage before destroying a netdev, in iface_destroy__(), the bridge calls tnl_port_map_delete_ipdev() which takes care of freeing the instatiated ip_dev structs that refer to a specific netdev. An extra test is also introduced which verifies that the resources used by OvS netdev datapath have been correctly cleaned up between OVS_TRAFFIC_VSWITCHD_STOP and AT_CLEANUP. Signed-off-by: Tiago Lam <tiago.lam@intel.com> Signed-off-by: Ben Pfaff <blp@ovn.org>
66 lines
1.7 KiB
Plaintext
66 lines
1.7 KiB
Plaintext
AT_BANNER([system-inteface])
|
|
|
|
dnl add a veth interface to br0, then delete and re-create
|
|
dnl the veth interface with the same name in the system
|
|
AT_SETUP([interface - add delete add same interface])
|
|
|
|
OVS_TRAFFIC_VSWITCHD_START()
|
|
|
|
AT_CHECK([ip link add ovs-veth0 type veth peer name ovs-veth1])
|
|
on_exit 'ip link del ovs-veth0'
|
|
|
|
AT_CHECK([ovs-vsctl add-port br0 ovs-veth0])
|
|
|
|
AT_CHECK([ip link del ovs-veth0])
|
|
AT_CHECK([ip link add ovs-veth0 type veth peer name ovs-veth1])
|
|
|
|
AT_CHECK([ovs-vsctl del-port br0 ovs-veth0])
|
|
|
|
OVS_TRAFFIC_VSWITCHD_STOP(["dnl
|
|
/could not open network device ovs-veth0/d
|
|
/cannot get .*STP status on nonexistent port/d
|
|
/ethtool command .*on network device ovs-veth0 failed/d
|
|
/error receiving .*ovs-veth0/d
|
|
/ovs-veth0: removing policing failed/d"])
|
|
|
|
AT_CLEANUP
|
|
|
|
dnl add a p1-0 interface to br-p1, then add a route to br-p1 and stop the OvS
|
|
dnl instance. Confirm br-p1 interface has been deleted from the system.
|
|
AT_SETUP([interface - add route to br and verify clean-up])
|
|
|
|
OVS_TRAFFIC_VSWITCHD_START()
|
|
|
|
HWADDR_BRP1=aa:55:00:00:00:01
|
|
|
|
dnl Create tap port to later add to br-p1
|
|
AT_CHECK([ip tuntap add name p1-0 mode tap])
|
|
AT_CHECK([ip link set p1-0 up])
|
|
on_exit 'ip link del p1-0'
|
|
|
|
AT_CHECK([
|
|
ovs-vsctl add-br br-p1 -- \
|
|
set bridge br-p1 datapath_type=netdev fail-mode=standalone other-config:hwaddr=$HWADDR_BRP1
|
|
|
|
ovs-vsctl add-port br-p1 p1-0
|
|
|
|
ovs-ofctl del-flows br-p1
|
|
], [0])
|
|
|
|
AT_CHECK([
|
|
ip addr add 10.0.0.1/24 dev br-p1
|
|
ip link set br-p1 up
|
|
], [0], [stdout])
|
|
|
|
AT_CHECK([
|
|
ovs-appctl ovs/route/add 10.0.0.0/24 br-p1
|
|
ovs-appctl tnl/arp/set br-p1 10.0.0.1 $HWADDR_BRP1
|
|
], [0], [stdout])
|
|
|
|
OVS_TRAFFIC_VSWITCHD_STOP
|
|
AT_CHECK([
|
|
ip link show br-p1], [1],
|
|
[stdout], [Device "br-p1" does not exist.]
|
|
)
|
|
AT_CLEANUP
|