From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: Continuing kallsyms failures - large kernels, XIP kernels, and large XIP kernels
Date: Wed, 04 Feb 2015 15:44:23 +0100 [thread overview]
Message-ID: <8085056.h8oiDn7EOb@wuerfel> (raw)
In-Reply-To: <20150204094414.GC8656@n2100.arm.linux.org.uk>
On Wednesday 04 February 2015 09:44:14 Russell King - ARM Linux wrote:
> On Tue, Feb 03, 2015 at 08:59:15PM -0500, Nicolas Pitre wrote:
> > On Wed, 4 Feb 2015, Russell King - ARM Linux wrote:
> >
> > > It looks like we have cases where this falsely triggers. Consider EFM32:
> > >
> > > CONFIG_DRAM_BASE=0x88000000
> > > CONFIG_DRAM_SIZE=0x00400000
> > > CONFIG_FLASH_MEM_BASE=0x8c000000
> > > CONFIG_FLASH_SIZE=0x01000000
> > >
> > > This means that we quite legally end up with the .data section below the
> > > .text section, which makes:
> > >
> > > ASSERT((_data >= __data_loc), "Text section oversize")
> > >
> > > falsely trigger.
> > >
> > > The linker has the capacity to specify regions of ROM and RAM in the
> > > linker file, I wonder if we should be using those for XIP. Merely
> > > adding the MEMORY table to the linker file is not good enough - we
> > > also need to explicitly tell the linker which memory regions to place
> > > the output sections, otherwise the linker ends up making assumptions.
> > >
> > > What that means is... asm-generic/vmlinux.lds.h breaks for us.
> > >
> > > Any ideas? I think using the MEMORY table would be the best approach,
> > > because that should allow us to properly verify that the resulting
> > > binary should fit in the memory regions.
> >
> > Maybe simply having an assert() on the size of the .text section could
> > be all that is needed. We already know the maximum size in advance.
>
> That doesn't work, it's not just the .text section but also .rodata,
> __bug_table, __ksymtab, __ksymtab_gpl, __kcrctab, __kcrctab_gpl,
> __ksymtab_strings, __param, __modver, __ex_table, .notes, .vectors,
> .stubs, .init.text, maybe .exit.text, .init.arch.info, .init.tagtable,
> .init.smpalt, .init.pv_table, and apparently .init.data (which is
> surely wrong?) The following is from Arnd's failing configuration:
>
> 18 .init.tagtable 00000040 80073a9c 80073a9c 0100ba9c 2**2
> CONTENTS, ALLOC, LOAD, READONLY, DATA
> 19 .init.data 000010e8 80073adc 80073adc 0100badc 2**2
> CONTENTS, ALLOC, LOAD, DATA
> 20 .data 003552c4 80008000 80074bc4 01010000 2**8
> CONTENTS, ALLOC, LOAD, DATA
>
> Hmm, if .init.data is contained in the flash section (which it seemingly
> is), it seems that XIP support is currently broken - that section is
> definitely a read/write section. No one has seemingly noticed that it's
> broken and it's been broken for a long time, so maybe the simple answer
> then is just to rip XIP support out?
>
> How does EFM32 work? Does it work?
I believe that Uwe has a patch series on top of mainline that he still
requires for using the platform.
The part I'm sure about is that it does not work without XIP_KERNEL,
given the tight memory constraints (2MB RAM?).
Arnd
next prev parent reply other threads:[~2015-02-04 14:44 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-30 14:56 Continuing kallsyms failures - large kernels, XIP kernels, and large XIP kernels Russell King - ARM Linux
2015-01-30 15:32 ` Russell King - ARM Linux
2015-01-30 15:34 ` Russell King - ARM Linux
2015-01-30 17:29 ` Nicolas Pitre
2015-01-31 0:22 ` Russell King - ARM Linux
2015-02-04 0:03 ` Russell King - ARM Linux
2015-02-04 1:59 ` Nicolas Pitre
2015-02-04 9:44 ` Russell King - ARM Linux
2015-02-04 13:50 ` Nicolas Pitre
2015-02-04 14:44 ` Arnd Bergmann [this message]
2015-02-05 8:43 ` Uwe Kleine-König
2015-02-06 14:20 ` Uwe Kleine-König
2015-02-06 16:14 ` Russell King - ARM Linux
2015-02-10 8:17 ` Uwe Kleine-König
2015-02-10 19:13 ` Russell King - ARM Linux
2015-02-11 20:46 ` Stephen Rothwell
2015-02-06 20:14 ` Stefan Agner
2015-02-06 14:25 ` Uwe Kleine-König
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=8085056.h8oiDn7EOb@wuerfel \
--to=arnd@arndb.de \
--cc=linux-arm-kernel@lists.infradead.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