From: "H. Peter Anvin" <hpa@zytor.com>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Keith Packard <keithp@keithp.com>,
linux-kernel@vger.kernel.org,
Matt Fleming <matt.fleming@intel.com>,
Matthew Garrett <mjg@redhat.com>, Zhang Rui <rui.zhang@intel.com>,
Huang Ying <huang.ying.caritas@gmail.com>,
Andrew Morton <akpm@linux-foundation.org>,
Ingo Molnar <mingo@elte.hu>
Subject: Re: [PATCH] Revert "x86, efi: Calling __pa() with an ioremap()ed address is invalid"
Date: Sun, 11 Dec 2011 19:01:45 -0800 [thread overview]
Message-ID: <4EE56E99.60801@zytor.com> (raw)
In-Reply-To: <CA+55aFydzKrzUaDtBaebjKKO4xVpubfZAXP6oSD-SDuDMq9eGw@mail.gmail.com>
On 12/11/2011 06:59 PM, Linus Torvalds wrote:
> On Sun, Dec 11, 2011 at 4:12 PM, Keith Packard <keithp@keithp.com> wrote:
>> This hangs my MacBook Air at boot time; I get no console messages at
>> all. I reverted this on top of -rc5 and my machine boots again.
>
> Interesting. Current -git works for me on my 11" Macbook Air. Which
> should be identical to yours. This is the 4GB RAM, 128GB disk version,
> no?
>
> But I'm running OpenSUSE, and you're running Debian, right? And our
> kernel configs are probably wildly different, since I tend to try to
> boot a fairly minimal set.
>
> That said, if you've verified that if fails for you, I guess we should
> revert regardless. Matt, any suggestions for tests to do?
>
Also... I'd like to know what machines, if any, would break without the
patch, so we can figure out what has to be tested.
-hpa
--
H. Peter Anvin, Intel Open Source Technology Center
I work for Intel. I don't speak on their behalf.
next prev parent reply other threads:[~2011-12-12 3:02 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-12-12 0:12 [PATCH] Revert "x86, efi: Calling __pa() with an ioremap()ed address is invalid" Keith Packard
2011-12-12 2:59 ` Linus Torvalds
2011-12-12 3:01 ` H. Peter Anvin [this message]
2011-12-12 3:11 ` Ingo Molnar
2011-12-12 3:31 ` Keith Packard
2011-12-12 3:32 ` H. Peter Anvin
2011-12-12 3:43 ` Keith Packard
2011-12-12 10:01 ` Matt Fleming
2011-12-12 16:35 ` Keith Packard
2011-12-12 16:50 ` Linus Torvalds
2011-12-12 17:09 ` H. Peter Anvin
2011-12-12 17:21 ` Keith Packard
2011-12-12 17:29 ` H. Peter Anvin
2011-12-12 17:37 ` Linus Torvalds
2011-12-15 15:29 ` Matt Fleming
2012-01-16 9:22 ` Matt Fleming
2012-01-16 9:31 ` Ingo Molnar
2012-01-16 9:49 ` Matt Fleming
2012-01-16 9:55 ` Ingo Molnar
2012-01-16 10:16 ` Matt Fleming
2012-01-16 14:54 ` [tip:x86/efi] x86/efi: Delete efi_ioremap() and fix CONFIG_X86_32 oops tip-bot for Matt Fleming
2012-02-08 12:46 ` [PATCH] Revert "x86, efi: Calling __pa() with an ioremap()ed address is invalid" Matt Fleming
2012-02-08 17:15 ` Keith Packard
2012-02-15 12:13 ` Matt Fleming
2012-02-17 11:21 ` Ingo Molnar
2011-12-12 10:01 ` Matt Fleming
2011-12-12 17:09 ` H. Peter Anvin
2011-12-12 17:26 ` Ingo Molnar
2011-12-12 3:06 ` Ingo Molnar
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=4EE56E99.60801@zytor.com \
--to=hpa@zytor.com \
--cc=akpm@linux-foundation.org \
--cc=huang.ying.caritas@gmail.com \
--cc=keithp@keithp.com \
--cc=linux-kernel@vger.kernel.org \
--cc=matt.fleming@intel.com \
--cc=mingo@elte.hu \
--cc=mjg@redhat.com \
--cc=rui.zhang@intel.com \
--cc=torvalds@linux-foundation.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.