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:00, "Keir Fraser" <keir.xen@xxxxxxxxx> wrote:
> 
>>> The problem is:
>>> With apicv support, the apic write is trap like vmexit. We cannot fallback 
>>> to
>>> guest to retry the instruction. So it will break current logic.
>> 
>> 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.

Best regards,
Yang



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