[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: [Patch]: Fix ld pr11138 FAILures on mips*.

On 12/05/2011 09:40 PM, Alan Modra wrote:
On Mon, Dec 05, 2011 at 04:49:35PM -0800, David Daney wrote:
The root cause of this is that the mips linker synthesizes a special
symbol "__RLD_MAP", and then sets MIPS_RLD_MAP to point to it.  When
a version script is present, this symbol gets versioned along with
all the rest, and when it is time to take its address, the symbol
can no longer be found as it has had version information appended to
its name.

Why not just change

	&&  (strcmp (name, "__rld_map") == 0
	      || strcmp (name, "__RLD_MAP") == 0))


	&&  (strncmp (name, "__rld_map", 9) == 0
	      || strncmp (name, "__RLD_MAP", 9) == 0))

in _bfd_mips_elf_finish_dynamic_symbol?  Perhaps the same for other
syms there too?

Because that doesn't work. Perhpas I should have been a bit more detailed in my description of what is happening (at least in one case).

If the version script contains something like:
        global: main;
        local: *;

Then "__RLD_MAP" gets hidden and we never see it in _bfd_mips_elf_finish_dynamic_symbol().

This hiding gets done precisely in _bfd_elf_link_assign_sym_version() after the version information is calculated. So as the patch stands, we bail out of _bfd_elf_link_assign_sym_version() before the symbol is hidden (or modified in any way).

It is possible that 'no_sym_version' is not the best name for the flag, but I think we really some sort of flag to exclude ABI symbols from being mangled in _bfd_elf_link_assign_sym_version().

David Daney

Reply to: