[ovs-git] Open vSwitch: cfm: Allow time for CCM reception after cfm_configure(); (master)

dev at openvswitch.org dev at openvswitch.org
Fri Apr 1 20:48:31 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, master has been updated
       via  aac19178a43f5ed3367e2d7a7a19ae34fdf56cfb (commit)
       via  9aa952b2dd7c091c63dbfc7becb3b3dcc9f7bfce (commit)
      from  43675e260ccc26f8278fd751436af125a18bee38 (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 aac19178a43f5ed3367e2d7a7a19ae34fdf56cfb
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=aac19178a43f5ed3367e2d7a7a19ae34fdf56cfb
Author: Ethan Jackson <ethan at nicira.com>
cfm: Allow time for CCM reception after cfm_configure();
Before this (and the previous) patch, whenever cfm_configure was
called it would set the fault_timer to expired.  Thus, the next
call to cfm_run would notice a lack of CCM reception and trigger a
faulted status.  This is a bug in and of itself, but normally would
not be a big deal because cfm_configure should only be called
infrequently (when the database changes).  However due to an
unrelated bug, cfm_configure() was getting called approximately once
per second.  This resulted in all monitors showing faults all of
the time.

This patch fixes the problem by not expiring the timer at
cfm_configure().  Instead it gives it the appropriate
fault_interval amount of time to miss heartbeats.

Bug #5244.

commit 9aa952b2dd7c091c63dbfc7becb3b3dcc9f7bfce
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=9aa952b2dd7c091c63dbfc7becb3b3dcc9f7bfce
Author: Ethan Jackson <ethan at nicira.com>
cfm: cfm_configure() only update when necessary.
Calling cfm_configure often could cause timers to be reset
resulting in unexpected behavior.  This commit only updates when
cfm configuration actually changed.

Bug #5244.


Summary of changes:
 lib/cfm.c |   46 +++++++++++++++++++++++++++-------------------
 1 files changed, 27 insertions(+), 19 deletions(-)

Open vSwitch

More information about the git mailing list