[ovs-git] Open vSwitch: netdev-vport: Checks tunnel status change when route-table is reset. (master)

dev at openvswitch.org dev at openvswitch.org
Fri May 2 22:04:04 UTC 2014


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, master has been updated
       via  41ca1e0afb4b261a217c9fdaf672ef606e8434f9 (commit)
      from  12eb035b810ff9d537d6ff9f1eb8ad5564c1f644 (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 41ca1e0afb4b261a217c9fdaf672ef606e8434f9
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=41ca1e0afb4b261a217c9fdaf672ef606e8434f9
Author: Alex Wang <alexw at nicira.com>
		
netdev-vport: Checks tunnel status change when route-table is reset.
		
Commit 3e912ffcbb (netdev: Add 'change_seq' back to netdev.) added per-
netdev change number for indicating status change.  Future commits used
this change number to optimize the netdev status update to database.
However, the work also introduced the bug in the following scenario:

- assume interface eth0 has address 1.2.3.4, eth1 has adddress 10.0.0.1.
- assume tunnel port p1 is set with remote_ip=10.0.0.5.
- after setup, 'ovs-vsctl list interface p1 status' should show the
  'tunnel_egress_iface="eth1"'.
- now if the address of eth1 is change to 0 via 'ifconfig eth1 0'.
- expectedly, after change, 'ovs-vsctl list interface p1 status' should
  show the 'tunnel_egress_iface="eth0"'

However, 'tunnel_egress_iface' will not be updated on current master.
This is in that, the 'netdev-vport' module corresponding to p1 does
not react to routing related changes.

To fix the bug, this commit adds a change sequence number in the route-
table module and makes netdev-vport check the sequence number for
tunnel status update.

Bug #1240626

Signed-off-by: Alex Wang <alexw at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>


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

Summary of changes:
 lib/netdev-provider.h  |    1 +
 lib/netdev-vport.c     |   96 +++++++++++++++++++++++++++++++++++++++++++-----
 lib/netdev-vport.h     |    1 +
 lib/netdev.c           |   35 ++++++++++++++++++
 lib/route-table-bsd.c  |    6 +++
 lib/route-table-stub.c |    8 +++-
 lib/route-table.c      |   17 ++++++++-
 lib/route-table.h      |    3 +-
 8 files changed, 154 insertions(+), 13 deletions(-)


hooks/post-receive
-- 
Open vSwitch



More information about the git mailing list