Re: [Xen-devel] [PATCH] qemu-xen: fix cpu hotplug

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

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