[ovs-git] Open vSwitch: datapath: Fix tunnel reconfiguration that does not change key data. (branch-1.3)

dev at openvswitch.org dev at openvswitch.org
Tue Oct 18 20:35:01 UTC 2011


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.3 has been updated
       via  d05cf9daaf98754c8025f4d6d6777851f479f4a2 (commit)
      from  653e2fefc2c24f815f1a108680726ac9a01e8350 (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 d05cf9daaf98754c8025f4d6d6777851f479f4a2
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=d05cf9daaf98754c8025f4d6d6777851f479f4a2
Author: Ben Pfaff <blp at nicira.com>
		
datapath: Fix tunnel reconfiguration that does not change key data.
		
Without this commit, a pair of commands like
    ovs-dpctl add-if br0 gre0,type=gre,remote_ip=192.168.5.2,csum=true
    ovs-dpctl set-if br0 gre0,csum=false
would result in a csum of "true" for gre0, that is, the second command
would silently have no effect.  This could also happen when the key data
(such as remote_ip) changed but the port hash just happened to have the
same value.

This also fixes a small kernel memory leak in this case.

An upcoming commit implements the "ovs-dpctl set-if" command mentioned
above.

Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Jesse Gross <jesse at nicira.com>


-----------------------------------------------------------------------

Summary of changes:
 datapath/tunnel.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)


hooks/post-receive
-- 
Open vSwitch



More information about the git mailing list