This is the mail archive of the
binutils@sources.redhat.com
mailing list for the binutils project.
Re: DT_TEXTREL/.dynamic issue with the binutils 2.15 linker on ARM and Linux
- From: Daniel Jacobowitz <drow at false dot org>
- To: David Poole <daveml at mbuf dot com>
- Cc: binutils at sources dot redhat dot com, nickc at redhat dot com, Richard dot Earnshaw at arm dot com
- Date: Fri, 29 Oct 2004 16:13:40 -0400
- Subject: Re: DT_TEXTREL/.dynamic issue with the binutils 2.15 linker on ARM and Linux
- References: <B55B7886-B0B6-11D8-8CFA-000393CDE218@mbuf.com> <20040528145608.GA556@nevyn.them.org> <20040529060633.GH3368@bubble.modra.org> <FF30F76A-B3EC-11D8-93CF-000393CDE218@mbuf.com>
On Tue, Jun 01, 2004 at 10:59:08AM -0600, David Poole wrote:
> On May 29, 2004, at 12:06 AM, Alan Modra wrote:
>
> >On Fri, May 28, 2004 at 10:56:09AM -0400, Daniel Jacobowitz wrote:
> >>I don't know why DT_TEXTREL isn't there for this case. It should be.
> >
> >Perhaps because elf32-arm.h doesn't have something like
> >readonly_dynrelocs as implemented in other backends.
>
> Looking at elf32-i386.c and elf32-s390.c, I copied the
> readonly_dynrelocs() and the call in
> elf_(i386|s390)_size_dynamic_sections.
>
The patch from the message I'm quoting looks right to me:
http://sources.redhat.com/ml/binutils/2004-06/msg00010.html
However, to apply it, two things need to happen:
- An ARM maintainer needs to review it.
- Someone needs to decide whether it requires a copyright assignment
(my instinct is no, but I don't make these decisions).
BTW, you said you still needed the patch with 2.15. I didn't try 2.15,
but you definitely don't need it for HEAD; you'll get a PLT entry for
the branch. However, if I change the test to use .word instead of bl,
then it's obvious that your patch is needed.
--
Daniel Jacobowitz