From: Mark Brown <broonie-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public•gmane.org>
To: Daniel Lezcano <daniel.lezcano-QSEj5FYQhm4dnm+yROfE0A@public•gmane.org>
Cc: Nicolas Pitre
<nicolas.pitre-QSEj5FYQhm4dnm+yROfE0A@public•gmane.org>,
Kevin Hilman <khilman-l0cyMroinI0@public•gmane.org>,
Stephen Rothwell <sfr-3FnU+UHB4dNDw9hX6IcOSA@public•gmane.org>,
linux-next-u79uwXL29TY76Z2rM5mHXA@public•gmane.org,
Jamie Iles <jamie-wmLquQDDieKakBO8gow8eQ@public•gmane.org>,
Lists Linaro-dev
<linaro-dev-cunTk1MwBs8s++Sfvej+rw@public•gmane.org>
Subject: Re: linux-next not booting on snowball
Date: Thu, 1 Dec 2011 14:58:01 +0000 [thread overview]
Message-ID: <20111201145801.GB2103@sirena.org.uk> (raw)
In-Reply-To: <4ED79454.1090304-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
On Thu, Dec 01, 2011 at 03:51:00PM +0100, Daniel Lezcano wrote:
> commit 549158d2ab01e8370d2773044fe09738a26f7086
> Author: Nicolas Pitre <nicolas.pitre-QSEj5FYQhm4dnm+yROfE0A@public•gmane.org>
> Date: Thu Aug 25 00:35:59 2011 -0400
> ARM: move iotable mappings within the vmalloc region
>
> In order to remove the build time variation between different SOCs
> with
> regards to VMALLOC_END, the iotable mappings are now allocated inside
> the vmalloc region. This allows for VMALLOC_END to be identical
> across
> all machines.
I recently reported an issue with this patch on s3c64xx which I'm
avoiding with the below change, I believe Nicolas folded this in to his
code but it's not propagated into -next yet.
>From d53e2ce3fb18e097678b324932591044eb80c0f1 Mon Sep 17 00:00:00 2001
From: Mark Brown <broonie-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public•gmane.org>
Date: Thu, 24 Nov 2011 12:46:04 +0000
Subject: [PATCH] ARM: Handle empty maps in iotable_init()
Some CPUs (such as the S3C6410) have been relying on being able to call
iotable_init() with no io_descs in order to simplify passing through
machine-specific io_descs. The changes in "ARM: move iotable mappings
within the vmalloc region" broke this by adding an early_alloc_aligned()
for an array of vm_structs. Fix this by returning early if no descriptors
have been passed.
I'm not sure if this is the most tasteful fix but it preserves existing
behaviour and allows boot to proceed on my system.
Signed-off-by: Mark Brown <broonie-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public•gmane.org>
---
arch/arm/mm/mmu.c | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/arch/arm/mm/mmu.c b/arch/arm/mm/mmu.c
index 06e2aef..94c5a0c 100644
--- a/arch/arm/mm/mmu.c
+++ b/arch/arm/mm/mmu.c
@@ -763,6 +763,9 @@ void __init iotable_init(struct map_desc *io_desc, int nr)
struct map_desc *md;
struct vm_struct *vm;
+ if (!nr)
+ return;
+
vm = early_alloc_aligned(sizeof(*vm) * nr, __alignof__(*vm));
for (md = io_desc; nr; md++, nr--) {
--
1.7.7.3
next prev parent reply other threads:[~2011-12-01 14:58 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-12-01 14:51 linux-next not booting on snowball Daniel Lezcano
[not found] ` <4ED79454.1090304-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2011-12-01 14:58 ` Mark Brown [this message]
2011-12-01 15:34 ` Daniel Lezcano
2011-12-01 19:03 ` Nicolas Pitre
2011-12-01 23:06 ` Daniel Lezcano
2011-12-02 0:11 ` Nicolas Pitre
2011-12-02 17:31 ` Daniel Lezcano
2011-12-07 4:09 ` Nicolas Pitre
2011-12-13 12:35 ` Linus Walleij
2011-12-14 8:24 ` Linus Walleij
2011-12-14 8:27 ` Mark Brown
2011-12-14 8:38 ` Linus Walleij
2011-12-12 15:41 ` Daniel Lezcano
2011-12-12 16:56 ` Nicolas Pitre
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=20111201145801.GB2103@sirena.org.uk \
--to=broonie-yzvpicuk2aatku/dhu1wvuem+bqzidxxqq4iyu8u01e@public$(echo .)gmane.org \
--cc=daniel.lezcano-QSEj5FYQhm4dnm+yROfE0A@public$(echo .)gmane.org \
--cc=jamie-wmLquQDDieKakBO8gow8eQ@public$(echo .)gmane.org \
--cc=khilman-l0cyMroinI0@public$(echo .)gmane.org \
--cc=linaro-dev-cunTk1MwBs8s++Sfvej+rw@public$(echo .)gmane.org \
--cc=linux-next-u79uwXL29TY76Z2rM5mHXA@public$(echo .)gmane.org \
--cc=nicolas.pitre-QSEj5FYQhm4dnm+yROfE0A@public$(echo .)gmane.org \
--cc=sfr-3FnU+UHB4dNDw9hX6IcOSA@public$(echo .)gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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