[ovs-git] Open vSwitch: bfd: Require bfd control packet received in forwarding_if_rx mode. (master)

[dev at openvswitch.org]

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  34c88624ad02129a1b477717fe5d3928530dccbe (commit)
       via  5767a79a40599e5e6e70662833b27ffdbdab7e37 (commit)
      from  6e9231f70f31317619d43838f54fc6abce3942ac (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 34c88624ad02129a1b477717fe5d3928530dccbe
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=34c88624ad02129a1b477717fe5d3928530dccbe
Author: Alex Wang <alexw at nicira.com>
		
bfd: Require bfd control packet received in forwarding_if_rx mode.
		
This commit adds a requirement that bfd session must receive at least
one bfd control packet every 100 * bfd->cfg_min_rx amount of time in
forwarding_if_rx mode.  Otherwise, even if the data packets are received
on the monitored interface, the bfd->forwarding is still false.

Since the datapath flow is not purged when the userspace Open Vswitch
crashes, data packet can still be forwarded through the tunnel and
fool the remote BFD session in forwarding_if_rx mode.  Thus, this commit
can prevent the remote BFD session from falsely declaring tunnel liveness
in this situation.

Signed-off-by: Alex Wang <alexw at nicira.com>
Acked-by: Ethan Jackson <ethan at nicira.com>



commit 5767a79a40599e5e6e70662833b27ffdbdab7e37
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=5767a79a40599e5e6e70662833b27ffdbdab7e37
Author: Alex Wang <alexw at nicira.com>
		
cfm: Require ccm received in demand mode.
		
This commit adds a new requirement that cfm session must receive
at least one ccm every 100 * cfm_interval amount of time in demand
mode.  Otherwise, even if the data packets are received on the
monitored interface, the cfm session still reports "[recv]" fault.

Since the datapath flow is not purged when the userspace Open Vswitch
crashes, data packet can still be forwarded through the tunnel and
fool the remote CFM session in demand mode.  Thus, this commit can
prevent the remote CFM session from falsely declaring tunnel liveness
in this situation.

Signed-off-by: Alex Wang <alexw at nicira.com>
Acked-by: Ethan Jackson <ethan at nicira.com>



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

Summary of changes:
 lib/bfd.c            |   27 ++++++++++----
 lib/cfm.c            |   12 +++++-
 tests/bfd.at         |  101 ++++++++++++++++++++++++++++++++++++++------------
 tests/cfm.at         |   75 +++++++++++++++++++++++++++++++++++++
 vswitchd/vswitch.xml |   18 +++++----
 5 files changed, 195 insertions(+), 38 deletions(-)


hooks/post-receive
-- 
Open vSwitch