From: Leif Lindholm <leif.lindholm@linaro.org>
To: Matt Sealey <neko@bakuhatsu.net>
Cc: "linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
linux-efi@vger.kernel.org,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Russell King <linux@arm.linux.org.uk>,
matt.fleming@intel.com, Grant Likely <grant.likely@linaro.org>,
Roy Franz <roy.franz@linaro.org>,
Mark Salter <msalter@redhat.com>,
Patch Tracking <patches@linaro.org>,
linaro-uefi@lists.linaro.org, Mark Rutland <mark.rutland@arm.com>,
Rob Landley <rob@landley.net>,
linux-doc@vger.kernel.org
Subject: Re: [PATCH v3 1/3] Documentation: arm: add UEFI support documentation
Date: Thu, 5 Dec 2013 13:58:06 +0100 [thread overview]
Message-ID: <20131205125805.GX24997@rocoto.smurfnet.nu> (raw)
In-Reply-To: <CAHCPf3tsWdV+dJGm6mX6N4Ou5EQoGukdLyRvsRX6VQFjtKgxtg@mail.gmail.com>
On Wed, Dec 04, 2013 at 03:06:47PM -0600, Matt Sealey wrote:
> By the time we get half-way through arm/kernel/head.S the cache and
> MMU has been turned off and on and off again by the decompressor, and
> after a large amount of guesswork and arbitrary restriction-based
> implementation, there's no guarantee that the kernel hasn't been
> decompressed over some important UEFI feature or some memory hasn't
> been trashed. You can't make that guarantee because by entering the
> plain zImage, you forfeited that information. This is at worst case
> going to be lots of blank screens and blinking serial console prompts
> and little more than frustration..
So, Grant covered the reason _why_ we coexist with zImage, so I won't
go into that. I will however point out that we are explicitly using the
UEFI interfaces to allocate the regions the zImage will decompress into.
This isn't guesswork, and has in fact already turned up issues with a
couple of UEFI board ports that reserved memory near 0 (which were
indeed previously being silently overwritten by the kernel
decompression).
We _are_ planning to do more development for subsequent patches, making
more use of the UEFI memory map. And by subsequent, I mean hopefully in
time for 3.14. I sneekily included this in the version of uefi.txt sent
out for separate review early November:
http://permalink.gmane.org/gmane.linux.kernel.efi/2657, but not in
the one included with this patch set (since the code isn't there yet).
But we considered it more important to get the basic support ready
first.
At that point, you will see the stub reading the dram_base from the
UEFI memory map rather than DT, and memblock_init getting its input
from there too.
Regards,
Leif
next prev parent reply other threads:[~2013-12-05 12:57 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-28 16:41 [PATCH v3 0/3] (U)EFI runtime services for arm Leif Lindholm
[not found] ` < 1385656883-4420-2-git-send-email-leif.lindholm@linaro.org>
[not found] ` < CAHCPf3t5dEzVWbYZ6DWcFCENNO2dbrx-YeZvKRk0MkEsJMTE3A@mail.gmail.com>
[not found] ` < 20131202210719.GQ24997@rocoto.smurfnet.nu>
[not found] ` < CAHCPf3tsWdV+dJGm6mX6N4Ou5EQoGukdLyRvsRX6VQFjtKgxtg@mail.gmail.com>
[not found] ` < 20131204224447.GA19265@srcf.ucam.org>
[not found] ` < CAHCPf3s_SXYXLRwvn7tzqTCQ-gB=emRMK_dXkEmw_HWigiD7Mw@mail.gmail.com>
[not found] ` < 20131210123035.D1586C40A27@trevor.secretlab.ca>
[not found] ` < 20131129115319.GC11775@console-pimps.org>
2013-11-28 16:41 ` [PATCH v3 1/3] Documentation: arm: add UEFI support documentation Leif Lindholm
2013-12-02 19:51 ` Matt Sealey
2013-12-02 21:07 ` Leif Lindholm
2013-12-04 21:06 ` Matt Sealey
2013-12-04 22:31 ` Mark Salter
2013-12-04 22:44 ` Matthew Garrett
2013-12-06 17:20 ` Matt Sealey
2013-12-10 12:30 ` Grant Likely
2013-12-10 18:29 ` Roy Franz
2013-12-10 22:42 ` Grant Likely
2013-12-05 11:08 ` Grant Likely
2013-12-05 12:58 ` Leif Lindholm [this message]
2013-12-05 10:55 ` Grant Likely
2013-12-05 11:16 ` Grant Likely
2013-11-28 16:41 ` [PATCH v3 2/3] arm: Add [U]EFI runtime services support Leif Lindholm
2013-11-29 16:10 ` Will Deacon
2013-11-29 17:43 ` Leif Lindholm
2013-12-06 12:20 ` Will Deacon
2013-12-05 11:59 ` Grant Likely
2013-12-06 1:59 ` Arnd Bergmann
2013-12-06 17:54 ` Leif Lindholm
2013-12-06 17:59 ` Arnd Bergmann
2013-11-28 16:41 ` [PATCH v3 3/3] init: efi: arm: enable (U)EFI runtime services on arm Leif Lindholm
2013-12-05 12:03 ` Grant Likely
2013-11-29 11:53 ` [PATCH v3 0/3] (U)EFI runtime services for arm Matt Fleming
2013-11-29 17:58 ` Leif Lindholm
2013-12-05 12:04 ` Grant Likely
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=20131205125805.GX24997@rocoto.smurfnet.nu \
--to=leif.lindholm@linaro.org \
--cc=grant.likely@linaro.org \
--cc=linaro-uefi@lists.linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-efi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=mark.rutland@arm.com \
--cc=matt.fleming@intel.com \
--cc=msalter@redhat.com \
--cc=neko@bakuhatsu.net \
--cc=patches@linaro.org \
--cc=rob@landley.net \
--cc=roy.franz@linaro.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
all inboxes | Powered by JetHome®