* Re: [PATCH V2] Xen: support maxvcpus in xm and xl config
[not found] <1450218051-25842-1-git-send-email-jfehlig@suse.com>
@ 2015-12-16 10:11 ` Ian Campbell
2015-12-16 11:09 ` Ian Campbell
2015-12-18 11:47 ` [libvirt] " Michal Privoznik
[not found] ` <5673F25B.70606@redhat.com>
2 siblings, 1 reply; 4+ messages in thread
From: Ian Campbell @ 2015-12-16 10:11 UTC (permalink / raw)
To: Jim Fehlig, libvir-list; +Cc: xen-devel
On Tue, 2015-12-15 at 15:20 -0700, Jim Fehlig wrote:
> From: Ian Campbell <ian.campbell@citrix.com>
>
> xend prior to 4.0 understands vcpus as maxvcpus and vcpu_avail
> as a bit map of which cpus are online (default is all).
>
> xend from 4.0 onwards understands maxvcpus as maxvcpus and
> vcpus as the number which are online (from 0..N-1). The
> upstream commit (68a94cf528e6 "xm: Add maxvcpus support")
> claims that if maxvcpus is omitted then the old behaviour
> (i.e. obeying vcpu_avail) is retained, but AFAICT it was not,
> in this case vcpu==maxcpus==online cpus. This is good for us
> because handling anything else would be fiddly.
>
> This patch changes parsing of the virDomainDef maxvcpus and vcpus
> entries to use the corresponding 'maxvcpus' and 'vcpus' settings
> from xm and xl config. It also drops use of the old Xen 3.x
> 'vcpu_avail' setting.
>
> The change also removes the maxvcpus limit of MAX_VIRT_VCPUS (since
> maxvcpus is simply a count, not a bit mask), which is particularly
> crucial on ARM where MAX_VIRT_CPUS == 1 (since all guests are
> expected to support vcpu placement, and therefore only the boot
> vcpu's info lives in the shared info page).
>
> Existing tests adjusted accordingly, and new tests added for the
> 'maxvcpus' setting.
>
> Signed-off-by: Jim Fehlig <jfehlig@suse.com>
Acked-by: Ian Campbell <Ian.campbell@citrix.com>
Tested-by: Ian Campbell <Ian.campbell@citrix.com>
(as far as "domxml-from-native xen-xl" goes, I seem to have another issue
actually starting a domain on ARM, which I'll investigate...)
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH V2] Xen: support maxvcpus in xm and xl config
2015-12-16 10:11 ` [PATCH V2] Xen: support maxvcpus in xm and xl config Ian Campbell
@ 2015-12-16 11:09 ` Ian Campbell
0 siblings, 0 replies; 4+ messages in thread
From: Ian Campbell @ 2015-12-16 11:09 UTC (permalink / raw)
To: Jim Fehlig, libvir-list; +Cc: xen-devel
On Wed, 2015-12-16 at 10:11 +0000, Ian Campbell wrote:
> On Tue, 2015-12-15 at 15:20 -0700, Jim Fehlig wrote:
> > From: Ian Campbell <ian.campbell@citrix.com>
> >
> > xend prior to 4.0 understands vcpus as maxvcpus and vcpu_avail
> > as a bit map of which cpus are online (default is all).
> >
> > xend from 4.0 onwards understands maxvcpus as maxvcpus and
> > vcpus as the number which are online (from 0..N-1). The
> > upstream commit (68a94cf528e6 "xm: Add maxvcpus support")
> > claims that if maxvcpus is omitted then the old behaviour
> > (i.e. obeying vcpu_avail) is retained, but AFAICT it was not,
> > in this case vcpu==maxcpus==online cpus. This is good for us
> > because handling anything else would be fiddly.
> >
> > This patch changes parsing of the virDomainDef maxvcpus and vcpus
> > entries to use the corresponding 'maxvcpus' and 'vcpus' settings
> > from xm and xl config. It also drops use of the old Xen 3.x
> > 'vcpu_avail' setting.
> >
> > The change also removes the maxvcpus limit of MAX_VIRT_VCPUS (since
> > maxvcpus is simply a count, not a bit mask), which is particularly
> > crucial on ARM where MAX_VIRT_CPUS == 1 (since all guests are
> > expected to support vcpu placement, and therefore only the boot
> > vcpu's info lives in the shared info page).
> >
> > Existing tests adjusted accordingly, and new tests added for the
> > 'maxvcpus' setting.
> >
> > Signed-off-by: Jim Fehlig <jfehlig@suse.com>
>
> Acked-by: Ian Campbell <Ian.campbell@citrix.com>
> Tested-by: Ian Campbell <Ian.campbell@citrix.com>
> (as far as "domxml-from-native xen-xl" goes, I seem to have another issue
> actually starting a domain on ARM, which I'll investigate...)
Turned out to be a mismatch between my build-time and run-time libxl
versions, fixed by a clean rebuild. So that's a full Tested-by.
I noticed that the newer cmdline= (inplace of root=+extra= etc) wasn't
supported. I'll knock something up.
Ian.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [libvirt] [PATCH V2] Xen: support maxvcpus in xm and xl config
[not found] <1450218051-25842-1-git-send-email-jfehlig@suse.com>
2015-12-16 10:11 ` [PATCH V2] Xen: support maxvcpus in xm and xl config Ian Campbell
@ 2015-12-18 11:47 ` Michal Privoznik
[not found] ` <5673F25B.70606@redhat.com>
2 siblings, 0 replies; 4+ messages in thread
From: Michal Privoznik @ 2015-12-18 11:47 UTC (permalink / raw)
To: Jim Fehlig, libvir-list; +Cc: ian.campbell, xen-devel
On 15.12.2015 23:20, Jim Fehlig wrote:
> From: Ian Campbell <ian.campbell@citrix.com>
>
> xend prior to 4.0 understands vcpus as maxvcpus and vcpu_avail
> as a bit map of which cpus are online (default is all).
>
> xend from 4.0 onwards understands maxvcpus as maxvcpus and
> vcpus as the number which are online (from 0..N-1). The
> upstream commit (68a94cf528e6 "xm: Add maxvcpus support")
> claims that if maxvcpus is omitted then the old behaviour
> (i.e. obeying vcpu_avail) is retained, but AFAICT it was not,
> in this case vcpu==maxcpus==online cpus. This is good for us
> because handling anything else would be fiddly.
>
> This patch changes parsing of the virDomainDef maxvcpus and vcpus
> entries to use the corresponding 'maxvcpus' and 'vcpus' settings
> from xm and xl config. It also drops use of the old Xen 3.x
> 'vcpu_avail' setting.
>
> The change also removes the maxvcpus limit of MAX_VIRT_VCPUS (since
> maxvcpus is simply a count, not a bit mask), which is particularly
> crucial on ARM where MAX_VIRT_CPUS == 1 (since all guests are
> expected to support vcpu placement, and therefore only the boot
> vcpu's info lives in the shared info page).
>
> Existing tests adjusted accordingly, and new tests added for the
> 'maxvcpus' setting.
>
> Signed-off-by: Jim Fehlig <jfehlig@suse.com>
> ---
>
> This is a respin of Ian's patch [1] to support parsing/formatting of the
> 'maxvcpus' xm and xl config setting. It goes a bit further and removes
> the old Xen 3.x 'vcpu_avail' setting and lifts the maxvcpus limit of
> MAX_VIRT_CPUS entirely.
>
> The patch is based on my recent series [2] removing xend config version from
> the codebase.
>
> [1] https://www.redhat.com/archives/libvir-list/2015-November/msg01155.html
> [2] https://www.redhat.com/archives/libvir-list/2015-December/msg00616.html
>
>
> src/xenconfig/xen_common.c | 25 +++++++++++----------
> tests/xlconfigdata/test-paravirt-maxvcpus.cfg | 13 +++++++++++
> tests/xlconfigdata/test-paravirt-maxvcpus.xml | 28 ++++++++++++++++++++++++
> tests/xlconfigtest.c | 1 +
> tests/xmconfigdata/test-paravirt-maxvcpus.cfg | 13 +++++++++++
> tests/xmconfigdata/test-paravirt-maxvcpus.xml | 31 +++++++++++++++++++++++++++
> tests/xmconfigdata/test-paravirt-vcpu.cfg | 4 ++--
> tests/xmconfigtest.c | 1 +
> 8 files changed, 101 insertions(+), 15 deletions(-)
ACK
Michal
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [libvirt] [PATCH V2] Xen: support maxvcpus in xm and xl config
[not found] ` <5673F25B.70606@redhat.com>
@ 2015-12-19 1:25 ` Jim Fehlig
0 siblings, 0 replies; 4+ messages in thread
From: Jim Fehlig @ 2015-12-19 1:25 UTC (permalink / raw)
To: Michal Privoznik, libvir-list; +Cc: ian.campbell, xen-devel
On 12/18/2015 04:47 AM, Michal Privoznik wrote:
> On 15.12.2015 23:20, Jim Fehlig wrote:
>> From: Ian Campbell <ian.campbell@citrix.com>
>>
>> xend prior to 4.0 understands vcpus as maxvcpus and vcpu_avail
>> as a bit map of which cpus are online (default is all).
>>
>> xend from 4.0 onwards understands maxvcpus as maxvcpus and
>> vcpus as the number which are online (from 0..N-1). The
>> upstream commit (68a94cf528e6 "xm: Add maxvcpus support")
>> claims that if maxvcpus is omitted then the old behaviour
>> (i.e. obeying vcpu_avail) is retained, but AFAICT it was not,
>> in this case vcpu==maxcpus==online cpus. This is good for us
>> because handling anything else would be fiddly.
>>
>> This patch changes parsing of the virDomainDef maxvcpus and vcpus
>> entries to use the corresponding 'maxvcpus' and 'vcpus' settings
>> from xm and xl config. It also drops use of the old Xen 3.x
>> 'vcpu_avail' setting.
>>
>> The change also removes the maxvcpus limit of MAX_VIRT_VCPUS (since
>> maxvcpus is simply a count, not a bit mask), which is particularly
>> crucial on ARM where MAX_VIRT_CPUS == 1 (since all guests are
>> expected to support vcpu placement, and therefore only the boot
>> vcpu's info lives in the shared info page).
>>
>> Existing tests adjusted accordingly, and new tests added for the
>> 'maxvcpus' setting.
>>
>> Signed-off-by: Jim Fehlig <jfehlig@suse.com>
>> ---
>>
>> This is a respin of Ian's patch [1] to support parsing/formatting of the
>> 'maxvcpus' xm and xl config setting. It goes a bit further and removes
>> the old Xen 3.x 'vcpu_avail' setting and lifts the maxvcpus limit of
>> MAX_VIRT_CPUS entirely.
>>
>> The patch is based on my recent series [2] removing xend config version from
>> the codebase.
>>
>> [1] https://www.redhat.com/archives/libvir-list/2015-November/msg01155.html
>> [2] https://www.redhat.com/archives/libvir-list/2015-December/msg00616.html
>>
>>
>> src/xenconfig/xen_common.c | 25 +++++++++++----------
>> tests/xlconfigdata/test-paravirt-maxvcpus.cfg | 13 +++++++++++
>> tests/xlconfigdata/test-paravirt-maxvcpus.xml | 28 ++++++++++++++++++++++++
>> tests/xlconfigtest.c | 1 +
>> tests/xmconfigdata/test-paravirt-maxvcpus.cfg | 13 +++++++++++
>> tests/xmconfigdata/test-paravirt-maxvcpus.xml | 31 +++++++++++++++++++++++++++
>> tests/xmconfigdata/test-paravirt-vcpu.cfg | 4 ++--
>> tests/xmconfigtest.c | 1 +
>> 8 files changed, 101 insertions(+), 15 deletions(-)
> ACK
Thanks, pushed now.
Regards,
Jim
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-12-19 1:25 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1450218051-25842-1-git-send-email-jfehlig@suse.com>
2015-12-16 10:11 ` [PATCH V2] Xen: support maxvcpus in xm and xl config Ian Campbell
2015-12-16 11:09 ` Ian Campbell
2015-12-18 11:47 ` [libvirt] " Michal Privoznik
[not found] ` <5673F25B.70606@redhat.com>
2015-12-19 1:25 ` Jim Fehlig
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.