This change enables SU repair after the described scenario yes so ack (comment is not updated though) BUT - how can AMF do failover when the SU is in TERM-FAILED pres state? That is totally wrong and needs to be addressed asap. Thanks, Hans
On 3 July 2013 13:12, praveen malviya <[email protected]> wrote: > Ack. > > -Praveen > On 02-Jul-13 12:26 PM, [email protected] wrote: >> osaf/services/saf/avsv/avnd/avnd_susm.c | 3 ++- >> 1 files changed, 2 insertions(+), 1 deletions(-) >> >> >> diff --git a/osaf/services/saf/avsv/avnd/avnd_susm.c >> b/osaf/services/saf/avsv/avnd/avnd_susm.c >> --- a/osaf/services/saf/avsv/avnd/avnd_susm.c >> +++ b/osaf/services/saf/avsv/avnd/avnd_susm.c >> @@ -1466,7 +1466,8 @@ uint32_t avnd_su_pres_st_chng_prc(AVND_C >> } >> >> /* terminating -> term-failed */ >> - if ((SA_AMF_PRESENCE_TERMINATING == prv_st) && >> (SA_AMF_PRESENCE_TERMINATION_FAILED == final_st)) { >> + if (((prv_st == SA_AMF_PRESENCE_RESTARTING) || >> (SA_AMF_PRESENCE_TERMINATING == prv_st)) >> + && (SA_AMF_PRESENCE_TERMINATION_FAILED == >> final_st)) { >> TRACE("Terminating -> Termination Failed"); >> m_AVND_SU_OPER_STATE_SET(su, >> SA_AMF_OPERATIONAL_DISABLED); >> m_AVND_SEND_CKPT_UPDT_ASYNC_UPDT(cb, su, >> AVND_CKPT_SU_OPER_STATE); > > > ------------------------------------------------------------------------------ > This SF.net email is sponsored by Windows: > > Build for Windows Store. > > http://p.sf.net/sfu/windows-dev2dev > _______________________________________________ > Opensaf-devel mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/opensaf-devel ------------------------------------------------------------------------------ This SF.net email is sponsored by Windows: Build for Windows Store. http://p.sf.net/sfu/windows-dev2dev _______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
