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] [Question] vcpu-set before or after xen_pause_requested

To: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Subject: RE: [Xen-devel] [Question] vcpu-set before or after xen_pause_requested
From: "Jiang, Yunhong" <yunhong.jiang@xxxxxxxxx>
Date: Thu, 29 Apr 2010 11:01:00 +0800
Accept-language: en-US
Acceptlanguage: en-US
Cc: "Liu, Jinsong" <jinsong.liu@xxxxxxxxx>, xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>, Keir Fraser <Keir.Fraser@xxxxxxxxxxxxx>, "Zhai, Edwin" <edwin.zhai@xxxxxxxxx>, "Ian.Campbell@xxxxxxxxxx" <Ian.Campbell@xxxxxxxxxx>
Delivery-date: Wed, 28 Apr 2010 20:03:04 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <19416.3675.401481.320777@xxxxxxxxxxxxxxxxxxxxxxxx>
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/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <EB8593BCECAB3D40A8248BE0B6400A38469C6F70@xxxxxxxxxxxxxxxxxxxxxxxxxxxxx> <19399.7706.36343.487108@xxxxxxxxxxxxxxxxxxxxxxxx> <BC00F5384FCFC9499AF06F92E8B78A9E026FAC6041@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <19404.31543.186398.77423@xxxxxxxxxxxxxxxxxxxxxxxx> <789F9655DD1B8F43B48D77C5D30659731D851D23@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <19416.3675.401481.320777@xxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcrmveSDDTz/NGARQGedhKTVeAmDYAAiXy/A
Thread-topic: [Xen-devel] [Question] vcpu-set before or after xen_pause_requested
Ian, thanks for your reply. Yes, I think the intermediate values is not 
important in general. If it is really important, the caller of "xm  vcpu-set" 
should take care of it, i.e. do xm vcpu-set, wait and check the vcpu's status 
(maybe from xen hypervisor) with timeout mechanism.

BTW, from the followed changeset in dom0's drivers/xen/cpu_hotplug.c , it means 
the xm vcpu-set may happen even before guest is fully up, at that time, it is 
difficult for xend to achieve the close loop.

commit d745562cc40bff60f0597004d8128fa0225cc267
Author: Ian Campbell <Ian.Campbell@xxxxxxxxxx>
Date:   Thu Apr 2 13:24:28 2009 +0100

    xen: honour VCPU availability on boot

    If a VM is booted with offline VCPUs then unplug them during boot. 
Determining
    the availability of a VCPU requires access to XenStore which is not 
available
    at the point smp_prepare_cpus() is called, therefore we bring up all VCPUS
    initially and unplug the offline ones as soon as XenStore becomes available.

    Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>

Thanks
--jyh

>-----Original Message-----
>From: Ian Jackson [mailto:Ian.Jackson@xxxxxxxxxxxxx]
>Sent: Wednesday, April 28, 2010 6:31 PM
>To: Jiang, Yunhong
>Cc: Liu, Jinsong; xen-devel; Keir Fraser; Zhai, Edwin
>Subject: RE: [Xen-devel] [Question] vcpu-set before or after 
>xen_pause_requested
>
>Jiang, Yunhong writes ("RE: [Xen-devel] [Question] vcpu-set before or after
>xen_pause_requested"):
>> For example, considering user do vcpu-set 2 (i.e set vcpu number to
>> 2), vcpu-set 4, and vcpu-set 2, and guest didn't take any action at
>> all, would it be really so important? Or, if it is 2->4->3, should
>> the xm vcpu-set itself really need care 4 vcpu is bringup in some
>> time?
>
>This is a good point.  If you are sure that intermediate values are
>not important, and that changes are idempotent, then the open loop
>approach is quite fine.
>
>Ian.

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