On Tue, Sep 20, 2016 at 01:25:06PM -0500, Eric Blake wrote:
On 09/19/2016 02:42 PM, Eduardo Habkost wrote:
> Add a new optional field to query-cpu-definitions schema:
> "unavailable-features". It will contain a list of QOM properties
> that prevent the CPU model from running in the current host.
>
> Cc: David Hildenbrand <dahi(a)linux.vnet.ibm.com>
> Cc: Michael Mueller <mimu(a)linux.vnet.ibm.com>
> Cc: Christian Borntraeger <borntraeger(a)de.ibm.com>
> Cc: Cornelia Huck <cornelia.huck(a)de.ibm.com>
> Cc: Jiri Denemark <jdenemar(a)redhat.com>
> Cc: libvir-list(a)redhat.com
> Signed-off-by: Eduardo Habkost <ehabkost(a)redhat.com>
> ---
> +++ b/qapi-schema.json
> @@ -3111,10 +3111,31 @@
> # QEMU version, machine type, machine options and accelerator options.
> # A static model is always migration-safe. (since 2.8)
> #
> +# @unavailable-features: #optional List of properties that prevent
> +# the CPU model from running in the current
> +# host. (since 2.7)
We've missed 2.7; this should probably be 2.8. Otherwise reads okay to me;
Reviewed-by: Eric Blake <eblake(a)redhat.com>
Oops! I will fix it before merging. Thanks for spotting!
--
Eduardo