Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 21 Feb 2002 17:26:21 -0500 (EST)
From:      Andrew Gallatin <gallatin@cs.duke.edu>
To:        Doug Rabson <dfr@nlsystems.com>
Cc:        peter@freebsd.org, jdp@freebsd.org, <freebsd-alpha@freebsd.org>
Subject:   Re: alpha klds and the new linker
Message-ID:  <15477.29709.908299.287618@grasshopper.cs.duke.edu>
In-Reply-To: <Pine.BSF.4.33.0202211623290.89657-100000@herring.nlsystems.com>
References:  <15477.3715.6908.110667@grasshopper.cs.duke.edu> <Pine.BSF.4.33.0202211623290.89657-100000@herring.nlsystems.com>

next in thread | previous in thread | raw e-mail | index | archive | help

Doug Rabson writes:
 > On Thu, 21 Feb 2002, Andrew Gallatin wrote:
 > 
 > >
 > > Doug Rabson writes:
 > >  > On Wed, 20 Feb 2002, Andrew Gallatin wrote:
 > >  >
 > >  > >
 > >  > > Doug,
 > >  > >
 > >  > > There's something about the new linker (from the latest binutils
 > >  > > import) which breaks klds on alpha.  If I attempt to load a module
 > >  > > built with the new binutils, things go kaboom like this:
 > >  >
 > >  > >From looking at this, I can't say much other than that it looks like the
 > >  > symbol lookup for "modmetadata_set" returned something bogus. Either that
 > >  > or maybe the linker sets didn't get relocated properly. What does
 > >  > 'objdump -p' and 'objdump -R' look like for the two files?
 > >
 > >
 > > Due to their size, I've left the objdump -R output on freefall:
 > >
 > >      http://people.freebsd.org/~gallatin/objdump-R_new
 > >      http://people.freebsd.org/~gallatin/objdump-R_old
 > 
 > I guess it might be something to do with all those RELATIVE relocs which
 > have addends. I can't see anything obviously wrong with the code but it
 > has never been called with non-zero addends so there could easily be a
 > problem there. Wasn't there something similar with rtld recently?


I think I've fixed it.  If I remove the '*where' from the addr
calculation, all seems to be fine.   I don't really know enough about
how elf is supposed to work to want to commit this without a review.

Anybody care to say this is "OK" ?

Index: elf_machdep.c
===================================================================
RCS file: /home/ncvs/src/sys/alpha/alpha/elf_machdep.c,v
retrieving revision 1.7
diff -u -r1.7 elf_machdep.c
--- elf_machdep.c	28 Aug 1999 00:38:25 -0000	1.7
+++ elf_machdep.c	21 Feb 2002 22:22:14 -0000
@@ -101,7 +101,7 @@
 			break;
 
 		case R_ALPHA_RELATIVE:
-			addr = relocbase + addend + *where;
+			addr = relocbase + addend;
 			if (*where != addr)
 				*where = addr;
 			break;


Thanks,


Drew

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-alpha" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?15477.29709.908299.287618>