[devel] [PATCH 1 of 1] cpsv: To update checkpoint user number for each node [#1669] V3

2016-05-03 Thread Nhat Pham
osaf/libs/common/cpsv/include/cpd_cb.h |2 + osaf/libs/common/cpsv/include/cpd_proc.h |3 + osaf/libs/common/cpsv/include/cpd_red.h | 14 ++ osaf/libs/common/cpsv/include/cpsv_evt.h |8 + osaf/services/saf/cpsv/cpd/cpd_db.c | 14 ++- osaf/services/saf/cpsv/cpd/cpd_evt.c

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

2016-05-03 Thread Nhat Pham
Summary: cpsv: To update checkpoint user number for each node [#1669] V3 Review request for Trac Ticket(s): 1669 Peer Reviewer(s): mahesh.va...@oracle.com; anders.wid...@ericsson.com Pull request to: mahesh.va...@oracle.com Affected branch(es): default, 5.0 Development branch: default

Re: [devel] [PATCH 1 of 1] cpd: To exit cpd_clm_init() after function succeeds. [#1806]

2016-05-03 Thread A V Mahesh
Hi Nhat PhamI, Verified , I am pushing it. -AVM On 5/4/2016 10:29 AM, A V Mahesh wrote: > Please hold > > is this patch on top of ckpt: init with clm before entering > initialize_for_assignment() [#1781] ? > > -AVM > > > On 5/4/2016 10:18 AM, A V Mahesh wrote: >> ACK >> >> Not tested. >> >> -A

Re: [devel] [PATCH 1 of 1] cpd: To exit cpd_clm_init() after function succeeds. [#1806]

2016-05-03 Thread A V Mahesh
Please hold is this patch on top of ckpt: init with clm before entering initialize_for_assignment() [#1781] ? -AVM On 5/4/2016 10:18 AM, A V Mahesh wrote: > ACK > > Not tested. > > -AVM > > On 5/4/2016 10:10 AM, Nhat Pham wrote: >>osaf/services/saf/cpsv/cpd/cpd_init.c | 2 ++ >>1 fil

Re: [devel] [PATCH 1 of 1] cpd: To exit cpd_clm_init() after function succeeds. [#1806]

2016-05-03 Thread A V Mahesh
ACK Not tested. -AVM On 5/4/2016 10:10 AM, Nhat Pham wrote: > osaf/services/saf/cpsv/cpd/cpd_init.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > > Problem: > > The function cpd_clm_init() doesn't exit after successfull operation. > This leads the clm handles finali

[devel] [PATCH 0 of 1] Review Request for cpd: To exit cpd_clm_init() after function succeeds. [#1806]

2016-05-03 Thread Nhat Pham
Summary: cpd: To exit cpd_clm_init() after function succeeds. [#1806] Review request for Trac Ticket(s): 1806 Peer Reviewer(s): mahesh.va...@oracle.com; anders.wid...@ericsson.com Pull request to: mahesh.va...@oracle.com Affected branch(es): default, 5.0 Development branch: default ---

[devel] [PATCH 1 of 1] cpd: To exit cpd_clm_init() after function succeeds. [#1806]

2016-05-03 Thread Nhat Pham
osaf/services/saf/cpsv/cpd/cpd_init.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) Problem: The function cpd_clm_init() doesn't exit after successfull operation. This leads the clm handles finalized after function exit. As result ckpt operations fails. Solution: -

Re: [devel] [PATCH 1 of 1] plm: disallow activation of blade if it is operationally disabled [#1751]

2016-05-03 Thread Mathivanan Naickan Palanivelu
Ack, This fits the SAF Admin state model too. Could you update the ticket with this approach instead of the v1 patch. Will push it tomorrow. Thanks, Mathi. > -Original Message- > From: Alex Jones [mailto:ajo...@genband.com] > Sent: Tuesday, April 26, 2016 7:03 PM > To: Mathivanan Naickan