diff options
| author | Giuseppe Lettieri <g.lettieri@iet.unipi.it> | 2012-05-09 12:17:15 +0200 |
|---|---|---|
| committer | Ben Pfaff <blp@nicira.com> | 2012-05-16 09:52:57 -0700 |
| commit | 2197d7abab6d765013399790d11290562a583b71 (patch) | |
| tree | 363f71cb80b6cfe72128c255b3651f9dd5e7e8a7 /lib/dpif-netdev.c | |
| parent | e6c5e5390395a2ef19676319caa97b4e5a6942e2 (diff) | |
dpif-netdev: allow for proper destruction of netdev datapaths
Until now, bridges with datapath_type=netdev did not destroy the datapath
when deleted. In particular, the tap device implementing the internal
interface was not close()d, and therefore the tap persists until
ovs-vswitchd exit()s.
This behaviour was caused by the missing callback for 'enumerate' in the
dpif-netdev class. Without this callback 'bridge_reconfigure' failed to
realize that there are datapaths with no bridge, and thus cannot destroy
them. Providing an 'enumerate' callback fixes this.
Signed-off-by: Giuseppe Lettieri <g.lettieri@iet.unipi.it>
Signed-off-by: Ben Pfaff <blp@nicira.com>
Diffstat (limited to 'lib/dpif-netdev.c')
| -rw-r--r-- | lib/dpif-netdev.c | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c index a6c06fe9..fb0a863e 100644 --- a/lib/dpif-netdev.c +++ b/lib/dpif-netdev.c @@ -165,6 +165,17 @@ get_dp_netdev(const struct dpif *dpif) return dpif_netdev_cast(dpif)->dp; } +static int +dpif_netdev_enumerate(struct sset *all_dps) +{ + struct shash_node *node; + + SHASH_FOR_EACH(node, &dp_netdevs) { + sset_add(all_dps, node->name); + } + return 0; +} + static struct dpif * create_dpif_netdev(struct dp_netdev *dp) { @@ -1241,7 +1252,7 @@ dp_netdev_execute_actions(struct dp_netdev *dp, const struct dpif_class dpif_netdev_class = { "netdev", - NULL, /* enumerate */ + dpif_netdev_enumerate, dpif_netdev_open, dpif_netdev_close, dpif_netdev_destroy, |