Re: [Patch v2]: Fix ld pr11138 FAILures on mips*.
- To: David Daney <david.daney@cavium.com>
- Cc: binutils <binutils@sourceware.org>, Alan Modra <amodra@gmail.com>, Manuel Lauss <manuel.lauss@googlemail.com>, Debian MIPS <debian-mips@lists.debian.org>, linux-mips <linux-mips@linux-mips.org>
- Subject: Re: [Patch v2]: Fix ld pr11138 FAILures on mips*.
- From: Richard Sandiford <rdsandiford@googlemail.com>
- Date: Sat, 10 Dec 2011 10:19:20 +0000
- Message-id: <[🔎] 87y5ukenkn.fsf@firetop.home>
- Mail-followup-to: David Daney <david.daney@cavium.com>,binutils <binutils@sourceware.org>, Alan Modra <amodra@gmail.com>, Manuel Lauss <manuel.lauss@googlemail.com>, Debian MIPS <debian-mips@lists.debian.org>, linux-mips <linux-mips@linux-mips.org>, rdsandiford@googlemail.com
- In-reply-to: <[🔎] 4EE2ACB9.9010301@cavium.com> (David Daney's message of "Fri, 09 Dec 2011 16:50:01 -0800")
- References: <[🔎] 4EE27012.5030508@cavium.com> <[🔎] 20111210003928.GC2461@bubble.grove.modra.org> <[🔎] 4EE2ACB9.9010301@cavium.com>
David Daney <david.daney@cavium.com> writes:
> I will wait a couple of days to give Richard a chance to object.
Looks good to me too. Thanks for doing this. I think it should go
on the 2.22 branch as well.
> I am especially concerned about what happens on IRIX where the symbol
> comes in from an external object rather than being generated by the
> linker itself. I had no way to test that.
Me neither, unfortunately. But I agree it looks right. There's a
possibility that we could create the dynamic sections before the
definition of __rld_obj_head has been read in (e.g. from the
check_relocs of a previous object). In that case it looks like
we would create the .rld_map section and __rld_map/__RLD_MAP symbol,
then later switch to __rld_obj_head. .rld_map would then be left
at zero size and the __rld_map/__RLD_MAP symbol would be ignored
when setting the tag value. But your patch preserves the
behaviour in that case too.
Richard
Reply to: