LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Daniel Kiper <dkiper@net-space.pl>
To: linux-kernel@vger.kernel.org, stable@kernel.org, x86@kernel.org,
xen-devel@lists.xensource.com, jeremy@goop.org,
konrad.wilk@oracle.com, mingo@redhat.com, mingo@elte.hu
Subject: [PATCH] xen: convert p2m to a 3 level tree - partial revert
Date: Thu, 27 Jan 2011 15:48:43 +0100 [thread overview]
Message-ID: <20110127144843.GA23746@router-fw-old.local.net-space.pl> (raw)
Hi,
Durning work on Xen memory hotplug I discoverd that
2.6.38-rc2 does not boot on domU. After some investigation
it appeared that 58e05027b530ff081ecea68e38de8d59db8f87e0
commit changed CONFIG_XEN_MAX_DOMAIN_MEMORY constant value
to 128. This change does not allow to boot kernel on domU
with small memory size (I could confirm that it is even
not possible to boot kernel on domU with 2 GiB). Guest
crash silently without any warning. Durning further
research I found out that there is another commit published
recently (8e1b4cf2108488ccfb9a3e7ed7cd85a435e01d4b) which attempts
to fix this issue, however, it does not work on my platform.
I decided to reenable CONFIG_XEN_MAX_DOMAIN_MEMORY option in
kernel config and enable users to choose resonable values for
their machines until better fix will be published. I think this
solution is good because allow users to boot domU with newest
kernel and allow developers to continue their work without
time presure which could lead to new bugs.
This patch applies cleanly to current Linus' kernel tree.
Signed-off-by: Daniel Kiper <dkiper@net-space.pl>
---
arch/x86/xen/Kconfig | 11 +++++++----
1 files changed, 7 insertions(+), 4 deletions(-)
diff --git a/arch/x86/xen/Kconfig b/arch/x86/xen/Kconfig
index 5b54892..a234b9a 100644
--- a/arch/x86/xen/Kconfig
+++ b/arch/x86/xen/Kconfig
@@ -29,12 +29,15 @@ config XEN_PVHVM
depends on X86_LOCAL_APIC
config XEN_MAX_DOMAIN_MEMORY
- int
- default 128
+ int "Maximum allowed size of a domain in gigabytes"
+ default 8 if X86_32
+ default 32 if X86_64
depends on XEN
help
- This only affects the sizing of some bss arrays, the unused
- portions of which are freed.
+ The pseudo-physical to machine address array is sized
+ according to the maximum possible memory size of a Xen
+ domain. This array uses 1 page per gigabyte, so there's no
+ need to be too stingy here.
config XEN_SAVE_RESTORE
bool
--
1.5.6.5
next reply other threads:[~2011-01-27 14:51 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-01-27 14:48 Daniel Kiper [this message]
2011-01-27 14:59 ` Stefan Bader
2011-01-27 15:02 ` Stefan Bader
2011-01-27 15:19 ` Daniel Kiper
2011-01-27 15:28 ` Stefan Bader
2011-01-27 15:25 ` [Xen-devel] " Konrad Rzeszutek Wilk
2011-01-31 11:21 ` Daniel Kiper
2011-01-31 14:27 ` Stefano Stabellini
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=20110127144843.GA23746@router-fw-old.local.net-space.pl \
--to=dkiper@net-space.pl \
--cc=jeremy@goop.org \
--cc=konrad.wilk@oracle.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=mingo@redhat.com \
--cc=stable@kernel.org \
--cc=x86@kernel.org \
--cc=xen-devel@lists.xensource.com \
--subject='Re: [PATCH] xen: convert p2m to a 3 level tree - partial revert' \
/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
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).