Re: [devel] [PATCH 1 of 1] amfnd: send recovery request to amfd for term-failed PI su [#2047] V2

2016-10-04 Thread praveen malviya
Hi Hans, Thanks for the patch. In order to avoid writing separate function for checking each prev_assign_state, function avnd_su_check_sis_previous_assign_state() should be generic. Attached is the patch 2047_ontop.patch on top of 2047_V2_suggestion.diff with this minor change. Thanks, Prav

Re: [devel] [PATCH 1 of 1] amfnd: send recovery request to amfd for term-failed PI su [#2047] V2

2016-10-04 Thread Hans Nordebäck
Hi Praveen, ack, code review only. Minor comment below. Small suggestions in attached patch also. /Thanks HansN On 10/03/2016 09:28 AM, praveen.malv...@oracle.com wrote: osaf/services/saf/amf/amfnd/clc.cc| 5 + osaf/services/saf/amf/amfnd/di.cc | 2 +- osaf/serv

[devel] [PATCH 1 of 1] amfnd: send recovery request to amfd for term-failed PI su [#2047] V2

2016-10-03 Thread praveen . malviya
osaf/services/saf/amf/amfnd/clc.cc| 5 + osaf/services/saf/amf/amfnd/di.cc | 2 +- osaf/services/saf/amf/amfnd/include/avnd_su.h | 3 +- osaf/services/saf/amf/amfnd/su.cc | 14 osaf/services/saf/amf/amfnd/susm.cc | 89 +

[devel] [PATCH 1 of 1] amfnd: send recovery request to amfd for term-failed PI su [#2047]

2016-09-27 Thread praveen . malviya
osaf/services/saf/amf/amfnd/include/avnd_su.h | 1 + osaf/services/saf/amf/amfnd/su.cc | 13 + osaf/services/saf/amf/amfnd/susm.cc | 36 +++--- 3 files changed, 45 insertions(+), 5 deletions(-) A SU having one NPI and one PI comp moved to ter