LKML Archive on
help / color / mirror / Atom feed
From: "Rafael J. Wysocki" <>
To: "SUZUKI, Kazuhiro" <>
Cc:,, Greg KH <>,
	LKML <>
Subject: Re: [PATCH 0/2] Fix hangup after creating checkpoint on Xen.
Date: Mon, 7 Feb 2011 10:35:29 +0100	[thread overview]
Message-ID: <> (raw)
In-Reply-To: <>

On Monday, February 07, 2011, SUZUKI, Kazuhiro wrote:
> Hi,


> The following patch series fixes hangup after creating checkpoint on
> Xen. The Linux Xen guest can be saved the state to restore later, and
> also created snapshot like checkpoint via the hypervisor.
> But, when the snapshot is created for the PV guest, it will hangup.
> We added 'PMSG_CANCEL' message and 'cancel' handler in dev_pm_ops
> struct in the pm-linux part.

Please don't do that, unless you can convince me there's no other way to fix
the problem you're trying to address.

In my opinion it's highly unrealistic to assume that device drivers
(or even subsystems) will implement the ->cancel() callback just for the
benefit of Xen.

And if the only subsystem that needs to implement ->cancel() is Xen, then the
issue should be addressed without modifying the device core code, in a different

> In creating checkpoint mode, the resume handler of xenbus should not
> be called. In this case, it is recognized that the suspend was canceled
> in drivers/xen/manage.c and call dpm_resume_end() with PMSG_CANCEL.
> If the 'cancel' handler is defined, it is called instead of resume().
> [1/2] - Fix hangup after creating checkpoint on Xen -- pm-linux part.
> [2/2] - Fix hangup after creating checkpoint on Xen -- Xen part.


       reply	other threads:[~2011-02-07  9:35 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <>
2011-02-07  9:35 ` Rafael J. Wysocki [this message]
2011-02-08 11:22   ` Ian Campbell
2011-02-08 16:46     ` [linux-pm] " Alan Stern
2011-02-08 17:35       ` Ian Campbell
2011-02-09 23:16         ` Brendan Cully
2011-02-09 23:42           ` Alan Stern
2011-02-10 11:40             ` [Xen-devel] " Ian Campbell
2011-02-10 16:00               ` Alan Stern
2011-02-10 16:26                 ` Rafael J. Wysocki
2011-02-10 16:34                 ` Ian Campbell
2011-02-10 17:01                   ` Rafael J. Wysocki
2011-02-10 18:56                   ` Alan Stern
2011-02-10 11:31           ` Ian Campbell
2011-02-10 17:53             ` Brendan Cully

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:

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \ \ \ \ \ \ \ \
    --subject='Re: [PATCH 0/2] Fix hangup after creating checkpoint on Xen.' \

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

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).