[PATCH] totemconfig: Change default FAIL_TO_RECV_CONST

[Jan Friesse <jfriesse@xxxxxxxxxx>]

Previous default (50) was too low for most modern switch hardware. This
may trigger abort because the aru doesn't increase for 50 token
rotations combined with a defect in how failed to recv conditions are
handled.  By increasing this tunable, the condition should no longer
trigger the errant code.

Signed-off-by: Jan Friesse <jfriesse@xxxxxxxxxx>
Reviewed-by: Steven Dake <sdake@xxxxxxxxxx>
(backported from flatiron commit a48c8e517d82d099bfd3f4a8ebc11716eeb3962b)
---
 branches/whitetank/exec/totemconfig.c |    2 +-
 branches/whitetank/man/openais.conf.5 |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/branches/whitetank/exec/totemconfig.c b/branches/whitetank/exec/totemconfig.c
index 567e01c..7c0a3fd 100644
--- a/branches/whitetank/exec/totemconfig.c
+++ b/branches/whitetank/exec/totemconfig.c
@@ -62,7 +62,7 @@
 #define CONSENSUS_TIMEOUT			800
 #define MERGE_TIMEOUT				200
 #define DOWNCHECK_TIMEOUT			1000
-#define FAIL_TO_RECV_CONST			50
+#define FAIL_TO_RECV_CONST			2500
 #define	SEQNO_UNCHANGED_CONST			30
 #define MINIMUM_TIMEOUT				(int)(1000/HZ)*3
 #define MAX_NETWORK_DELAY			50
diff --git a/branches/whitetank/man/openais.conf.5 b/branches/whitetank/man/openais.conf.5
index 499c430..d8eaf86 100644
--- a/branches/whitetank/man/openais.conf.5
+++ b/branches/whitetank/man/openais.conf.5
@@ -332,7 +332,7 @@ This constant specifies how many rotations of the token without receiving any
 of the messages when messages should be received may occur before a new
 configuration is formed.
 
-The default is 50 failures to receive a message.
+The default is 2500 failures to receive a message.
 
 .TP
 seqno_unchanged_const
-- 
1.7.4.1

_______________________________________________
discuss mailing list
discuss@xxxxxxxxxxxx
http://lists.corosync.org/mailman/listinfo/discuss