[Openais] Re: A bug for AMF
sakai.miyotaka at nttcom.co.jp
Mon Oct 18 16:02:27 PDT 2004
I would like your comment and advice on the patch
that was attacted to the E-mail I sent.
I don't mind whether you give me severe comment.
I'll reconsider and fix that.
By the way, I have other jobs on bugzilla that are 140 and 157.
I'll correct these bug first ?
(I'm going to do that ,after I fix this bug.)
Miyotaka Sakai wrote:
> Could you review the patch attached to E-mail instead of last one.
> Some change is added.
> And response in line.
> Miyotaka Sakai wrote:
>> I made the patch ,and attached this Email.
>> This patch include 2 bugs (142,143).
>> response in line .
>> Steven Dake wrote:
>>> I'm not sure if I was clear with dsmDisabledUnlockedFailed. This is why
>>> SA_AMF_NOT_RESPONDING must be removed from your patch to work. The
>>> reason is because you are sending a message with HIGH priority in this
>>> case when doing RECOVERY messages which causes the state machine to
>>> break down.
>>> By encoding the priority into the state machine,
>>> dsmDisabledUnlockedFailed will have the priority value to cause send
>>> the right hastatesetcluster or readinesssattesetcluster messages.
>> I agree with you .
>> I'll avoid using RECOVERY priority in readinessSetClusetr and
>> hastateSetCluseter .
>> Regarding SA_AMF_NOT_RESPONDING :
>> In component_unregister () ,SA_AMF_NOT_RESPONDING is needed .
>> In this case ,all messages must not be deliverded to library.
>> But in component_register () ,SA_AMF_NOT_RESPONDING isn't needed .
>> In this case ,some messages has to be deliverded to library.
>> For example HealthchekCallback and OtherCallbacks.
>> Of course ,program shouldn't send response to saAmfComponentUnregister.
> I misunderstood SA_AMF_NOT_RESPONDING.
> When SA_AMF_NOT_RESPONDING is set, HealethcheckcCallback is delibderd.
> I'm sorry about that.
> But this case ,SA_AMF_NOT_RESPONDING isn't needed.
> Becase in order to avoid sending response to saAmfComponentUnregister,
> this patch take another way.
> - Miyotaka Sakai
>> This Patch is added mesage IF (req_exec_amf_componentregister) change.
>> In order to scynchronize component date quickly ,this change is neede
>> ,I think. (Should I add another message IF ? )
>> This patch work well comparably.
>> - Miyotaka
More information about the Openais