[devel] [PATCH 3 of 4] amfd: use std::vector to replace realloc [#713]

2014-01-14 Thread Hans Nordeback
osaf/services/saf/amf/amfd/include/node.h | 3 +- osaf/services/saf/amf/amfd/nodegroup.cc | 12 ++ osaf/services/saf/amf/amfd/su.cc | 32 ++ 3 files changed, 20 insertions(+), 27 deletions(-) diff --git

[devel] [PATCH 4 of 4] amfnd: use std::vector instead of realloc [#713]

2014-01-14 Thread Hans Nordeback
osaf/services/saf/amf/amfnd/clc.cc | 19 --- 1 files changed, 8 insertions(+), 11 deletions(-) diff --git a/osaf/services/saf/amf/amfnd/clc.cc b/osaf/services/saf/amf/amfnd/clc.cc --- a/osaf/services/saf/amf/amfnd/clc.cc +++ b/osaf/services/saf/amf/amfnd/clc.cc @@ -33,6 +33,7

[devel] [PATCH 2 of 4] amfnd: use bool instead of SaBoolT [#713]

2014-01-14 Thread Hans Nordeback
osaf/services/saf/amf/amfnd/clc.cc | 6 +++--- osaf/services/saf/amf/amfnd/clm.cc | 8 osaf/services/saf/amf/amfnd/comp.cc | 4 +--- osaf/services/saf/amf/amfnd/compdb.cc | 14 ++ osaf/services/saf/amf/amfnd/di.cc

[devel] [PATCH 2 of 4] amfnd: use bool instead of SaBoolT [#713]

2014-01-14 Thread Hans Nordeback
osaf/services/saf/amf/amfnd/clc.cc | 6 +++--- osaf/services/saf/amf/amfnd/clm.cc | 8 osaf/services/saf/amf/amfnd/comp.cc | 4 +--- osaf/services/saf/amf/amfnd/compdb.cc | 14 ++ osaf/services/saf/amf/amfnd/di.cc

[devel] [PATCH 3 of 4] amfd: use std::vector to replace realloc [#713]

2014-01-14 Thread Hans Nordeback
osaf/services/saf/amf/amfd/include/node.h | 3 +- osaf/services/saf/amf/amfd/nodegroup.cc | 12 ++ osaf/services/saf/amf/amfd/su.cc | 32 ++ 3 files changed, 20 insertions(+), 27 deletions(-) diff --git

[devel] [PATCH 4 of 4] amfnd: use std::vector instead of realloc [#713]

2014-01-14 Thread Hans Nordeback
osaf/services/saf/amf/amfd/include/node.h | 1 + osaf/services/saf/amf/amfnd/clc.cc| 19 --- 2 files changed, 9 insertions(+), 11 deletions(-) diff --git a/osaf/services/saf/amf/amfd/include/node.h b/osaf/services/saf/amf/amfd/include/node.h ---

Re: [devel] [PATCH 2 of 4] amfnd: use bool instead of SaBoolT [#713]

2014-01-14 Thread Hans Feldt
Ack with comment: - there are 2 occurrences of x != true, please change to x == false Thanks, Hans -Original Message- From: Hans Nordebäck Sent: den 14 januari 2014 13:24 To: Hans Feldt; praveen.malv...@oracle.com; nagendr...@oracle.com Cc: opensaf-devel@lists.sourceforge.net

Re: [devel] [PATCH 1 of 4] amfd: use bool instead of SaBoolT [#713]

2014-01-14 Thread Anders Björnerstedt
Or just if (!avnd-node_info.member) :-) /AndersBj -Original Message- From: Hans Feldt [mailto:hans.fe...@ericsson.com] Sent: den 14 januari 2014 13:49 To: Hans Nordebäck; praveen.malv...@oracle.com; nagendr...@oracle.com Cc: opensaf-devel@lists.sourceforge.net Subject: Re:

[devel] [PATCH 0 of 1] Review Request for amf #713

2014-01-14 Thread Hans Feldt
Summary: use map in app Review request for Trac Ticket(s): 713 Peer Reviewer(s): Hans N, Praveen Pull request to: LIST THE PERSON WITH PUSH ACCESS HERE Affected branch(es): default Development branch: IF ANY GIVE THE REPO URL Impacted area Impact y/n

[devel] [PATCH 1 of 1] amfd: use stl:map in app [#713]

2014-01-14 Thread Hans Feldt
osaf/services/saf/amf/amfd/app.cc| 72 --- osaf/services/saf/amf/amfd/ckpt_enc.cc | 12 +--- osaf/services/saf/amf/amfd/include/app.h | 13 +++-- osaf/services/saf/amf/amfd/util.cc | 15 +++--- 4 files changed, 47 insertions(+), 65 deletions(-)

Re: [devel] [PATCH 1 of 1] amfd: use stl:map in app [#713]

2014-01-14 Thread Hans Nordebäck
as and alternative to use a global g_app_db you can use a class, e.g.: h file: class AmfApplicationDb { public: static ApplicationMap application_map_db; static void add(AmfApplication* application); : }; cc file: ApplicationMap AmfApplicationDb::application_map_db; void

[devel] checkpoint section create performance

2014-01-14 Thread Alex Jones
AVM, In my 5+1 setup, when I have the standby node open all the checkpoints and read from them, as well as open the hot-standby callback, the section creates done on the other active nodes can take a very long time. (For 40k sections, it can sometimes take over 2 minutes). Once the

Re: [devel] checkpoint section create performance

2014-01-14 Thread Alex Jones
3.7.1 saCkptSectionCreate() If the checkpoint was created with the SA_CKPT_WR_ALL_REPLICAS property, the section is created in all of the checkpoint replicas when the invocation returns; otherwise, the section has been created at least in the active checkpoint replica when the invocation