WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

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

To: "Keir Fraser" <Keir.Fraser@xxxxxxxxxxxx>, "Jiang, Yunhong" <yunhong.jiang@xxxxxxxxx>
Subject: RE: [Xen-devel] [PATCH] Pin vcpu for VMX domain
From: "Dong, Eddie" <eddie.dong@xxxxxxxxx>
Date: Wed, 8 Feb 2006 18:25:39 +0800
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Wed, 08 Feb 2006 10:36:47 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcYslbnUSShMDgb8T9CbE1kWXKabggAA055g
Thread-topic: [Xen-devel] [PATCH] Pin vcpu for VMX domain
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