virtualization.lists.linux-foundation.org archive mirror
 help / color / mirror / Atom feed
From: Jeremy Fitzhardinge <jeremy@goop.org>
To: Andi Kleen <ak@suse.de>
Cc: virtualization@lists.osdl.org,
	"Eric W. Biederman" <ebiederm@xmission.com>,
	Chris Wright <chrisw@sous-sol.org>,
	"H. Peter Anvin" <hpa@zytor.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	Linus Torvalds <torvalds@linux-foundation.org>,
	lkml <linux-kernel@vger.kernel.org>
Subject: [PATCH 10/28] i386: map enough initial memory to create lowmem mappings
Date: Sat, 14 Apr 2007 13:42:04 -0700	[thread overview]
Message-ID: <20070414204924.125386443@goop.org> (raw)
In-Reply-To: 20070414204154.871250608@goop.org

[-- Attachment #1: x86-map-enough-initial-memory.patch --]
[-- Type: text/plain, Size: 3558 bytes --]

head.S creates the very initial pagetable for the kernel.  This just
maps enough space for the kernel itself, and an allocation bitmap.
The amount of mapped memory is rounded up to 4Mbytes, and so this
typically ends up mapping 8Mbytes of memory.

When booting, pagetable_init() needs to create mappings for all
lowmem, and the pagetables for these mappings are allocated from the
free pages around the kernel in low memory.  If the number of
pagetable pages + kernel size exceeds head.S's initial mapping, it
will end up faulting on an unmapped page.  This will only happen with
specific combinations of kernel size and memory size.

This patch makes sure that head.S also maps enough space to fit the
kernel pagetables as well as the kernel itself.  It ends up using an
additional two pages of unreclaimable memory.

Signed-off-by: Jeremy Fitzhardinge <jeremy@xensource.com>
Acked-by: "H. Peter Anvin" <hpa@zytor.com>
Cc: Andi Kleen <ak@suse.de>
Cc: Zachary Amsden <zach@vmware.com>
Cc: Chris Wright <chrisw@sous-sol.org>
Cc: "Eric W. Biederman" <ebiederm@xmission.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,

---
 arch/i386/kernel/asm-offsets.c |    6 ++++++
 arch/i386/kernel/head.S        |   25 ++++++++++++++++++++-----
 2 files changed, 26 insertions(+), 5 deletions(-)

===================================================================
--- a/arch/i386/kernel/asm-offsets.c
+++ b/arch/i386/kernel/asm-offsets.c
@@ -11,6 +11,7 @@
 #include <linux/suspend.h>
 #include <asm/ucontext.h>
 #include "sigframe.h"
+#include <asm/pgtable.h>
 #include <asm/fixmap.h>
 #include <asm/processor.h>
 #include <asm/thread_info.h>
@@ -96,6 +97,11 @@ void foo(void)
 		 sizeof(struct tss_struct));
 
 	DEFINE(PAGE_SIZE_asm, PAGE_SIZE);
+	DEFINE(PAGE_SHIFT_asm, PAGE_SHIFT);
+	DEFINE(PTRS_PER_PTE, PTRS_PER_PTE);
+	DEFINE(PTRS_PER_PMD, PTRS_PER_PMD);
+	DEFINE(PTRS_PER_PGD, PTRS_PER_PGD);
+
 	DEFINE(VDSO_PRELINK_asm, VDSO_PRELINK);
 
 	OFFSET(crypto_tfm_ctx_offset, crypto_tfm, __crt_ctx);
===================================================================
--- a/arch/i386/kernel/head.S
+++ b/arch/i386/kernel/head.S
@@ -34,17 +34,32 @@
 
 /*
  * This is how much memory *in addition to the memory covered up to
- * and including _end* we need mapped initially.  We need one bit for
- * each possible page, but only in low memory, which means
- * 2^32/4096/8 = 128K worst case (4G/4G split.)
+ * and including _end* we need mapped initially.
+ * We need:
+ *  - one bit for each possible page, but only in low memory, which means
+ *     2^32/4096/8 = 128K worst case (4G/4G split.)
+ *  - enough space to map all low memory, which means
+ *     (2^32/4096) / 1024 pages (worst case, non PAE)
+ *     (2^32/4096) / 512 + 4 pages (worst case for PAE)
+ *  - a few pages for allocator use before the kernel pagetable has
+ *     been set up
  *
  * Modulo rounding, each megabyte assigned here requires a kilobyte of
  * memory, which is currently unreclaimed.
  *
  * This should be a multiple of a page.
  */
-#define INIT_MAP_BEYOND_END	(128*1024)
-
+LOW_PAGES = 1<<(32-PAGE_SHIFT_asm)
+
+#if PTRS_PER_PMD > 1
+PAGE_TABLE_SIZE = (LOW_PAGES / PTRS_PER_PMD) + PTRS_PER_PGD
+#else
+PAGE_TABLE_SIZE = (LOW_PAGES / PTRS_PER_PGD)
+#endif
+BOOTBITMAP_SIZE = LOW_PAGES / 8
+ALLOCATOR_SLOP = 4
+
+INIT_MAP_BEYOND_END = BOOTBITMAP_SIZE + (PAGE_TABLE_SIZE + ALLOCATOR_SLOP)*PAGE_SIZE_asm
 
 /*
  * 32-bit kernel entrypoint; only used by the boot CPU.  On entry,

-- 

  parent reply	other threads:[~2007-04-14 20:42 UTC|newest]

Thread overview: 70+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-14 20:41 [PATCH 00/28] Updates for firstfloor paravirt-ops patches Jeremy Fitzhardinge
2007-04-14 20:41 ` [PATCH 01/28] revert account-for-module-percpu-space-separately-from-kernel-percpu Jeremy Fitzhardinge
2007-04-14 20:41 ` [PATCH 02/28] Account for module percpu space separately from kernel percpu Jeremy Fitzhardinge
2007-04-14 20:41 ` [PATCH 03/28] fix allow-percpu-variables-to-be-page-aligned.patch Jeremy Fitzhardinge
2007-04-14 20:41 ` [PATCH 04/28] deflate stack usage in lib/inflate.c Jeremy Fitzhardinge
2007-04-14 20:41 ` [PATCH 05/28] Page-align the GDT Jeremy Fitzhardinge
2007-04-14 20:42 ` [PATCH 06/28] Convert PDA into the percpu section Jeremy Fitzhardinge
2007-04-14 20:42 ` [PATCH 07/28] cleanups to help using per-cpu variables from asm Jeremy Fitzhardinge
2007-04-14 20:42 ` [PATCH 08/28] Define per_cpu_offset Jeremy Fitzhardinge
2007-04-14 20:42 ` [PATCH 09/28] Fix UP gdt bugs Jeremy Fitzhardinge
2007-04-14 20:42 ` Jeremy Fitzhardinge [this message]
2007-04-14 22:04   ` [PATCH 10/28] i386: map enough initial memory to create lowmem mappings H. Peter Anvin
2007-04-15  9:46     ` Jan Engelhardt
2007-04-15 10:17       ` Andreas Schwab
2007-04-19 20:47   ` Chuck Ebbert
2007-04-19 20:50     ` Andi Kleen
2007-04-19 20:55       ` H. Peter Anvin
2007-04-19 21:04         ` Andi Kleen
2007-04-19 21:11           ` H. Peter Anvin
2007-04-19 21:22             ` Chuck Ebbert
2007-04-19 21:35               ` Jeremy Fitzhardinge
2007-04-23  9:12                 ` Eric W. Biederman
2007-04-23 16:01                   ` H. Peter Anvin
2007-04-23 16:34                   ` Jeremy Fitzhardinge
2007-04-23 16:42                     ` H. Peter Anvin
2007-04-23 17:02                       ` Jeremy Fitzhardinge
2007-04-23 17:22                         ` H. Peter Anvin
2007-04-23 18:00                           ` Eric W. Biederman
2007-04-23 17:31                         ` Eric W. Biederman
2007-04-23 17:45                           ` H. Peter Anvin
2007-04-23 17:52                             ` Eric W. Biederman
2007-04-23 17:54                             ` Andi Kleen
2007-04-23 17:21                       ` Eric W. Biederman
2007-04-23 18:06                         ` Jeremy Fitzhardinge
2007-04-23 18:54                           ` Eric W. Biederman
2007-04-23 19:10                             ` Jeremy Fitzhardinge
2007-04-23 19:14                               ` H. Peter Anvin
2007-04-23 19:21                                 ` Jeremy Fitzhardinge
2007-04-23 19:39                                   ` Eric W. Biederman
2007-04-23 20:41                                     ` H. Peter Anvin
2007-04-25 20:54                 ` Eric W. Biederman
2007-04-25 21:31                   ` Jeremy Fitzhardinge
2007-04-25 22:00                     ` Eric W. Biederman
2007-04-25 22:06                       ` Jeremy Fitzhardinge
2007-04-25 22:18                         ` Eric W. Biederman
2007-04-25 22:52                           ` Jeremy Fitzhardinge
2007-04-25 23:33                             ` Eric W. Biederman
2007-04-25 23:41                               ` Jeremy Fitzhardinge
2007-04-26  0:33                                 ` Chris Wright
2007-04-26  0:55                                   ` Jeremy Fitzhardinge
2007-04-29 16:44                         ` Eric W. Biederman
2007-04-29 16:55                           ` Jeremy Fitzhardinge
2007-04-14 20:42 ` [PATCH 11/28] x86: incremental update for i386 and x86-64 check_bugs Jeremy Fitzhardinge
2007-04-14 20:42 ` [PATCH 12/28] i386: now its ok to use identify_boot_cpu Jeremy Fitzhardinge
2007-04-14 20:42 ` [PATCH 13/28] paravirt: flush lazy mmu updates on kunmap_atomic Jeremy Fitzhardinge
2007-04-14 20:42 ` [PATCH 14/28] fix paravirt-documentation Jeremy Fitzhardinge
2007-04-14 20:42 ` [PATCH 15/28] In compat mode, the return value here was uninitialized Jeremy Fitzhardinge
2007-04-14 20:42 ` [PATCH 16/28] kRemove a warning about unused variable in !CONFIG_ACPI compilation Jeremy Fitzhardinge
2007-04-14 20:42 ` [PATCH 17/28] x86: cleanup arch/i386/kernel/cpu/mcheck/p4.c Jeremy Fitzhardinge
2007-04-14 20:42 ` [PATCH 18/28] Copying of the pgd range must happen under the pgd_lock Jeremy Fitzhardinge
2007-04-14 20:42 ` [PATCH 19/28] Dont implement native_kmap_atomic_pte for !HIGHPTE Jeremy Fitzhardinge
2007-04-14 20:42 ` [PATCH 20/28] Now that the VDSO can be relocated, we can support it in VMI configurations Jeremy Fitzhardinge
2007-04-14 20:42 ` [PATCH 21/28] Implement vmi_kmap_atomic_pte Jeremy Fitzhardinge
2007-04-14 20:42 ` [PATCH 22/28] Convert VMI timer to use clock events Jeremy Fitzhardinge
2007-04-14 20:42 ` [PATCH 23/28] Fix BusLogic to stop using check_region Jeremy Fitzhardinge
2007-04-14 20:42 ` [PATCH 24/28] paravirt: drop unused ptep_get_and_clear Jeremy Fitzhardinge
2007-04-14 20:42 ` [PATCH 25/28] From: Jeremy Fitzhardinge <jeremy@goop.org> Jeremy Fitzhardinge
2007-04-14 20:42 ` [PATCH 26/28] From: Andrew Morton <akpm@linux-foundation.org> Jeremy Fitzhardinge
2007-04-14 20:42 ` [PATCH 27/28] paravirt: little compile fixes for vmi.c Jeremy Fitzhardinge
2007-04-14 20:42 ` [PATCH 28/28] Add a sched_clock paravirt_op Jeremy Fitzhardinge

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=20070414204924.125386443@goop.org \
    --to=jeremy@goop.org \
    --cc=ak@suse.de \
    --cc=akpm@linux-foundation.org \
    --cc=chrisw@sous-sol.org \
    --cc=ebiederm@xmission.com \
    --cc=hpa@zytor.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=torvalds@linux-foundation.org \
    --cc=virtualization@lists.osdl.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 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).