From: Jeremy Fitzhardinge <jeremy@goop.org>
To: Chuck Ebbert <cebbert@redhat.com>
Cc: lkml <linux-kernel@vger.kernel.org>,
Chris Wright <chrisw@sous-sol.org>,
virtualization@lists.osdl.org,
"Eric W. Biederman" <ebiederm@xmission.com>,
"H. Peter Anvin" <hpa@zytor.com>,
Andrew Morton <akpm@linux-foundation.org>,
Linus Torvalds <torvalds@linux-foundation.org>
Subject: Re: [PATCH 10/28] i386: map enough initial memory to create lowmem mappings
Date: Thu, 19 Apr 2007 14:35:21 -0700 [thread overview]
Message-ID: <4627E099.209@goop.org> (raw)
In-Reply-To: <4627DDAD.4070805@redhat.com>
Chuck Ebbert wrote:
> H. Peter Anvin wrote:
>
>> Andi Kleen wrote:
>>
>>> Then we would have seen reports surely?
>>>
Yes, I would have thought so. It surprised me that such an obvious bug
could be there, apparently for a long time. But it's real, and
potentially affects everyone. It probably doesn't affect highly modular
distros much, since the kernel itself will be relatively small.
> I never saw a description of the symptoms of encountering this bug.
> Does it just hang, or what?
>
You get an early-fault message on-screen, assuming that's enabled;
otherwise it will just appear to hang. It happens in pagetable_init,
when it allocates a new pagetable above the head.S mapping (8M in my
case). It will only hit if the kernel size approaches a 4M boundary,
since it won't leave enough space mapped to construct the lowmem mappings.
It only affects native booting, since under Xen all those mappings have
already been constructed. It happened to me with a paravirt kernel that
happened to Xen compiled into it, but that was irrelevent (though
misleading; the 40k difference in kernel size was enough to make it not
happen in a non-Xen kernel).
J
next prev parent reply other threads:[~2007-04-19 21:35 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 ` [PATCH 10/28] i386: map enough initial memory to create lowmem mappings Jeremy Fitzhardinge
2007-04-14 22:04 ` 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 [this message]
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=4627E099.209@goop.org \
--to=jeremy@goop.org \
--cc=akpm@linux-foundation.org \
--cc=cebbert@redhat.com \
--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).