All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ed White <edmund.h.white@intel.com>
To: Tim Deegan <tim@xen.org>, Andrew Cooper <amc96@cam.ac.uk>
Cc: ravi.sahita@intel.com, Keir Fraser <keir@xen.org>,
	Ian Campbell <ian.campbell@citrix.com>,
	Ian Jackson <ian.jackson@eu.citrix.com>,
	Jan Beulich <jbeulich@suse.com>,
	xen-devel@lists.xenproject.org
Subject: Re: Alternate p2m design specification
Date: Thu, 11 Jun 2015 09:23:46 -0700	[thread overview]
Message-ID: <5579B612.1060901@intel.com> (raw)
In-Reply-To: <20150611120613.GB1818@deinos.phlegethon.org>

On 06/11/2015 05:06 AM, Tim Deegan wrote:
> At 00:09 +0100 on 11 Jun (1433981379), Andrew Cooper wrote:
>> On 10/06/15 20:41, Ed White wrote:
>>> On 06/10/2015 11:23 AM, Andrew Cooper wrote:
>>>> Also, hardware accelerated altp2m is mutually exclusive with EPT PML, as
>>>> we have no way of determining which translation was in use when a gpa
>>>> was appended to the buffer.  We are going to have to maintain a feature
>>>> compatibility matrix.  Even for non-accelerated altp2m, the cost of
>>>> working out the real gpa is likely prohibitive, and we should probably
>>>> resort to declaring logdirty and altp2m as exclusive features.
>>>>
>>>> ~Andrew
>>>>
>>> I haven't investigated the PML code, but just to be clear, log-dirty
>>> without PML is compatible with altp2m.
>>
>> The logdirty code is built around the notion of a single linear idea of
>> a guests physical address space.  Altp2m, by its very nature, introduces
>> non-linearities into a guests physical address space.
> 
> All current users of the log-dirty code, except for PML, operate on
> MFNs and use the M2P to get a PFN so use for the dirty-bitmap
> operation.  Those paths should all be able to operate with altp2m
> active.  (IOW the single linear address space is the 'host' p2m).
> 

I was going to write something this morning to point this out.

The nested page fault handler does a gfn->mfn translation using
the p2m the hardware is currently using, and then the non-PML
log-dirty code does a mfn->gfn translation that is valid for the
host p2m and uses those gfn's to update the log-dirty bitmap and
change the state of the pages in the host p2m.

However, migration still won't work with altp2m active, because
of the extra state that it knows nothing about.

Ed

      reply	other threads:[~2015-06-11 16:24 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-10  0:09 Alternate p2m design specification Ed White
2015-06-10  7:43 ` Jan Beulich
2015-06-10 16:39   ` Ed White
2015-06-11  7:05     ` Jan Beulich
2015-06-11 16:30       ` Ed White
2015-06-10 18:23 ` Andrew Cooper
2015-06-10 19:41   ` Ed White
2015-06-10 23:09     ` Andrew Cooper
2015-06-10 23:41       ` Ed White
2015-06-11 12:06       ` Tim Deegan
2015-06-11 16:23         ` Ed White [this message]

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=5579B612.1060901@intel.com \
    --to=edmund.h.white@intel.com \
    --cc=amc96@cam.ac.uk \
    --cc=ian.campbell@citrix.com \
    --cc=ian.jackson@eu.citrix.com \
    --cc=jbeulich@suse.com \
    --cc=keir@xen.org \
    --cc=ravi.sahita@intel.com \
    --cc=tim@xen.org \
    --cc=xen-devel@lists.xenproject.org \
    /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.