changeset: 7644:c893131ce311 user: Nhat Pham <[email protected]> date: Wed May 18 10:31:16 2016 +0530 summary: cpnd: To indicate clm_updated_sel_obj in cpnd_clm_init_thread [#1830]
changeset: 7645:ea188b1ec260 branch: opensaf-5.0.x tag: tip parent: 7642:83ee28ff2284 user: Nhat Pham <[email protected]> date: Wed May 18 10:32:37 2016 +0530 summary: cpnd: To indicate clm_updated_sel_obj in cpnd_clm_init_thread [#1830] -AVM On 5/18/2016 11:30 AM, Nhat Pham wrote: > Hi Mahesh, > > It was tested. > Could you please help to push the patch? Thanks. > > Best regards, > Nhat Pham > > -----Original Message----- > From: A V Mahesh [mailto:[email protected]] > Sent: Wednesday, May 18, 2016 12:15 PM > To: Nhat Pham <[email protected]>; [email protected] > Cc: [email protected] > Subject: Re: [PATCH 1 of 1] cpnd: To indicate clm_updated_sel_obj in > cpnd_clm_init_thread [#1830] > > Hi Nhat Pham, > > I hope you have tested the testcase of #1821 case with this patch > (saClmDispatch() fail case). > > -AVM > > On 5/18/2016 10:20 AM, Nhat Pham wrote: >> osaf/services/saf/cpsv/cpnd/cpnd_init.c | 8 +++++--- >> 1 files changed, 5 insertions(+), 3 deletions(-) >> >> >> Problem: >> -------- >> The message "ncs_sel_obj_ind: write failed - Bad file descriptor" > sometimes happens when the node reboot. >> This is because indicating clm_updated_sel_obj is executed in >> cpnd_clm_init() at start up. In this case the clm_updated_sel_obj is not > created yet. >> Solution: >> --------- >> To execute indicating clm_updated_sel_obj in the >> cpnd_clm_init_thread() >> >> diff --git a/osaf/services/saf/cpsv/cpnd/cpnd_init.c >> b/osaf/services/saf/cpsv/cpnd/cpnd_init.c >> --- a/osaf/services/saf/cpsv/cpnd/cpnd_init.c >> +++ b/osaf/services/saf/cpsv/cpnd/cpnd_init.c >> @@ -659,9 +659,6 @@ static SaAisErrorT cpnd_clm_init(void) >> goto cpnd_clm_fail; >> } >> >> - /* Notify main process to update clm select object */ >> - ncs_sel_obj_ind(&cb->clm_updated_sel_obj); >> - >> TRACE_LEAVE(); >> return rc; >> >> @@ -686,6 +683,8 @@ static SaAisErrorT cpnd_clm_init(void) >> > **************************************************************************** > */ >> static void* cpnd_clm_init_thread(void* arg) >> { >> + CPND_CB *cb = m_CPND_TAKE_CPND_CB; >> + >> TRACE_ENTER(); >> >> SaAisErrorT rc = cpnd_clm_init(); >> @@ -693,6 +692,9 @@ static void* cpnd_clm_init_thread(void* >> exit(EXIT_FAILURE); >> } >> >> + /* Notify main process to update clm select object */ >> + ncs_sel_obj_ind(&cb->clm_updated_sel_obj); >> + >> TRACE_LEAVE(); >> return NULL; >> } > ------------------------------------------------------------------------------ Mobile security can be enabling, not merely restricting. Employees who bring their own devices (BYOD) to work are irked by the imposition of MDM restrictions. Mobile Device Manager Plus allows you to control only the apps on BYO-devices by containerizing them, leaving personal data untouched! https://ad.doubleclick.net/ddm/clk/304595813;131938128;j _______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
