Attachment 575387 Details for Bug 671575 – Backported flatiron patch to increase failed to recv constant

[patch] Backported flatiron patch to increase failed to recv constant

2012-04-05-0001-totemconfig-Change-default-FAIL_TO_RECV_CONST.patch (text/plain), 1.85 KB, created by Jan Friesse on 2012-04-05 12:24:56 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jan Friesse

Created: 2012-04-05 12:24:56 UTC

Size: 1.85 KB

patch

obsolete

>From aae64a1d9c5d93d12b53c18a21d4092dacfc20a3 Mon Sep 17 00:00:00 2001
>From: Jan Friesse <jfriesse@redhat.com>
>Date: Thu, 5 Apr 2012 14:22:28 +0200
>Subject: [PATCH] totemconfig: Change default FAIL_TO_RECV_CONST
>
>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@redhat.com>
>Reviewed-by: Steven Dake <sdake@redhat.com>
>(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
>

Actions: View | Diff

Attachments on bug 671575: 474700 | 477494 | 575387