aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/xen/manage.c
diff options
context:
space:
mode:
authorIan Campbell <ian.campbell@citrix.com>2011-02-17 11:04:20 +0000
committerStefano Stabellini <stefano.stabellini@eu.citrix.com>2011-02-25 16:43:11 +0000
commitceb180294790c8a6a437533488616f6b591b49d0 (patch)
tree6520c8f2a4a83ed36998ac5d92ea9f97250af2ec /drivers/xen/manage.c
parentxen: suspend: use HYPERVISOR_suspend for PVHVM case instead of open coding (diff)
downloadlinux-dev-ceb180294790c8a6a437533488616f6b591b49d0.tar.xz
linux-dev-ceb180294790c8a6a437533488616f6b591b49d0.zip
xen: suspend: refactor cancellation flag into a structure
Will add extra fields in subsequent patches. Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Reviewed-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Diffstat (limited to 'drivers/xen/manage.c')
-rw-r--r--drivers/xen/manage.c32
1 files changed, 19 insertions, 13 deletions
diff --git a/drivers/xen/manage.c b/drivers/xen/manage.c
index 4dd01865ad18..5c0184fb9d84 100644
--- a/drivers/xen/manage.c
+++ b/drivers/xen/manage.c
@@ -34,11 +34,15 @@ enum shutdown_state {
/* Ignore multiple shutdown requests. */
static enum shutdown_state shutting_down = SHUTDOWN_INVALID;
+struct suspend_info {
+ int cancelled;
+};
+
#ifdef CONFIG_PM_SLEEP
static int xen_hvm_suspend(void *data)
{
+ struct suspend_info *si = data;
int err;
- int *cancelled = data;
BUG_ON(!irqs_disabled());
@@ -54,12 +58,12 @@ static int xen_hvm_suspend(void *data)
* or the domain was merely checkpointed, and 0 if it
* is resuming in a new domain.
*/
- *cancelled = HYPERVISOR_suspend(0UL);
+ si->cancelled = HYPERVISOR_suspend(0UL);
- xen_hvm_post_suspend(*cancelled);
+ xen_hvm_post_suspend(si->cancelled);
gnttab_resume();
- if (!*cancelled) {
+ if (!si->cancelled) {
xen_irq_resume();
xen_console_resume();
xen_timer_resume();
@@ -72,8 +76,8 @@ static int xen_hvm_suspend(void *data)
static int xen_suspend(void *data)
{
+ struct suspend_info *si = data;
int err;
- int *cancelled = data;
BUG_ON(!irqs_disabled());
@@ -93,13 +97,13 @@ static int xen_suspend(void *data)
* or the domain was merely checkpointed, and 0 if it
* is resuming in a new domain.
*/
- *cancelled = HYPERVISOR_suspend(virt_to_mfn(xen_start_info));
+ si->cancelled = HYPERVISOR_suspend(virt_to_mfn(xen_start_info));
- xen_post_suspend(*cancelled);
+ xen_post_suspend(si->cancelled);
gnttab_resume();
xen_mm_unpin_all();
- if (!*cancelled) {
+ if (!si->cancelled) {
xen_irq_resume();
xen_console_resume();
xen_timer_resume();
@@ -113,7 +117,7 @@ static int xen_suspend(void *data)
static void do_suspend(void)
{
int err;
- int cancelled = 1;
+ struct suspend_info si;
shutting_down = SHUTDOWN_SUSPEND;
@@ -143,20 +147,22 @@ static void do_suspend(void)
goto out_resume;
}
+ si.cancelled = 1;
+
if (xen_hvm_domain())
- err = stop_machine(xen_hvm_suspend, &cancelled, cpumask_of(0));
+ err = stop_machine(xen_hvm_suspend, &si, cpumask_of(0));
else
- err = stop_machine(xen_suspend, &cancelled, cpumask_of(0));
+ err = stop_machine(xen_suspend, &si, cpumask_of(0));
dpm_resume_noirq(PMSG_RESUME);
if (err) {
printk(KERN_ERR "failed to start xen_suspend: %d\n", err);
- cancelled = 1;
+ si.cancelled = 1;
}
out_resume:
- if (!cancelled) {
+ if (!si.cancelled) {
xen_arch_resume();
xs_resume();
} else