All of lore.kernel.org
 help / color / mirror / Atom feed
From: Chris Lalancette <clalance@redhat.com>
To: Keir Fraser <keir.fraser@eu.citrix.com>
Cc: Jiri Denemark <jdenemar@redhat.com>,
	"xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>,
	Gerd Hoffmann <kraxel@redhat.com>
Subject: Re: [PATCH] Fix xenconsole after aborted save
Date: Wed, 22 Apr 2009 16:24:20 +0200	[thread overview]
Message-ID: <49EF2894.4090208@redhat.com> (raw)
In-Reply-To: <C614AE59.9293%keir.fraser@eu.citrix.com>

Keir Fraser wrote:
> On 22/04/2009 10:49, "Chris Lalancette" <clalance@redhat.com> wrote:
> 
>> Keir Fraser wrote:
>>> Another thing to note is I think this problem can only occur if the domU
>>> does not support suspend cancellation (advertised as SUSPEND_CANCEL in
>>> kernel elf notes -- see xen/xend/XendDomainInfo.py:resumeDomain()). Your
>>> kernels should support that feature -- suspend cancellation (a.k.a. Resume)
>>> is very likely to be hit-or-miss without it!
>> Could you elaborate a bit on this?  I was under the impression that suspend
>> cancellation was there mostly for the netaccel bits, but I have to admit I
>> didn't look at it very closely.  What scenarios do the suspend cancel bits
>> help?
> 
> Suspend failures (failure to save or to migrate). Also for live
> checkpointing/snapshotting. The feature indicates that the guest is happy
> for the suspend hypercall to return indicating 'failure/cancelled' and in
> which case it can pretty much resume whatever it was doing without any of
> the usual resume logic. The alternative is for the toolstack to make it look
> like the domain has been restored/migrated, by resetting PV devices and the
> like, and this is not much tested and almost inherently fragile.
> 
> The suspend_cancel hook you are thinking of is for any PV devices which *do*
> need to know that a suspend was cancelled. Netaccel does for some reason
> which I cannot recall.
> 
> Anyway, it is pretty easy and pretty important to support SUSPEND_CANCEL!

OK, that makes sense.  Thanks Keir!

-- 
Chris Lalancette

  parent reply	other threads:[~2009-04-22 14:24 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-20 13:23 [PATCH] Fix xenconsole after aborted save Jiri Denemark
2009-04-20 13:38 ` Gerd Hoffmann
2009-04-20 14:04   ` Jiri Denemark
2009-04-20 14:09     ` Gerd Hoffmann
2009-04-20 14:41     ` Keir Fraser
2009-04-20 15:58       ` Jiri Denemark
2009-04-22  9:49       ` Chris Lalancette
2009-04-22 10:22         ` Keir Fraser
2009-04-22 12:41           ` John Levon
2009-04-22 14:24           ` Chris Lalancette [this message]
2009-04-22 16:15           ` Neil Turton

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=49EF2894.4090208@redhat.com \
    --to=clalance@redhat.com \
    --cc=jdenemar@redhat.com \
    --cc=keir.fraser@eu.citrix.com \
    --cc=kraxel@redhat.com \
    --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: link
Be 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.