RE: heads up: apparent GAS bug in current (CVS) binutils

From: David Mosberger <davidm_at_napali.hpl.hp.com>
Date: 2005-05-18 06:53:51
>>>>> On Tue, 17 May 2005 13:48:51 -0700, "Luck, Tony" <tony.luck@intel.com> said:

  >> +++ linux/arch/ia64/kernel/smpboot.c	2005-05-13 
  >> -struct pt_regs * __devinit idle_regs(struct pt_regs *regs)
  >> +struct pt_regs *
  >> +__devinit __comdatfunc (idle_regs)
  >> +idle_regs(struct pt_regs *regs)

  >> +++ linux/kernel/fork.c	2005-05-13 13:19:40.000000000 -0700
  >> -struct pt_regs * __devinit __attribute__((weak)) 
  >> idle_regs(struct pt_regs *regs)
  >> +struct pt_regs *
  >> +__devinit __comdatfunc (idle_regs)
  >> +idle_regs(struct pt_regs *regs)

  Tony> So is there some deeper magic that I don't understand, or is this
  Tony> wrong?  After this patch is applied, both of these definitions look
  Tony> identical to me.  So how does the linker know which one to use?

Please do not apply this patch.  The bug is in the toolchain.
(The kernel may be ugly in this corner and we may want to fix
that by getting rid of the weak definitions but that issue is
entirely independent of the toolchain bug.)

	--david
-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Received on Tue May 17 16:54:05 2005

This archive was generated by hypermail 2.1.8 : 2005-08-02 09:20:39 EST