]> git.ipfire.org Git - thirdparty/glibc.git/commitdiff
Update.
authorUlrich Drepper <drepper@redhat.com>
Mon, 8 Jun 1998 13:37:25 +0000 (13:37 +0000)
committerUlrich Drepper <drepper@redhat.com>
Mon, 8 Jun 1998 13:37:25 +0000 (13:37 +0000)
1998-06-08 13:34  Ulrich Drepper  <drepper@cygnus.com>

* elf/sprof.c: Implement call graph profiling.

* sysdeps/generic/getenv.c: Optimize to use strncmp less often.

1998-06-07  Andreas Jaeger  <aj@arthur.rhein-neckar.de>

* pwd/fgetpwent_r.c (__fgetpwent_r): Correct buffer overflow fix.
* grp/fgetgrent_r.c (__fgetgrent_r): Likewise.
* shadow/fgetspent_r.c (__fgetspent_r): Likewise.
Noticed by Jake Garver <garver@valkyrie.net>.

1998-06-07  Andreas Schwab  <schwab@issan.informatik.uni-dortmund.de>

* libio/genops.c (__underflow): Read character from read pointer
as unsigned.
(__uflow): Likewise.

ChangeLog
elf/sprof.c
grp/fgetgrent_r.c
libio/genops.c
pwd/fgetpwent_r.c
shadow/fgetspent_r.c
sysdeps/generic/getenv.c

index cc8ba2f5f8063b65beb5e145982c958c40f717e1..f50ad340024d26103f25b4d39e18f8d20d1e925e 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,22 @@
+1998-06-08 13:34  Ulrich Drepper  <drepper@cygnus.com>
+
+       * elf/sprof.c: Implement call graph profiling.
+
+       * sysdeps/generic/getenv.c: Optimize to use strncmp less often.
+
+1998-06-07  Andreas Jaeger  <aj@arthur.rhein-neckar.de>
+
+       * pwd/fgetpwent_r.c (__fgetpwent_r): Correct buffer overflow fix.
+       * grp/fgetgrent_r.c (__fgetgrent_r): Likewise.
+       * shadow/fgetspent_r.c (__fgetspent_r): Likewise.
+       Noticed by Jake Garver <garver@valkyrie.net>.
+
+1998-06-07  Andreas Schwab  <schwab@issan.informatik.uni-dortmund.de>
+
+       * libio/genops.c (__underflow): Read character from read pointer
+       as unsigned.
+       (__uflow): Likewise.
+
 1998-06-07 13:32  Ulrich Drepper  <drepper@cygnus.com>
 
        * libc.map: Add _dl_profile.
index 477d95a283f8f58caf0fc0a1867a9815b4d89aa5..f54a61657fd83963be5cf63af269196b729b9c3a 100644 (file)
@@ -76,6 +76,7 @@ static const struct argp_option options[] =
   { NULL, 0, NULL, 0, N_("Output selection:") },
   { "flat-profile", 'p', NULL, 0,
     N_("generate flat profile with counts and ticks") },
+  { "graph", 'q', NULL, 0, N_("generate call graph") },
 
   { "test", OPT_TEST, NULL, OPTION_HIDDEN, NULL },
   { NULL, 0, NULL, 0, NULL }
@@ -102,8 +103,9 @@ static enum
 {
   NONE = 0,
   FLAT_MODE = 1 << 0,
+  CALL_GRAPH_MODE = 1 << 1,
 
-  DEFAULT_MODE = FLAT_MODE
+  DEFAULT_MODE = FLAT_MODE | CALL_GRAPH_MODE
 } mode;
 
 /* If nonzero the total number of invocations of a function is emitted.  */
@@ -114,20 +116,32 @@ int do_test;
 
 /* Strcuture describing calls.  */
 struct here_fromstruct
-  {
-    struct here_cg_arc_record volatile *here;
-    uint16_t link;
-  };
+{
+  struct here_cg_arc_record volatile *here;
+  uint16_t link;
+};
 
 /* We define a special type to address the elements of the arc table.
    This is basically the `gmon_cg_arc_record' format but it includes
    the room for the tag and it uses real types.  */
 struct here_cg_arc_record
-  {
-    uintptr_t from_pc;
-    uintptr_t self_pc;
-    uint32_t count;
-  } __attribute__ ((packed));
+{
+  uintptr_t from_pc;
+  uintptr_t self_pc;
+  uint32_t count;
+} __attribute__ ((packed));
+
+
+struct known_symbol;
+struct arc_list
+{
+  size_t idx;
+  uintmax_t count;
+
+  struct arc_list *next;
+};
+
+static struct obstack ob_list;
 
 
 struct known_symbol
@@ -138,6 +152,9 @@ struct known_symbol
 
   uintmax_t ticks;
   uintmax_t calls;
+
+  struct arc_list *froms;
+  struct arc_list *tos;
 };
 
 
@@ -198,7 +215,9 @@ static void unload_profdata (struct profdata *profdata);
 static void count_total_ticks (struct shobj *shobj, struct profdata *profdata);
 static void count_calls (struct shobj *shobj, struct profdata *profdata);
 static void read_symbols (struct shobj *shobj);
+static void add_arcs (struct profdata *profdata);
 static void generate_flat_profile (struct profdata *profdata);
+static void generate_call_graph (struct profdata *profdata);
 
 
 int
@@ -278,6 +297,9 @@ no filename for profiling data given and shared object `%s' has no soname"),
   /* Count the calls.  */
   count_calls (shobj_handle, profdata_handle);
 
+  /* Add the arc information.  */
+  add_arcs (profdata_handle);
+
   /* If no mode is specified fall back to the default mode.  */
   if (mode == NONE)
     mode = DEFAULT_MODE;
@@ -286,6 +308,9 @@ no filename for profiling data given and shared object `%s' has no soname"),
   if (mode & FLAT_MODE)
     generate_flat_profile (profdata_handle);
 
+  if (mode & CALL_GRAPH_MODE)
+    generate_call_graph (profdata_handle);
+
   /* Free the resources.  */
   unload_shobj (shobj_handle);
   unload_profdata (profdata_handle);
@@ -300,6 +325,12 @@ parse_opt (int key, char *arg, struct argp_state *state)
 {
   switch (key)
     {
+    case 'p':
+      mode |= FLAT_MODE;
+      break;
+    case 'q':
+      mode |= CALL_GRAPH_MODE;
+      break;
     case OPT_TEST:
       do_test = 1;
       break;
@@ -800,7 +831,7 @@ count_total_ticks (struct shobj *shobj, struct profdata *profdata)
 }
 
 
-static struct known_symbol *
+static size_t
 find_symbol (uintptr_t addr)
 {
   size_t sidx = 0;
@@ -811,7 +842,7 @@ find_symbol (uintptr_t addr)
       uintptr_t end = start + sortsym[sidx]->size;
 
       if (addr >= start && addr < end)
-       return sortsym[sidx];
+       return sidx;
 
       if (addr < start)
        break;
@@ -819,7 +850,7 @@ find_symbol (uintptr_t addr)
       ++sidx;
     }
 
-  return NULL;
+  return (size_t) -1l;
 }
 
 
@@ -833,12 +864,12 @@ count_calls (struct shobj *shobj, struct profdata *profdata)
   for (cnt = 0; cnt < narcs; ++cnt)
     {
       uintptr_t here = data[cnt].self_pc;
-      struct known_symbol *symbol;
+      size_t symbol_idx;
 
       /* Find the symbol for this address.  */
-      symbol = find_symbol (here);
-      if (symbol != NULL)
-       symbol->calls += data[cnt].count;
+      symbol_idx = find_symbol (here);
+      if (symbol_idx != (size_t) -1l)
+       sortsym[symbol_idx]->calls += data[cnt].count;
     }
 }
 
@@ -846,8 +877,8 @@ count_calls (struct shobj *shobj, struct profdata *profdata)
 static int
 symorder (const void *o1, const void *o2)
 {
-  const struct known_symbol *p1 = (struct known_symbol *) o1;
-  const struct known_symbol *p2 = (struct known_symbol *) o2;
+  const struct known_symbol *p1 = (const struct known_symbol *) o1;
+  const struct known_symbol *p2 = (const struct known_symbol *) o2;
 
   return p1->addr - p2->addr;
 }
@@ -872,6 +903,7 @@ read_symbols (struct shobj *shobj)
 #define obstack_chunk_free free
   obstack_init (&shobj->ob_str);
   obstack_init (&shobj->ob_sym);
+  obstack_init (&ob_list);
 
   /* Process the symbols.  */
   if (shobj->symtab)
@@ -947,6 +979,8 @@ read_symbols (struct shobj *shobj)
              newsym->addr = symtab->st_value;
              newsym->size = symtab->st_size;
              newsym->ticks = 0;
+             newsym->froms = NULL;
+             newsym->tos = NULL;
 
              existp = tfind (newsym, &symroot, symorder);
              if (existp == NULL)
@@ -979,6 +1013,85 @@ read_symbols (struct shobj *shobj)
 }
 
 
+static void
+add_arcs (struct profdata *profdata)
+{
+  uint32_t narcs = profdata->narcs;
+  struct here_cg_arc_record *data = profdata->data;
+  uint32_t cnt;
+
+  for (cnt = 0; cnt < narcs; ++cnt)
+    {
+      /* First add the incoming arc.  */
+      size_t sym_idx = find_symbol (data[cnt].self_pc);
+
+      if (sym_idx != (size_t) -1l)
+       {
+         struct known_symbol *sym = sortsym[sym_idx];
+         struct arc_list *runp = sym->froms;
+
+         while (runp != NULL
+                && ((data[cnt].from_pc == 0 && runp->idx != (size_t) -1l)
+                    || (data[cnt].from_pc != 0
+                        && (runp->idx == (size_t) -1l
+                            || data[cnt].from_pc < sortsym[runp->idx]->addr
+                            || (data[cnt].from_pc
+                                >= (sortsym[runp->idx]->addr
+                                    + sortsym[runp->idx]->size))))))
+           runp = runp->next;
+
+         if (runp == NULL)
+           {
+             /* We need a new entry.  */
+             struct arc_list *newp = (struct arc_list *)
+               obstack_alloc (&ob_list, sizeof (struct arc_list));
+
+             if (data[cnt].from_pc == 0)
+               newp->idx = (size_t) -1l;
+             else
+               newp->idx = find_symbol (data[cnt].from_pc);
+             newp->count = data[cnt].count;
+             newp->next = sym->froms;
+             sym->froms = newp;
+           }
+         else
+           /* Increment the counter for the found entry.  */
+           runp->count += data[cnt].count;
+       }
+
+      /* Now add it to the appropriate outgoing list.  */
+      sym_idx = find_symbol (data[cnt].from_pc);
+      if (sym_idx != (size_t) -1l)
+       {
+         struct known_symbol *sym = sortsym[sym_idx];
+         struct arc_list *runp = sym->tos;
+
+         while (runp != NULL
+                && (runp->idx == (size_t) -1l
+                    || data[cnt].self_pc < sortsym[runp->idx]->addr
+                    || data[cnt].self_pc >= (sortsym[runp->idx]->addr
+                                             + sortsym[runp->idx]->size)))
+           runp = runp->next;
+
+         if (runp == NULL)
+           {
+             /* We need a new entry.  */
+             struct arc_list *newp = (struct arc_list *)
+               obstack_alloc (&ob_list, sizeof (struct arc_list));
+
+             newp->idx = find_symbol (data[cnt].self_pc);
+             newp->count = data[cnt].count;
+             newp->next = sym->tos;
+             sym->tos = newp;
+           }
+         else
+           /* Increment the counter for the found entry.  */
+           runp->count += data[cnt].count;
+       }
+    }
+}
+
+
 static int
 countorder (const void *p1, const void *p2)
 {
@@ -1007,13 +1120,13 @@ printflat (const void *node, VISIT value, int level)
 
       cumu_ticks += s->ticks;
 
-      printf ("%6.2f%10.2f%9.2f%9" PRIdMAX "%9.2f%9.2f  %s\n",
+      printf ("%6.2f%10.2f%9.2f%9" PRIdMAX "%9.2f           %s\n",
              total_ticks ? (100.0 * s->ticks) / total_ticks : 0.0,
              tick_unit * cumu_ticks,
              tick_unit * s->ticks,
              s->calls,
              s->calls ? (s->ticks * 1000000) * tick_unit / s->calls : 0,
-             0.0,      /* FIXME: don't know about called functions.  */
+             /* FIXME: don't know about called functions.  */
              s->name);
     }
 }
@@ -1049,3 +1162,71 @@ generate_flat_profile (struct profdata *profdata)
 
   tdestroy (data, freenoop);
 }
+
+
+static void
+generate_call_graph (struct profdata *profdata)
+{
+  size_t cnt;
+
+  puts ("\nindex % time    self  children    called     name\n");
+
+  for (cnt = 0; cnt < symidx; ++cnt)
+    if (sortsym[cnt]->froms != NULL || sortsym[cnt]->tos != NULL)
+      {
+       struct arc_list *runp;
+       size_t n;
+
+       /* First print the from-information.  */
+       runp = sortsym[cnt]->froms;
+       while (runp != NULL)
+         {
+           printf ("            %8.2f%8.2f%9" PRIdMAX "/%-9" PRIdMAX "   %s",
+                   (runp->idx != (size_t) -1l
+                    ? sortsym[runp->idx]->ticks * tick_unit : 0.0),
+                   0.0, /* FIXME: what's time for the childern, recursive */
+                   runp->count, sortsym[cnt]->calls,
+                   (runp->idx != (size_t) -1l ?
+                    sortsym[runp->idx]->name : "<UNKNOWN>"));
+
+           if (runp->idx != (size_t) -1l)
+             printf (" [%Zd]", runp->idx);
+           putchar_unlocked ('\n');
+
+           runp = runp->next;
+         }
+
+       /* Info abount the function itself.  */
+       n = printf ("[%d]", cnt);
+       printf ("%*s%5.1f%8.2f%8.2f%9" PRIdMAX "         %s [%Zd]\n",
+               7 - n, " ",
+               total_ticks ? (100.0 * sortsym[cnt]->ticks) / total_ticks : 0,
+               sortsym[cnt]->ticks * tick_unit,
+               0.0, /* FIXME: what's time for the childern, recursive */
+               sortsym[cnt]->calls,
+               sortsym[cnt]->name, cnt);
+
+       /* Info about the functions this function calls.  */
+       runp = sortsym[cnt]->tos;
+       while (runp != NULL)
+         {
+           printf ("            %8.2f%8.2f%9" PRIdMAX "/",
+                   (runp->idx != (size_t) -1l
+                    ? sortsym[runp->idx]->ticks * tick_unit : 0.0),
+                   0.0, /* FIXME: what's time for the childern, recursive */
+                   runp->count);
+
+           if (runp->idx != (size_t) -1l)
+             printf ("%-9" PRIdMAX "   %s [%Zd]\n",
+                     sortsym[runp->idx]->calls,
+                     sortsym[runp->idx]->name,
+                     runp->idx);
+           else
+             fputs ("???         <UNKNOWN>\n\n", stdout);
+
+           runp = runp->next;
+         }
+
+       fputs ("-----------------------------------------------\n", stdout);
+      }
+}
index de11326139726674274af17bf10241e37b4e7cf1..0c8781506438035386b79b4de8457f9e8d5f239f 100644 (file)
@@ -61,14 +61,14 @@ __fgetgrent_r (FILE *stream, struct group *resbuf, char *buffer, size_t buflen,
 
   do
     {
-      buffer[buflen] = '\xff';
+      buffer[buflen - 1] = '\xff';
       p = fgets (buffer, buflen, stream);
       if (p == NULL && feof (stream))
        {
          *result = NULL;
          return errno;
        }
-      if (p == NULL || buffer[buflen] != '\xff')
+      if (p == NULL || buffer[buflen - 1] != '\xff')
        {
          *result = NULL;
          return errno = ERANGE;
index b0334d6d0eb221dde7c23263f862d2bec895c79e..4286eef6c22ace47f1ae587e60d57db9903ff204 100644 (file)
@@ -273,7 +273,7 @@ __underflow (fp)
     {
       _IO_switch_to_main_get_area (fp);
       if (fp->_IO_read_ptr < fp->_IO_read_end)
-       return *fp->_IO_read_ptr;
+       return *(unsigned char *) fp->_IO_read_ptr;
     }
   if (_IO_have_markers (fp))
     {
@@ -298,7 +298,7 @@ __uflow (fp)
     {
       _IO_switch_to_main_get_area (fp);
       if (fp->_IO_read_ptr < fp->_IO_read_end)
-       return *fp->_IO_read_ptr++;
+       return *(unsigned char *) fp->_IO_read_ptr++;
     }
   if (_IO_have_markers (fp))
     {
index 534afd58f51e75d293422f28bc2feffb8ac91911..87ea1dedda92a63ad1c47b7f9c18094893cc2e32 100644 (file)
@@ -74,14 +74,14 @@ __fgetpwent_r (FILE *stream, struct passwd *resbuf, char *buffer,
 
   do
     {
-      buffer[buflen] = '\xff';
+      buffer[buflen - 1] = '\xff';
       p = fgets (buffer, buflen, stream);
       if (p == NULL && feof (stream))
        {
          *result = NULL;
          return errno;
        }
-      if (p == NULL || buffer[buflen] != '\xff')
+      if (p == NULL || buffer[buflen - 1] != '\xff')
        {
          *result = NULL;
          return errno = ERANGE;
index cb14a230373e9df7eb2eb93fa5ce593272f2fbf7..bf4a3f8c0845251d7a90dc77595d14dc69df2cef 100644 (file)
@@ -41,14 +41,14 @@ __fgetspent_r (FILE *stream, struct spwd *resbuf, char *buffer, size_t buflen,
 
   do
     {
-      buffer[buflen] = '\xff';
+      buffer[buflen - 1] = '\xff';
       p = fgets (buffer, buflen, stream);
       if (p == NULL && feof (stream))
        {
          *result = NULL;
          return errno;
        }
-      if (p == NULL || buffer[buflen] != '\xff')
+      if (p == NULL || buffer[buflen - 1] != '\xff')
        {
          *result = NULL;
          return errno = ERANGE;
index 990ddfd93ff6912fd8f7e8559d1c9dbc05f5127d..b6f405df08078ec21f32063f38c8ba953dc24bd5 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (C) 1991, 1992, 1994, 1996 Free Software Foundation, Inc.
+/* Copyright (C) 1991, 1992, 1994, 1996, 1998 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
@@ -16,7 +16,9 @@
    write to the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
    Boston, MA 02111-1307, USA.  */
 
+#include <endian.h>
 #include <errno.h>
+#include <stdint.h>
 #include <stdlib.h>
 #include <string.h>
 #include <unistd.h>
 #define        __environ       environ
 #endif
 
-/* Return the value of the environment variable NAME.  */
+/* Return the value of the environment variable NAME.  This implementation
+   is tuned a bit in that it assumes no environment variable has an empty
+   name which of course should always be true.  We have a special case for
+   one character names so that for the general case we can assume at least
+   two characters which we can access.  By doing this we can avoid using the
+   `strncmp' most of the time.  */
 char *
 getenv (name)
      const char *name;
 {
   const size_t len = strlen (name);
   char **ep;
+  uint16_t name_start;
 
-  if (__environ == NULL)
+  if (__environ == NULL || name[0] == '\0')
     return NULL;
 
-  for (ep = __environ; *ep != NULL; ++ep)
-    if (!strncmp (*ep, name, len) && (*ep)[len] == '=')
-      return &(*ep)[len + 1];
+  if (name[1] == '\0')
+    {
+      /* The name of the variable consists of only one character.  Therefore
+        the first two characters of the environment entry are this character
+        and a '=' character.  */
+#if __BYTE_ORDER == __LITTLE_ENDIAN
+      name_start = ('=' << 8) | *(const unsigned char *) name;
+#else
+# if __BYTE_ORDER == __BIG_ENDIAN
+      name_start = '=' | ((*(const unsigned char *) name) << 8);
+# else
+ #error "Funny byte order."
+# endif
+#endif
+      for (ep = __environ; *ep != NULL; ++ep)
+       {
+#if _STRING_ARCH_unaligned
+         uint16_t ep_start = *(uint16_t *) *ep;
+#else
+         uint16_t ep_start = (((unsigned char *) *ep)[0]
+                              | (((unsigned char *) *ep)[1] << 8));
+#endif
+         if (name_start == ep_start)
+           return &(*ep)[2];
+       }
+    }
+  else
+    {
+#if _STRING_ARCH_unaligned
+      name_start = *(const uint16_t *) name;
+#else
+      name_start = (((const unsigned char *) name)[0]
+                   | (((const unsigned char *) name)[1] << 8));
+#endif
+      len -= 2;
+      name += 2;
+
+      for (ep = __environ; *ep != NULL; ++ep)
+       {
+#if _STRING_ARCH_unaligned
+         uint16_t ep_start = *(uint16_t *) *ep;
+#else
+         uint16_t ep_start = (((unsigned char *) *ep)[0]
+                              | (((unsigned char *) *ep)[1] << 8));
+#endif
+
+         if (name_start == ep_start && !strncmp (*ep + 2, name, len)
+             && (*ep)[len + 2] == '=')
+           return &(*ep)[len + 3];
+       }
+    }
 
   return NULL;
 }