From: Keir Fraser <keir.fraser@eu.citrix.com>
To: "Cui, Dexuan" <dexuan.cui@intel.com>,
Ross Philipson <Ross.Philipson@citrix.com>,
"xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>
Subject: Re: [PATCH] Included reserved memory regions in dom0 iommu mappings
Date: Thu, 26 Feb 2009 15:30:25 +0000 [thread overview]
Message-ID: <C5CC6611.3412%keir.fraser@eu.citrix.com> (raw)
In-Reply-To: <EADF0A36011179459010BDF5142A457507965BD7@pdsmsx502.ccr.corp.intel.com>
I'm not keen on Ross's patch anyway. I think if you have such a broken
system, the cmdline option should simply identity-map everything below 4GB,
unconditionally. Easy.
-- Keir
On 26/02/2009 09:20, "Cui, Dexuan" <dexuan.cui@intel.com> wrote:
> The patch is useful when BIOS doesn’t report RMRR correctly; however the patch
> may not help in some situations.
> E.g., to work around a buggy BIOS, we may have to map such an E820 entry
> (XEN) 00000000cff0b000 - 00000000d0000000 (reserved).
> But if we use a xen parameter "mem=512m", or, if the host only has 512m
> memory, the variable 'max_page" is 512m/PAGE_SIZE, so even with the patch, the
> E820 entry would not be mapped, and Xen would still hang.
>
> -- Dexuan
>
> ________________________________________
> From: xen-devel-bounces@lists.xensource.com
> [mailto:xen-devel-bounces@lists.xensource.com] On Behalf Of Ross Philipson
> Sent: 2009年2月24日 4:01
> To: xen-devel@lists.xensource.com
> Subject: [Xen-devel] [PATCH] Included reserved memory regions in dom0 iommu
> mappings
>
> This adds a boolean boot parameter to xen to allow reserved memory
> regions to be added to the iommu mappings for dom0. The parameter
> is "iommu_include_reserved" and is off by default. A warning is
> also traced when incorrect RMRR to system memory map values are
> detected. This is being added to address some incorrect BIOS's that
> do not report correctly the requied reserved memory ranges in
> the RMRRs. When this occurs it currently can cause early boot hangs
> and crashes.
>
> Signed-off-by: Ross Philipson <ross.philipson@citrix.com>
>
> Based on changeset 19238
>
> Thanks
> Ross
>
>
> Ross Philipson
> Senior Software Engineer
> Citrix Systems, Inc
> 14 Crosby Drive
> Bedford, MA 01730
> 781-301-7949
> ross.philipson@citrix.com
>
next prev parent reply other threads:[~2009-02-26 15:30 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-02-23 20:00 [PATCH] Included reserved memory regions in dom0 iommu mappings Ross Philipson
2009-02-26 9:20 ` Cui, Dexuan
2009-02-26 14:05 ` Ross Philipson
2009-02-26 15:30 ` Keir Fraser [this message]
2009-02-26 15:38 ` Ross Philipson
2009-02-26 16:11 ` Keir Fraser
2009-02-26 16:15 ` Ross Philipson
2009-02-26 17:34 ` Cihula, Joseph
2009-02-26 18:11 ` Ross Philipson
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=C5CC6611.3412%keir.fraser@eu.citrix.com \
--to=keir.fraser@eu.citrix.com \
--cc=Ross.Philipson@citrix.com \
--cc=dexuan.cui@intel.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.