From: Jeroen Hofstee <jeroen@myspectrum.nl>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v1] Refactor linker-generated arrays
Date: Sat, 02 Feb 2013 19:42:40 +0100 [thread overview]
Message-ID: <510D5E20.2090406@myspectrum.nl> (raw)
In-Reply-To: <1359824574-32627-2-git-send-email-albert.u.boot@aribaud.net>
Hello Albert,
On 02/02/2013 06:02 PM, Albert ARIBAUD wrote:
> Refactor linker-generated array code so that symbols
> which were previously linker-generated are now compiler-
> generated. This causes relocation records of type
> R_ARM_ABS32 to become R_ARM_RELATIVE, which makes
> code which uses LGA able to run before relocation as
> well as after.
>
> Note: this affects more than ARM targets, as linker-
> lists span possibly all target architectures, notably
> PowerPC.
>
> Signed-off-by: Albert ARIBAUD <albert.u.boot@aribaud.net>
> ---
Command completion currently traps with U-Boot master on a
twister board, as mentioned in [1]. Depending on random,
valid changes in the code, the name pointers in the command
list can become corrupted. Although this patch shouldn't effect
code running after relocation, it does prevent the incorrect
pointers and solves mentioned problem. The exact reason is
unknown, but it might be that code running before relocation
corrupts the data used after relocation.
At least this patch fixes the behaviour for the twister and prevents
that I get completely insane, thanks!
For the twister:
Tested-by: Jeroen Hofstee <jhofstee@myspectrum.nl>
Thanks,
Jeroen
[1] http://lists.denx.de/pipermail/u-boot/2013-February/145709.html
next prev parent reply other threads:[~2013-02-02 18:42 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-02-02 17:02 [U-Boot] [PATCH v1 0/1] Get rid of R_ARM_ABS32 relocation records Albert ARIBAUD
2013-02-02 17:02 ` [U-Boot] [PATCH v1] Refactor linker-generated arrays Albert ARIBAUD
2013-02-02 17:51 ` Daniel Schwierzeck
2013-02-02 17:53 ` Marek Vasut
2013-02-02 18:30 ` Heiko Schocher
2013-02-02 18:42 ` Jeroen Hofstee [this message]
2013-02-04 8:21 ` Heiko Schocher
2013-02-04 8:51 ` Albert ARIBAUD
2013-02-04 12:41 ` Andreas Bießmann
2013-02-04 14:22 ` Albert ARIBAUD
2013-02-16 18:20 ` Albert ARIBAUD
2013-02-18 10:39 ` Andreas Bießmann
2013-02-18 10:42 ` Andreas Bießmann
2013-02-18 16:48 ` Albert ARIBAUD
2013-02-25 10:58 ` [U-Boot] [PATCH v2 0/4] Get rid of R_ARM_ABS32 relocation records Albert ARIBAUD
2013-02-25 10:58 ` [U-Boot] [PATCH v2 1/4] arm: omap: map u_boot_lists section to .sram Albert ARIBAUD
2013-02-25 10:58 ` [U-Boot] [PATCH v2 2/4] Remove linker lists (LGAs) from SPL linker scripts Albert ARIBAUD
2013-02-25 10:58 ` [U-Boot] [PATCH v2 3/4] arm: make __bss_start and __bss_end__ compiler-generated Albert ARIBAUD
2013-02-25 10:59 ` [U-Boot] [PATCH v2 4/4] Refactor linker-generated arrays Albert ARIBAUD
2013-02-25 11:32 ` [U-Boot] [PATCH v2 0/4] Get rid of R_ARM_ABS32 relocation records Marek Vasut
2013-02-25 15:19 ` Tom Rini
2013-02-25 16:20 ` Daniel Schwierzeck
2013-03-13 18:39 ` Tom Rini
2013-02-25 17:22 ` Albert ARIBAUD
2013-03-13 21:15 ` Albert ARIBAUD
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=510D5E20.2090406@myspectrum.nl \
--to=jeroen@myspectrum.nl \
--cc=u-boot@lists.denx.de \
/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