This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: ld -z options


On Thu, Jul 14, 2011 at 04:06:03AM -0400, Hans-Peter Nilsson wrote:
> Talking to myself...
> 
> On Thu, 14 Jul 2011, Hans-Peter Nilsson wrote:
> > This caused several test-cases to go from PASS to UNRESOLVED for
> > cris-axis-linux-gnu
> 
> It seems this would happen for e.g. ia64-*-linux-gnu too.

Yes.

> > All preceded by a linker warning ".../ld-new: warning: -z relro
> > ignored." in ld.log.  The UNRESOLVED status then cause the
> > check-ld to return error.  According to the dejagnu docs, the
> > UNRESOLVED is an indication of an error in the testsuite (likely
> > the unexpected warning).
> 
> That's a bug in ld-elf/binutils.exp or the new warning message;
> binutils.exp tests [string match "*not supported*" $link_output]
>              || [string match "*unrecognized option*" $link_output]
> which doesn't match "warning: -z relro ignored.".
> 
> Which one should be adjusted?

Probably the new one.  Patch preapproved if you'd like to do it.

> > Hm, at a second glance many other ports set
> > COMMONPAGESIZE="CONSTANT (COMMONPAGESIZE)" which for some reason
> > triggers the recognition of e.g. "-z relro" but why would I have
> > to set that when there's just one page-size?
> 
> This seems the wrong default, and emulparams/elf64_ia64.sh
> should set something to disable -z relro instead of everyone
> else setting some vaguely related variable.  Or is that's too
> linux-centric?  Maybe create a ${srcdir}/emulparams/linux-common.sh
> for emulparams/*.sh to include?

I had the same idea, but see my other mail about bfd backend and ld
needing to agree.

-- 
Alan Modra
Australia Development Lab, IBM


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]