On 11/26/2013 01:04 PM, Nagendra Kumar wrote:
> Yes, should be committed in all branches.
>
> But, I had a question: The condition for PI was committed by you as part of 
> #1194, CS #2378. Was there any specific reason for the same ?

Not that I am aware...
Just looks like a mistake to me
/Hans

>
> Thanks
> -Nagu
>
> -----Original Message-----
> From: Hans Feldt [mailto:hans.fe...@ericsson.com]
> Sent: 26 November 2013 17:22
> To: Nagendra Kumar; hans.nordeb...@ericsson.com
> Cc: opensaf-devel@lists.sourceforge.net
> Subject: Re: [PATCH 1 of 1] amfd: enable NPI component oper state during 
> admin repair [#182]
>
> Ack, but this should be fixed in all branches right?
> Currently the milestone is set to 4.4.FC, should be 4.2.5 I think.
> Thanks,
> Hans
>
> On 11/26/2013 07:02 AM, nagendr...@oracle.com wrote:
>>    osaf/services/saf/amf/amfnd/su.cc |  6 ++----
>>    1 files changed, 2 insertions(+), 4 deletions(-)
>>
>>
>> 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
>> @@ -490,10 +490,8 @@ uint32_t avnd_evt_su_admin_op_req(AVND_C
>>                      m_AVND_COMP_STATE_RESET(comp);
>>                      avnd_comp_pres_state_set(comp, 
>> SA_AMF_PRESENCE_UNINSTANTIATED);
>>                      
>> -                    if (m_AVND_COMP_TYPE_IS_PREINSTANTIABLE(comp)) {
>> -                            m_AVND_COMP_OPER_STATE_SET(comp, 
>> SA_AMF_OPERATIONAL_ENABLED);
>> -                            avnd_di_uns32_upd_send(AVSV_SA_AMF_COMP, 
>> saAmfCompOperState_ID, &comp->name, comp->oper);
>> -                    }
>> +                    m_AVND_COMP_OPER_STATE_SET(comp, 
>> SA_AMF_OPERATIONAL_ENABLED);
>> +                    avnd_di_uns32_upd_send(AVSV_SA_AMF_COMP, 
>> saAmfCompOperState_ID, &comp->name, comp->oper);
>>              }
>>
>>              m_AVND_SU_STATE_RESET(su);
>>
>>
>
>

------------------------------------------------------------------------------
Shape the Mobile Experience: Free Subscription
Software experts and developers: Be at the forefront of tech innovation.
Intel(R) Software Adrenaline delivers strategic insight and game-changing 
conversations that shape the rapidly evolving mobile landscape. Sign up now. 
http://pubads.g.doubleclick.net/gampad/clk?id=63431311&iu=/4140/ostg.clktrk
_______________________________________________
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to