Re: [devel] [PATCH 1 of 1] cpnd: use shared memory based on ckpt name length [#2108]

2016-10-17 Thread Vo Minh Hoang
Dear Mahesh, >> [AVM] A non-collated Ckpt will have two replicas on both Active and standby. Each node will receive one CPND_EVT_D2ND_CKPT_CREATE message so it handles swapping itself and does not affect each other nor another. >> [AVM] piratically we can have large size data & transit time, if

Re: [devel] [PATCH 1 of 1] cpnd: use shared memory based on ckpt name length [#2108]

2016-10-17 Thread A V Mahesh
Hi Hoan, On 10/18/2016 9:59 AM, Vo Minh Hoang wrote: > Dear Mahesh, > > Thank you very much for your comments. > > I would like to explain my understanding and reason for this solution. > Please correct me if I am wrong. > > - This memory swapping works on single node alone, it will occur

Re: [devel] [PATCH 1 of 1] cpnd: use shared memory based on ckpt name length [#2108]

2016-10-17 Thread Vo Minh Hoang
Dear Mahesh, Thank you very much for your comments. I would like to explain my understanding and reason for this solution. Please correct me if I am wrong. - This memory swapping works on single node alone, it will occur maximum once per node in open/create checkpoint process. - This swapping

Re: [devel] [PATCH 1 of 1] cpnd: use shared memory based on ckpt name length [#2108]

2016-10-17 Thread A V Mahesh
Hi Hoang, On 10/13/2016 12:44 PM, Vo Minh Hoang wrote: > No, old checkpoint data is converted to `big format`. > So all of them will be stored in `big format`. [AVM] This approach is introducing NEW transit , so far application are aware of switch-over & fail-over transit and TRY-AGAIN is

[devel] [PATCH 3 of 3] dtm: Use inotify to improve response time for transport monitor process V2 [#2091]

2016-10-17 Thread Hans Nordeback
osaf/services/infrastructure/dtms/transport/tests/Makefile.am | 3 +- osaf/services/infrastructure/dtms/transport/tests/transport_monitor_test.cc | 4 +- osaf/services/infrastructure/dtms/transport/transport_monitor.cc| 38 ++--- 3 files changed, 30

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

2016-10-17 Thread Hans Nordeback
osaf/libs/core/cplusplus/base/Makefile.am|2 + osaf/libs/core/cplusplus/base/file_notify.cc | 175 +++ osaf/libs/core/cplusplus/base/file_notify.h | 87 + 3 files changed, 264 insertions(+), 0 deletions(-) diff --git

[devel] [PATCH 0 of 3] Review Request for dtm: Use inotify to improve response time for transport monitor process V2 [#2091]

2016-10-17 Thread Hans Nordeback
Summary: dtm: Use inotify to improve response time for transport monitor process V2 Review request for Trac Ticket(s): #2091 Peer Reviewer(s): AndersW, Ramesh, Mahesh Pull request to: Affected branch(es): default Development branch: default Impacted area

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

2016-10-17 Thread Hans Nordeback
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/osaf/libs/core/cplusplus/base/tests/Makefile.am

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

2016-10-17 Thread Hans Nordebäck
Hi Anders, thanks for pointing out, I missed the term_fd_. I'll send out a new version. /Thanks HansN On 10/14/2016 10:34 AM, Anders Widell wrote: > Hi! > > This code has degraded in one aspect: it does not return so quickly > (or at all) when term_fd_ has been signalled. Could you try to fix

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

2016-10-17 Thread Hans Nordebäck
Hi Anders, good comments! I'll update and send out a new version. I'll keep the dirname and basename for now, (the c++ version requires some more work, boost filesystem path may be used but it requires building). /Thanks HansN On 10/14/2016 10:27 AM, Anders Widell wrote: > > Ack with

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

2016-10-17 Thread A V Mahesh
Hi Hoang, ACK Not tested V5 ( tested V4 ) -AVM > > > On 9/19/2016 1:19 PM, Vo Minh Hoang wrote: >> Dear Mahesh, >> >> I run osaftest (ckpttest) with 1 SC is old version and others are new >> version. >> I cannot test for [fail over /switch over] cases in my environment. >> >> Thank you and

Re: [devel] [PATCH 0 of 1] Review Request for log: improve test cases for log service [#1913] V2

2016-10-17 Thread Canh Truong
Thanks Lennart, I will check these functions and move some of them if needed. Regards, Canh -Original Message- From: Lennart Lund [mailto:lennart.l...@ericsson.com] Sent: Thursday, October 13, 2016 5:38 PM To: Canh Van Truong; mahesh.va...@oracle.com; Vu Minh Nguyen Cc:

[devel] [PATCH 0 of 1] Review Request for imm: Dont send ND2ND_ADMOP_RSP message when OI and OM client are on the same node [#2113]

2016-10-17 Thread Hung Nguyen
Summary: imm: Don't send ND2ND_ADMOP_RSP message when OI and OM client are on the same node [#2113] Review request for Trac Ticket(s): 2113 Peer Reviewer(s): Zoran, Neel Pull request to: Affected branch(es): 5.0, 5.1, 5.2 Development branch: 5.2 Impacted area

[devel] [PATCH 1 of 1] imm: Don't send ND2ND_ADMOP_RSP message when OI and OM client are on the same node [#2113]

2016-10-17 Thread Hung Nguyen
osaf/services/saf/immsv/immnd/immnd_evt.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) Don't send ND2ND_ADMOP_RSP message when OI and OM client are on the same node. diff --git a/osaf/services/saf/immsv/immnd/immnd_evt.c b/osaf/services/saf/immsv/immnd/immnd_evt.c ---