RE: [Xen-devel] [PATCH] Pin vcpu for VMX domain

["Dong, Eddie" <eddie.dong@xxxxxxxxx>]

Keir:
        Just some curious question.

Keir Fraser wrote:
> On 8 Feb 2006, at 09:28, Jiang, Yunhong wrote:
> 
> You don't need to __vmpclear at the time the affinity is changed. You
> can still do it from within arch_vmx_do_resume() -- it is valid to
> call smp_call_function() there. That avoids having yet another HVM
> function, and avoids calling yet another arch_* function from common
> code. 
Good point. Do u mean we check v->arch.hvm_vmx.launch_cpu with
 v->processor in arch_vmx_do_resume and send IPI if they are not equal?
If yes, is it worth to exchage common code change with the frequent
check 
at every resume?
> 
> Rather than calling vmx_remove_timers() from your SMP call function,
> can you not just call it from vmx_reinstall_timers()? You explain why
Sure will look into details.
Do u mean removing ac_timer on remote physical processor? Just a little
bit worry about the corner case. 
> the pit and hlt timers do not need to be re-activated in
> vmx_reinstall_timer() -- what about the APIC timer?
Oo, thanks, a mistake here.

> There's no need to check active_timer() before calling stop_timer() --
> stop_timer does the check for you.
Yes, sir!
> 
Eddie

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel