[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 11/12 v2] qmp: add cpu-set qmp command
From: |
Luiz Capitulino |
Subject: |
Re: [Qemu-devel] [PATCH 11/12 v2] qmp: add cpu-set qmp command |
Date: |
Mon, 25 Mar 2013 16:09:59 -0400 |
On Mon, 25 Mar 2013 16:35:11 +0100
Igor Mammedov <address@hidden> wrote:
> Signed-off-by: Igor Mammedov <address@hidden>
> ---
> v2:
> * s/cpu_set/cpu-set/
> * qmp doc style fix
> * use bool type instead of opencodding online/offline string
> suggested-by: Eric Blake <address@hidden>
> changes are on WIP branch: https://github.com/imammedo/qemu/tree/cpu_set.WIP
>
> ---
> include/sysemu/sysemu.h | 2 ++
> qapi-schema.json | 12 ++++++++++++
> qmp-commands.hx | 24 ++++++++++++++++++++++++
> qmp.c | 9 +++++++++
> stubs/Makefile.objs | 1 +
> stubs/do_cpu_hot_add.c | 7 +++++++
> 6 files changed, 55 insertions(+), 0 deletions(-)
> create mode 100644 stubs/do_cpu_hot_add.c
>
> diff --git a/include/sysemu/sysemu.h b/include/sysemu/sysemu.h
> index 4b8f721..8bcaf26 100644
> --- a/include/sysemu/sysemu.h
> +++ b/include/sysemu/sysemu.h
> @@ -156,6 +156,8 @@ void drive_hot_add(Monitor *mon, const QDict *qdict);
> void qemu_register_cpu_add_notifier(Notifier *notifier);
> void qemu_system_cpu_hotplug_request(uint32_t id);
>
> +void do_cpu_hot_add(const int64_t id, Error **errp);
> +
> /* pcie aer error injection */
> void pcie_aer_inject_error_print(Monitor *mon, const QObject *data);
> int do_pcie_aer_inject_error(Monitor *mon,
> diff --git a/qapi-schema.json b/qapi-schema.json
> index 088f4e1..aa5f3dc 100644
> --- a/qapi-schema.json
> +++ b/qapi-schema.json
> @@ -1385,6 +1385,18 @@
> { 'command': 'cpu', 'data': {'index': 'int'} }
>
> ##
> +# @cpu-set
> +#
> +# Sets specified cpu to online/offline mode
> +#
> +# @id: cpu id to be updated
> +#
> +# @online: true to put the cpu online, false to take it offline
> +#
> +##
> +{ 'command': 'cpu-set', 'data': {'id': 'int', 'online': 'bool'} }
> +
> +##
> # @memsave:
> #
> # Save a portion of guest memory to a file.
> diff --git a/qmp-commands.hx b/qmp-commands.hx
> index b370060..2f9c256 100644
> --- a/qmp-commands.hx
> +++ b/qmp-commands.hx
> @@ -385,6 +385,30 @@ Note: CPUs' indexes are obtained with the 'query-cpus'
> command.
> EQMP
>
> {
> + .name = "cpu-set",
> + .args_type = "id:i,online:b",
> + .mhandler.cmd_new = qmp_marshal_input_cpu_set,
> + },
> +
> +SQMP
> +cpu-set
> +-------
> +
> +Sets virtual cpu to online/ofline mode
> +
> +Arguments:
> +
> +- "id": cpu id (json-int)
> +- "online": true to put the cpu online, false to take it offline (json-bool)
> +
> +Example:
> +
> +-> { "execute": "cpu-set", "arguments": { "id": 2, "online": true } }
> +<- { "return": {} }
> +
> +EQMP
> +
> + {
> .name = "memsave",
> .args_type = "val:l,size:i,filename:s,cpu:i?",
> .mhandler.cmd_new = qmp_marshal_input_memsave,
> diff --git a/qmp.c b/qmp.c
> index 55b056b..c211da5 100644
> --- a/qmp.c
> +++ b/qmp.c
> @@ -108,6 +108,15 @@ void qmp_cpu(int64_t index, Error **errp)
> /* Just do nothing */
> }
>
> +void qmp_cpu_set(int64_t id, const bool online, Error **errp)
> +{
> + if (online) {
> + do_cpu_hot_add(id, errp);
> + } else {
> + error_setg(errp, "Unplug is not implemented");
> + }
> +}
As a general rule, we don't allow command extensions to be done this
way because this is not queriable. A client would have to try online=off
to see if QEMU version X supports it, worse: the client would have to
parse the error message to be sure the failure actually corresponds
to unplug not implemented.
The alternative is to have cpu-set-online and later cpu-set-offline. Quite
verbose, but doesn't have issues.
Eric, what do you think?
> +
> #ifndef CONFIG_VNC
> /* If VNC support is enabled, the "true" query-vnc command is
> defined in the VNC subsystem */
> diff --git a/stubs/Makefile.objs b/stubs/Makefile.objs
> index 6a492f5..4154a2b 100644
> --- a/stubs/Makefile.objs
> +++ b/stubs/Makefile.objs
> @@ -26,3 +26,4 @@ stub-obj-$(CONFIG_WIN32) += fd-register.o
> stub-obj-y += resume_vcpu.o
> stub-obj-y += get_icc_bus.o
> stub-obj-y += qemu_system_cpu_hotplug_request.o
> +stub-obj-y += do_cpu_hot_add.o
> diff --git a/stubs/do_cpu_hot_add.c b/stubs/do_cpu_hot_add.c
> new file mode 100644
> index 0000000..1f6d7a6
> --- /dev/null
> +++ b/stubs/do_cpu_hot_add.c
> @@ -0,0 +1,7 @@
> +#include "qapi/error.h"
> +#include "sysemu/sysemu.h"
> +
> +void do_cpu_hot_add(const int64_t id, Error **errp)
> +{
> + error_setg(errp, "Not implemented");
> +}
- [Qemu-devel] [PATCH 09/12] rtc: update rtc_cmos on CPU hot-plug, (continued)
- [Qemu-devel] [PATCH 09/12] rtc: update rtc_cmos on CPU hot-plug, Igor Mammedov, 2013/03/21
- [Qemu-devel] [PATCH 06/12] target-i386: replace FROM_SYSBUS() with QOM type cast, Igor Mammedov, 2013/03/21
- [Qemu-devel] [PATCH 08/12] introduce CPU hot-plug notifier, Igor Mammedov, 2013/03/21
- [Qemu-devel] [PATCH 10/12] acpi_piix4: add infrastructure to send CPU hot-plug GPE to guest, Igor Mammedov, 2013/03/21
- [Qemu-devel] [PATCH 11/12] qmp: add cpu_set qmp command, Igor Mammedov, 2013/03/21
- Re: [Qemu-devel] [PATCH 11/12] qmp: add cpu_set qmp command, Eric Blake, 2013/03/21
- [Qemu-devel] [PATCH 11/12 v2] qmp: add cpu-set qmp command, Igor Mammedov, 2013/03/25
- Re: [Qemu-devel] [PATCH 11/12 v2] qmp: add cpu-set qmp command,
Luiz Capitulino <=
- Re: [Qemu-devel] [PATCH 11/12 v2] qmp: add cpu-set qmp command, Eric Blake, 2013/03/25
- Re: [Qemu-devel] [PATCH 11/12 v2] qmp: add cpu-set qmp command, Igor Mammedov, 2013/03/26
- Re: [Qemu-devel] [PATCH 11/12 v2] qmp: add cpu-set qmp command, Luiz Capitulino, 2013/03/26
- Re: [Qemu-devel] [PATCH 11/12 v2] qmp: add cpu-set qmp command, Eric Blake, 2013/03/26
Re: [Qemu-devel] [PATCH 11/12] qmp: add cpu_set qmp command, Paolo Bonzini, 2013/03/27
[Qemu-devel] [PATCH 12/12] target-i386: implement CPU hot-add, Igor Mammedov, 2013/03/21
Re: [Qemu-devel] [RFC 00/12] target-i386: CPU hot-add with cpu_set QMP command, Eric Blake, 2013/03/21