All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jeremy Fitzhardinge <jeremy@goop.org>
To: Keir Fraser <keir@xen.org>
Cc: "xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>,
	Stefano Stabellini <Stefano.Stabellini@eu.citrix.com>,
	Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>,
	"bruce.edge@gmail.com" <bruce.edge@gmail.com>,
	Ian Campbell <Ian.Campbell@eu.citrix.com>,
	Gianni Tedesco <gianni.tedesco@citrix.com>
Subject: Re: [RFC Patch] Support for making an E820 PCI hole in toolstack (xl + xm)
Date: Tue, 16 Nov 2010 10:01:33 -0800	[thread overview]
Message-ID: <4CE2C6FD.8060504@goop.org> (raw)
In-Reply-To: <C9080945.A1AE%keir@xen.org>

On 11/16/2010 02:11 AM, Keir Fraser wrote:
> On 16/11/2010 10:02, "Ian Campbell" <Ian.Campbell@eu.citrix.com> wrote:
>
>>> The right answer is probably to refuse save/restore/migrate when devices are
>>> passed through.
>> Absolutely. 
>>
>> However we are talking about setting up a 1-1 mapping in the P2M region
>> corresponding to the PCI hole at guest boot and preserving that until
>> such a time as a device is plugged in, which may be after a migration.
>>
>> I don't think it matters that no device is passed through at the time of
>> the migration, in this configuration we still need arrange for the
>> relevant P2M entries to be correct after the migration (or at least
>> before the device gets plugged in, perhaps we can leave holes and only
>> establish the 1-1 p2m on demand in pcifront?).
> Leave the hole empty and populate on demand when devices are passed through
> would seem sensible.

Actually I was originally thinking that the hole would all be
INVALID_MFN but then pfn_to_mfn() would translate that to being an
identity translation.  But that's pretty hacky, and only works if you
actually want identity.  On-demand population of regions is much cleaner.

    J

  reply	other threads:[~2010-11-16 18:01 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-12 23:08 [RFC Patch] Support for making an E820 PCI hole in toolstack (xl + xm) Konrad Rzeszutek Wilk
2010-11-12 23:16 ` Konrad Rzeszutek Wilk
2010-11-13  7:40 ` Keir Fraser
2010-11-15 17:03   ` Konrad Rzeszutek Wilk
2010-11-15 17:20     ` Ian Campbell
2010-11-15 17:28       ` Konrad Rzeszutek Wilk
2010-11-15 17:48     ` Keir Fraser
2010-11-15 18:15       ` Konrad Rzeszutek Wilk
2010-11-15 18:41         ` Keir Fraser
2010-11-15 19:32           ` Jeremy Fitzhardinge
2010-11-15 19:57             ` Keir Fraser
2010-11-15 23:11               ` Konrad Rzeszutek Wilk
2010-11-16  1:06                 ` Jeremy Fitzhardinge
2010-11-16  9:26                   ` Ian Campbell
2010-11-16  9:52                     ` Keir Fraser
2010-11-16 10:02                       ` Ian Campbell
2010-11-16 10:11                         ` Keir Fraser
2010-11-16 18:01                           ` Jeremy Fitzhardinge [this message]
2010-11-16 15:50                     ` Konrad Rzeszutek Wilk
2010-11-17 14:23                       ` Ian Campbell
2010-11-16  7:40                 ` Keir Fraser
2010-11-15 19:30         ` Jeremy Fitzhardinge
2010-11-17 11:14 ` Gianni Tedesco
2010-11-17 11:43   ` Ian Campbell
2010-11-17 13:37     ` Gianni Tedesco

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=4CE2C6FD.8060504@goop.org \
    --to=jeremy@goop.org \
    --cc=Ian.Campbell@eu.citrix.com \
    --cc=Stefano.Stabellini@eu.citrix.com \
    --cc=bruce.edge@gmail.com \
    --cc=gianni.tedesco@citrix.com \
    --cc=keir@xen.org \
    --cc=konrad.wilk@oracle.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.