Re: [devel] [PATCH 0 of 3] Review Request for log: implement SaLogFilterSetCallbackT and version handling V4 [#2146]

2017-02-21 Thread Canh Truong
Hi Mahesh, SaLogFilterSetCallbackT was already declared before . So we don't have to add more one header file. Regards Canh -Original Message- From: Canh Truong [mailto:canh.v.tru...@dektech.com.au] Sent: Tuesday, February 21, 2017 4:41 PM To: 'A V Mahesh'; 'lennart.l...@ericsson.com

Re: [devel] [PATCH 0 of 3] Review Request for log: implement SaLogFilterSetCallbackT and version handling V4 [#2146]

2017-02-21 Thread Canh Truong
/ for version handling. Regards Canh -Original Message- From: A V Mahesh [mailto:mahesh.va...@oracle.com] Sent: Tuesday, February 21, 2017 4:31 PM To: Canh Truong; lennart.l...@ericsson.com; vu.m.ngu...@dektech.com.au Cc: opensaf-devel@lists.sourceforge.net Subject: Re: [devel] [PATCH 0 of 3] Review

[devel] FW: [PATCH 1 of 3] log: add alternative destinations of log records [#2258]

2017-02-21 Thread Canh Truong
Hi Vu, Please see some my comments with [Canh] Regards Canh -Original Message- From: Vu Minh Nguyen [mailto:vu.m.ngu...@dektech.com.au] Sent: Thursday, February 16, 2017 5:30 PM To: lennart.l...@ericsson.com; mahesh.va...@oracle.com; canh.v.tru...@dektech.com.au Cc:

Re: [devel] [PATCH 0 of 3] Review Request for log: implement SaLogFilterSetCallbackT and version handling V4 [#2146]

2017-02-20 Thread Canh Truong
Hi Mahesh, The test case " logtest 1 10" and " logtest 1 11" are created for version handling. In standby node, it has new code for log agent (with #2146), that means that version handling in log agent have been added. The log agent check if minor version bigger than the agent support, it will

Re: [devel] [PATCH 1 of 1] ntf: workaround to fix failed upgrade [#2300]

2017-02-15 Thread Canh Truong
Hi aMinh, NTFS has already re-assigned length of buffer to MAX_ADDITIONAL_TEXT_LENGTH if it is exceeds to MAX_ADDITIONAL_TEXT_LENGTH. So we don't need to add any more for this check. Regards Canh -Original Message- From: minh chau [mailto:minh.c...@dektech.com.au] Sent: Wednesday,

Re: [devel] [PATCH 1 of 1] log: implement SaLogFilterSetCallbackT [#2146]

2017-01-05 Thread Canh Truong
Hi Lennart, Thanks for your comments. As I understand you mean that the agent shall send to the server with the minor version that is initialized with client [in minor version] , except if minor = 0, it will be set to 1. But as section of 3.5.1 AIS document " minorVersion = highest value of

Re: [devel] [PATCH 1 of 1] log: fix bad file discriptor error when changing imm attributes[#2215]

2016-12-13 Thread Canh Truong
Hi Mahesh, Could you explain more clear about this? The first ` if (*stream->p_fd == -1) {` is to avoid closing and rename log file if file descriptor is -1. The second is checked if open log if open log file fail. Regards Canh -Original Message- From: A V Mahesh

Re: [devel] [PATCH 1 of 1] ntfd: process NTFA down event with high priority[#2206]

2016-11-29 Thread Canh Truong
Hi Praveen, When app do saNtfInitialize(), the ntfa will send initialized message with normal priority (MDS_SEND_PRIORITY_MEDIUM = 2) to NTFS. NTFS pick this message up in mds callback and put to mbx with same priority . if app is dead suddenly after initializing, NTFS will receive both

Re: [devel] [PATCH 1 of 1] log: implement SaLogFilterSetCallbackT [#2146]

2016-11-15 Thread Canh Truong
Hi Lennart, I already did the upgrade test as your below and it was passed from my side. I will check and run all test again Regards, Canh -Original Message- From: Lennart Lund [mailto:lennart.l...@ericsson.com] Sent: Tuesday, November 15, 2016 11:16 PM To: Canh Van Truong; Vu Minh

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:

Re: [devel] [PATCH 1 of 1] log: fix cppcheck with performance severity [#1975]

2016-10-11 Thread Canh Truong
Thanks Lennart, I will add this before pushing code. Regards, Canh. -Original Message- From: Lennart Lund [mailto:lennart.l...@ericsson.com] Sent: Tuesday, October 11, 2016 8:51 PM To: Canh Van Truong; Vu Minh Nguyen; mahesh.va...@oracle.com Cc: opensaf-devel@lists.sourceforge.net

Re: [devel] [PATCH 1 of 1] ntf: fix ntfa does not remove subscriber in subscriberNoList at finalize [#1978]

2016-10-05 Thread Canh Truong
Hi Praveen, What I understand about deadlock as below: When other API and saNtfFinalize() called in parallel, both take the ntfhandle by ncshm_take_hdl(), the cell->use_ct increase (and > 1). The saNtfFinalize() call ncshm_destroy_hdl() to destroy ntfhandle, but ncshm_destroy_hdl() will check

Re: [devel] [PATCH 1 of 1] ntf: fix to keep MDS connection when last client finalize [#1895]

2016-09-13 Thread Canh Truong
Hi Praveen, The problem is not always happen. After #1818 is fixed, it's still happen sometimes. I cannot reproduce this problem, just have trace log (attached here: https://sourceforge.net/p/opensaf/tickets/1895/). The test case in this patch may not reproduce the problem. It just proves that

Re: [devel] [PATCH 1 of 1] log: fix cppcheck with performance severity [#1975]

2016-09-01 Thread Canh Truong
Hi Mahesh, I run cppcheck with latest version 1.75. I got from: https://sourceforge.net/projects/cppcheck/files/cppcheck/1.75/ And build, then run with option --enable=performance. Regards, Canh. -Original Message- From: A V Mahesh [mailto:mahesh.va...@oracle.com] Sent: Thursday,

Re: [devel] [PATCH 1 of 1] log: fix errors reported by cppcheck version 1.75 [#1985]

2016-09-01 Thread Canh Truong
Hi Mahesh, Please see my comment inline. Regards, Canh. -Original Message- From: A V Mahesh [mailto:mahesh.va...@oracle.com] Sent: Thursday, September 01, 2016 10:59 AM To: Canh Van Truong; vu.m.ngu...@dektech.com.au; lennart.l...@ericsson.com Cc: opensaf-devel@lists.sourceforge.net

Re: [devel] [PATCH 1 of 1] log: fix issue with priority of messages adding to mailbox in the server[1396]

2016-08-04 Thread Canh Truong
opensaf-devel@lists.sourceforge.net > Subject: Re: [PATCH 1 of 1] log: fix issue with priority of messages > adding to mailbox in the server[1396] > > Hi Canh, > > It seems Lennart already provide comments please check. > > -AVM > > > On 6/21/2016 12:07 PM,

Re: [devel] [PATCH 1 of 1] log: fix issue with priority of messages adding to mailbox in the server[1396]

2016-06-21 Thread Canh Truong
] Sent: Tuesday, June 21, 2016 11:47 AM To: Canh Truong; vu.m.ngu...@dektech.com.au; lennart.l...@ericsson.com Cc: opensaf-devel@lists.sourceforge.net Subject: Re: [PATCH 1 of 1] log: fix issue with priority of messages adding to mailbox in the server[1396] Hi Canh, It will be good to send V2 patch

Re: [devel] [PATCH 1 of 1] log: fix issue with priority of messages adding to mailbox in the server[1396]

2016-06-14 Thread Canh Truong
Hi Mahesh, I have created new test case for multiple threads. Please help check new patch in attachment. Thanks, Canh. -Original Message- From: A V Mahesh [mailto:mahesh.va...@oracle.com] Sent: Tuesday, June 14, 2016 11:03 AM To: Canh Van Truong; vu.m.ngu...@dektech.com.au;

[devel] [PATCH 0 of 1] Review Request for log: fix trouble when the number app streams reachs limitation [#1446]

2016-03-19 Thread Canh Truong
n Tests n Other n Comments (indicate scope for each "y" above): - <> changeset 7214b787efa1088e407c89b016d0d16f30254c1a Author: Canh Truong <canh.v.tru...@dektech.com.au> Date: Wed, 16 M

[devel] [PATCH 1 of 1] log: fix trouble when the number app streams reachs limitation [#1446]

2016-03-19 Thread Canh Truong
osaf/services/saf/logsv/lgs/lgs_evt.cc| 6 ++ osaf/services/saf/logsv/lgs/lgs_imm.cc| 7 ++- osaf/services/saf/logsv/lgs/lgs_stream.cc | 12 + osaf/services/saf/logsv/lgs/lgs_stream.h | 1 + tests/logsv/tet_LogOiOps.c| 74 +++

[devel] [PATCH 0 of 1] Review Request for log: fix info represents by millisecond token is not correct [#1619]

2016-02-22 Thread Canh Truong
Tests n Other n Comments (indicate scope for each "y" above): - <> changeset ba42ccf646021ea8b705b451431999d6fe4d77d7 Author: Canh Truong <canh.v.tru...@dektech.com.au> Date: Tue, 16 F