* [Qemu-devel] [PATCH v2 0/1] Export machine type deprecation info through QMP @ 2019-06-08 23:34 Eduardo Habkost 2019-06-08 23:34 ` [Qemu-devel] [PATCH v2 1/1] qmp: Add deprecation information to query-machines Eduardo Habkost 2019-06-24 18:03 ` [Qemu-devel] [PATCH v2 0/1] Export machine type deprecation info through QMP Eduardo Habkost 0 siblings, 2 replies; 6+ messages in thread From: Eduardo Habkost @ 2019-06-08 23:34 UTC (permalink / raw) To: qemu-devel Cc: Thomas Huth, Daniel P. Berrange, Markus Armbruster, Wainer dos Santos Moschetta, Michal Privoznik, Paolo Bonzini, Philippe Mathieu-Daudé Changes v1 -> v2: * I've decided to get rid of the status-message and suggested-alternative fields, to avoid more bikeshedding. This series adds machine type deprecation information to the output of the `query-machines` QMP command. With this, libvirt and management software will be able to show this information to users and/or suggest changes to VM configuration to avoid deprecated machine types. Eduardo Habkost (1): qmp: Add deprecation information to query-machines qapi/misc.json | 7 ++++++- vl.c | 1 + 2 files changed, 7 insertions(+), 1 deletion(-) -- 2.18.0.rc1.1.g3f1ff2140 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Qemu-devel] [PATCH v2 1/1] qmp: Add deprecation information to query-machines 2019-06-08 23:34 [Qemu-devel] [PATCH v2 0/1] Export machine type deprecation info through QMP Eduardo Habkost @ 2019-06-08 23:34 ` Eduardo Habkost 2019-06-25 8:37 ` Marcel Apfelbaum 2019-06-24 18:03 ` [Qemu-devel] [PATCH v2 0/1] Export machine type deprecation info through QMP Eduardo Habkost 1 sibling, 1 reply; 6+ messages in thread From: Eduardo Habkost @ 2019-06-08 23:34 UTC (permalink / raw) To: qemu-devel Cc: Thomas Huth, Daniel P. Berrange, Markus Armbruster, Wainer dos Santos Moschetta, Michal Privoznik, Paolo Bonzini, Philippe Mathieu-Daudé Export machine type deprecation status through the query-machines QMP command. With this, libvirt and management software will be able to show this information to users and/or suggest changes to VM configuration to avoid deprecated machines. Signed-off-by: Eduardo Habkost <ehabkost@redhat.com> --- Changes v1 -> v2: * Take a much simpler approach: add a simple `deprecated` bool field, instead of including a human-readable message and suggested alternatives --- qapi/misc.json | 7 ++++++- vl.c | 1 + 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/qapi/misc.json b/qapi/misc.json index 8b3ca4fdd3..fca793d39c 100644 --- a/qapi/misc.json +++ b/qapi/misc.json @@ -2018,12 +2018,17 @@ # # @hotpluggable-cpus: cpu hotplug via -device is supported (since 2.7.0) # +# @deprecated: if true, the machine type is deprecated and may be removed +# in future versions of QEMU according to the QEMU deprecation +# policy (since 4.1.0) +# # Since: 1.2.0 ## { 'struct': 'MachineInfo', 'data': { 'name': 'str', '*alias': 'str', '*is-default': 'bool', 'cpu-max': 'int', - 'hotpluggable-cpus': 'bool'} } + 'hotpluggable-cpus': 'bool', + 'deprecated': 'bool' } } ## # @query-machines: diff --git a/vl.c b/vl.c index cd1fbc4cdc..f825d2159c 100644 --- a/vl.c +++ b/vl.c @@ -1428,6 +1428,7 @@ MachineInfoList *qmp_query_machines(Error **errp) info->name = g_strdup(mc->name); info->cpu_max = !mc->max_cpus ? 1 : mc->max_cpus; info->hotpluggable_cpus = mc->has_hotpluggable_cpus; + info->deprecated = !!mc->deprecation_reason; entry = g_malloc0(sizeof(*entry)); entry->value = info; -- 2.18.0.rc1.1.g3f1ff2140 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH v2 1/1] qmp: Add deprecation information to query-machines 2019-06-08 23:34 ` [Qemu-devel] [PATCH v2 1/1] qmp: Add deprecation information to query-machines Eduardo Habkost @ 2019-06-25 8:37 ` Marcel Apfelbaum 0 siblings, 0 replies; 6+ messages in thread From: Marcel Apfelbaum @ 2019-06-25 8:37 UTC (permalink / raw) To: Eduardo Habkost, qemu-devel Cc: Thomas Huth, Daniel P. Berrange, Philippe Mathieu-Daudé, Markus Armbruster, Wainer dos Santos Moschetta, Michal Privoznik, Paolo Bonzini On 6/9/19 2:34 AM, Eduardo Habkost wrote: > Export machine type deprecation status through the query-machines > QMP command. With this, libvirt and management software will be > able to show this information to users and/or suggest changes to > VM configuration to avoid deprecated machines. > > Signed-off-by: Eduardo Habkost <ehabkost@redhat.com> > --- > Changes v1 -> v2: > * Take a much simpler approach: add a simple `deprecated` bool > field, instead of including a human-readable message and > suggested alternatives > --- > qapi/misc.json | 7 ++++++- > vl.c | 1 + > 2 files changed, 7 insertions(+), 1 deletion(-) > > diff --git a/qapi/misc.json b/qapi/misc.json > index 8b3ca4fdd3..fca793d39c 100644 > --- a/qapi/misc.json > +++ b/qapi/misc.json > @@ -2018,12 +2018,17 @@ > # > # @hotpluggable-cpus: cpu hotplug via -device is supported (since 2.7.0) > # > +# @deprecated: if true, the machine type is deprecated and may be removed > +# in future versions of QEMU according to the QEMU deprecation > +# policy (since 4.1.0) > +# > # Since: 1.2.0 > ## > { 'struct': 'MachineInfo', > 'data': { 'name': 'str', '*alias': 'str', > '*is-default': 'bool', 'cpu-max': 'int', > - 'hotpluggable-cpus': 'bool'} } > + 'hotpluggable-cpus': 'bool', > + 'deprecated': 'bool' } } > > ## > # @query-machines: > diff --git a/vl.c b/vl.c > index cd1fbc4cdc..f825d2159c 100644 > --- a/vl.c > +++ b/vl.c > @@ -1428,6 +1428,7 @@ MachineInfoList *qmp_query_machines(Error **errp) > info->name = g_strdup(mc->name); > info->cpu_max = !mc->max_cpus ? 1 : mc->max_cpus; > info->hotpluggable_cpus = mc->has_hotpluggable_cpus; > + info->deprecated = !!mc->deprecation_reason; > > entry = g_malloc0(sizeof(*entry)); > entry->value = info; Reviewed-by: Marcel Apfelbaum <marcel.apfelbaum@gmail.com> Thanks, Marcel ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH v2 0/1] Export machine type deprecation info through QMP 2019-06-08 23:34 [Qemu-devel] [PATCH v2 0/1] Export machine type deprecation info through QMP Eduardo Habkost 2019-06-08 23:34 ` [Qemu-devel] [PATCH v2 1/1] qmp: Add deprecation information to query-machines Eduardo Habkost @ 2019-06-24 18:03 ` Eduardo Habkost 2019-06-24 22:48 ` Wainer dos Santos Moschetta 2019-06-25 12:00 ` Markus Armbruster 1 sibling, 2 replies; 6+ messages in thread From: Eduardo Habkost @ 2019-06-24 18:03 UTC (permalink / raw) To: qemu-devel Cc: Thomas Huth, Daniel P. Berrange, Markus Armbruster, Wainer dos Santos Moschetta, Michal Privoznik, Paolo Bonzini, Philippe Mathieu-Daudé Any objections to this? I'm planning to merge it this week. On Sat, Jun 08, 2019 at 08:34:46PM -0300, Eduardo Habkost wrote: > Changes v1 -> v2: > * I've decided to get rid of the status-message and > suggested-alternative fields, to avoid more bikeshedding. > > This series adds machine type deprecation information to the > output of the `query-machines` QMP command. With this, libvirt > and management software will be able to show this information to > users and/or suggest changes to VM configuration to avoid > deprecated machine types. > > Eduardo Habkost (1): > qmp: Add deprecation information to query-machines > > qapi/misc.json | 7 ++++++- > vl.c | 1 + > 2 files changed, 7 insertions(+), 1 deletion(-) > > -- > 2.18.0.rc1.1.g3f1ff2140 > -- Eduardo ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH v2 0/1] Export machine type deprecation info through QMP 2019-06-24 18:03 ` [Qemu-devel] [PATCH v2 0/1] Export machine type deprecation info through QMP Eduardo Habkost @ 2019-06-24 22:48 ` Wainer dos Santos Moschetta 2019-06-25 12:00 ` Markus Armbruster 1 sibling, 0 replies; 6+ messages in thread From: Wainer dos Santos Moschetta @ 2019-06-24 22:48 UTC (permalink / raw) To: Eduardo Habkost, qemu-devel Cc: Thomas Huth, Daniel P. Berrange, Michal Privoznik, Markus Armbruster, Paolo Bonzini, Philippe Mathieu-Daudé On 06/24/2019 03:03 PM, Eduardo Habkost wrote: > Any objections to this? I'm planning to merge it this week. IMHO, 1+. So I don't have objections. - Wainer > > On Sat, Jun 08, 2019 at 08:34:46PM -0300, Eduardo Habkost wrote: >> Changes v1 -> v2: >> * I've decided to get rid of the status-message and >> suggested-alternative fields, to avoid more bikeshedding. >> >> This series adds machine type deprecation information to the >> output of the `query-machines` QMP command. With this, libvirt >> and management software will be able to show this information to >> users and/or suggest changes to VM configuration to avoid >> deprecated machine types. >> >> Eduardo Habkost (1): >> qmp: Add deprecation information to query-machines >> >> qapi/misc.json | 7 ++++++- >> vl.c | 1 + >> 2 files changed, 7 insertions(+), 1 deletion(-) >> >> -- >> 2.18.0.rc1.1.g3f1ff2140 >> ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH v2 0/1] Export machine type deprecation info through QMP 2019-06-24 18:03 ` [Qemu-devel] [PATCH v2 0/1] Export machine type deprecation info through QMP Eduardo Habkost 2019-06-24 22:48 ` Wainer dos Santos Moschetta @ 2019-06-25 12:00 ` Markus Armbruster 1 sibling, 0 replies; 6+ messages in thread From: Markus Armbruster @ 2019-06-25 12:00 UTC (permalink / raw) To: Eduardo Habkost Cc: Thomas Huth, Daniel P. Berrange, Michal Privoznik, qemu-devel, Wainer dos Santos Moschetta, Paolo Bonzini, Philippe Mathieu-Daudé Eduardo Habkost <ehabkost@redhat.com> writes: > Any objections to this? I'm planning to merge it this week. No objection, in fact: Reviewed-by: Markus Armbruster <armbru@redhat.com> The QAPI feature flags work, once finished, might render this redundant, but I'd rather not have my unfinished solution for a wider problem block your working solution for a narrower problem. ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-06-25 12:01 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-06-08 23:34 [Qemu-devel] [PATCH v2 0/1] Export machine type deprecation info through QMP Eduardo Habkost 2019-06-08 23:34 ` [Qemu-devel] [PATCH v2 1/1] qmp: Add deprecation information to query-machines Eduardo Habkost 2019-06-25 8:37 ` Marcel Apfelbaum 2019-06-24 18:03 ` [Qemu-devel] [PATCH v2 0/1] Export machine type deprecation info through QMP Eduardo Habkost 2019-06-24 22:48 ` Wainer dos Santos Moschetta 2019-06-25 12:00 ` Markus Armbruster
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).