> -----Original Message----- > From: Nagendra Kumar [mailto:[email protected]] > Sent: den 5 juli 2013 06:15 > To: Hans Feldt; Praveen Malviya > Cc: [email protected] > Subject: Re: [devel] [PATCH 1 of 1] amfnd: set SU DISABLED at TERM-FAILED > [#481] > > Hi Hans, > I guess this is Ack and going forward we need to consolidate > such kind of scenario when su goes into term failure and we need to fix, test > them in all redundancy models.
Yes and yes /Hans > > Thanks > -Nagu > > -----Original Message----- > From: Hans Feldt [mailto:[email protected]] > Sent: 04 July 2013 17:46 > To: praveen malviya > Cc: Nagendra Kumar; [email protected] > Subject: Re: [devel] [PATCH 1 of 1] amfnd: set SU DISABLED at TERM-FAILED > [#481] > > 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 ------------------------------------------------------------------------------ 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
