From: Carlos O'Donell <carlos@systemhalted.org>
To: John David Anglin <dave.anglin@bell.net>
Cc: John David Anglin <dave.anglin@nrc.ca>,
linux-parisc <linux-parisc@vger.kernel.org>
Subject: Re: Sort output dynamic relocations.
Date: Mon, 11 Jul 2011 09:55:26 -0400 [thread overview]
Message-ID: <4E1B00CE.9030707@systemhalted.org> (raw)
In-Reply-To: <4E1AFB99.5090402@bell.net>
On 7/11/2011 9:33 AM, John David Anglin wrote:
> On 7/10/2011 9:27 PM, Carlos O'Donell wrote:
>> On Sat, Jul 9, 2011 at 11:08 AM, John David
>> Anglin<dave.anglin@bell.net> wrote:
>>> As far as I can tell, most targets don't sort dynamic
>>> relocations. The exceptions that I see are mips and score.
>>> Doing this will increase link time, so I tend to think it should
>>> be avoided if possible. Isn't the dynamic loader a special
>>> case?
>> Ulrich Drepper says that relative relocations must *always* be
>> first, it doesn't require a complete sort.
>>
> I'm not knowledgeable about x86 but I don't believe it has a
> relocation equivalent to the PLABEL32 reloc. Everything in .rel.dyn
> may be relative. However, R_386_RELATIVE seems to occur before
> R_386_GLOB_DATA. I'm not sure how this is done.
It might just be a happy accident of the input order.
> If we need to sort, combreloc may need to be disabled because it
> might do an incompatible sort.
OK.
>> The dynamic loader is a special case, but there are some
>> requirements.
>>
>>> Have you tried the ld "-z combreloc" option to see if that works?
>>> It combines reloc sections and sorts them.
>> I haven't tried that yet, are you saying we should change binutils
>> to *default* to "-z combreloc?"
> No. I was just thinking of trying to link the dynamic loader with
> this option. Didn't notice any other targets making it the default.
Right! That's a good idea, I'll give this a try.
Yes, the dynamic linker is the only thing that has this requirement.
Cheers,
Carlos.
prev parent reply other threads:[~2011-07-11 13:55 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-07-07 15:29 Sort output dynamic relocations Carlos O'Donell
2011-07-07 17:44 ` John David Anglin
2011-07-07 23:13 ` Carlos O'Donell
2011-07-09 15:08 ` John David Anglin
2011-07-11 1:27 ` Carlos O'Donell
2011-07-11 13:33 ` John David Anglin
2011-07-11 13:55 ` Carlos O'Donell [this message]
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=4E1B00CE.9030707@systemhalted.org \
--to=carlos@systemhalted.org \
--cc=dave.anglin@bell.net \
--cc=dave.anglin@nrc.ca \
--cc=linux-parisc@vger.kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.