All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tim Deegan <Tim.Deegan@eu.citrix.com>
To: Kieran Mansley <kmansley@solarflare.com>
Cc: xen-devel@lists.xensource.com
Subject: Re: Live migration with MMIO pages
Date: Thu, 1 Nov 2007 16:43:28 +0000	[thread overview]
Message-ID: <20071101164328.GD15566@york.uk.xensource.com> (raw)
In-Reply-To: <1193934965.4327.39.camel@moonstone.uk.level5networks.com>

At 16:36 +0000 on 01 Nov (1193934965), Kieran Mansley wrote:
> One question in my mind is whether the tests of iomem_access_permitted()
> in _sh_propogate() would be better replaced with the more general
> shadow_mode_translate().  The former seemed less risky to me as

You could do that.  I think the correct test is shadow_mode_refcounts();
as Keir pointed out, guests whose refcounting isn't done by the shadow
code will have had their pagetables validated bby the PV MMU interface.

The test around the log-dirty code can just be on mfn_valid().

Cheers,

Tim.

-- 
Tim Deegan <Tim.Deegan@eu.citrix.com>
Principal Software Engineer, Citrix Systems.
[Company #5334508: XenSource UK Ltd, reg'd c/o EC2Y 5EB, UK.]

  reply	other threads:[~2007-11-01 16:43 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-10-31 12:19 Live migration with MMIO pages Kieran Mansley
2007-10-31 13:32 ` Keir Fraser
2007-10-31 14:03   ` Kieran Mansley
2007-10-31 14:08     ` Keir Fraser
2007-10-31 16:34       ` Kieran Mansley
2007-10-31 16:44         ` Keir Fraser
2007-10-31 17:14           ` Kieran Mansley
2007-10-31 17:31           ` Tim Deegan
2007-10-31 18:23             ` Keir Fraser
2007-11-01 11:18             ` Kieran Mansley
2007-11-01 11:34               ` Tim Deegan
2007-11-01 16:36                 ` Kieran Mansley
2007-11-01 16:43                   ` Tim Deegan [this message]
2007-11-02  9:31                     ` Kieran Mansley
2007-11-02 10:42                       ` Keir Fraser
2007-11-02 10:51                         ` 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=20071101164328.GD15566@york.uk.xensource.com \
    --to=tim.deegan@eu.citrix.com \
    --cc=kmansley@solarflare.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.