Re: [Openais] [PATCH 4/4] Remove -lcoroipcc from tools/Makefile.am notifyd

2011-08-09 Thread Angus Salkeld
On Tue, Aug 09, 2011 at 02:55:06PM -0700, Steven Dake wrote: > Signed-off-by: Steven Dake Reviewed-by: Angus Salkeld > --- > tools/Makefile.am |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/tools/Makefile.am b/tools/Makefile.am > index f88e741..2699519 100644 >

Re: [Openais] [PATCH 3/4] properly define rec_token_cq_send_event_fn

2011-08-09 Thread Angus Salkeld
On Tue, Aug 09, 2011 at 02:55:05PM -0700, Steven Dake wrote: > Signed-off-by: Steven Dake Reviewed-by: Angus Salkeld > --- > exec/totemiba.c |5 - > 1 files changed, 4 insertions(+), 1 deletions(-) > > diff --git a/exec/totemiba.c b/exec/totemiba.c > index 008018a..ffcfceb 100644 > --

Re: [Openais] [PATCH 2/4] Define totemiba_log_printf properly

2011-08-09 Thread Angus Salkeld
On Tue, Aug 09, 2011 at 02:55:04PM -0700, Steven Dake wrote: > Signed-off-by: Steven Dake Reviewed-by: Angus Salkeld > --- > exec/totemiba.c |8 +--- > 1 files changed, 5 insertions(+), 3 deletions(-) > > diff --git a/exec/totemiba.c b/exec/totemiba.c > index a16f88a..008018a 100644 >

Re: [Openais] [PATCH 1/4] Fix problem in totemiba where incorrect define is used (and also not defined)

2011-08-09 Thread Angus Salkeld
On Tue, Aug 09, 2011 at 02:55:03PM -0700, Steven Dake wrote: > Signed-off-by: Steven Dake Reviewed-by: Angus Salkeld > --- > exec/totemiba.c |4 +++- > 1 files changed, 3 insertions(+), 1 deletions(-) > > diff --git a/exec/totemiba.c b/exec/totemiba.c > index 2d8c690..a16f88a 100644 > --

[Openais] [PATCH 3/4] properly define rec_token_cq_send_event_fn

2011-08-09 Thread Steven Dake
Signed-off-by: Steven Dake --- exec/totemiba.c |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/exec/totemiba.c b/exec/totemiba.c index 008018a..ffcfceb 100644 --- a/exec/totemiba.c +++ b/exec/totemiba.c @@ -562,7 +562,10 @@ static int mcast_rdma_event_fn (int events,

[Openais] [PATCH 4/4] Remove -lcoroipcc from tools/Makefile.am notifyd

2011-08-09 Thread Steven Dake
Signed-off-by: Steven Dake --- tools/Makefile.am |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/tools/Makefile.am b/tools/Makefile.am index f88e741..2699519 100644 --- a/tools/Makefile.am +++ b/tools/Makefile.am @@ -55,7 +55,7 @@ corosync_quorumtool_LDADD = -lconfdb -lc

[Openais] [PATCH 2/4] Define totemiba_log_printf properly

2011-08-09 Thread Steven Dake
Signed-off-by: Steven Dake --- exec/totemiba.c |8 +--- 1 files changed, 5 insertions(+), 3 deletions(-) diff --git a/exec/totemiba.c b/exec/totemiba.c index a16f88a..008018a 100644 --- a/exec/totemiba.c +++ b/exec/totemiba.c @@ -187,13 +187,15 @@ struct totemiba_instance { str

[Openais] [PATCH 1/4] Fix problem in totemiba where incorrect define is used (and also not defined)

2011-08-09 Thread Steven Dake
Signed-off-by: Steven Dake --- exec/totemiba.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/exec/totemiba.c b/exec/totemiba.c index 2d8c690..a16f88a 100644 --- a/exec/totemiba.c +++ b/exec/totemiba.c @@ -70,6 +70,8 @@ #include #include #include + +#include #

Re: [Openais] [PATCH 2/2] cfg: Handle errors from totem_mcast

2011-08-09 Thread Steven Dake
On second consideration this patch is Reviewed-by: Steven Dake On 08/08/2011 09:15 AM, Steven Dake wrote: > Before accepting an IPC message, ipc checks that the totem queue has > available room for new messages. As a result this patch is either not > necessary or fixes the wrong thing. > > See

Re: [Openais] [PATCH 1/2] cpg: Handle errors from totem_mcast

2011-08-09 Thread Steven Dake
On second consideration this patch is Reviewed-by: Steven Dake On 08/08/2011 09:11 AM, Steven Dake wrote: > On 07/28/2011 07:20 AM, Jan Friesse wrote: >> totem_mcast function can return -1 if corosync is overloaded. Sadly in >> many calls of this functions was error code ether not handled at all

Re: [Openais] [PATCH] Allow compile master on RHEL 6

2011-08-09 Thread Angus Salkeld
On Tue, Aug 09, 2011 at 09:23:50AM +0200, Jan Friesse wrote: > corosync_timer_handle_t is know conditionally defined to prevent double > definition causing compile fault on RHEL 6 systems. Reviewed-by: Angus Salkeld > > Signed-off-by: Jan Friesse > --- > exec/timer.h |

[Openais] [PATCH] Allow compile master on RHEL 6

2011-08-09 Thread Jan Friesse
corosync_timer_handle_t is know conditionally defined to prevent double definition causing compile fault on RHEL 6 systems. Signed-off-by: Jan Friesse --- exec/timer.h |3 +++ include/corosync/engine/coroapi.h |4 2 files changed, 7 insertions(+), 0 deletions(-)