Re: [Xen-devel] use tasklet to handle init/sipi?

["Zhang, Yang Z" <yang.z.zhang@xxxxxxxxx>]

Keir Fraser wrote on 2013-03-26:
> On 26/03/2013 07:17, "Zhang, Yang Z" <yang.z.zhang@xxxxxxxxx> wrote:
> 
>>>> Oh, I see. Well I think it is fine to have
>>>> vlapic_schedule_init_sipi_tasklet() return X86EMUL_OKAY rather than
>>>> X86EMUL_RETRY. We used to need to return RETRY, but the code got
>>>> simplified and now it is actually unnecessary.
>>>> 
>>>> That should make your patch a lot simpler eh? ;)
>>> 
>>> Given that you ignore the return code on the apicv call path, is there
>>> currently a bug at all for you? Seems what is there already must work
>>> for you?
>> It do cause bug after we change to use seabios. For seabios, it will
>> send INIT/SIPI to all vcpus via broadcasting. And there only one vcpu
>> is waken up via tasklet with current logic. That's the reason why I
>> want to wakeup all vcpus on one callback. Just change X86EMUL_RETRY to
>> OK cannot solve the problem. still need the logic I mentioned above.
> 
> Ok, wait a sec, I will sort out a patch for you to try...
Thanks. Actually, I have patch on hand and testing it now. But it's ok if you 
can provide a more better solution.

Best regards,
Yang



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel