osaf/services/saf/amf/amfnd/su.cc |  8 +++++---
 1 files changed, 5 insertions(+), 3 deletions(-)


This scenario happen if unlock-in SU before going headless.
After headless, amfnd sends SU oper state DISABLE in recovery data.

The patch comments out the suspicious setting SU's oper state to DISABLED
while knowing that SU is not FAILED.

diff --git a/osaf/services/saf/amf/amfnd/su.cc 
b/osaf/services/saf/amf/amfnd/su.cc
--- a/osaf/services/saf/amf/amfnd/su.cc
+++ b/osaf/services/saf/amf/amfnd/su.cc
@@ -574,9 +574,11 @@ uint32_t avnd_su_curr_info_del(AVND_CB *
                /* stop su_err_esc_tmr TBD Later */
 
                /* disable the oper state (if pi su) */
-               if (m_AVND_SU_IS_PREINSTANTIABLE(su) && (su->admin_op_Id != 
SA_AMF_ADMIN_RESTART)) {
-                       m_AVND_SU_OPER_STATE_SET(su, 
SA_AMF_OPERATIONAL_DISABLED);
-               }
+               // TODO: reset healthy SU to DISABLED doesn't seem to be right, 
double check 
+               // how SU Operational state being set in other places
+               //if (m_AVND_SU_IS_PREINSTANTIABLE(su) && (su->admin_op_Id != 
SA_AMF_ADMIN_RESTART)) {
+               //      m_AVND_SU_OPER_STATE_SET(su, 
SA_AMF_OPERATIONAL_DISABLED);
+               //}
        }
 
        /* scan & delete the current info store in each component */

------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140
_______________________________________________
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to