Ack, code review only. Thanks, Praveen
On 19-Mar-15 2:54 PM, [email protected] wrote: > osaf/services/saf/amf/amfd/sgproc.cc | 1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > > Amfd checkpoints node admin state as lock when clm lock > operation starts, but it doesn't send checkpoint for the same > for unlock. > This doesn't allow assignment to be given after clm unlock on > payload because node remains in lock state on newly Act Amfd. > So, when Amfd marks node admin state as unlocked, Amfd should > send checkpoint to its peer. > > diff --git a/osaf/services/saf/amf/amfd/sgproc.cc > b/osaf/services/saf/amf/amfd/sgproc.cc > --- a/osaf/services/saf/amf/amfd/sgproc.cc > +++ b/osaf/services/saf/amf/amfd/sgproc.cc > @@ -1366,6 +1366,7 @@ void avd_su_si_assign_evh(AVD_CL_CB *cb, > /* clm admin lock/shutdown operations were on, > so we need to reset > node admin state.*/ > su->su_on_node->saAmfNodeAdminState = > SA_AMF_ADMIN_UNLOCKED; > + m_AVSV_SEND_CKPT_UPDT_ASYNC_UPDT(avd_cb, > su->su_on_node, AVSV_CKPT_AVND_ADMIN_STATE); > clm_node_terminate(su->su_on_node); > } else if (n2d_msg->msg_info.n2d_su_si_assign.error != > NCSCC_RC_SUCCESS) { > /* clm admin lock/shutdown operations were on, > so we need to reset > ------------------------------------------------------------------------------ Dive into the World of Parallel Programming The Go Parallel Website, sponsored by Intel and developed in partnership with Slashdot Media, is your hub for all things parallel software development, from weekly thought leadership blogs to news, videos, case studies, tutorials and more. Take a look and join the conversation now. http://goparallel.sourceforge.net/ _______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
