Re: [devel] [PATCH 1 of 3] mds: Convert the mds_log.c file to C++ [#2015]

2016-11-03 Thread Anders Widell
Hi! Yes agreed, using this mechanism also for trace is a logical next step. It should be fairly easy to implement once this MDS logging mechanism is fully in place. regards, Anders Widell On 11/03/2016 06:14 AM, A V Mahesh wrote: > Hi Anders Widell, > > >>Using a standardized format means tha

Re: [devel] [PATCH 1 of 3] base: Use inotify to improve response time for transport monitor process V4 [#2091]

2016-11-03 Thread ramesh betham
Ack with very minor comments inline. Regards, Ramesh. On 10/27/2016 8:55 PM, Hans Nordeback wrote: > osaf/libs/core/cplusplus/base/Makefile.am|2 + > osaf/libs/core/cplusplus/base/file_notify.cc | 190 > +++ > osaf/libs/core/cplusplus/base/file_notify.h | 1

Re: [devel] [PATCH 2 of 3] base: Unit tests for FileNotify V4 [#2091]

2016-11-03 Thread ramesh betham
Ack. Thanks, Ramesh. On 10/27/2016 8:55 PM, Hans Nordeback wrote: > osaf/libs/core/cplusplus/base/tests/Makefile.am | 7 +- > osaf/libs/core/cplusplus/base/tests/file_notify_test.cc | 83 > + > 2 files changed, 88 insertions(+), 2 deletions(-) > > > diff --git a/os

[devel] [PATCH 0 of 1] Review Request for amfd: add support of saAmfSUAssignedSIs [#905]

2016-11-03 Thread nagendra . k
Summary: amfd: add support of saAmfSUAssignedSIs [#905] Review request for Trac Ticket(s): #905 Peer Reviewer(s): Amf Dev Pull request to: <> Affected branch(es): Default Development branch: Default Impacted area Impact y/n D

[devel] [PATCH 1 of 1] amfd: add support of saAmfSUAssignedSIs [#905]

2016-11-03 Thread nagendra . k
osaf/services/saf/amf/amfd/imm.cc| 63 osaf/services/saf/amf/amfd/include/imm.h | 7 +++ osaf/services/saf/amf/amfd/su.cc | 31 --- 3 files changed, 95 insertions(+), 6 deletions(-) diff --git a/osaf/services/saf/amf/amfd/imm.cc b

[devel] [PATCH 1 of 5] imm: Compile the IMM library using the C++ compiler [#2142]

2016-11-03 Thread Hung Nguyen
osaf/libs/agents/saf/imma/Makefile.am | 24 ++-- osaf/libs/agents/saf/imma/imma_db.c | 0 osaf/libs/agents/saf/imma/imma_init.c | 0 osaf/libs/agents/saf/imma/imma_mds.c| 0 osaf/libs/agents/saf/imma/imma_oi_api.c | 0 osaf/libs/agents/saf/imma/imma_om_

[devel] [PATCH 0 of 5] Review Request for imm: Compile the IMM library using the C++ compiler [#2142]

2016-11-03 Thread Hung Nguyen
Summary: imm: Compile the IMM library using the C++ compiler [#2142] Review request for Trac Ticket(s): 2142 Peer Reviewer(s): Zoran, Neel Pull request to: Affected branch(es): 5.2 Development branch: 5.2 Impacted area Impact y/n -

[devel] [PATCH 3 of 5] imm: Fix "invalid conversion" errors. [#2142]

2016-11-03 Thread Hung Nguyen
osaf/libs/agents/saf/imma/imma_db.cc |7 +- osaf/libs/agents/saf/imma/imma_init.cc |4 +- osaf/libs/agents/saf/imma/imma_mds.cc|4 +- osaf/libs/agents/saf/imma/imma_oi_api.cc | 34 +++--- osaf/libs/agents/saf/imma/imma_om_api.cc | 144 +++--- osaf/l

[devel] [PATCH 4 of 5] imm: Fix "comparison between signed and unsigned integer" errors [#2142]

2016-11-03 Thread Hung Nguyen
osaf/libs/agents/saf/imma/imma_cb.h | 2 +- osaf/libs/agents/saf/imma/imma_db.cc | 2 +- osaf/libs/agents/saf/imma/imma_oi_api.cc | 2 +- osaf/libs/agents/saf/imma/imma_proc.cc | 4 ++-- 4 files changed, 5 insertions(+), 5 deletions(-) Fix "comparison between signed and unsigned i

[devel] [PATCH 2 of 5] imm: Fix "crosses initialization" errors [#2142]

2016-11-03 Thread Hung Nguyen
osaf/libs/agents/saf/imma/imma_db.cc | 3 +-- osaf/libs/agents/saf/imma/imma_oi_api.cc | 7 +++ osaf/libs/agents/saf/imma/imma_om_api.cc | 24 +--- osaf/libs/agents/saf/imma/imma_proc.cc | 6 -- 4 files changed, 21 insertions(+), 19 deletions(-) Fix "cros

[devel] [PATCH 5 of 5] imm: Fix linkage errors [#2142]

2016-11-03 Thread Hung Nguyen
osaf/libs/agents/saf/imma/imma_init.cc | 1 - osaf/libs/agents/saf/imma/imma_om_api.cc | 1 - osaf/libs/core/mds/include/mds_dl_api.h | 9 + osaf/libs/saf/libSaImm/Makefile.am | 4 ++-- osaf/libs/saf/libSaImm/libSaImmOm.map| 14 -- 5 files changed, 19 inse

[devel] [PATCH 0 of 5] Review Request for mds: Make the MdsLog implementation lock-free [#2127]

2016-11-03 Thread Anders Widell
Summary: mds: Make the MdsLog implementation lock-free [#2127] Review request for Trac Ticket(s): 2127 Peer Reviewer(s): Mahesh, Ramesh Pull request to: Affected branch(es): default(5.2) Development branch: default Impacted area Impact y/n --

[devel] [PATCH 3 of 5] base: Make the UnixSocket class thread-safe [#2127]

2016-11-03 Thread Anders Widell
osaf/libs/core/cplusplus/base/unix_client_socket.cc | 28 --- osaf/libs/core/cplusplus/base/unix_client_socket.h | 2 +- osaf/libs/core/cplusplus/base/unix_server_socket.cc | 19 +-- osaf/libs/core/cplusplus/base/unix_server_socket.h | 4 +- osaf/libs/core/cplusplus/base/unix_s

[devel] [PATCH 5 of 5] mds: Use the va_list version of the Buffer::Write() method [#2127]

2016-11-03 Thread Anders Widell
osaf/libs/core/mds/mds_log.cc | 42 -- 1 files changed, 16 insertions(+), 26 deletions(-) Use the va_list version of Buffer::Write() to avoid the need for formatting the message into a temporary buffer. diff --git a/osaf/libs/core/mds/mds_log.cc b/osaf/l

[devel] [PATCH 4 of 5] mds: Make the MdsLog implementation lock-free [#2127]

2016-11-03 Thread Anders Widell
osaf/libs/core/mds/include/mds_log.h | 10 osaf/libs/core/mds/mds_log.cc| 37 +++ 2 files changed, 13 insertions(+), 34 deletions(-) Remove the mutex from the MdsLog class, by allocating a temporary Buffer instance on the stack and relying on th

[devel] [PATCH 2 of 5] base: Add a Buffer::Write() method taking a format string and a va_list parameter [#2127]

2016-11-03 Thread Anders Widell
osaf/libs/core/cplusplus/base/log_message.h | 56 + 1 files changed, 56 insertions(+), 0 deletions(-) Make it possible to send a format string and a va_list to Buffer::Write, so that it can be used more efficiently in a printf-style function. diff --git a/osaf/libs/

[devel] [PATCH 1 of 5] base: Remove dynamic memory allocation from the Buffer class [#2127]

2016-11-03 Thread Anders Widell
osaf/libs/core/cplusplus/base/buffer.h | 26 +- osaf/libs/core/cplusplus/base/log_message.cc| 106 - osaf/libs/core/cplusplus/base/log_message.h | 126 +++- osaf/libs/core/cplusplus/base/tests/log_message_test.cc | 14 +- osaf/

Re: [devel] [PATCH 1 of 3] mds: Convert the mds_log.c file to C++ [#2015]

2016-11-03 Thread A V Mahesh
Ack tested. -AVM On 11/3/2016 1:58 PM, Anders Widell wrote: > Hi! > > Yes agreed, using this mechanism also for trace is a logical next > step. It should be fairly easy to implement once this MDS logging > mechanism is fully in place. > > regards, > > Anders Widell > > On 11/03/2016 06:14 AM,

[devel] [PATCH 0 of 1] Review Request for imm: Fix external linkage in OI library [#2167]

2016-11-03 Thread Hung Nguyen
Summary: imm: Fix external linkage in OI library [#2167] Review request for Trac Ticket(s): 2167 Peer Reviewer(s): Zoran, Neel Pull request to: Affected branch(es): 5.0, 5.1, 5.2 Development branch: 5.2 Impacted area Impact y/n ---

[devel] [PATCH 1 of 1] imm: Fix external linkage in OI library [#2167]

2016-11-03 Thread Hung Nguyen
osaf/libs/agents/saf/imma/imma_oi_api.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) Fix external linkage in OI library. diff --git a/osaf/libs/agents/saf/imma/imma_oi_api.c b/osaf/libs/agents/saf/imma/imma_oi_api.c --- a/osaf/libs/agents/saf/imma/imma_oi_api.c +++ b/osaf/libs/age