Update.
authorUlrich Drepper <drepper@redhat.com>
Wed, 15 Mar 2000 05:32:11 +0000 (05:32 +0000)
committerUlrich Drepper <drepper@redhat.com>
Wed, 15 Mar 2000 05:32:11 +0000 (05:32 +0000)
* 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
elf/dl-deps.c
elf/dl-open.c
elf/ldsodefs.h
elf/rtld.c

index 56c2112..0d362cb 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,11 @@
 2000-03-14  Ulrich Drepper  <drepper@redhat.com>
 
+       * 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
index 68e4921..57d1056 100644 (file)
@@ -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;
 }
index f3d8368..2a03921 100644 (file)
@@ -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
index 15b7cc6..15f89db 100644 (file)
@@ -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.  */
index 20a6b31..94b9386 100644 (file)
@@ -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);