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: PATCH: PR ld/10433: Latest ld fails to link ldconfig properly


On 22.07.2009 23:11, Mike Frysinger wrote:
On Wednesday 22 July 2009 20:41:23 David Daney wrote:
H.J. Lu wrote:
On Wed, Jul 22, 2009 at 4:34 PM, Alan Modra wrote:
On Wed, Jul 22, 2009 at 02:32:27PM -0700, H.J. Lu wrote:
A STT_GNU_IFUNC symbol goes through PLT only if it is ever referenced.
I checked in this patch to fix it.

You don't have global authority to commit patches. I'd normally let this sort of patch pass, but I quite clearly explained why I removed your test of ref_regular before. Please revert this patch and instead fix it in your target finish_dynamic_symbol.

I disagree. If ref_regular isn't set properly, IFUNC may not work properly at all. I will revert my patch and move it to the Linux binutils instead.

Granted all GNU/Linux systems use your binutils fork, but could you explain why the issue is important enough to fix in your binutils fork, but doesn't merit fixing in the sourceware.org version?

that's not really true. we've switched Gentoo to the GNU releases by default (while still making the HJL versions available). mips tended to be screwed up in the non-GNU releases, and havent found any real pressing need to use the HJL versions in general. plus, it's a lot easier to stabilize on a stable release than one made against the latest development tree. -mike

I'm using the FSF releases/branches for Debian as well; currently using trunk updated to the same base that it used for the HJL releases until the next FSF release. This works well except for mips, which shoes regressions in 2.19 and trunk (PR10144), and in trunk (-fPIE broken, http://bugs.debian.org/532821).


Ubuntu does use a subset of the patches, which H.J. includes separately in the patches directory. AFAICS Fedora does do the same (at least reverting some patches).

Matthias


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