Re: [Openais] How to tune corosync heartbeat timer ?

2010-05-17 Thread Alain.Moulle
Hi again, I 've checked the man corosync.conf and seen many parameters around token timers etc. but I can't see how to increase the heartbeat timeout. When testing, it occurs that timeout is between 10s and 12s before a node decides to fence another one in the cluster (when for example I force a

Re: [Openais] [PATCH 1/3] cov 10405: remove unused pointer from totemsrp.

2010-05-17 Thread Steven Dake
good for merge On Mon, 2010-05-17 at 12:01 +1000, Angus Salkeld wrote: Signed-off-by: Angus Salkeld asalk...@redhat.com --- exec/totemsrp.c |4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff --git a/exec/totemsrp.c b/exec/totemsrp.c index 8786859..93e8769 100644 ---

Re: [Openais] [PATCH 3/3] cov 10387 (and others): make _logsys_subsys_create() return int

2010-05-17 Thread Steven Dake
good for merge On Mon, 2010-05-17 at 12:02 +1000, Angus Salkeld wrote: in the body it can return -1, and callers check for -1. but the return type is unsigned int? Signed-off-by: Angus Salkeld asalk...@redhat.com --- exec/logsys.c|2 +-

[Openais] corosync trunk: flush log messages if worker thread for logsys hasn't been started

2010-05-17 Thread Steven Dake
This patch flushes log messages in the case of an error condition that occurs before the worker thread responsible for log flushing is activated. Regards -steve Index: exec/logsys.c === --- exec/logsys.c (revision 2843) +++

Re: [Openais] [PATCH 1/3] cov 10405: remove unused pointer from totemsrp.

2010-05-17 Thread Angus Salkeld
On Mon, May 17, 2010 at 02:24:56PM -0700, Steven Dake wrote: good for merge Comitted. On Mon, 2010-05-17 at 12:01 +1000, Angus Salkeld wrote: Signed-off-by: Angus Salkeld asalk...@redhat.com --- exec/totemsrp.c |4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff

Re: [Openais] [PATCH 2/3] cov 10404: don't assign loc pointer when not used.

2010-05-17 Thread Angus Salkeld
On Mon, May 17, 2010 at 02:26:22PM -0700, Steven Dake wrote: good for merge Committed. loc is reused On Mon, 2010-05-17 at 12:02 +1000, Angus Salkeld wrote: Signed-off-by: Angus Salkeld asalk...@redhat.com --- exec/coroparse.c |2 +- 1 files changed, 1 insertions(+), 1

Re: [Openais] [PATCH 1/3] cov 10391: allow assert to check for a negitive number

2010-05-17 Thread Steven Dake
good for merge On Tue, 2010-05-18 at 10:07 +1000, Angus Salkeld wrote: Signed-off-by: Angus Salkeld asalk...@redhat.com --- exec/totemsrp.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/exec/totemsrp.c b/exec/totemsrp.c index 93e8769..6dfd42d 100644 ---

Re: [Openais] [PATCH 1/3] cov 10391: allow assert to check for a negitive number

2010-05-17 Thread Angus Salkeld
On Mon, May 17, 2010 at 05:21:18PM -0700, Steven Dake wrote: good for merge Committed. On Tue, 2010-05-18 at 10:07 +1000, Angus Salkeld wrote: Signed-off-by: Angus Salkeld asalk...@redhat.com --- exec/totemsrp.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff

Re: [Openais] [PATCH 2/3] cov (many): make sure all _set() functions return a signed int

2010-05-17 Thread Angus Salkeld
On Mon, May 17, 2010 at 05:21:53PM -0700, Steven Dake wrote: good for merge committed. ___ Openais mailing list Openais@lists.linux-foundation.org https://lists.linux-foundation.org/mailman/listinfo/openais

[Openais] [PATCH] add __attribute__((noreturn)) to functions that always exit.

2010-05-17 Thread Angus Salkeld
we had some __attribute__((__noreturn__)) and some__attribute__((noreturn)) I made them all: __attribute__((noreturn)) Signed-off-by: Angus Salkeld asalk...@redhat.com --- exec/apidef.c |2 +- exec/logsys.c |2 +- exec/main.c