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

2016-08-25 Thread minh . chau
Hi Praveen, Long is preparing the patch adding sentinel element. Thanks, Minh > Hi Minh, > > I think this is subject to interpretation. After finalize(), handle > becomes invalid. So an application cannot call Free_4() to free any > memory. In such a case, freeing any resources associated with

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

2016-08-25 Thread praveen malviya
Hi Minh, I think this is subject to interpretation. After finalize(), handle becomes invalid. So an application cannot call Free_4() to free any memory. In such a case, freeing any resources associated with this handle in finalize() seems to be ok. Anyways freeing in finalize() can be

Re: [devel] [PATCH 0 of 7] Review Request for Support for CSI attribute change callback. [#1553].

2016-08-25 Thread Gary Lee
Hi Praveen Some initial comments below: Thanks Gary 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/libs/agents/saf/amfa/amf_agent.cc @@ -2812,6 +2812,7 @@ */ SaAisErrorT

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

2016-08-25 Thread minh chau
Hi Praveen, Just to confirm if I understand correctly the problem you mentioned in saAmfFinalize(). As the specification says application should call free()/Free_4() to release the allocated memory, if application does not release memory then it's most likely application misuses API. Or do you

Re: [devel] [PATCH 2 of 2] AMFND: Admin operation continuation if csi callback completes during headless [#1725 part 1] V1

2016-08-25 Thread minh chau
Hi, The test failed because two reasons: 1. There are two places that nodegroup operation borrows 2N SG FSM, but the AdminState of SG is not stored to IMM saAmfSGAdminState = ng->saAmfNGAdminState; ... su->sg_of_su->saAmfSGAdminState = SA_AMF_ADMIN_UNLOCKED; This setting needs to

Re: [devel] [PATCH 2 of 2] AMFND: Admin operation continuation if csi callback completes during headless [#1725 part 1] V1

2016-08-25 Thread Nagendra Kumar
Further testing results: Node group lock has resulted in SG unstable. Logs and configuration file attached. Configuration : SC-1, PL-3 and PL-4. Steps: 1. Unlock SU1(on PL-3), SU2 and SU3 (Both on PL-4). 2. Create node group of PL-3 and PL-4: 3. Lock the node group. amf-adm lock

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

2016-08-25 Thread Anders Widell
Hi! This is what the TIPC user documentation says about TIPC_DEST_DROPPABLE: "This option governs the handling of messages sent by the socket if the message cannot be delivered to its destination, either because the receiver is congested or because the specified receiver does not exist. If

Re: [devel] [PATCH 2 of 2] AMFND: Admin operation continuation if csi callback completes during headless [#1725 part 1] V1

2016-08-25 Thread minh chau
Hi Praveen, I think we need to come back a bit to non-headless feature. The cluster init timer expiry ensures all nodes having MW SUs assigned and node state are PRESENT. It's the unique entry point to non-ncs SU assignment phase. We also need to keep this principle in headless for #1725. ...

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

2016-08-25 Thread praveen malviya
Hi Minh, AMFA currently does not remember the allocated memory. It relies on the application always to free the memory. In saAmfFinalize() also, it does not free the memory. I think AMFA should remember memory by associating it with handle because process which is starting PG tracking may not

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

2016-08-25 Thread A V Mahesh
Hi HansN, On 8/23/2016 5:22 PM, Hans Nordebäck wrote: > Hi Mahesh, > > Yes, this is my understanding too, if TIPC_DROPPABLE = true tipc may drop > messages silently, at receive sock buffer full condition, but do not return > any ancillary message. > If TIPC_DROPPABLE = false tipc may drop

Re: [devel] Review Request for ntf: update PR documentation [#1952]

2016-08-25 Thread minh chau
Hi Vu, Ack from me for PR doc. I see this limitation still documented in README, but I think it's just fine to be there (under 4.5). You can also make another paragraph to say it's removed in 5.1. I'm ok with both, since nothing is interesting in 5.1 to say. Thanks, Minh On 25/08/16 12:39,

Re: [devel] [PATCH 1 of 1] amf: README file for long DN support [#1642]

2016-08-25 Thread Long Nguyen
Thanks Praveen, Actually, the PR doc only contains the implementation detail part in readme file. I will remove the internal amfd and amfnd changes in the PR doc. Best regards, Long Nguyen. On 8/25/2016 11:42 AM, praveen malviya wrote: > Ack. > > I think implementation details should go in PR

Re: [devel] [PATCH 2 of 2] AMFND: Admin operation continuation if csi callback completes during headless [#1725 part 1] V1

2016-08-25 Thread praveen malviya
Hi Minh, One minor correction is still needed. node_up event comes very early. In case atleast one node up event has come from all amfnds then AMFD stops Node sync timer very early even before cluster timer has started: if (rc_node_up == sync_nd_size) { if