From: Alexander Holler <holler@ahsoftware.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH V3 1/3] arm926ejs: fix linker file for newer ld support
Date: Thu, 04 Nov 2010 20:39:24 +0100 [thread overview]
Message-ID: <4CD30BEC.1050203@ahsoftware.de> (raw)
In-Reply-To: <4CD3076C.3080905@free.fr>
Hello,
Am 04.11.2010 20:20, schrieb Albert ARIBAUD:
>> If I read that patch (66_*) correctly, it's the same as when using
>> --warn-shared-textrel in LDFLAGS.
>
> All right. I don't like getting a warning like that, but as I cannot
> reproduce your toolchain, I cannot avoid it.
I think they enabbled this by default to do some kind of QA, because
those textrels are forcing the dynamic linker to resolve some stuff
which otherwise (without those textrels) isn't needed. And this results
in longer load times.
Sidenote: My toolchain is reproducable, but that means you have to
compile a Gentoo system, because Gentoo is a source only distribution.
So it isn't as comfortable as using binary distributions (here), but it
has other advantages.
> Regarding the build you sent me, the fixup for monitor_flash_len is
> present in the fixup table at the 11th entry at 00752fb8 and the fixup
> table is correctly accessible using _rel_dyn_start_ofs and
> _rel_dyn_end_ofs. That means your code should have done the fixup if the
> relocation loop was run correctly.
>
> Can you do a step-by-step run of the fixup loop under a debugger?
Sorry, no. That thing has 2mm-header for the jtag and I don't have any
prefabricated cable for that. Maybe someone else with binutils 2.20.1
could check your V3. I don't know if I will find the time test your v3
in the next days with some other hw where I can attach a jtag.
Regards,
Alexander
next prev parent reply other threads:[~2010-11-04 19:39 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-11-03 18:24 [U-Boot] [PATCH V3 1/3] arm926ejs: fix linker file for newer ld support Albert Aribaud
2010-11-03 18:24 ` [U-Boot] [PATCH V3 2/3] tx25: " Albert Aribaud
2010-11-03 18:24 ` [U-Boot] [PATCH V3 3/3] tx25: increase u-boot NAND size to match current build sizes Albert Aribaud
2010-11-03 18:27 ` [U-Boot] [PATCH V3 1/3] arm926ejs: fix linker file for newer ld support Albert ARIBAUD
2010-11-04 10:26 ` Alexander Holler
2010-11-04 10:46 ` Alexander Holler
2010-11-04 12:40 ` Albert ARIBAUD
2010-11-04 12:47 ` Alexander Holler
2010-11-04 12:55 ` Albert ARIBAUD
2010-11-04 13:18 ` Alexander Holler
2010-11-04 16:56 ` Albert ARIBAUD
2010-11-04 18:37 ` Alexander Holler
2010-11-04 18:42 ` Alexander Holler
2010-11-04 19:20 ` Albert ARIBAUD
2010-11-04 19:39 ` Alexander Holler [this message]
2010-11-04 22:06 ` 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=4CD30BEC.1050203@ahsoftware.de \
--to=holler@ahsoftware.de \
--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