[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/3] qapi: Report support for -device cpu hotplu
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 1/3] qapi: Report support for -device cpu hotplug in query-machines |
Date: |
Thu, 23 Jun 2016 21:49:25 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 06/23/2016 08:56 PM, David Gibson wrote:
> On Thu, 23 Jun 2016 22:23:23 +0200
> Peter Krempa <address@hidden> wrote:
>
>> For management apps it's very useful to know whether the selected
>> machine type supports cpu hotplug via the new -device approach. Using
>> the presence of 'query-hotpluggable-cpus' is enough for a withess.
>>
>
> I'd been under the impression that there was a general way of detecting
> the availability of a particular qmp command. Was I mistaken?
You are correct - query-commands says whether 'query-hotpluggable-cpus'
exists as a command. But that is insufficient. See my review, or the
v2 patch, where the above poor wording was corrected to say what was
really meant: knowing whether query-hotpluggable-cpus exists is
insufficient to tell you whether a given cpu type can be hotplugged. So
adding one more piece of witness (for every type of cpu supported, we
also advertise if it is hotpluggable) is enough for libvirt to
efficiently take advantage of the new query-hotpluggable-cpus command.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH 0/3] qapi: Fix up cpu hotplug property names and add witness for cpu hotplug support, Peter Krempa, 2016/06/23
- [Qemu-devel] [PATCH 1/3] qapi: Report support for -device cpu hotplug in query-machines, Peter Krempa, 2016/06/23
- Re: [Qemu-devel] [PATCH 1/3] qapi: Report support for -device cpu hotplug in query-machines, Eric Blake, 2016/06/23
- Re: [Qemu-devel] [PATCH 1/3] qapi: Report support for -device cpu hotplug in query-machines, David Gibson, 2016/06/23
- Re: [Qemu-devel] [PATCH 1/3] qapi: Report support for -device cpu hotplug in query-machines,
Eric Blake <=
- Re: [Qemu-devel] [PATCH 1/3] qapi: Report support for -device cpu hotplug in query-machines, David Gibson, 2016/06/24
- Re: [Qemu-devel] [PATCH 1/3] qapi: Report support for -device cpu hotplug in query-machines, Igor Mammedov, 2016/06/24
- Re: [Qemu-devel] [PATCH 1/3] qapi: Report support for -device cpu hotplug in query-machines, Peter Krempa, 2016/06/24
- Re: [Qemu-devel] [PATCH 1/3] qapi: Report support for -device cpu hotplug in query-machines, David Gibson, 2016/06/24
- Re: [Qemu-devel] [PATCH 1/3] qapi: Report support for -device cpu hotplug in query-machines, Peter Krempa, 2016/06/24
- Re: [Qemu-devel] [PATCH 1/3] qapi: Report support for -device cpu hotplug in query-machines, David Gibson, 2016/06/26
[Qemu-devel] [PATCH 2/3] [VARIANT 1] qapi: Change 'core' to 'core-id' in 'CpuInstanceProperties', Peter Krempa, 2016/06/23
[Qemu-devel] [PATCH 3/3] [VARIANT 2] qapi: Change 'core-id' to 'core' in 'struct CPUCore', Peter Krempa, 2016/06/23