All of lore.kernel.org
 help / color / mirror / Atom feed
From: Keir Fraser <keir@xensource.com>
To: Brendan Cully <brendan@cs.ubc.ca>, xen-devel@lists.xensource.com
Subject: Re: [PATCH 0 of 5] Make checkpointing backwards compatible with older kernels
Date: Sat, 13 Jan 2007 09:28:55 +0000	[thread overview]
Message-ID: <C1CE56D7.71F3%keir@xensource.com> (raw)
In-Reply-To: <patchbomb.1168642987@ventoux.cs.ubc.ca>

On 12/1/07 10:03 pm, "Brendan Cully" <brendan@cs.ubc.ca> wrote:

> This set of patches makes checkpointing backwards compatible with
> older kernels. Checkpointable kernels now create a xenstore entry
> called "control/feature-resumable" which xc save -c checks before
> attempting to checkpoint a domain.
> 
> It also writes a new SIF_RESUMABLE flag into the suspend record,
> which xc_linux_restore uses to decide whether it is safe to set the
> hypercall return value.

A single unified new elfnote would probably be better. We would like a new
libxc function to pull image metadata (inc. Elfnotes) out of a kernel image
and provide a comma-separated key-value list, turned into a dictionary for
Python consumption. This would then be persisted by xend (e.g., be added to
save/restore stream format) and can be used to decide whether 'xm save -c'
is okay, and would allow info to be passed to xc_save/xc_restore about
whether suspend is resumable. We already have another intended use for this
mechanism, which is to provide a pte mask to use to decide whether or not a
not-present pte contains an MFN (needing conversion to PFN on xc_save).
Unfortunately although this method is cleanest there is some tedious
plumbing to be done and I'm not sure how to represent the Elf notes in a
save image format (perhaps it can be merged into the image sxpr).

Yes -- returning 1 on resume (rather than restore) will allow new guest
kernels to run properly on old Xen and tools which do not understand resume,
so that would be the better way round.

 -- Keir

  parent reply	other threads:[~2007-01-13  9:28 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-01-12 22:03 [PATCH 0 of 5] Make checkpointing backwards compatible with older kernels Brendan Cully
2007-01-12 22:03 ` [PATCH 1 of 5] Advertise resumability feature in guest kernel Brendan Cully
2007-01-12 22:03 ` [PATCH 2 of 5] Disallow xm save -c unless the guest advertises that it is resumable Brendan Cully
2007-01-12 22:03 ` [PATCH 3 of 5] Add SIF_RESUMABLE flag to tell restore that a guest " Brendan Cully
2007-01-12 22:03 ` [PATCH 4 of 5] Set SIF_RESUMABLE flag in suspend Brendan Cully
2007-01-12 22:03 ` [PATCH 5 of 5] Do not set suspend return value unless SIF_RESUMABLE is set Brendan Cully
2007-01-13  9:28 ` Keir Fraser [this message]
2007-01-13  9:32 ` [PATCH 0 of 5] Make checkpointing backwards compatible with older kernels Keir Fraser

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=C1CE56D7.71F3%keir@xensource.com \
    --to=keir@xensource.com \
    --cc=brendan@cs.ubc.ca \
    --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.