All of lore.kernel.org
 help / color / mirror / Atom feed
From: John Byrne <john.l.byrne@hp.com>
To: Ian Pratt <m+Ian.Pratt@cl.cam.ac.uk>
Cc: xen-devel <xen-devel@lists.xensource.com>
Subject: Re: Live migration leaves page tables read-only?
Date: Sat, 13 Jan 2007 20:11:15 -0800	[thread overview]
Message-ID: <45A9AD63.6070308@hp.com> (raw)
In-Reply-To: <8A87A9A84C201449A0C56B728ACF491E04EE29@liverpoolst.ad.cl.cam.ac.uk>


Ian,

I haven't noticed a fix. Is someone working on this bug or should I open 
a bugzilla for it, so it isn't forgotten?

John Byrne

Ian Pratt wrote:
>  
>> I finally ran down the problem. SAP is protecting the pages PROT_NONE,
>> so the page-present bit in the pte is not set and
>> canonicalize/uncanonicalize code in save/restore ignore the pte. I've
>> attached a patch. It is possible that this change should be made to
> the
>> l1e tests in xc_ptrace.c; I'm not sure.
> 
> That's a good catch, thanks. Interesting that we hadn't seen this
> before.
> 
> Although your patch works today, it will break when we add PSE (super
> page) support for PV guests as it will confuse PROT_NONE with PSE.
> Assuming PROT_NONE only makes sense for L1 entries, we can probably gate
> the tests on whether the page table page is an L1 or not to fix this.
> 
> However, it does point out an issue for other OSes: Taking this patch
> effectively makes Linux's PROT_NONE (flags 0x80 for a not present PTE)
> part of the Xen API. We need to find out whether this is compatible with
> *BSD and Solaris' use of flags for not present ptes.
> 
> Ian
> 

  parent reply	other threads:[~2007-01-14  4:11 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-11-29  0:13 Live migration leaves page tables read-only? John Byrne
2006-11-29  0:22 ` John Byrne
2006-11-29  1:36   ` Ian Pratt
2006-11-29  2:52     ` John Byrne
2006-11-29  7:42       ` Keir Fraser
2006-11-29 16:49         ` John Byrne
2006-11-30 23:36       ` John Byrne
2006-12-01  1:13         ` Ian Pratt
2006-12-09  5:40           ` John Byrne
2006-12-09  5:44             ` John Byrne
2006-12-09  8:33             ` Ian Pratt
2006-12-09  9:22               ` Keir Fraser
2006-12-09  9:34                 ` Keir Fraser
2006-12-09  9:48                   ` Keir Fraser
2006-12-11 17:00               ` Joe Bonasera
2006-12-11 18:29                 ` Ian Pratt
2006-12-11 19:55                   ` John Byrne
2006-12-11 21:30                   ` Joe Bonasera
2007-01-14  4:11               ` John Byrne [this message]
2007-01-14  8:21                 ` Ian Pratt

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=45A9AD63.6070308@hp.com \
    --to=john.l.byrne@hp.com \
    --cc=m+Ian.Pratt@cl.cam.ac.uk \
    --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.