[devel] [PATCH 0 of 1] Review Request for amf: Fix all Cppcheck 1.77 issues [#2341]

2017-03-08 Thread mahesh . valla
Summary:amf: Fix all Cppcheck 1.77 issues [#2341] Review request for Trac Ticket(s): #2341 Peer Reviewer(s): Amf Dev Pull request to: <> Affected branch(es): <> Development branch: default Impacted area Impact y/n Docs

[devel] [PATCH 1 of 1] mds: reduce log level to Err for TIPC_ERR_OVERLOAD in syslog [#2350]

2017-03-06 Thread mahesh . valla
src/mds/mds_dt_tipc.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) Note : MDS still maintained as MDS_LOG_CRITICAL diff --git a/src/mds/mds_dt_tipc.c b/src/mds/mds_dt_tipc.c --- a/src/mds/mds_dt_tipc.c +++ b/src/mds/mds_dt_tipc.c @@ -590,7 +590,7 @@ ssize_t

[devel] [PATCH 0 of 1] Review Request for mds: reduce log level to Err for TIPC_ERR_OVERLOAD in syslog [#2350]

2017-03-06 Thread mahesh . valla
Summary:mds: reduce log level to Err for TIPC_ERR_OVERLOAD in syslog [#2350] Review request for Trac Ticket(s): #2350 Peer Reviewer(s): Hans N Pull request to: <> Affected branch(es): default , 5.2, 5.1 Development branch: default Impacted area Impact y/n

[devel] [PATCH 0 of 1] Review Request for log: Fix all Cppcheck 1.77 issues [#2326]

2017-02-27 Thread mahesh . valla
Summary:log: Fix all Cppcheck 1.77 issues [#2326] Review request for Trac Ticket(s): #2326 Peer Reviewer(s): Lennart, Canh, Vu Pull request to: <> Affected branch(es):default Development branch: default Impacted area Impact y/n

[devel] [PATCH 1 of 1] log: Fix all Cppcheck 1.77 issues [#2326]

2017-02-27 Thread mahesh . valla
src/log/agent/lga_api.c | 3 +-- src/log/agent/lga_mds.c | 6 ++ src/log/agent/lga_state.c| 3 +-- src/log/agent/lga_util.c | 4 +--- src/log/logd/lgs_clm.cc | 8 +++- src/log/logd/lgs_config.cc | 33 -

[devel] [PATCH 0 of 1] Review Request for logd: replace client DB patricia tree with cpp Map [#1984] V1

2017-02-23 Thread mahesh . valla
Summary: logd: replace client DB patricia tree with cpp Map [#1984] V1 Review request for Trac Ticket(s): #1984 Peer Reviewer(s): Vu, Lennart, Canh Pull request to: <> Affected branch(es): default Development branch: default Impacted area Impact y/n

[devel] [PATCH 1 of 1] logd: replace client DB patricia tree with cpp Map [#1984] V1

2017-02-23 Thread mahesh . valla
src/log/README|4 +- src/log/README_LONGDN | 15 +--- src/log/logd/lgs_cb.h |2 - src/log/logd/lgs_clm.cc | 25 +++- src/log/logd/lgs_evt.cc | 123 ++--- src/log/logd/lgs_evt.h|7 ++ src/log/logd/lgs_mbcsv.cc

[devel] [PATCH 1 of 1] logd: replace client DB patricia tree with cpp Map [#1984]

2017-02-12 Thread mahesh . valla
src/log/README|4 +- src/log/README_LONGDN | 15 +--- src/log/logd/lgs_cb.h |2 - src/log/logd/lgs_clm.cc | 25 +++- src/log/logd/lgs_evt.cc | 123 ++--- src/log/logd/lgs_evt.h|7 ++ src/log/logd/lgs_mbcsv.cc

[devel] [PATCH 0 of 1] Review Request for logd: replace client DB patricia tree with cpp Map [#1984]

2017-02-12 Thread mahesh . valla
Summary:logd: replace client DB patricia tree with cpp Map [#1984] Review request for Trac Ticket(s): #1984 Peer Reviewer(s):Vu, Lennart, Canh Pull request to: <> Affected branch(es): <> Development branch: <> Impacted area Impact y/n

[devel] [PATCH 1 of 1] logd: implement cfg files rotation [#2239]

2017-02-05 Thread mahesh . valla
src/log/logd/lgs_file.cc|4 + src/log/logd/lgs_file.h |1 + src/log/logd/lgs_filehdl.cc | 182 ++- src/log/logd/lgs_filehdl.h |2 + src/log/logd/lgs_stream.cc | 88 +++- 5 files changed, 262 insertions(+), 15

[devel] [PATCH 0 of 1] Review Request for logd: implement cfg files rotation [#2239]

2017-02-05 Thread mahesh . valla
Summary:logd: implement cfg files rotation [#2239] Review request for Trac Ticket(s): #2239 Peer Reviewer(s): Vu, Lennart, Canh Pull request to: <> Affected branch(es): default Development branch: default Impacted area Impact y/n

[devel] [PATCH 0 of 1] Review Request for cpd: syncup active standby mbcsv dtat for non-colcated ckpt above 3 replicas[#2253]

2017-01-06 Thread mahesh . valla
Summary: cpd: syncup active standby mbcsv dtat for non-colcated ckpt above 3 replicas[#2253] Review request for Trac Ticket(s): #2253 Peer Reviewer(s): Hoang Pull request to: <> Affected branch(es): defualt to 5.1 Development branch: defult Impacted area

[devel] [PATCH 1 of 1] cpd: syncup active standby mbcsv dtat for non-colcated ckpt above 3 replicas[#2253]

2017-01-06 Thread mahesh . valla
src/ckpt/ckptd/cpd_sbevt.c | 30 +- 1 files changed, 21 insertions(+), 9 deletions(-) Issue :According to Ckpt non-collocated ckpt implementation the cluster can have max 3 replicas and minimum of 2 replicas,if the

[devel] [PATCH 3 of 3] mqsv : update mqsv error handing based on leap changes [#2202]

2016-11-29 Thread mahesh . valla
osaf/services/saf/glsv/glnd/glnd_shm.c | 2 +- osaf/services/saf/mqsv/mqnd/mqnd_shm.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/osaf/services/saf/glsv/glnd/glnd_shm.c b/osaf/services/saf/glsv/glnd/glnd_shm.c --- a/osaf/services/saf/glsv/glnd/glnd_shm.c +++

[devel] [PATCH 2 of 3] cpsv : update cpsv error handing based on leap changes [#2202]

2016-11-29 Thread mahesh . valla
osaf/libs/common/cpsv/include/cpnd_cb.h | 4 +- osaf/libs/common/cpsv/include/cpnd_init.h | 8 +- osaf/libs/common/cpsv/include/cpnd_sec.h | 2 +- osaf/libs/core/include/ncs_osprm.h| 2 +- osaf/services/saf/cpsv/cpnd/cpnd_db.c | 12 ++--

[devel] [PATCH 0 of 3] Review Request for leap : now leap library ensure shm availability before writing [#2202]

2016-11-29 Thread mahesh . valla
Summary:leap : now leap library ensure shm availability before writing [#2202] Review request for Trac Ticket(s): #2202 Peer Reviewer(s): Hoang / Ramesh Pull request to: <> Affected branch(es): <> Development branch: <> Impacted area Impact y/n

[devel] [PATCH 1 of 3] leap : now leap library ensure shm availability before writing [#2202]

2016-11-29 Thread mahesh . valla
osaf/libs/core/leap/os_defs.c | 20 ++-- 1 files changed, 18 insertions(+), 2 deletions(-) Issue : If OSAF_CKPT_SHM_ALLOC_GUARANTEE is NOT set and SHM is 100% used in system , pnd Segmentation fault (core dumped) at LEAP memcpy(). Fix : Now LEAP library ensures shm free

[devel] [PATCH 0 of 1] Review Request for cpsv: on cpnd down fist remove child safReplica object then parent safCkpt object [#2189]

2016-11-16 Thread mahesh . valla
Summary:v: on cpnd down fist remove child safReplica object then parent safCkpt object [#2189] Review request for Trac Ticket(s): #2189 Peer Reviewer(s): Hoang Vo Pull request to: <> Affected branch(es): default , 5.1 Development branch: default Impacted area

[devel] [PATCH 1 of 1] cpsv: on cpnd down fist remove child safReplica object then parent safCkpt object [#2189]

2016-11-16 Thread mahesh . valla
osaf/services/saf/cpsv/cpd/cpd_imm.c | 20 osaf/services/saf/cpsv/cpd/cpd_proc.c | 9 - 2 files changed, 20 insertions(+), 9 deletions(-) Bug : While cpd processing cpnd down for COLLOCATED cktp and that checkpoint only exist on the went down cpnd ( no others

[devel] [PATCH 0 of 1] Review Request for cpa: provide support for full range of timeout for CkptSynchronize API [#2044]

2016-09-19 Thread mahesh . valla
Summary:cpa: provide support for full range of timeout for CkptSynchronize API [#2044] Review request for Trac Ticket(s): #2044 Peer Reviewer(s): Hoang Pull request to: <> Affected branch(es): 4.7 to default Development branch: default Impacted area Impact

[devel] [PATCH 1 of 1] cpa: provide support for full range of timeout for CkptSynchronize API [#2044]

2016-09-19 Thread mahesh . valla
osaf/libs/agents/saf/cpa/cpa_api.c | 13 ++--- 1 files changed, 6 insertions(+), 7 deletions(-) saCkptCheckpointSynchronize() API functions that take an explicit timeout parameter are currently expect the user to always set this parameter to MIN timeout value. Now this patch

[devel] [PATCH 0 of 1] Review Request for dtm: TCP Improve node fail fast with TCP_USER_TIMEOUT [#2014]

2016-09-13 Thread mahesh . valla
Summary:dtm: TCP Improve node failFast with TCP_USER_TIMEOUT [#2014] Review request for Trac Ticket(s): #2014 Peer Reviewer(s): Jonas, Anders widell Pull request to: <> Affected branch(es): defulat, 5.1 Development branch: default Impacted area Impact y/n

[devel] [PATCH 0 of 1] Review Request for lga: ignore CLM status if client Initialize with A.2.1 [#1999]

2016-09-07 Thread mahesh . valla
Summary:lga: ignore CLM status if client Initialize with A.2.1 [#1999] Review request for Trac Ticket(s): #1999 Peer Reviewer(s): Vu, Lennart,praveen Pull request to: <> Affected branch(es): 5.1 default Development branch: default Impacted area Impact y/n

[devel] [PATCH 1 of 1] lga: ignore CLM status if client Initialize with A.2.1 [#1999]

2016-09-07 Thread mahesh . valla
osaf/libs/agents/saf/lga/lga.h | 1 + osaf/libs/agents/saf/lga/lga_api.c | 4 +++- osaf/libs/agents/saf/lga/lga_mds.c | 12 +--- tests/logsv/tet_Log_clm.c | 35 ++- 4 files changed, 47 insertions(+), 5 deletions(-) Issue : If client

[devel] [PATCH 1 of 1] osaf: configuration support for OS whic has TIPC builtin kernel module [#1951] V1

2016-09-06 Thread mahesh . valla
00-README.conf | 11 ++- osaf/services/infrastructure/nid/scripts/configure_tipc.in | 51 - 2 files changed, 39 insertions(+), 23 deletions(-) Issue: In some OS like Montavista, TIPC is built in kernel module, the default node address

[devel] [PATCH 0 of 1] Review Request for osaf: configuration support for OS whic has TIPC builtin kernel module [#1951] V1

2016-09-06 Thread mahesh . valla
Summary:osaf: configuration support for OS whic has TIPC builtin kernel module [#1951] V1 Review request for Trac Ticket(s): #1951 Peer Reviewer(s): Anders Widell /Srikanth Pull request to: <> Affected branch(es): default Development branch: default Impacted

[devel] [PATCH 0 of 1] Review Request for osaf: configuration support for OS whic has TIPC builtin kernel module [#1951]

2016-09-02 Thread mahesh . valla
Summary:osaf: configuration support for OS whic has TIPC builtin kernel module [#1951] Review request for Trac Ticket(s): #1951 Peer Reviewer(s): Srikanth / Anders Widell Pull request to: <> Affected branch(es): default, 5.1 Development branch: default Impacted

[devel] [PATCH 1 of 1] osaf: configuration support for OS whic has TIPC builtin kernel module [#1951]

2016-09-02 Thread mahesh . valla
osaf/services/infrastructure/nid/scripts/configure_tipc.in | 38 - 1 files changed, 23 insertions(+), 15 deletions(-) Issue: In some OS like Montavista, TIPC is built in kernel module, the default node address will be <0.0.0>. Fix : This patch determines such OS which has TIPC

[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

[devel] [PATCH 00 of 13] Review Request for osaf: support for SAF SaTimeT variable in MDS transport changes [#1658]

2016-08-09 Thread mahesh . valla
Summary:osaf: support for SAF SaTimeT variable in MDS transport changes [#1658] Review request for Trac Ticket(s): #1658 Peer Reviewer(s):Ramesh & All Service owners Pull request to: Affected branch(es): default Development branch: default Impacted area

[devel] [PATCH 08 of 13] plmsv: support for SAF SaTimeT timeout variable in MDS transport [#1658]

2016-08-09 Thread mahesh . valla
osaf/libs/common/plmsv/include/plma.h| 2 +- osaf/libs/common/plmsv/include/plms.h| 2 +- osaf/libs/common/plmsv/include/plms_hrb.h| 2 +- osaf/libs/common/plmsv/plms_common_mds.c | 2 +-

[devel] [PATCH 02 of 13] clmsv: support for SAF SaTimeT timeout variable in MDS transport [#1658]

2016-08-09 Thread mahesh . valla
osaf/libs/agents/saf/clma/clma.h | 2 +- osaf/libs/agents/saf/clma/clma_api.c | 10 +- osaf/libs/agents/saf/clma/clma_mds.c | 2 +- osaf/services/saf/clmsv/nodeagent/main.c | 2 +- 4 files changed, 4 insertions(+), 12 deletions(-) diff --git

[devel] [PATCH 11 of 13] mqsv: support for SAF SaTimeT timeout variable in MDS transport [#1658]

2016-08-09 Thread mahesh . valla
osaf/libs/agents/saf/mqa/mqa_api.c | 20 ++-- osaf/libs/agents/saf/mqa/mqa_mds.c | 6 +++--- osaf/libs/common/mqsv/include/mqa_def.h | 6 +++--- osaf/libs/common/mqsv/include/mqnd_mds.h | 2 +- osaf/services/saf/mqsv/mqd/mqd_tmr.c | 4 ++--

[devel] [PATCH 01 of 13] immsv: support for SAF SaTimeT timeout variable in MDS transport [#1658]

2016-08-09 Thread mahesh . valla
osaf/libs/agents/saf/imma/imma.h| 2 +- osaf/libs/agents/saf/imma/imma_cb.h | 4 +- osaf/libs/agents/saf/imma/imma_mds.c| 2 +- osaf/libs/agents/saf/imma/imma_mds.h| 2 +- osaf/libs/agents/saf/imma/imma_oi_api.c | 12 osaf/libs/agents/saf/imma/imma_om_api.c |

[devel] [PATCH 10 of 13] amf: support for SAF SaTimeT timeout variable in MDS transport [#1658]

2016-08-09 Thread mahesh . valla
osaf/services/saf/amf/amfd/timer.cc | 4 ++-- osaf/services/saf/amf/amfnd/clc.cc | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/osaf/services/saf/amf/amfd/timer.cc b/osaf/services/saf/amf/amfd/timer.cc --- a/osaf/services/saf/amf/amfd/timer.cc +++

[devel] [PATCH 12 of 13] glsv: support for SAF SaTimeT timeout variable in MDS transport [#1658]

2016-08-09 Thread mahesh . valla
osaf/libs/agents/saf/gla/gla_api.c | 6 +++--- osaf/libs/agents/saf/gla/gla_mds.c | 2 +- osaf/libs/agents/saf/gla/gla_tmr.c | 6 +++--- osaf/libs/common/glsv/include/gla_mds.h | 2 +- osaf/services/saf/glsv/gld/gld_tmr.c| 4 ++-- osaf/services/saf/glsv/glnd/glnd_res.c |

[devel] [PATCH 09 of 13] smf: support for SAF SaTimeT timeout variable in MDS transport [#1658]

2016-08-09 Thread mahesh . valla
osaf/libs/common/smfsv/include/smfsv_evt.h | 2 +- osaf/libs/common/smfsv/smfsv_evt.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/osaf/libs/common/smfsv/include/smfsv_evt.h b/osaf/libs/common/smfsv/include/smfsv_evt.h ---

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

2016-08-09 Thread mahesh . valla
osaf/libs/agents/saf/eda/eda_mds.c | 2 +- osaf/libs/agents/saf/eda/eda_saf_api.c | 7 --- osaf/libs/common/edsv/include/eda_mds.h | 2 +- osaf/libs/common/edsv/include/eds_mds.h | 2 +- osaf/services/saf/edsv/eds/eds_ckpt.c | 2 +- osaf/services/saf/edsv/eds/eds_mds.c| 2 +-

[devel] [PATCH 03 of 13] lgs: support for SAF SaTimeT timeout variable in MDS transport [#1658]

2016-08-09 Thread mahesh . valla
osaf/libs/agents/saf/lga/lga.h | 2 +- osaf/libs/agents/saf/lga/lga_api.c | 4 ++-- osaf/libs/agents/saf/lga/lga_mds.c | 2 +- osaf/libs/agents/saf/lga/lga_state.c | 4 ++-- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/osaf/libs/agents/saf/lga/lga.h

[devel] [PATCH 06 of 13] mds: support for SAF SaTimeT timeout variable in MDS transport [#1658]

2016-08-09 Thread mahesh . valla
osaf/libs/core/include/mds_papi.h | 14 +++--- osaf/libs/core/mds/include/mds_dl_api.h | 4 ++-- osaf/libs/core/mds/mds_c_api.c | 2 +- osaf/libs/core/mds/mds_c_sndrcv.c | 7 ---

[devel] [PATCH 04 of 13] cpsv: support for SAF SaTimeT timeout variable in MDS transport [#1658]

2016-08-09 Thread mahesh . valla
osaf/libs/agents/saf/cpa/cpa_api.c| 18 +- osaf/libs/agents/saf/cpa/cpa_mds.c| 2 +- osaf/libs/agents/saf/cpa/cpa_tmr.c| 6 +++--- osaf/libs/common/cpsv/include/cpa_mds.h | 2 +- osaf/libs/common/cpsv/include/cpa_tmr.h | 2 +-

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

2016-08-09 Thread mahesh . valla
osaf/libs/core/common/include/osaf_poll.h| 4 ++-- osaf/libs/core/common/include/osaf_secutil.h | 2 +- osaf/libs/core/common/osaf_poll.c| 9 + osaf/libs/core/common/osaf_secutil.c | 8 osaf/libs/core/include/ncs_osprm.h | 2 +-

[devel] [PATCH 05 of 13] ntfsv: support for SAF SaTimeT timeout variable in MDS transport [#1658]

2016-08-09 Thread mahesh . valla
osaf/libs/agents/saf/ntfa/ntfa.h | 2 +- osaf/libs/agents/saf/ntfa/ntfa_api.c | 12 ++-- osaf/libs/agents/saf/ntfa/ntfa_mds.c | 2 +- 3 files changed, 8 insertions(+), 8 deletions(-) diff --git a/osaf/libs/agents/saf/ntfa/ntfa.h b/osaf/libs/agents/saf/ntfa/ntfa.h ---

[devel] [PATCH 0 of 4] Review Request for lgsv: Log Service CLM integration [#1638] V4

2016-08-05 Thread mahesh . valla
Summary:lgsv: Log Service CLM integration [#1638] V4 Review request for Trac Ticket(s): #1638 Peer Reviewer(s): Vu, Lennart,Anders Widell Pull request to: <> Affected branch(es): default Development branch: defult Impacted area Impact y/n

[devel] [PATCH 4 of 4] tests: added missing saf_error strings [#1638] V4

2016-08-05 Thread mahesh . valla
tests/unit_test_fw/src/util.c | 13 + 1 files changed, 13 insertions(+), 0 deletions(-) diff --git a/tests/unit_test_fw/src/util.c b/tests/unit_test_fw/src/util.c --- a/tests/unit_test_fw/src/util.c +++ b/tests/unit_test_fw/src/util.c @@ -52,6 +52,19 @@ static const char

[devel] [PATCH 1 of 4] lga: agent Cluster Membership (CLM) integration [#1638] V4

2016-08-05 Thread mahesh . valla
osaf/libs/agents/saf/lga/lga.h | 3 + osaf/libs/agents/saf/lga/lga_api.c | 52 ++ osaf/libs/agents/saf/lga/lga_mds.c | 69 ++ osaf/libs/agents/saf/lga/lga_util.c| 2 +

[devel] [PATCH 2 of 4] lgs: director Cluster Membership (CLM) integration [#1638] V4

2016-08-05 Thread mahesh . valla
osaf/services/saf/logsv/lgs/Makefile.am |7 +- osaf/services/saf/logsv/lgs/lgs_cb.h|4 + osaf/services/saf/logsv/lgs/lgs_clm.cc | 395 osaf/services/saf/logsv/lgs/lgs_clm.h | 39 +++ osaf/services/saf/logsv/lgs/lgs_evt.cc | 15 +

[devel] [PATCH 3 of 4] logtest: change related to Cluster Membership (CLM) integration [#1638] V4

2016-08-05 Thread mahesh . valla
tests/logsv/Makefile.am |3 +- tests/logsv/logtest.c |2 +- tests/logsv/logtestfr.c |2 +- tests/logsv/saflogtest.c |2 +- tests/logsv/tet_Log_clm.c | 134 ++ 5 files changed, 139 insertions(+), 4 deletions(-) Test

[devel] [PATCH 2 of 3] lgs: director Cluster Membership (CLM) integration [#1638] V3

2016-08-04 Thread mahesh . valla
osaf/services/saf/logsv/lgs/Makefile.am |7 +- osaf/services/saf/logsv/lgs/lgs_cb.h|9 + osaf/services/saf/logsv/lgs/lgs_clm.cc | 403 osaf/services/saf/logsv/lgs/lgs_clm.h | 39 +++ osaf/services/saf/logsv/lgs/lgs_evt.cc | 15 +

[devel] [PATCH 1 of 3] lga: agent Cluster Membership (CLM) integration [#1638] V3

2016-08-04 Thread mahesh . valla
osaf/libs/agents/saf/lga/lga.h | 3 + osaf/libs/agents/saf/lga/lga_api.c | 52 ++ osaf/libs/agents/saf/lga/lga_mds.c | 69 ++ osaf/libs/agents/saf/lga/lga_util.c| 2 +

[devel] [PATCH 0 of 3] Review Request for lgsv: Log Service CLM integration [#1638] V3

2016-08-04 Thread mahesh . valla
Summary:lgsv: Log Service CLM integration [#1638] V3 Review request for Trac Ticket(s): #1638 Peer Reviewer(s): Vu, Lennart,Anders Widell Pull request to: <> Affected branch(es): default Development branch: default Impacted area Impact y/n

[devel] [PATCH 3 of 3] logtest: change related to Cluster Membership (CLM) integration [#1638] V2

2016-08-02 Thread mahesh . valla
tests/logsv/saflogtest.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) Test : #amf-adm lock safNode=PL-4,safCluster=myClmCluster failed with code SA_AIS_ERR_UNAVAILABLE (31). #amf-adm unlock safNode=PL-4,safCluster=myClmCluster failed with code SA_AIS_ERR_UNAVAILABLE

[devel] [PATCH 1 of 3] lga: agent Cluster Membership (CLM) integration [#1638] V2

2016-08-02 Thread mahesh . valla
osaf/libs/agents/saf/lga/lga.h | 2 + osaf/libs/agents/saf/lga/lga_api.c | 47 osaf/libs/agents/saf/lga/lga_mds.c | 69 ++ osaf/libs/agents/saf/lga/lga_util.c| 2 +

[devel] [PATCH 0 of 3] Review Request for lgsv: Log Service CLM integration [#1638] V2

2016-08-02 Thread mahesh . valla
Summary:lgsv: Log Service CLM integration [#1638] V2 Review request for Trac Ticket(s): #1638 Peer Reviewer(s): Vu, Lennart,Anders Widell Pull request to: <> Affected branch(es): default Development branch: defult Impacted area Impact y/n

[devel] [PATCH 1 of 3] lga: agent Cluster Membership (CLM) integration [#1638] V2

2016-08-01 Thread mahesh . valla
osaf/libs/agents/saf/lga/lga.h | 2 + osaf/libs/agents/saf/lga/lga_api.c | 47 osaf/libs/agents/saf/lga/lga_mds.c | 69 ++ osaf/libs/agents/saf/lga/lga_util.c| 2 +

[devel] [PATCH 0 of 3] Review Request for lgsv: Log Service CLM integration [#1638] V2

2016-08-01 Thread mahesh . valla
Summary:lga: agent Cluster Membership (CLM) integration [#1638] V2 Review request for Trac Ticket(s): #1638 Peer Reviewer(s):Vu, Lennart,Anders Widell Pull request to: <> Affected branch(es): default Development branch: default Impacted area Impact y/n

[devel] [PATCH 3 of 3] logtest: change related to Cluster Membership (CLM) integration [#1638] V2

2016-08-01 Thread mahesh . valla
tests/logsv/saflogtest.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) Test : #amf-adm lock safNode=PL-4,safCluster=myClmCluster failed with code SA_AIS_ERR_UNAVAILABLE (31). #amf-adm unlock safNode=PL-4,safCluster=myClmCluster failed with code SA_AIS_ERR_UNAVAILABLE

[devel] [PATCH 0 of 3] Review Request for lgsv: Log Service CLM integration [#1638] V1

2016-07-26 Thread mahesh . valla
Summary:lgsv: Log Service CLM integration [#1638] V1 Review request for Trac Ticket(s): #1638 Peer Reviewer(s): Vu, Lennart Pull request to: <> Affected branch(es): Default Development branch: Default Impacted area Impact y/n

[devel] [PATCH 2 of 3] lgs: director Cluster Membership (CLM) integration [#1638] V1

2016-07-26 Thread mahesh . valla
osaf/services/saf/logsv/lgs/Makefile.am |4 +- osaf/services/saf/logsv/lgs/lgs.h |5 + osaf/services/saf/logsv/lgs/lgs_cb.h| 15 +++ osaf/services/saf/logsv/lgs/lgs_clm.cc | 143 osaf/services/saf/logsv/lgs/lgs_evt.cc | 142

[devel] [PATCH 1 of 3] lga: agent Cluster Membership (CLM) integration [#1638] V1

2016-07-26 Thread mahesh . valla
osaf/libs/agents/saf/lga/lga.h | 2 + osaf/libs/agents/saf/lga/lga_api.c | 37 ++ osaf/libs/agents/saf/lga/lga_mds.c | 75 ++ osaf/libs/agents/saf/lga/lga_util.c| 2 + osaf/libs/common/logsv/include/lgsv_defs.h |

[devel] [PATCH 3 of 3] logtest: change related to Cluster Membership (CLM) integration [#1638] V1

2016-07-26 Thread mahesh . valla
tests/logsv/saflogtest.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) Test : #amf-adm lock safNode=PL-4,safCluster=myClmCluster failed with code SA_AIS_ERR_UNAVAILABLE (31). #amf-adm unlock safNode=PL-4,safCluster=myClmCluster failed with code SA_AIS_ERR_UNAVAILABLE

[devel] [PATCH 2 of 3] lgs: director Cluster Membership (CLM) integration [#1638]

2016-07-26 Thread mahesh . valla
osaf/services/saf/logsv/lgs/Makefile.am |4 +- osaf/services/saf/logsv/lgs/lgs.h |5 + osaf/services/saf/logsv/lgs/lgs_cb.h| 15 +++ osaf/services/saf/logsv/lgs/lgs_evt.cc | 142 osaf/services/saf/logsv/lgs/lgs_evt.h |1 +

[devel] [PATCH 0 of 3] Review Request for lgsv: Log Service CLM integration [#1638]

2016-07-26 Thread mahesh . valla
Summary:lgsv: Log Service CLM integration [#1638] Review request for Trac Ticket(s): #1638 Peer Reviewer(s): Vu, Lennar Pull request to: Affected branch(es): default Development branch: default Impacted area Impact y/n

[devel] [PATCH 1 of 3] lga: agent Cluster Membership (CLM) integration [#1638]

2016-07-26 Thread mahesh . valla
osaf/libs/agents/saf/lga/lga.h | 2 + osaf/libs/agents/saf/lga/lga_api.c | 37 ++ osaf/libs/agents/saf/lga/lga_mds.c | 75 ++ osaf/libs/agents/saf/lga/lga_util.c| 2 + osaf/libs/common/logsv/include/lgsv_defs.h |

[devel] [PATCH 1 of 1] mds:non-root- don't delete pid info to avoid race condition if re-registers [#1825]

2016-05-31 Thread mahesh . valla
osaf/libs/core/mds/mds_dt_common.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) MDS while deleting a down process completely, fist determines the process doesn't exist forever, if process not exist any more , then only it deletes the PID onfo ,to avoid race with a client that

[devel] [PATCH 0 of 1] Review Request for mds:non-root- dont delete pid info to avoid race condition if re-registers [#1825]

2016-05-31 Thread mahesh . valla
Summary:mds:non-root- don't delete pid info to avoid race condition if re-registers [#1825] Review request for Trac Ticket(s): #1825 Peer Reviewer(s): Mathi Pull request to: Mahesh Affected branch(es): defult , 4.x 5.x Development branch: defult Impacted area

[devel] [PATCH 1 of 1] mds-tcp: adjust the TCP fragamnet size according to new mds hdr size [#1694]

2016-03-08 Thread mahesh . valla
osaf/libs/core/include/mds_papi.h | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) After the changes of MDS enhancement ticket #1522, MDS TCP application getting segmentation if application trying to send messge size > 65535 Bug : MDS:TCP code changes included the node name as a

[devel] [PATCH 0 of 1] Review Request for mds-tcp: adjust the TCP fragamnet size according to new mds hdr size [#1694]

2016-03-08 Thread mahesh . valla
Summary:mds-tcp: adjust the TCP fragamnet size according to new mds hdr size [#1694] Review request for Trac Ticket(s): #1694 Peer Reviewer(s): Mathi Pull request to: <> Affected branch(es): 5.0 default Development branch: default Impacted area Impact y/n

[devel] [PATCH 0 of 1] Review Request for osaf: provide support for the full range of timeout values for SAF API [#1657]

2016-02-09 Thread mahesh . valla
Summary:osaf: provide support for the full range of timeout values for SAF API [#1657] Review request for Trac Ticket(s): #1657 Peer Reviewer(s):Neel-IMMA /Mathi-CLMA /Ramesh-CPA Pull request to: <> Affected branch(es): 4.6 4.7 ,default Development branch: default

[devel] [PATCH 1 of 1] osaf: provide support for the full range of timeout values for SAF API [#1657]

2016-02-09 Thread mahesh . valla
osaf/libs/agents/saf/clma/clma_api.c| 22 +++--- osaf/libs/agents/saf/cpa/cpa_api.c | 21 + osaf/libs/agents/saf/imma/imma_om_api.c | 24 osaf/libs/core/include/mds_papi.h | 3 +++ 4 files changed, 51 insertions(+),

[devel] [PATCH 1 of 3] MDS:TIPC include node name as a part of callback_info events [#1522]

2016-02-04 Thread mahesh . valla
osaf/libs/core/include/mds_papi.h |8 ++- osaf/libs/core/mds/include/mds_core.h |4 +- osaf/libs/core/mds/include/mds_dt.h |5 +- osaf/libs/core/mds/include/mds_dt2c.h |6 +- osaf/libs/core/mds/mds_c_api.c| 14 ++- osaf/libs/core/mds/mds_c_sndrcv.c |5 +

[devel] [PATCH 2 of 3] MDS:TCP include node name as a part of callback_info events [#1522]

2016-02-04 Thread mahesh . valla
osaf/libs/core/mds/mds_dt_trans.c | 108 osaf/services/infrastructure/dtms/dtm/dtm_intra.c |8 +- osaf/services/infrastructure/dtms/dtm/dtm_intra_svc.c | 20 +- osaf/services/infrastructure/dtms/dtm/dtm_node.c |4 +-

[devel] [PATCH 0 of 3] Review Request for MDS: Include node name as a part of Transport callback_info events [#1522]

2016-02-04 Thread mahesh . valla
Summary:MDS: Include node name as a part of Transport callback_info events [#1522] Review request for Trac Ticket(s): #1522 Peer Reviewer(s): Lennart /Beatriz /Mathi /Ramesh Pull request to: avm Affected branch(es): default Development branch: default Impacted

[devel] [PATCH 3 of 3] MDS: use case examples of node_name from NCSMDS_CALLBACK_INFO [#1522]

2016-02-04 Thread mahesh . valla
osaf/services/saf/clmsv/clms/clms_mds.c | 1 + osaf/services/saf/logsv/lgs/lgs_mds.cc | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) Example node_name from ncsmds_callback_info (NCSMDS_CALLBACK_INFO) use case this will not be committed , just to show use case. diff --git

[devel] [PATCH 0 of 1] Review Request for mds: move tzset call to mds_log_init to reduce overhead [#1672]

2016-01-29 Thread mahesh . valla
Summary:mds: move tzset call to mds_log_init to reduce overhead [#1672] Review request for Trac Ticket(s): #1672 Peer Reviewer(s): Anders Widell / Nagu Pull request to: <> Affected branch(es): default /4.7/4.6 Development branch: default Impacted area

[devel] [PATCH 1 of 1] mds: move tzset call to mds_log_init to reduce overhead [#1672]

2016-01-29 Thread mahesh . valla
osaf/libs/core/mds/mds_log.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) calling tzset() in log_mds() for each log is creating a lot of overhead in MDS logging, taking locks, reading from file etc. This seems unnecessary just for supporting changed time zones which should be a

[devel] [PATCH 0 of 1] Review Request for cpsv: validated SaTimeT timeout argument value for all CPA APIs [#1583]

2015-11-05 Thread mahesh . valla
Summary:cpsv: validated SaTimeT timeout argument value for all CPA APIs [#1583] Review request for Trac Ticket(s): #1583 Peer Reviewer(s): Nagu/Ramesh Pull request to: <> Affected branch(es): default 4.7 4.6 Development branch: default Impacted area Impact

[devel] [PATCH 1 of 1] cpsv: validated SaTimeT timeout argument value for all CPA APIs [#1583]

2015-11-05 Thread mahesh . valla
osaf/libs/agents/saf/cpa/cpa_api.c | 111 ++-- 1 files changed, 91 insertions(+), 20 deletions(-) If we test upper and lower limits of SaTimeT timeout arguments to CPSV API , some of CPA APIs are doesn't returns or hungs. Current cpa code doesn't validating or

[devel] [PATCH 0 of 1] Review Request for base: reduced duplicate discovery tipc Link tolerance time to minimum [#1549]

2015-10-20 Thread mahesh . valla
Summary:base: reduced duplicate discovery tipc Link tolerance time to minimum [#1549] Review request for Trac Ticket(s): #1549 Peer Reviewer(s): srikanth Pull request to: <> Affected branch(es): 4.7 & default Development branch: default Impacted area

[devel] [PATCH 1 of 1] base: reduced duplicate discovery tipc Link tolerance time to minimum [#1549]

2015-10-20 Thread mahesh . valla
osaf/services/infrastructure/nid/scripts/configure_tipc.in | 8 ++-- 1 files changed, 6 insertions(+), 2 deletions(-) This will reduce the duplicate discovery time 1.2 seconds The sys log show ,duplicate discovery log event and Opensaf start-up event in sequence Used different temporary

[devel] [PATCH 0 of 1] Review Request for cpsv: introduced null check for SecPtr to prevent ckptnd crashed (multithreading)app V2 [#242]

2015-10-09 Thread mahesh . valla
Summary: cpsv: introduced null check for SecPtr to prevent ckptnd crashed (multithreading)app V2 [#242] Review request for Trac Ticket(s): #242 Peer Reviewer(s): Ramesh Pull request to: <> Affected branch(es): All Development branch: default Impacted area

[devel] [PATCH 0 of 1] Review Request for cpsv: introduced null check for SecPtr to prevent ckptnd crashed (multithreading)app [#242]

2015-10-09 Thread mahesh . valla
Summary:cpsv: introduced null check for SecPtr to prevent ckptnd crashed (multithreading)app [#242] Review request for Trac Ticket(s): #242 Peer Reviewer(s): Remesh Pull request to: <> Affected branch(es): 4.5 4.6 4.7 & default Development branch: defult

[devel] [PATCH 1 of 1] cpsv: introduced null check for SecPtr to prevent ckptnd crashed (multithreading)app [#242]

2015-10-09 Thread mahesh . valla
osaf/services/saf/cpsv/cpnd/cpnd_db.c | 10 +- osaf/services/saf/cpsv/cpnd/cpnd_evt.c | 11 +-- osaf/services/saf/cpsv/cpnd/cpnd_proc.c | 5 + 3 files changed, 23 insertions(+), 3 deletions(-) Their is NULL check missing in CPND code while processing section get next

[devel] [PATCH 0 of 1] Review Request for mds: fixed compilation issue gcc version 4.8.3 [#1487]

2015-09-29 Thread mahesh . valla
Summary:mds: fixed compilation issue gcc version 4.8.3 [#1487] Review request for Trac Ticket(s): #1487 Peer Reviewer(s): Neel Pull request to: <> Affected branch(es): default Development branch: default Impacted area Impact y/n

[devel] [PATCH 0 of 1] Review Request for improved mds logging [#520]

2015-09-16 Thread mahesh . valla
Summary:mds: improved mds logging [#520] Review request for Trac Ticket(s): #520 Peer Reviewer(s): Zoran / Nagu Pull request to: Affected branch(es): default Development branch: default Impacted area Impact y/n Docs

[devel] [PATCH 0 of 1] Review Request for cpsv: raised track callback for non-collo ckpt section create/delete [#1305]

2015-09-15 Thread mahesh . valla
Summary: cpsv: raised track callback for non-collo ckpt section create/delete [#1305] Review request for Trac Ticket(s): #1305 Peer Reviewer(s): Srikanth/Ramesh Pull request to: <> Affected branch(es): 4.5 4.6 4.7 Development branch: default Impacted area

[devel] [PATCH 0 of 1] Review Request for opensaf: prevented cluster re-start on misconfigured slot_id [#464/#414]

2015-09-03 Thread mahesh . valla
Summary:opensaf: prevented cluster re-start on misconfigured slot_id [#464/#414] Review request for Trac Ticket(s): #464/#414 Peer Reviewer(s): Mathi/Praveen/Srikanth Pull request to: <> Affected branch(es): default Development branch: default Impacted area

[devel] [PATCH 1 of 1] opensaf: prevented cluster re-start on misconfigured slot_id [#464/#414]

2015-09-03 Thread mahesh . valla
00-README.conf | 8 +- osaf/services/infrastructure/nid/scripts/configure_tipc.in | 83 +- 2 files changed, 87 insertions(+), 4 deletions(-) Issue : - Currently in Opensaf duplicate tipc-id is identified on network and

[devel] [PATCH 1 of 1] Opensaf : Optimized fcntl() call for sckts/fd FD_CLOEXEC [#1337]

2015-08-31 Thread mahesh . valla
osaf/libs/core/leap/os_defs.c | 20 +--- osaf/libs/core/leap/sysf_mem.c | 2 +- osaf/libs/core/mds/mds_dt_tcp.c | 16 +--- 3 files changed, 3 insertions(+), 35 deletions(-) Since Linux 2.6.27, socketpair() supports the SOCK_NONBLOCK and SOCK_CLOEXEC flags

[devel] [PATCH 0 of 1] Review Request for Opensaf : optimized fcntl() call for sckts/fd FD_CLOEXEC [#1337]

2015-08-31 Thread mahesh . valla
Summary:Opensaf : Optimized fcntl() call for sckts/fd FD_CLOEXEC [#1337] Review request for Trac Ticket(s): #1337 Peer Reviewer(s): Ramesh Pull request to: <> Affected branch(es): default Development branch: default Impacted area Impact y/n

[devel] [PATCH 0 of 1] Review Request for cpsv: prevented multiple write request with same checkpointHandle at cpnd [#1467]

2015-08-30 Thread mahesh . valla
Summary:cpsv: prevented multiple write request with same checkpointHandle at cpnd [#1467] Review request for Trac Ticket(s): #1467 Peer Reviewer(s): Nagu Pull request to: LIST THE PERSON WITH PUSH ACCESS HERE Affected branch(es): all Development branch: default

[devel] [PATCH 1 of 1] cpsv: prevented multiple write request with same checkpointHandle at cpnd [#1467]

2015-08-30 Thread mahesh . valla
possible that the checkpointHandle is being passed as |0 osaf/services/saf/cpsv/cpnd/cpnd_evt.c| 18 +++--- osaf/services/saf/cpsv/cpnd/cpnd_proc.c |3 ++- 3 files changed, 17 insertions(+), 4 deletions(-) Bug Analysis : checkpointHandle -

[devel] [PATCH 1 of 1] mds: Optimized the mds mutex locks code [#1338]

2015-08-26 Thread mahesh . valla
osaf/libs/core/mds/mds_c_sndrcv.c | 40 +- 1 files changed, 2 insertions(+), 38 deletions(-) Made centralized locks unlock wherever possible, for better debugging mds code and to reduces scope of locks unlock code errors diff --git

[devel] [PATCH 0 of 1] Review Request for mds: Optimized the mds mutex locks code [#1338]

2015-08-26 Thread mahesh . valla
Summary:mds: Optimized the mds mutex locks code [#1338] Review request for Trac Ticket(s): #1338 Peer Reviewer(s): Nagu/Ramesh Pull request to: LIST THE PERSON WITH PUSH ACCESS HERE Affected branch(es): default Development branch: default Impacted area

[devel] [PATCH 1 of 1] mds: fixed mismatch arguments for m_MDS_LOG [#1440]

2015-08-10 Thread mahesh . valla
osaf/libs/core/mds/mds_c_api.c | 2 +- osaf/libs/core/mds/mds_c_db.c | 12 ++-- osaf/libs/core/mds/mds_c_sndrcv.c | 18 +- osaf/libs/core/mds/mds_dt_common.c | 12 ++-- osaf/libs/core/mds/mds_dt_tipc.c | 2 +- 5 files changed, 23 insertions(+),

[devel] [PATCH 0 of 1] Review Request for mds: fixed mismatch arguments for m_MDS_LOG [#1440]

2015-08-10 Thread mahesh . valla
Summary: mds: fixed mismatch arguments for m_MDS_LOG [#1440] Review request for Trac Ticket(s): #1440 Peer Reviewer(s): Nagu Pull request to: LIST THE PERSON WITH PUSH ACCESS HERE Affected branch(es): LIST ALL AFFECTED BRANCH(ES) Development branch: default

[devel] [PATCH 0 of 1] Review Request for cpsv : fix CPA CPND memleak [#1373]

2015-05-15 Thread mahesh . valla
Summary: cpsv : fix CPA CPND memleak [#1373] Review request for Trac Ticket(s): #1373 Peer Reviewer(s): Ramesh Pull request to: LIST THE PERSON WITH PUSH ACCESS HERE Affected branch(es): 4.5 4.6 default Development branch: default Impacted area Impact y/n

[devel] [PATCH 0 of 1] Review Request for samples: ckpt_demo- set Section expirationTime to absolute time[#1270]

2015-05-15 Thread mahesh . valla
Summary: samples: ckpt_demo- set Section expirationTime to absolute time[#1270] Review request for Trac Ticket(s): #1270 Peer Reviewer(s): Nagu Pull request to: PERSON WITH PUSH ACCESS HERE Affected branch(es): all Development branch: default Impacted area

[devel] [PATCH 1 of 1] samples: ckpt_demo- set Section expirationTime to absolute time[#1270]

2015-05-15 Thread mahesh . valla
samples/cpsv/ckpt_demo/cpsv_test_app.c | 5 - samples/cpsv/ckpt_track_demo/cpsv_test_track_app.c | 5 - 2 files changed, 8 insertions(+), 2 deletions(-) Cpsv expects as absolute time check section 3.4.3.2 SaCkptSectionCreationAttributesT of CKPT Specification for more

[devel] [PATCH 1 of 1] cpsv : fix CPA CPND memleak [#1373]

2015-05-15 Thread mahesh . valla
osaf/services/saf/cpsv/cpnd/cpnd_proc.c | 1 - osaf/services/saf/cpsv/cpnd/cpnd_tmr.c | 5 + 2 files changed, 5 insertions(+), 1 deletions(-) 1) if ckpt application registered for saCkptCheckpointTrackCallback the cpnd_proc_ckpt_arrival_info_ntfy() will be triggered for that cpa from

  1   2   >