[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-devel] RE: [ PATCH 4/4 ] HVM vcpu add/remove: qemu logic for vcpu add/revmoe


  • To: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>, xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • From: "Liu, Jinsong" <jinsong.liu@xxxxxxxxx>
  • Date: Tue, 15 Dec 2009 19:31:33 +0800
  • Accept-language: en-US
  • Acceptlanguage: en-US
  • Cc: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
  • Delivery-date: Tue, 15 Dec 2009 03:33:27 -0800
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: Acp8HtRWWs29ZQxZRTqrRQcWGhMbYwAdUEXqAAMX2aUANmNFgA==
  • Thread-topic: [ PATCH 4/4 ] HVM vcpu add/remove: qemu logic for vcpu add/revmoe

Keir Fraser wrote:
> On 14/12/2009 08:04, "Keir Fraser" <keir.fraser@xxxxxxxxxxxxx> wrote:
> 
>> On 13/12/2009 18:05, "Liu, Jinsong" <jinsong.liu@xxxxxxxxx> wrote:
>> 
>>> HVM vcpu add/remove: qemu logic for vcpu add/revmoe
>>> 
>>> -- at qemu side, get vcpu_avail which used for original cpu avail
>>> map; 
>>> -- setup gpe ioread/iowrite at qmeu;
>>> -- setup vcpu add/remove user interface through monitor;
>>> -- setup SCI logic;
>> 
>> I'm guessing because this adds a new command-line option that I need
>> this checked into the qemu tree before I can apply your first patch
>> (1/4)? Otherwise that patch will break domain creation as qemu will
>> exit with an 'unrecognised option' error. So I need Ian Jackson to
>> apply this one and send me an updated QEMU_TAG first.
> 
> As of c/s 20640 all your Xen patches are checked in. I modified them
> a bit so you may want to take a look. I commented out the one line
> that actually sets the new qemu option, until that option is
> supported by our qemu. I think there is a question over whether the
> new qemu option should (a) have a better name (I called it
> vcpu_online[] in hvm_info structure); and (b) should have a more
> user-friendly format (currently passing a decimal number interpreted
> as a bitmap - perhaps should be a list of vcpus instead). 
> 
>  -- Keir

Just go through your modification, bios_info is really good, I don't know its 
usage before, thanks!

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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.