Re: [libvirt] PATCH: Support initial boot time CPU affinity mask

2008-05-22 Thread Daniel P. Berrange
On Wed, May 21, 2008 at 11:33:33AM -0400, Daniel Veillard wrote: On Fri, May 16, 2008 at 10:50:20PM +0100, Daniel P. Berrange wrote: +/* Extract domain vcpu info */ +obj = xmlXPathEval(BAD_CAST string(/domain/vcpu[1]/@cpuset), ctxt); +if ((obj == NULL) || (obj-type !=

Re: [libvirt] PATCH: Support initial boot time CPU affinity mask

2008-05-21 Thread Daniel Veillard
On Fri, May 16, 2008 at 10:50:20PM +0100, Daniel P. Berrange wrote: The XML format allows for an initial CPU mask to be specified for a guests vCPUs. eg with this XML: vcpu cpuset='1-4,8-20,525'1/vcpu what about output. In the xen case we went though the exercise to dump a cpuset string

[libvirt] PATCH: Support initial boot time CPU affinity mask

2008-05-16 Thread Daniel P. Berrange
The XML format allows for an initial CPU mask to be specified for a guests vCPUs. eg with this XML: vcpu cpuset='1-4,8-20,525'1/vcpu Since we have CPU pinning support from my previous patch, adding in the initial pinning is fairly easy. We first pass the '-S' arg to QEMU when forking it. This

Re: [libvirt] PATCH: Support initial boot time CPU affinity mask

2008-05-16 Thread Stefan de Konink
On Fri, 16 May 2008, Daniel P. Berrange wrote: The XML format allows for an initial CPU mask to be specified for a guests vCPUs. eg with this XML: vcpu cpuset='1-4,8-20,525'1/vcpu Since we have CPU pinning support from my previous patch, adding in the initial pinning is fairly easy. We

Re: [libvirt] PATCH: Support initial boot time CPU affinity mask

2008-05-16 Thread Cole Robinson
Stefan de Konink wrote: On Fri, 16 May 2008, Daniel P. Berrange wrote: The XML format allows for an initial CPU mask to be specified for a guests vCPUs. eg with this XML: vcpu cpuset='1-4,8-20,525'1/vcpu Since we have CPU pinning support from my previous patch, adding in the initial

Re: [libvirt] PATCH: Support initial boot time CPU affinity mask

2008-05-16 Thread Jim Paris
Daniel P. Berrange wrote: Since we have CPU pinning support from my previous patch, adding in the initial pinning is fairly easy. We first pass the '-S' arg to QEMU when forking it. If -S is always added, this becomes unnecessary (qemu_conf.c:2823): if (vm-migrateFrom[0]) { if