From: Gleb Natapov <gleb@redhat.com> To: Gerd Hoffmann <kraxel@redhat.com> Cc: xen-devel@lists.xensource.com, qemu-devel@nongnu.org Subject: Re: [Qemu-devel] [PATCH v3 1/6] suspend: add infrastructure Date: Thu, 9 Feb 2012 10:48:16 +0200 [thread overview] Message-ID: <20120209084816.GB18866@redhat.com> (raw) In-Reply-To: <1328698819-31269-2-git-send-email-kraxel@redhat.com> On Wed, Feb 08, 2012 at 12:00:14PM +0100, Gerd Hoffmann wrote: > This patch adds some infrastructure to handle suspend and resume to > qemu. First there are two functions to switch state and second there > is a suspend notifier: > > * qemu_system_suspend_request() is supposed to be called when the > guest asks for being be suspended, for example via ACPI. > > * qemu_system_wakeup_request is supposed to be called on events which > should wake up the guest. > qemu_system_wakeup_request() should get wakeup source as a parameter. There are ways to report it to a guest. > * qemu_register_suspend_notifier can be used to register a notifier > which will be called when the guest is suspended. Machine types > and device models can hook in there to modify state if needed. > > Signed-off-by: Gerd Hoffmann <kraxel@redhat.com> > --- > sysemu.h | 3 +++ > vl.c | 28 ++++++++++++++++++++++++++++ > 2 files changed, 31 insertions(+), 0 deletions(-) > > diff --git a/sysemu.h b/sysemu.h > index 9d5ce33..3b9d7f5 100644 > --- a/sysemu.h > +++ b/sysemu.h > @@ -39,6 +39,9 @@ void vm_stop(RunState state); > void vm_stop_force_state(RunState state); > > void qemu_system_reset_request(void); > +void qemu_system_suspend_request(void); > +void qemu_register_suspend_notifier(Notifier *notifier); > +void qemu_system_wakeup_request(void); > void qemu_system_shutdown_request(void); > void qemu_system_powerdown_request(void); > void qemu_system_debug_request(void); > diff --git a/vl.c b/vl.c > index 63dd725..822fd58 100644 > --- a/vl.c > +++ b/vl.c > @@ -1283,6 +1283,9 @@ static int shutdown_requested, shutdown_signal = -1; > static pid_t shutdown_pid; > static int powerdown_requested; > static int debug_requested; > +static bool is_suspended; > +static NotifierList suspend_notifiers = > + NOTIFIER_LIST_INITIALIZER(suspend_notifiers); > static RunState vmstop_requested = RUN_STATE_MAX; > > int qemu_shutdown_requested_get(void) > @@ -1398,6 +1401,31 @@ void qemu_system_reset_request(void) > qemu_notify_event(); > } > > +void qemu_system_suspend_request(void) > +{ > + if (is_suspended) { > + return; > + } > + cpu_stop_current(); > + notifier_list_notify(&suspend_notifiers, NULL); > + is_suspended = true; > +} > + > +void qemu_register_suspend_notifier(Notifier *notifier) > +{ > + notifier_list_add(&suspend_notifiers, notifier); > +} > + > +void qemu_system_wakeup_request(void) > +{ > + if (!is_suspended) { > + return; > + } > + reset_requested = 1; > + qemu_notify_event(); > + is_suspended = false; > +} > + > void qemu_system_killed(int signal, pid_t pid) > { > shutdown_signal = signal; > -- > 1.7.1 > > -- Gleb.
WARNING: multiple messages have this Message-ID (diff)
From: Gleb Natapov <gleb@redhat.com> To: Gerd Hoffmann <kraxel@redhat.com> Cc: xen-devel@lists.xensource.com, qemu-devel@nongnu.org Subject: Re: [PATCH v3 1/6] suspend: add infrastructure Date: Thu, 9 Feb 2012 10:48:16 +0200 [thread overview] Message-ID: <20120209084816.GB18866@redhat.com> (raw) In-Reply-To: <1328698819-31269-2-git-send-email-kraxel@redhat.com> On Wed, Feb 08, 2012 at 12:00:14PM +0100, Gerd Hoffmann wrote: > This patch adds some infrastructure to handle suspend and resume to > qemu. First there are two functions to switch state and second there > is a suspend notifier: > > * qemu_system_suspend_request() is supposed to be called when the > guest asks for being be suspended, for example via ACPI. > > * qemu_system_wakeup_request is supposed to be called on events which > should wake up the guest. > qemu_system_wakeup_request() should get wakeup source as a parameter. There are ways to report it to a guest. > * qemu_register_suspend_notifier can be used to register a notifier > which will be called when the guest is suspended. Machine types > and device models can hook in there to modify state if needed. > > Signed-off-by: Gerd Hoffmann <kraxel@redhat.com> > --- > sysemu.h | 3 +++ > vl.c | 28 ++++++++++++++++++++++++++++ > 2 files changed, 31 insertions(+), 0 deletions(-) > > diff --git a/sysemu.h b/sysemu.h > index 9d5ce33..3b9d7f5 100644 > --- a/sysemu.h > +++ b/sysemu.h > @@ -39,6 +39,9 @@ void vm_stop(RunState state); > void vm_stop_force_state(RunState state); > > void qemu_system_reset_request(void); > +void qemu_system_suspend_request(void); > +void qemu_register_suspend_notifier(Notifier *notifier); > +void qemu_system_wakeup_request(void); > void qemu_system_shutdown_request(void); > void qemu_system_powerdown_request(void); > void qemu_system_debug_request(void); > diff --git a/vl.c b/vl.c > index 63dd725..822fd58 100644 > --- a/vl.c > +++ b/vl.c > @@ -1283,6 +1283,9 @@ static int shutdown_requested, shutdown_signal = -1; > static pid_t shutdown_pid; > static int powerdown_requested; > static int debug_requested; > +static bool is_suspended; > +static NotifierList suspend_notifiers = > + NOTIFIER_LIST_INITIALIZER(suspend_notifiers); > static RunState vmstop_requested = RUN_STATE_MAX; > > int qemu_shutdown_requested_get(void) > @@ -1398,6 +1401,31 @@ void qemu_system_reset_request(void) > qemu_notify_event(); > } > > +void qemu_system_suspend_request(void) > +{ > + if (is_suspended) { > + return; > + } > + cpu_stop_current(); > + notifier_list_notify(&suspend_notifiers, NULL); > + is_suspended = true; > +} > + > +void qemu_register_suspend_notifier(Notifier *notifier) > +{ > + notifier_list_add(&suspend_notifiers, notifier); > +} > + > +void qemu_system_wakeup_request(void) > +{ > + if (!is_suspended) { > + return; > + } > + reset_requested = 1; > + qemu_notify_event(); > + is_suspended = false; > +} > + > void qemu_system_killed(int signal, pid_t pid) > { > shutdown_signal = signal; > -- > 1.7.1 > > -- Gleb.
next prev parent reply other threads:[~2012-02-09 8:48 UTC|newest] Thread overview: 58+ messages / expand[flat|nested] mbox.gz Atom feed top 2012-02-08 11:00 [Qemu-devel] [PATCH v3 0/6] initial suspend support Gerd Hoffmann 2012-02-08 11:00 ` Gerd Hoffmann 2012-02-08 11:00 ` [Qemu-devel] [PATCH v3 1/6] suspend: add infrastructure Gerd Hoffmann 2012-02-08 11:00 ` Gerd Hoffmann 2012-02-09 8:48 ` Gleb Natapov [this message] 2012-02-09 8:48 ` Gleb Natapov 2012-02-09 10:45 ` [Qemu-devel] " Gerd Hoffmann 2012-02-09 10:45 ` Gerd Hoffmann 2012-02-09 11:19 ` [Qemu-devel] " Gleb Natapov 2012-02-09 11:19 ` Gleb Natapov 2012-02-09 12:02 ` [Qemu-devel] " Gerd Hoffmann 2012-02-09 12:02 ` Gerd Hoffmann 2012-02-09 12:37 ` [Qemu-devel] " Gleb Natapov 2012-02-09 12:37 ` Gleb Natapov 2012-02-09 12:46 ` [Qemu-devel] " Gerd Hoffmann 2012-02-09 12:46 ` Gerd Hoffmann 2012-02-09 13:17 ` [Qemu-devel] " Gleb Natapov 2012-02-09 13:17 ` Gleb Natapov 2012-02-09 13:29 ` [Qemu-devel] " Gerd Hoffmann 2012-02-09 13:29 ` Gerd Hoffmann 2012-02-09 16:00 ` [Qemu-devel] " Gerd Hoffmann 2012-02-09 16:00 ` Gerd Hoffmann 2012-02-09 16:05 ` [Qemu-devel] " Gleb Natapov 2012-02-09 16:05 ` Gleb Natapov 2012-02-08 11:00 ` [Qemu-devel] [PATCH v3 2/6] suspend: switch acpi s3 to new infrastructure Gerd Hoffmann 2012-02-08 11:00 ` Gerd Hoffmann 2012-02-09 8:53 ` [Qemu-devel] " Gleb Natapov 2012-02-09 8:53 ` Gleb Natapov 2012-02-09 10:51 ` [Qemu-devel] " Gerd Hoffmann 2012-02-09 10:51 ` Gerd Hoffmann 2012-02-09 11:14 ` [Qemu-devel] " Gleb Natapov 2012-02-09 11:14 ` Gleb Natapov 2012-02-09 11:13 ` [Qemu-devel] [Xen-devel] " Stefano Stabellini 2012-02-09 11:13 ` Stefano Stabellini 2012-02-09 11:17 ` [Qemu-devel] " Paolo Bonzini 2012-02-09 11:17 ` Paolo Bonzini 2012-02-09 12:31 ` [Qemu-devel] " Gleb Natapov 2012-02-09 12:31 ` Gleb Natapov 2012-02-09 12:47 ` [Qemu-devel] " Paolo Bonzini 2012-02-09 12:47 ` Paolo Bonzini 2012-02-09 12:53 ` [Qemu-devel] " Gleb Natapov 2012-02-09 12:53 ` Gleb Natapov 2012-02-08 11:00 ` [Qemu-devel] [PATCH v3 3/6] suspend: add system_wakeup monitor command Gerd Hoffmann 2012-02-08 11:00 ` Gerd Hoffmann 2012-02-09 8:54 ` [Qemu-devel] " Gleb Natapov 2012-02-09 8:54 ` Gleb Natapov 2012-02-08 11:00 ` [Qemu-devel] [PATCH v3 4/6] suspend: make ps/2 devices wakeup the guest Gerd Hoffmann 2012-02-08 11:00 ` Gerd Hoffmann 2012-02-08 11:00 ` [Qemu-devel] [PATCH v3 5/6] suspend: make serial ports " Gerd Hoffmann 2012-02-08 11:00 ` Gerd Hoffmann 2012-02-08 11:00 ` [Qemu-devel] [PATCH v3 6/6] suspend: make rtc alarm " Gerd Hoffmann 2012-02-08 11:00 ` Gerd Hoffmann 2012-02-09 8:56 ` [Qemu-devel] " Gleb Natapov 2012-02-09 8:56 ` Gleb Natapov 2012-02-09 12:00 ` [Qemu-devel] " Gerd Hoffmann 2012-02-09 12:00 ` Gerd Hoffmann 2012-02-09 12:24 ` [Qemu-devel] " Gleb Natapov 2012-02-09 12:24 ` Gleb Natapov
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20120209084816.GB18866@redhat.com \ --to=gleb@redhat.com \ --cc=kraxel@redhat.com \ --cc=qemu-devel@nongnu.org \ --cc=xen-devel@lists.xensource.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.