|
|
|
|
|
|
|
|
|
|
xen-devel
Re: [Xen-devel] [PATCH] qemu-xen: fix cpu hotplug
To: |
Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx> |
Subject: |
Re: [Xen-devel] [PATCH] qemu-xen: fix cpu hotplug |
From: |
Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx> |
Date: |
Thu, 2 Sep 2010 18:30:01 +0100 |
Cc: |
"Liu, Jinsong" <jinsong.liu@xxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxx, "Li, Xin" <xin.li@xxxxxxxxx>, "Jiang, Yunhong" <yunhong.jiang@xxxxxxxxx> |
Delivery-date: |
Thu, 02 Sep 2010 10:30:47 -0700 |
Envelope-to: |
www-data@xxxxxxxxxxxxxxxxxxx |
In-reply-to: |
<alpine.DEB.2.00.1009021416500.2714@kaball-desktop> |
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> |
Newsgroups: |
chiark.mail.xen.devel |
References: |
<alpine.DEB.2.00.1009021416500.2714@kaball-desktop> |
Sender: |
xen-devel-bounces@xxxxxxxxxxxxxxxxxxx |
Stefano Stabellini writes ("[Xen-devel] [PATCH] qemu-xen: fix cpu hotplug"):
> qemu-xen: fix cpu hotplug
>
> The current xenstore watch path for a vcpu-set event is wrong and is
> also wrong the code to parse it. This patch fixes both of them:
Thanks. So it seems you're saying it's completely broken in
xen-unstable.
I have CC'd a bunch of Intel folks who were doing some other
work on cpu hotplug. They were dealing with a race when multiple CPUs
were added at once.
So I think there must be some confusion. Perhaps xl and xend have
different ideas about what the xenstore syntax is for these operations ?
Jinsong Liu et al: would you care to comment ? Particularly about this:
> A xenstore vcpu hotplug command is of the following form:
> path: /local/domain/DOMID/cpu/VCPU_NUMBER/availability
> values: "online" or "offline"
That doesn't seem to match up with what's in xend, which seems to
write "cpu_avail" in the vm tree.
Thanks,
Ian.
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
|
|
|
|