[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 07/12] suspend: add system_wakeup monitor com
From: |
Luiz Capitulino |
Subject: |
Re: [Qemu-devel] [PATCH v5 07/12] suspend: add system_wakeup monitor command |
Date: |
Fri, 17 Feb 2012 15:32:21 -0200 |
On Wed, 15 Feb 2012 11:28:16 +0100
Gerd Hoffmann <address@hidden> wrote:
> This patch adds the system_wakeup monitor command which will simply
> wake up suspended guests.
>
> Signed-off-by: Gerd Hoffmann <address@hidden>
> ---
> hmp-commands.hx | 14 ++++++++++++++
> hmp.c | 5 +++++
> hmp.h | 1 +
> qapi-schema.json | 11 +++++++++++
> qmp-commands.hx | 21 +++++++++++++++++++++
> qmp.c | 5 +++++
> 6 files changed, 57 insertions(+), 0 deletions(-)
>
> diff --git a/hmp-commands.hx b/hmp-commands.hx
> index 573b823..64b3656 100644
> --- a/hmp-commands.hx
> +++ b/hmp-commands.hx
> @@ -352,6 +352,20 @@ Resume emulation.
> ETEXI
>
> {
> + .name = "system_wakeup",
> + .args_type = "",
> + .params = "",
> + .help = "wakeup guest from suspend",
> + .mhandler.cmd = hmp_system_wakeup,
> + },
> +
> +STEXI
> address@hidden system_wakeup
> address@hidden system_wakeup
> +Wakeup guest from suspend.
> +ETEXI
> +
> + {
> .name = "gdbserver",
> .args_type = "device:s?",
> .params = "[device]",
> diff --git a/hmp.c b/hmp.c
> index 8ff8c94..3a54455 100644
> --- a/hmp.c
> +++ b/hmp.c
> @@ -632,6 +632,11 @@ void hmp_cont(Monitor *mon, const QDict *qdict)
> }
> }
>
> +void hmp_system_wakeup(Monitor *mon, const QDict *qdict)
> +{
> + qmp_system_wakeup(NULL);
> +}
> +
> void hmp_inject_nmi(Monitor *mon, const QDict *qdict)
> {
> Error *errp = NULL;
> diff --git a/hmp.h b/hmp.h
> index 18eecbd..5409464 100644
> --- a/hmp.h
> +++ b/hmp.h
> @@ -41,6 +41,7 @@ void hmp_cpu(Monitor *mon, const QDict *qdict);
> void hmp_memsave(Monitor *mon, const QDict *qdict);
> void hmp_pmemsave(Monitor *mon, const QDict *qdict);
> void hmp_cont(Monitor *mon, const QDict *qdict);
> +void hmp_system_wakeup(Monitor *mon, const QDict *qdict);
> void hmp_inject_nmi(Monitor *mon, const QDict *qdict);
> void hmp_set_link(Monitor *mon, const QDict *qdict);
> void hmp_block_passwd(Monitor *mon, const QDict *qdict);
> diff --git a/qapi-schema.json b/qapi-schema.json
> index d02ee86..226c1da 100644
> --- a/qapi-schema.json
> +++ b/qapi-schema.json
> @@ -999,6 +999,17 @@
> { 'command': 'cont' }
>
> ##
> +# @system_wakeup:
> +#
> +# Wakeup guest from suspend
> +#
> +# Since: 1.1
> +#
> +# Returns: nothing.
Would be nice to note that this command does nothing if the guest is already
suspended (btw, does is_suspend account for guest initiated suspends too?).
> +##
> +{ 'command': 'system_wakeup' }
> +
> +##
> # @inject-nmi:
> #
> # Injects an Non-Maskable Interrupt into all guest's VCPUs.
> diff --git a/qmp-commands.hx b/qmp-commands.hx
> index b5e2ab8..f5081e2 100644
> --- a/qmp-commands.hx
> +++ b/qmp-commands.hx
> @@ -212,6 +212,27 @@ Example:
> EQMP
>
> {
> + .name = "system_wakeup",
> + .args_type = "",
> + .mhandler.cmd_new = qmp_marshal_input_system_wakeup,
> + },
> +
> +SQMP
> +system_wakeup
> +-------------
> +
> +Wakeup guest from suspend.
> +
> +Arguments: None.
> +
> +Example:
> +
> +-> { "execute": "system_wakeup" }
> +<- { "return": {} }
> +
> +EQMP
> +
> + {
> .name = "system_reset",
> .args_type = "",
> .mhandler.cmd_new = qmp_marshal_input_system_reset,
> diff --git a/qmp.c b/qmp.c
> index 1f64844..a182b51 100644
> --- a/qmp.c
> +++ b/qmp.c
> @@ -163,6 +163,11 @@ void qmp_cont(Error **errp)
> vm_start();
> }
>
> +void qmp_system_wakeup(Error **errp)
> +{
> + qemu_system_wakeup_request(QEMU_WAKEUP_REASON_OTHER);
qemu_system_wakeup_request() does:
if (!(wakeup_reason_mask & (1 << reason))) {
return;
}
But I think you never disable REASON_OTHER, meaning that this will
never be true when the wakeup request comes from the command, right?
If that's right, this patch is fine with me. Otherwise we need to
return an error.
> +}
> +
> ObjectPropertyInfoList *qmp_qom_list(const char *path, Error **errp)
> {
> Object *obj;
- [Qemu-devel] [PATCH v5 00/12] initial suspend support, Gerd Hoffmann, 2012/02/15
- [Qemu-devel] [PATCH v5 04/12] acpi: add acpi_pm1_evt_write_en, Gerd Hoffmann, 2012/02/15
- [Qemu-devel] [PATCH v5 08/12] suspend: make ps/2 devices wakeup the guest, Gerd Hoffmann, 2012/02/15
- [Qemu-devel] [PATCH v5 01/12] acpi: move around structs, Gerd Hoffmann, 2012/02/15
- [Qemu-devel] [PATCH v5 03/12] acpi: don't pass overflow_time to acpi_pm1_evt_get_sts, Gerd Hoffmann, 2012/02/15
- [Qemu-devel] [PATCH v5 11/12] suspend: make acpi timer wakeup the guest., Gerd Hoffmann, 2012/02/15
- [Qemu-devel] [PATCH v5 07/12] suspend: add system_wakeup monitor command, Gerd Hoffmann, 2012/02/15
- Re: [Qemu-devel] [PATCH v5 07/12] suspend: add system_wakeup monitor command,
Luiz Capitulino <=
- [Qemu-devel] [PATCH v5 02/12] acpi: add ACPIREGS, Gerd Hoffmann, 2012/02/15
- [Qemu-devel] [PATCH v5 09/12] suspend: make serial ports wakeup the guest., Gerd Hoffmann, 2012/02/15
- [Qemu-devel] [PATCH v5 10/12] suspend: make rtc alarm wakeup the guest., Gerd Hoffmann, 2012/02/15
- [Qemu-devel] [PATCH v5 12/12] suspend: add qmp events, Gerd Hoffmann, 2012/02/15