[uClibc-cvs] uClibc/ldso/ldso/cris dl-sysdep.h, 1.4, 1.5 elfinterp.c, 1.7, 1.8

[Erik Andersen]

Update of /var/cvs/uClibc/ldso/ldso/cris
In directory nail:/tmp/cvs-serv9702/ldso/cris

Modified Files:
	dl-sysdep.h elfinterp.c 
Log Message:
Joakim Tjernlund writes:

Hi it is me again.

This is the latest ldso patch. the NEW weak symbol handling works now
with a little special handling in _dl_find_hash(). You get to chose
if you want the new or old handling :)

There was 2 missing _dl_check_if_named_library_is_loaded() calls in _dlopen().

I then disabled the _dl_check_if_named_library_is_loaded() in dl-elf.c since
it is rendundant.

Question, why does some _dl_linux_resolver(), like i386, have 2 calls
to _dl_find_hash()? I think that is wrong, isn't it?

I really hope you can check this out soon ...



Index: dl-sysdep.h
===================================================================
RCS file: /var/cvs/uClibc/ldso/ldso/cris/dl-sysdep.h,v
retrieving revision 1.4
retrieving revision 1.5
diff -u -d -r1.4 -r1.5
--- dl-sysdep.h	10 Feb 2004 09:26:48 -0000	1.4
+++ dl-sysdep.h	14 Feb 2004 11:30:32 -0000	1.5
@@ -110,3 +110,19 @@
 #define PAGE_ALIGN 0xffffe000
 #define ADDR_ALIGN 0x1fff
 #define OFFS_ALIGN 0xffffe000
+
+/* The union of reloc-type-classes where the reloc TYPE is a member.
+
+   TYPE is in the class ELF_RTYPE_CLASS_PLT if it can describe a
+   relocation for a PLT entry, that is, for which a PLT entry should not
+   be allowed to define the value.  The GNU linker for CRIS can merge a
+   .got.plt entry (R_CRIS_JUMP_SLOT) with a .got entry (R_CRIS_GLOB_DAT),
+   so we need to match both these reloc types.
+
+   TYPE is in the class ELF_RTYPE_CLASS_NOCOPY if it should not be allowed
+   to resolve to one of the main executable's symbols, as for a COPY
+   reloc.  */
+#define elf_machine_type_class(type)				\
+  ((((((type) == R_CRIS_JUMP_SLOT))				\
+     || ((type) == R_CRIS_GLOB_DAT)) * ELF_RTYPE_CLASS_PLT)	\
+   | (((type) == R_CRIS_COPY) * ELF_RTYPE_CLASS_COPY))

Index: elfinterp.c
===================================================================
RCS file: /var/cvs/uClibc/ldso/ldso/cris/elfinterp.c,v
retrieving revision 1.7
retrieving revision 1.8
diff -u -d -r1.7 -r1.8
--- elfinterp.c	29 Jan 2004 12:18:51 -0000	1.7
+++ elfinterp.c	14 Feb 2004 11:30:32 -0000	1.8
@@ -142,16 +142,10 @@
 	got_addr = (char **) instr_addr;
 
 	/* Fetch the address of the GOT entry. */
-	new_addr = _dl_find_hash(symname, tpnt->symbol_scope, tpnt, resolver);
-
+	new_addr = _dl_find_hash(symname, tpnt->symbol_scope, ELF_RTYPE_CLASS_PLT);
 	if (!new_addr) {
-		new_addr = _dl_find_hash(symname, NULL, NULL, resolver);
-
-		if (new_addr)
-			return (unsigned long) new_addr;
-
-		_dl_dprintf(2, "%s: Can't resolv symbol '%s'\n", _dl_progname, symname);
-		_dl_exit(1);
+	    _dl_dprintf(2, "%s: Can't resolv symbol '%s'\n", _dl_progname, symname);
+	    _dl_exit(1);
 	}
 
 #if defined (__SUPPORT_LD_DEBUG__)
@@ -261,7 +255,7 @@
 		}
 		else {
 			symbol_addr = (unsigned long) _dl_find_hash(symname, scope,
-				(reloc_type == R_CRIS_JUMP_SLOT ? tpnt : NULL), symbolrel);
+								    elf_machine_type_class(reloc_type));
 		}
 
 		if (!symbol_addr && ELF32_ST_BIND(symtab[symtab_index].st_info) == STB_GLOBAL) {
@@ -366,7 +360,7 @@
 	goof = 0;
 
 	if (symtab_index) {
-		symbol_addr = (unsigned long) _dl_find_hash(symname, scope, NULL, copyrel);
+		symbol_addr = (unsigned long) _dl_find_hash(symname, scope, ELF_RTYPE_CLASS_COPY);
 
 		if (!symbol_addr)
 			goof++;