From d9cb1a7dad1f2638cdaa12ff8ade54f9c24c64bc Mon Sep 17 00:00:00 2001 From: Ulrich Drepper Date: Wed, 15 Mar 2000 05:32:11 +0000 Subject: [PATCH] Update. * elf/dl-deps.c (_dl_map_object_deps): Change return type and remove last parameter. Move code to add map to global scope from here... * elf/dl-open.c (dl_open_worker): ...here. * elf/ldsodefs.h: Adjust declaration of _dl_map_object_deps. * elf/rtld.c: Adjust call to _dl_map_object_deps. --- ChangeLog | 6 ++++ elf/dl-deps.c | 86 ++-------------------------------------------------------- elf/dl-open.c | 83 ++++++++++++++++++++++++++++++++++++++++++++++++++++---- elf/ldsodefs.h | 9 +++--- elf/rtld.c | 2 +- 5 files changed, 92 insertions(+), 94 deletions(-) diff --git a/ChangeLog b/ChangeLog index 56c2112..0d362cb 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,11 @@ 2000-03-14 Ulrich Drepper + * elf/dl-deps.c (_dl_map_object_deps): Change return type and remove + last parameter. Move code to add map to global scope from here... + * elf/dl-open.c (dl_open_worker): ...here. + * elf/ldsodefs.h: Adjust declaration of _dl_map_object_deps. + * elf/rtld.c: Adjust call to _dl_map_object_deps. + * locale/programs/ld-collate.c (collate_read): Fix warning. * sysdeps/unix/sysv/linux/i386/sys/ucontext.h: Prepend REG_ to enum diff --git a/elf/dl-deps.c b/elf/dl-deps.c index 68e4921..57d1056 100644 --- a/elf/dl-deps.c +++ b/elf/dl-deps.c @@ -1,5 +1,5 @@ /* Load the dependencies of a mapped object. - Copyright (C) 1996, 1997, 1998, 1999 Free Software Foundation, Inc. + Copyright (C) 1996, 1997, 1998, 1999, 2000 Free Software Foundation, Inc. This file is part of the GNU C Library. The GNU C Library is free software; you can redistribute it and/or @@ -131,16 +131,15 @@ struct list __result; }) -unsigned int +void internal_function _dl_map_object_deps (struct link_map *map, struct link_map **preloads, unsigned int npreloads, - int trace_mode, int global_scope) + int trace_mode) { struct list known[1 + npreloads + 1]; struct list *runp, *utail, *dtail; unsigned int nlist, nduplist, i; - unsigned int to_add = 0; inline void preload (struct link_map *map) { @@ -480,83 +479,4 @@ _dl_map_object_deps (struct link_map *map, else map->l_searchlist.r_duplist[cnt++] = runp->map; } - - /* Now that all this succeeded put the objects in the global scope if - this is necessary. We put the original object and all the dependencies - in the global scope. If an object is already loaded and not in the - global scope we promote it. */ - if (global_scope) - { - unsigned int cnt; - struct link_map **new_global; - - /* Count the objects we have to put in the global scope. */ - for (cnt = 0; cnt < nlist; ++cnt) - if (map->l_searchlist.r_list[cnt]->l_global == 0) - ++to_add; - - /* The symbols of the new objects and its dependencies are to be - introduced into the global scope that will be used to resolve - references from other dynamically-loaded objects. - - The global scope is the searchlist in the main link map. We - extend this list if necessary. There is one problem though: - since this structure was allocated very early (before the libc - is loaded) the memory it uses is allocated by the malloc()-stub - in the ld.so. When we come here these functions are not used - anymore. Instead the malloc() implementation of the libc is - used. But this means the block from the main map cannot be used - in an realloc() call. Therefore we allocate a completely new - array the first time we have to add something to the locale scope. */ - - if (_dl_global_scope_alloc == 0) - { - /* This is the first dynamic object given global scope. */ - _dl_global_scope_alloc = _dl_main_searchlist->r_nlist + to_add + 8; - new_global = (struct link_map **) - malloc (_dl_global_scope_alloc * sizeof (struct link_map *)); - if (new_global == NULL) - { - _dl_global_scope_alloc = 0; - nomem: - _dl_signal_error (ENOMEM, map->l_libname->name, - "cannot extend global scope"); - return 0; - } - - /* Copy over the old entries. */ - memcpy (new_global, _dl_main_searchlist->r_list, - (_dl_main_searchlist->r_nlist * sizeof (struct link_map *))); - - _dl_main_searchlist->r_list = new_global; - } - else if (_dl_main_searchlist->r_nlist + to_add > _dl_global_scope_alloc) - { - /* We have to extend the existing array of link maps in the - main map. */ - new_global = (struct link_map **) - realloc (_dl_main_searchlist->r_list, - ((_dl_global_scope_alloc + to_add + 8) - * sizeof (struct link_map *))); - if (new_global == NULL) - goto nomem; - - _dl_global_scope_alloc += to_add + 8; - _dl_main_searchlist->r_list = new_global; - } - - /* Now add the new entries. */ - to_add = 0; - for (cnt = 0; cnt < nlist; ++cnt) - if (map->l_searchlist.r_list[cnt]->l_global == 0) - { - _dl_main_searchlist->r_list[_dl_main_searchlist->r_nlist + to_add] - = map->l_searchlist.r_list[cnt]; - ++to_add; - } - - /* XXX Do we have to add something to r_dupsearchlist??? --drepper */ - } - - return to_add; } diff --git a/elf/dl-open.c b/elf/dl-open.c index f3d8368..2a03921 100644 --- a/elf/dl-open.c +++ b/elf/dl-open.c @@ -1,5 +1,5 @@ /* Load a shared object at runtime, relocate it, and run its initializer. - Copyright (C) 1996, 1997, 1998, 1999 Free Software Foundation, Inc. + Copyright (C) 1996, 1997, 1998, 1999, 2000 Free Software Foundation, Inc. This file is part of the GNU C Library. The GNU C Library is free software; you can redistribute it and/or @@ -87,7 +87,6 @@ dl_open_worker (void *a) struct link_map *new, *l; ElfW(Addr) init; struct r_debug *r; - unsigned int global_add; const char *dst; int lazy; @@ -147,7 +146,7 @@ dl_open_worker (void *a) return; /* Load that object's dependencies. */ - global_add = _dl_map_object_deps (new, NULL, 0, 0, mode & RTLD_GLOBAL); + _dl_map_object_deps (new, NULL, 0, 0); /* So far, so good. Now check the versions. */ (void) _dl_check_all_versions (new, 0); @@ -208,8 +207,82 @@ dl_open_worker (void *a) __environ); /* Now we can make the new map available in the global scope. */ - while (global_add-- > 0) - _dl_main_searchlist->r_list[_dl_main_searchlist->r_nlist++]->l_global = 1; + if (mode & RTLD_GLOBAL) + { + struct link_map **new_global; + unsigned int to_add = 0; + unsigned int cnt; + + /* Count the objects we have to put in the global scope. */ + for (cnt = 0; cnt < new->l_searchlist.r_nlist; ++cnt) + if (new->l_searchlist.r_list[cnt]->l_global == 0) + ++to_add; + + /* The symbols of the new objects and its dependencies are to be + introduced into the global scope that will be used to resolve + references from other dynamically-loaded objects. + + The global scope is the searchlist in the main link map. We + extend this list if necessary. There is one problem though: + since this structure was allocated very early (before the libc + is loaded) the memory it uses is allocated by the malloc()-stub + in the ld.so. When we come here these functions are not used + anymore. Instead the malloc() implementation of the libc is + used. But this means the block from the main map cannot be used + in an realloc() call. Therefore we allocate a completely new + array the first time we have to add something to the locale scope. */ + + if (_dl_global_scope_alloc == 0) + { + /* This is the first dynamic object given global scope. */ + _dl_global_scope_alloc = _dl_main_searchlist->r_nlist + to_add + 8; + new_global = (struct link_map **) + malloc (_dl_global_scope_alloc * sizeof (struct link_map *)); + if (new_global == NULL) + { + _dl_global_scope_alloc = 0; + nomem: + _dl_signal_error (ENOMEM, new->l_libname->name, + "cannot extend global scope"); + return; + } + + /* Copy over the old entries. */ + memcpy (new_global, _dl_main_searchlist->r_list, + (_dl_main_searchlist->r_nlist * sizeof (struct link_map *))); + + _dl_main_searchlist->r_list = new_global; + } + else if (_dl_main_searchlist->r_nlist + to_add > _dl_global_scope_alloc) + { + /* We have to extend the existing array of link maps in the + main map. */ + new_global = (struct link_map **) + realloc (_dl_main_searchlist->r_list, + ((_dl_global_scope_alloc + to_add + 8) + * sizeof (struct link_map *))); + if (new_global == NULL) + goto nomem; + + _dl_global_scope_alloc += to_add + 8; + _dl_main_searchlist->r_list = new_global; + } + + /* Now add the new entries. */ + for (cnt = 0; cnt < new->l_searchlist.r_nlist; ++cnt) + { + struct link_map *map = new->l_searchlist.r_list[cnt]; + + if (map->l_global == 0) + { + map->l_global = 1; + _dl_main_searchlist->r_list[_dl_main_searchlist->r_nlist] = map; + ++_dl_main_searchlist->r_nlist; + } + } + + /* XXX Do we have to add something to r_dupsearchlist??? --drepper */ + } if (_dl_sysdep_start == NULL) /* We must be the static _dl_open in libc.a. A static program that diff --git a/elf/ldsodefs.h b/elf/ldsodefs.h index 15b7cc6..15f89db 100644 --- a/elf/ldsodefs.h +++ b/elf/ldsodefs.h @@ -1,5 +1,5 @@ /* Run-time dynamic linker data structures for loaded ELF shared objects. - Copyright (C) 1995, 1996, 1997, 1998, 1999 Free Software Foundation, Inc. + Copyright (C) 1995-1999, 2000 Free Software Foundation, Inc. This file is part of the GNU C Library. The GNU C Library is free software; you can redistribute it and/or @@ -238,10 +238,9 @@ extern struct link_map *_dl_map_object (struct link_map *loader, MAP->l_searchlist. PRELOADS points to a vector of NPRELOADS previously loaded objects that will be inserted into MAP->l_searchlist after MAP but before its dependencies. */ -extern unsigned int _dl_map_object_deps (struct link_map *map, - struct link_map **preloads, - unsigned int npreloads, - int trace_mode, int global_scope) +extern void _dl_map_object_deps (struct link_map *map, + struct link_map **preloads, + unsigned int npreloads, int trace_mode) internal_function; /* Cache the locations of MAP's hash table. */ diff --git a/elf/rtld.c b/elf/rtld.c index 20a6b31..94b9386 100644 --- a/elf/rtld.c +++ b/elf/rtld.c @@ -776,7 +776,7 @@ of this helper program; chances are you did not intend to run this program.\n\ specified some libraries to load, these are inserted before the actual dependencies in the executable's searchlist for symbol resolution. */ HP_TIMING_NOW (start); - _dl_map_object_deps (_dl_loaded, preloads, npreloads, mode == trace, 0); + _dl_map_object_deps (_dl_loaded, preloads, npreloads, mode == trace); HP_TIMING_NOW (stop); HP_TIMING_DIFF (diff, start, stop); HP_TIMING_ACCUM_NT (load_time, diff); -- 2.7.4