[ovs-git] Open vSwitch: dpif-linux: Close channel Netlink sockets when a port number gets recycled. (branch-1.10)

dev at openvswitch.org dev at openvswitch.org
Thu May 2 17:13:20 UTC 2013

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Open vSwitch".

The branch, branch-1.10 has been updated
       via  8ac6e0e15dafd1b41001fdc52ea00cc42906d5a9 (commit)
      from  d4c5b6423aa063eaf296ec8cf7d1a50197863cec (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 8ac6e0e15dafd1b41001fdc52ea00cc42906d5a9
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=8ac6e0e15dafd1b41001fdc52ea00cc42906d5a9
Author: Ben Pfaff <blp at nicira.com>
dpif-linux: Close channel Netlink sockets when a port number gets recycled.
When ovs-vswitchd deletes a port with dpif_linux_port_del(), that function
uses del_channel() to delete the corresponding channel, including closing
its Netlink socket fd.  However, if the vport gets removed by some other
process (e.g. "ip link delete" for veths) then this function never gets
called and thus the channel never gets deleted.

This commit partially fixes the problem.  Now, if a port number gets
reused, add_channel() closes the old Netlink socket assigned to that port
before it installs the new one.

Bug #16784.
Reported-by: Paul Ingram <paul at nicira.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>


Summary of changes:
 lib/dpif-linux.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

Open vSwitch

More information about the git mailing list