[devel] [PATCH 3 of 8] cpa: Add support for extended SaNameT [#1574] v1

2016-08-18 Thread Hoang Vo
osaf/libs/agents/saf/cpa/Makefile.am | 1 + osaf/libs/agents/saf/cpa/cpa_api.c | 36 osaf/libs/agents/saf/cpa/cpa_db.c| 2 + osaf/libs/agents/saf/cpa/cpa_mds.c | 2 +- osaf/libs/agents/saf/cpa/cpa_proc.c | 2 +- osaf/libs/commo

[devel] [PATCH 2 of 8] cpnd: Add support for extended SaNameT [#1574] v3

2016-08-18 Thread Hoang Vo
osaf/libs/common/cpsv/include/cpnd.h | 1 + osaf/libs/common/cpsv/include/cpnd_cb.h | 5 +- osaf/libs/common/cpsv/include/cpnd_init.h | 3 +- osaf/libs/common/cpsv/include/cpsv_shm.h | 6 +- osaf/services/saf/cpsv/cpnd/Makefile.am | 1 + osaf/services/saf/cpsv/cpnd/cpnd_db.c

[devel] [PATCH 4 of 8] cpsv: Add new message to support extended SaNameT [#1574] v3

2016-08-18 Thread Hoang Vo
osaf/libs/common/cpsv/cpsv_evt.c | 439 +- osaf/libs/common/cpsv/include/cpsv_evt.h | 10 + osaf/services/saf/cpsv/cpd/cpd_mds.c | 84 +- osaf/services/saf/cpsv/cpnd/cpnd_mds.c | 86 +- 4 files changed, 581 insertions(+), 38 deletions(-) d

[devel] [PATCH 1 of 8] cpd: Add support for extended SaNameT [#1574] v3

2016-08-18 Thread Hoang Vo
osaf/libs/common/cpsv/include/cpd.h |1 + osaf/libs/common/cpsv/include/cpd_cb.h | 17 +- osaf/libs/common/cpsv/include/cpd_imm.h |4 +- osaf/libs/common/cpsv/include/cpd_mem.h | 25 ++- osaf/libs/common/cpsv/include/cpd_proc.h |2 +- osaf/services/saf/cpsv/cpd/Makefile.am

[devel] [PATCH 6 of 8] cpsv: Apply new messages supporting extended SaNameT to CPD, CPND, and CPA [#1574] v4

2016-08-18 Thread Hoang Vo
osaf/libs/agents/saf/cpa/cpa_api.c | 12 osaf/libs/agents/saf/cpa/cpa_mds.c | 2 +- osaf/libs/common/cpsv/cpsv_evt.c| 1 + osaf/services/saf/cpsv/cpd/cpd_proc.c | 2 +- osaf/services/saf/cpsv/cpnd/cpnd_evt.c | 2 ++ osaf/services/saf/cpsv/cpnd/cpnd_proc.c

[devel] [PATCH 8 of 8] cpnd: add support for shm recovery for in-service update without restarting node [#1574] v1

2016-08-18 Thread Hoang Vo
osaf/libs/common/cpsv/include/cpsv_shm.h | 18 + osaf/services/saf/cpsv/cpnd/cpnd_proc.c |1 - osaf/services/saf/cpsv/cpnd/cpnd_res.c | 821 +++--- 3 files changed, 633 insertions(+), 207 deletions(-) diff --git a/osaf/libs/common/cpsv/include/cpsv_shm.h b/osaf

[devel] [PATCH 7 of 8] ckpt: Add new test cases to verify long DN feature on CPSV [#1574] v1

2016-08-18 Thread Hoang Vo
tests/cpsv/Makefile.am |1 + tests/cpsv/test_cpa.c | 180 +++- tests/cpsv/test_cpa_util.c | 16 +++- tests/cpsv/test_cpsv.h |6 + tests/cpsv/test_cpsv_conf.h |3 + 5 files changed, 201 insertions(+), 5 deletions(-) diff --gi

[devel] [PATCH 5 of 8] cpd: Add new mbcsv messages supporting extended SaNameT [#1574] v2

2016-08-18 Thread Hoang Vo
osaf/services/saf/cpsv/cpd/cpd_mbcsv.c | 22 ++ 1 files changed, 22 insertions(+), 0 deletions(-) diff --git a/osaf/services/saf/cpsv/cpd/cpd_mbcsv.c b/osaf/services/saf/cpsv/cpd/cpd_mbcsv.c --- a/osaf/services/saf/cpsv/cpd/cpd_mbcsv.c +++ b/osaf/services/saf/cpsv/cpd/cpd_m

[devel] [PATCH 0 of 8] Review Request for CKPT: Support DNs longer than 255 bytes [#1574] v5

2016-08-18 Thread Hoang Vo
Summary: CKPT: Support DNs longer than 255 bytes {#1574} Review request for Trac Ticket(s): 1574 Peer Reviewer(s): mahesh.va...@oracle.com; anders.wid...@ericsson.com Pull request to: mahesh.va...@oracle.com Affected branch(es): default Development branch: default

[devel] [PATCH 1 of 1] amfd: ensure long DNs are sent correctly to NTF [#1642]

2016-08-18 Thread Gary Lee
osaf/services/saf/amf/amfd/include/ntf.h |1 - osaf/services/saf/amf/amfd/ntf.cc| 105 +++--- osaf/services/saf/amf/amfd/si.cc |2 +- 3 files changed, 41 insertions(+), 67 deletions(-) Ensure long DNs are sent correctly in NTF additional info Add

[devel] [PATCH 0 of 1] Review Request for amfd: ensure long DNs are sent correctly to NTF [#1642]

2016-08-18 Thread Gary Lee
Summary: amfd: ensure long DNs are sent correctly to NTF [#1642] Review request for Trac Ticket(s): 1642 Peer Reviewer(s): <> Pull request to: <> Affected branch(es): default Development branch: default Impacted area Impact y/n ---

[devel] [PATCH 0 of 1] Review Request for MDS: Log TIPC dropped messages [#1957]

2016-08-18 Thread Hans Nordeback
Summary: MDS: Log TIPC dropped messages Review request for Trac Ticket(s): #1957 Peer Reviewer(s): AndersW, Mathi, Mahesh Pull request to: Affected branch(es): default Development branch: default Impacted area Impact y/n Doc

[devel] [PATCH 1 of 1] MDS: Log TIPC dropped messages [#1957]

2016-08-18 Thread Hans Nordeback
osaf/libs/core/mds/mds_dt_tipc.c | 32 +--- 1 files changed, 25 insertions(+), 7 deletions(-) diff --git a/osaf/libs/core/mds/mds_dt_tipc.c b/osaf/libs/core/mds/mds_dt_tipc.c --- a/osaf/libs/core/mds/mds_dt_tipc.c +++ b/osaf/libs/core/mds/mds_dt_tipc.c @@ -320,6 +320

Re: [devel] [PATCH 1 of 1] MDS: Log TIPC dropped messages [#1957]

2016-08-18 Thread A V Mahesh
Hi HansN, It seem you missed to see below : On 8/12/2016 9:11 AM, A V Mahesh wrote: > Hi HansN, > > We were having ticket for this raised by Hans Feldt > `https://sourceforge.net/p/opensaf/tickets/634/` > > at that time i have give my analysis base the MDS code at that time as > below please

[devel] [PATCH 0 of 1] Review Request for cpa: remove multiple sync_send() calls in case of multiple vector write [#1849]

2016-08-18 Thread mahesh . valla
Summary:cpa: remove multiple sync_send() calls in case of multiple vector write [#1849] Review request for Trac Ticket(s): #1849 Peer Reviewer(s): Nhat,Hoang Pull request to: <> Affected branch(es): default, 5.0 Development branch: default Impacted area Im

[devel] [PATCH 1 of 1] cpa: remove multiple sync_send() calls in case of multiple vector write [#1849]

2016-08-18 Thread mahesh . valla
osaf/libs/agents/saf/cpa/cpa_api.c | 17 - 1 files changed, 8 insertions(+), 9 deletions(-) Issue : In current saCkptCheckpointWrite() , in case of multiple ioVector write case ( > 1 ) multiple calls made for cpa_mds_msg_sync_send() , which is cause performance issue . Fix

Re: [devel] [PATCH 07 of 13] leap: support for SAF SaTimeT timeout variable in MDS transport [#1658]

2016-08-18 Thread ramesh betham
Hi Mahesh, Good job! Ack with the following review comments. Not tested. * hj_tmr.c: rp_tmr_time_left_in_sec(time_t tmr_trig_at, *uint32**_t* tmr_value) ==> change uint32_t to int64_t. * ncs_tmr.h: #define m_RP_TMR_LOG_MSG(str, val) printf("%s :: %x\n", str, *(uint32_t*)val); ==> has

Re: [devel] [PATCH 0 of 1] Review Request for imm: Return implementer ID to timed out client [#1948]

2016-08-18 Thread Neelakanta Reddy
Hi Hung, Reviewed and tested the patch. Ack. /Neel. On 2016/08/11 02:05 PM, Hung Nguyen wrote: > Summary: imm: Return implementer ID to timed out client [#1948] > Review request for Trac Ticket(s): 1948 > Peer Reviewer(s): Zoran, Neel > Pull request to: > Affected branch(es): 4.7, 5.0, 5.1 > Dev

Re: [devel] [PATCH 0 of 1] Review Request for immtools: Allow immadm to explicitly set ReleaseOnFinalize [#1937]

2016-08-18 Thread Neelakanta Reddy
Hi Hung, Reviewed the patch. Ack. /Neel. On 2016/08/09 01:48 PM, Hung Nguyen wrote: > Summary: immtools: Allow immadm to explicitly set ReleaseOnFinalize [#1937] > Review request for Trac Ticket(s): 1937 > Peer Reviewer(s): Zoran, Neel > Pull request to: > Affected branch(es): 5.1 > Development

[devel] [PATCH 1 of 1] imm: fix string format for size_t type [#195]

2016-08-18 Thread Zoran Milinkovic
osaf/services/saf/immsv/immnd/ImmModel.cc | 8 1 files changed, 4 insertions(+), 4 deletions(-) Fix string format for size_t type that IMM can be compiled with 32-bit compiler. diff --git a/osaf/services/saf/immsv/immnd/ImmModel.cc b/osaf/services/saf/immsv/immnd/ImmModel.cc --- a/os

[devel] [PATCH 0 of 1] Review Request for imm: fix string format for size_t type [#195]

2016-08-18 Thread Zoran Milinkovic
Summary: imm: fix string format for size_t type [#195] Review request for Trac Ticket(s): 195 Peer Reviewer(s): Neelakanta, Hung Pull request to: Zoran Affected branch(es): default(5.1) Development branch: default(5.1) Impacted area Impact y/n ---

Re: [devel] [PATCH 0 of 1] Review Request for imm: Do not revert isApplier and isPbe flag when timeout occurs [#1943]

2016-08-18 Thread Neelakanta Reddy
Hi Hung, Reviewed and tested the patch. Ack. /Neel. On 2016/08/11 12:39 PM, Hung Nguyen wrote: > Summary: imm: Do not revert isApplier and isPbe flag when timeout occurs > [#1943] > Review request for Trac Ticket(s): 1943 > Peer Reviewer(s): Zoran, Neel > Pull request to: > Affected branch(es):

Re: [devel] [PATCH 0 of 1] Review Request for imm: fix string format for size_t type [#195]

2016-08-18 Thread Neelakanta Reddy
Hi zoran, Reviewed the patch. Ack. /Neel. On 2016/08/18 05:46 PM, Zoran Milinkovic wrote: > Summary: imm: fix string format for size_t type [#195] > Review request for Trac Ticket(s): 195 > Peer Reviewer(s): Neelakanta, Hung > Pull request to: Zoran > Affected branch(es): default(5.1) > Developm

Re: [devel] [PATCH 1 of 1] log: fix failed to recover app streams [#1941]

2016-08-18 Thread Lennart Lund
Hi Vu Ack Thanks Lennart > -Original Message- > From: Vu Minh Nguyen [mailto:vu.m.ngu...@dektech.com.au] > Sent: den 10 augusti 2016 08:40 > To: Lennart Lund ; A V Mahesh > > Cc: opensaf-devel@lists.sourceforge.net > Subject: [PATCH 1 of 1] log: fix failed to recover app streams [#1941]

Re: [devel] [PATCH 0 of 1] Review Request for smf: Incorrect notification about AMF OI error in syslog [#1953]

2016-08-18 Thread Neelakanta Reddy
Hi Lennart, Reviewed the patch. Ack. /Neel. On 2016/08/15 04:30 PM, Lennart Lund wrote: > Summary: smf: Incorrect notification about AMF OI error in syslog > Review request for Trac Ticket(s): #1953 > Peer Reviewer(s): reddy.neelaka...@oracle.com, rafael.odza...@ericsson.com > Pull request to: <

[devel] [PATCH 0 of 1] Review Request for cpsv: To update checkpoint user number for each node [#1669] V5

2016-08-18 Thread Hoang Vo
Summary: cpsv: To update checkpoint user number for each node [#1669] V5 Review request for Trac Ticket(s): 1669 Peer Reviewer(s): mahesh.va...@oracle.com; anders.wid...@ericsson.com Pull request to: mahesh.va...@oracle.com; Affected branch(es): default Development branch: default

[devel] [PATCH 1 of 1] cpsv: To update checkpoint user number for each node [#1669] V5

2016-08-18 Thread Hoang Vo
osaf/libs/common/cpsv/include/cpd_cb.h |6 +- osaf/libs/common/cpsv/include/cpd_proc.h |3 + osaf/libs/common/cpsv/include/cpd_red.h | 13 ++ osaf/libs/common/cpsv/include/cpsv_evt.h |8 + osaf/services/saf/cpsv/cpd/cpd_db.c | 14 ++- osaf/services/saf/cpsv/cpd/cpd_evt.c

[devel] [PATCH 1 of 1] amfa: fixed freeing notification buff [#1642]

2016-08-18 Thread Long HB Nguyen
osaf/libs/agents/saf/amfa/amf_agent.cc | 1 + osaf/libs/agents/saf/amfa/ava_hdl.cc | 2 -- 2 files changed, 1 insertions(+), 2 deletions(-) diff --git a/osaf/libs/agents/saf/amfa/amf_agent.cc b/osaf/libs/agents/saf/amfa/amf_agent.cc --- a/osaf/libs/agents/saf/amfa/amf_agent.cc +++ b/osaf/li

[devel] [PATCH 0 of 1] Review Request for amfa: fixed freeing notification buff [#1642]

2016-08-18 Thread Long HB Nguyen
Summary: amfa: fixed freeing notification buff [#1642] Review request for Trac Ticket(s): 1642 Peer Reviewer(s): AMF devs Pull request to: Amf maintainers Affected branch(es): default Development branch: default Impacted area Impact y/n --

[devel] [PATCH 1 of 1] amfnd: remove used code in util.cc [#1642]

2016-08-18 Thread Long HB Nguyen
osaf/services/saf/amf/amfnd/util.cc | 546 1 files changed, 0 insertions(+), 546 deletions(-) diff --git a/osaf/services/saf/amf/amfnd/util.cc b/osaf/services/saf/amf/amfnd/util.cc --- a/osaf/services/saf/amf/amfnd/util.cc +++ b/osaf/services/saf/amf/amfnd/

[devel] [PATCH 0 of 1] Review Request for amfnd: remove used code in util.cc [#1642]

2016-08-18 Thread Long HB Nguyen
Summary: amfnd: remove used code in util.cc [#1642] Review request for Trac Ticket(s): 1642 Peer Reviewer(s): AMF devs Pull request to: AMF maintainers Affected branch(es): default Development branch: default Impacted area Impact y/n -