From: Jeremy Fitzhardinge <jeremy@goop.org>
To: Stable Kernel <stable@kernel.org>
Cc: "Ingo Molnar" <mingo@elte.hu>,
"Daniel Schröder" <mail@dschroeder.info>,
"Yinghai Lu" <yinghai@kernel.org>,
"Linux Kernel Mailing List" <linux-kernel@vger.kernel.org>
Subject: Change 5c371b31be3203 in stable breaks Xen
Date: Tue, 25 Nov 2008 13:26:37 -0800 [thread overview]
Message-ID: <492C6D8D.3040600@goop.org> (raw)
I have a report of Xen breaking between 2.6.27.5 and .6. I bisected it
down to change:
commit 5c371b31be32033b0a4a993431484da8a2305369
Author: Yinghai Lu <yhlu.kernel@gmail.com>
Date: Mon Sep 22 02:52:26 2008 -0700
x86: fix CONFIG_X86_RESERVE_LOW_64K=y
commit 2216d199b1430d1c0affb1498a9ebdbd9c0de439 upstream
The bad_bios_dmi_table() quirk never triggered because we do DMI setup
too late. Move it a bit earlier.
Also change the CONFIG_X86_RESERVE_LOW_64K quirk to operate on the e820
table directly instead of messing with early reservations - this handles
overlaps (which do occur in this low range of RAM) more gracefully.
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
I haven't completely root-caused why this is OK in mainline but failing
in stable, but this patch works around the problem.
Signed-off-by: Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com>
diff --git a/drivers/firmware/dmi_scan.c b/drivers/firmware/dmi_scan.c
index 455575b..abed1dd 100644
--- a/drivers/firmware/dmi_scan.c
+++ b/drivers/firmware/dmi_scan.c
@@ -7,6 +7,7 @@
#include <linux/bootmem.h>
#include <linux/slab.h>
#include <asm/dmi.h>
+#include <asm/xen/hypervisor.h>
/*
* DMI stands for "Desktop Management Interface". It is part
@@ -364,6 +365,11 @@ void __init dmi_scan_machine(void)
char __iomem *p, *q;
int rc;
+#ifdef CONFIG_XEN
+ if (is_running_on_xen())
+ return;
+#endif
+
if (efi_enabled) {
if (efi.smbios == EFI_INVALID_TABLE_ADDR)
goto out;
next reply other threads:[~2008-11-25 21:26 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-11-25 21:26 Jeremy Fitzhardinge [this message]
2008-11-25 23:06 ` Change 5c371b31be3203 in stable breaks Xen Ian Campbell
2008-11-25 23:25 ` Jeremy Fitzhardinge
2008-11-26 0:03 ` [stable] " Greg KH
2008-11-26 20:41 ` 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=492C6D8D.3040600@goop.org \
--to=jeremy@goop.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mail@dschroeder.info \
--cc=mingo@elte.hu \
--cc=stable@kernel.org \
--cc=yinghai@kernel.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.