From: Alexey G <x1917x@gmail.com>
To: "Roger Pau Monné" <roger.pau@citrix.com>
Cc: Juergen Gross <jgross@suse.com>,
xen-devel@lists.xenproject.org, Jan Beulich <jbeulich@suse.com>,
Andrew Cooper <andrew.cooper3@citrix.com>
Subject: Re: [PATCH for-4.11] x86/dom0: add extra RAM regions as RESERVED for PVH memory map
Date: Thu, 3 May 2018 22:02:47 +1000 [thread overview]
Message-ID: <20180503220247.00001e80@gmail.com> (raw)
In-Reply-To: <20180503111518.ac2z7fo6f2lei7vx@MacBook-Pro-de-Roger.local>
On Thu, 3 May 2018 12:15:18 +0100
Roger Pau Monné <roger.pau@citrix.com> wrote:
>On Thu, May 03, 2018 at 08:55:14PM +1000, Alexey G wrote:
>> On Thu, 3 May 2018 10:56:40 +0100
>> Roger Pau Monne <roger.pau@citrix.com> wrote:
>>
>> >When running as PVH Dom0 the native memory map is used in order to
>> >craft a tailored memory map for Dom0 taking into account it's memory
>> >limit.
>> >
>> >Dom0 memory is always going to be smaller than the total amount
>> >of memory present on the host, so in order to prevent Dom0 from
>> >relocating PCI BARs over RAM regions mark all the RAM regions not
>> >available to Dom0 as RESERVED in the memory map.
>> >---
>> >NB: I haven't seen any system where Dom0 would relocate the BARs
>> >over RAM regions, but AFAICT given the current memory map provided
>> >to Dom0 this is a possibility that should be avoided.
>>
>> Guest OSes typically use information from ACPI to learn where PCI
>> BARs can (or cannot) be relocated.
>
>I think it's better to be safe than sorry, so IMO the host RAM regions
>should be added to the memory map as RESERVED.
>
>Roger.
I assume host's DSDT passed through to PVH Dom0 as is? In this case
Dom0 will see PCI holes matching those of the host and shouldn't make
any attempts to place BARs outside provided PCI holes (except "nocrs"
given).
As long as Dom0 P2M map prevent using these host ranges we shouldn't
worry if it is marked as reserved in e820 map I think. It's kinda
excessive information for Dom0 -- knowing about host RAM ranges which
he cannot touch anyway due to lack of corresponding p2m mappings.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
next prev parent reply other threads:[~2018-05-03 12:03 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-05-03 9:56 [PATCH for-4.11] x86/dom0: add extra RAM regions as RESERVED for PVH memory map Roger Pau Monne
2018-05-03 10:55 ` Alexey G
2018-05-03 11:15 ` Roger Pau Monné
2018-05-03 12:02 ` Alexey G [this message]
2018-05-03 14:02 ` Roger Pau Monné
2018-05-03 16:41 ` Alexey G
2018-05-03 11:46 ` Jan Beulich
2018-05-03 12:59 ` Roger Pau Monné
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=20180503220247.00001e80@gmail.com \
--to=x1917x@gmail.com \
--cc=andrew.cooper3@citrix.com \
--cc=jbeulich@suse.com \
--cc=jgross@suse.com \
--cc=roger.pau@citrix.com \
--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.