OSDN Git Service

Deal with the inferior unloading shared objects.
authorjimb <jimb>
Wed, 15 Mar 2000 16:55:07 +0000 (16:55 +0000)
committerjimb <jimb>
Wed, 15 Mar 2000 16:55:07 +0000 (16:55 +0000)
* solib.c (current_sos): New function, replacing find_solib.
(find_solib): Deleted.
(free_so): New function.
(clear_solib): Call free_so, instead of writing it out.
(solib_add): Rewritten: compare the inferior's current list of
shared objects with GDB's list, and do the required loads and
unloads.
(info_sharedlibrary_command, solib_address): Don't use find_solib
to walk the list of shared libraries: call solib_add, and then
walk the list at so_list_head normally.
* objfiles.c (free_objfile): Don't call CLEAR_SOLIB, and don't
detach the core target.  These tasks are taken care of elsewhere.
* target.c (remove_target_sections): New function.
* target.h (remove_target_sections): New declaration.

* solib.c (symbol_add_stub): Check whether we've already created
an objfile for this shared object first, before doing all that
work to compute section addresses, etc.

* objfiles.c (unlink_objfile): Report an internal error if objfile
doesn't occur in the object_files list.

* solib.c (special_symbol_handling): Delete argument; it's not
used.

* solib.c (SOLIB_EXTRACT_ADDRESS):  New macro to extract addresses
from solib structures. Use it throughout solib.c, get rid of all
CORE_ADDR casts.
(struct so_list):  Change type of lmaddr to CORE_ADDR.
(first_link_map_member):  Change return value type to CORE_ADDR,
update callers.
(solib_add_common_symbols):  Change parameter type to CORE_ADDR,
update callers.
(open_symbol_file_object, find_solib):  Change type of lm variable
to CORE_ADDR.

gdb/ChangeLog
gdb/objfiles.c
gdb/solib.c
gdb/target.c
gdb/target.h

index 357ec84..fa2686e 100644 (file)
@@ -1,3 +1,44 @@
+2000-03-15  Jim Blandy  <jimb@redhat.com>
+
+       Deal with the inferior unloading shared objects.
+       * solib.c (current_sos): New function, replacing find_solib.
+       (find_solib): Deleted.
+       (free_so): New function.
+       (clear_solib): Call free_so, instead of writing it out.
+       (solib_add): Rewritten: compare the inferior's current list of
+       shared objects with GDB's list, and do the required loads and
+       unloads.
+       (info_sharedlibrary_command, solib_address): Don't use find_solib
+       to walk the list of shared libraries: call solib_add, and then
+       walk the list at so_list_head normally.
+       * objfiles.c (free_objfile): Don't call CLEAR_SOLIB, and don't
+       detach the core target.  These tasks are taken care of elsewhere.
+       * target.c (remove_target_sections): New function.
+       * target.h (remove_target_sections): New declaration.
+
+       * solib.c (symbol_add_stub): Check whether we've already created
+       an objfile for this shared object first, before doing all that
+       work to compute section addresses, etc.
+
+       * objfiles.c (unlink_objfile): Report an internal error if objfile
+       doesn't occur in the object_files list.
+
+       * solib.c (special_symbol_handling): Delete argument; it's not
+       used.
+       
+       Changes from Peter Schauer <pes@regent.e-technik.tu-muenchen.de>:
+
+       * solib.c (SOLIB_EXTRACT_ADDRESS):  New macro to extract addresses
+       from solib structures. Use it throughout solib.c, get rid of all
+       CORE_ADDR casts.
+       (struct so_list):  Change type of lmaddr to CORE_ADDR.
+       (first_link_map_member):  Change return value type to CORE_ADDR,
+       update callers.
+       (solib_add_common_symbols):  Change parameter type to CORE_ADDR,
+       update callers.
+       (open_symbol_file_object, find_solib):  Change type of lm variable
+       to CORE_ADDR.
+
 2000-03-15  Eli Zaretskii  <eliz@is.elta.co.il>
 
        * ser-go32.c (dos_noop, dos_raw, dos_noflush_set_tty_state)
index bcd047c..41ec317 100644 (file)
@@ -370,9 +370,11 @@ unlink_objfile (objfile)
        {
          *objpp = (*objpp)->next;
          objfile->next = NULL;
-         break;
+         return;
        }
     }
+
+  internal_error ("objfiles.c (unlink_objfile): objfile already unlinked");
 }
 
 
@@ -436,17 +438,6 @@ free_objfile (objfile)
      is unknown, but we play it safe for now and keep each action until
      it is shown to be no longer needed. */
 
-#if defined (CLEAR_SOLIB)
-  CLEAR_SOLIB ();
-  /* CLEAR_SOLIB closes the bfd's for any shared libraries.  But
-     the to_sections for a core file might refer to those bfd's.  So
-     detach any core file.  */
-  {
-    struct target_ops *t = find_core_target ();
-    if (t != NULL)
-      (t->to_detach) (NULL, 0);
-  }
-#endif
   /* I *think* all our callers call clear_symtab_users.  If so, no need
      to call this here.  */
   clear_pc_function_cache ();
index f6e36a0..c9f1e14 100644 (file)
@@ -109,13 +109,22 @@ static char *main_name_list[] =
 
 /* local data declarations */
 
+/* Macro to extract an address from a solib structure.
+   When GDB is configured for some 32-bit targets (e.g. Solaris 2.7
+   sparc), BFD is configured to handle 64-bit targets, so CORE_ADDR is
+   64 bits.  We have to extract only the significant bits of addresses
+   to get the right address when accessing the core file BFD.  */
+
+#define SOLIB_EXTRACT_ADDRESS(member) \
+  extract_address (&member, sizeof (member))
+
 #ifndef SVR4_SHARED_LIBS
 
-#define LM_ADDR(so) ((so) -> lm.lm_addr)
-#define LM_NEXT(so) ((so) -> lm.lm_next)
-#define LM_NAME(so) ((so) -> lm.lm_name)
+#define LM_ADDR(so) (SOLIB_EXTRACT_ADDRESS ((so) -> lm.lm_addr))
+#define LM_NEXT(so) (SOLIB_EXTRACT_ADDRESS ((so) -> lm.lm_next))
+#define LM_NAME(so) (SOLIB_EXTRACT_ADDRESS ((so) -> lm.lm_name))
 /* Test for first link map entry; first entry is a shared library. */
-#define IGNORE_FIRST_LINK_MAP_ENTRY(x) (0)
+#define IGNORE_FIRST_LINK_MAP_ENTRY(so) (0)
 static struct link_dynamic dynamic_copy;
 static struct link_dynamic_2 ld_2_copy;
 static struct ld_debug debug_copy;
@@ -124,11 +133,12 @@ static CORE_ADDR flag_addr;
 
 #else /* SVR4_SHARED_LIBS */
 
-#define LM_ADDR(so) ((so) -> lm.l_addr)
-#define LM_NEXT(so) ((so) -> lm.l_next)
-#define LM_NAME(so) ((so) -> lm.l_name)
+#define LM_ADDR(so) (SOLIB_EXTRACT_ADDRESS ((so) -> lm.l_addr))
+#define LM_NEXT(so) (SOLIB_EXTRACT_ADDRESS ((so) -> lm.l_next))
+#define LM_NAME(so) (SOLIB_EXTRACT_ADDRESS ((so) -> lm.l_name))
 /* Test for first link map entry; first entry is the exec-file. */
-#define IGNORE_FIRST_LINK_MAP_ENTRY(x) ((x).l_prev == NULL)
+#define IGNORE_FIRST_LINK_MAP_ENTRY(so) \
+  (SOLIB_EXTRACT_ADDRESS ((so) -> lm.l_prev) == 0)
 static struct r_debug debug_copy;
 char shadow_contents[BREAKPOINT_MAX];  /* Stash old bkpt addr contents */
 
@@ -136,18 +146,36 @@ char shadow_contents[BREAKPOINT_MAX];     /* Stash old bkpt addr contents */
 
 struct so_list
   {
+    /* The following fields of the structure come directly from the
+       dynamic linker's tables in the inferior, and are initialized by
+       current_sos.  */
+
     struct so_list *next;      /* next structure in linked list */
     struct link_map lm;                /* copy of link map from inferior */
-    struct link_map *lmaddr;   /* addr in inferior lm was read from */
+    CORE_ADDR lmaddr;          /* addr in inferior lm was read from */
+
+    /* Shared object file name, exactly as it appears in the
+       inferior's link map.  This may be a relative path, or something
+       which needs to be looked up in LD_LIBRARY_PATH, etc.  We use it
+       to tell which entries in the inferior's dynamic linker's link
+       map we've already loaded.  */
+    char so_original_name[MAX_PATH_SIZE];
+
+    /* shared object file name, expanded to something GDB can open */
+    char so_name[MAX_PATH_SIZE];
+
+    /* The following fields of the structure are built from
+       information gathered from the shared object file itself, and
+       are initialized when we actually add it to our symbol tables.  */
+
+    bfd *abfd;
     CORE_ADDR lmend;           /* upper addr bound of mapped object */
-    char so_name[MAX_PATH_SIZE];       /* shared object lib name (FIXME) */
     char symbols_loaded;       /* flag: symbols read in yet? */
     char from_tty;             /* flag: print msgs? */
     struct objfile *objfile;   /* objfile for loaded lib */
     struct section_table *sections;
     struct section_table *sections_end;
     struct section_table *textsection;
-    bfd *abfd;
   };
 
 static struct so_list *so_list_head;   /* List of known shared objects */
@@ -168,7 +196,7 @@ static int
 match_main PARAMS ((char *));
 
 static void
-special_symbol_handling PARAMS ((struct so_list *));
+special_symbol_handling PARAMS ((void));
 
 static void
 sharedlibrary_command PARAMS ((char *, int));
@@ -181,10 +209,7 @@ info_sharedlibrary_command PARAMS ((char *, int));
 
 static int symbol_add_stub PARAMS ((PTR));
 
-static struct so_list *
-  find_solib PARAMS ((struct so_list *));
-
-static struct link_map *
+static CORE_ADDR
   first_link_map_member PARAMS ((void));
 
 static CORE_ADDR
@@ -199,6 +224,9 @@ static CORE_ADDR
 
 #else
 
+static struct so_list *current_sos (void);
+static void free_so (struct so_list *node);
+
 static int
 disable_break PARAMS ((void));
 
@@ -206,7 +234,7 @@ static void
 allocate_rt_common_objfile PARAMS ((void));
 
 static void
-solib_add_common_symbols PARAMS ((struct rtc_symb *));
+solib_add_common_symbols (CORE_ADDR);
 
 #endif
 
@@ -338,9 +366,9 @@ solib_map_sections (arg)
       /* Relocate the section binding addresses as recorded in the shared
          object's file by the base address to which the object was actually
          mapped. */
-      p->addr += (CORE_ADDR) LM_ADDR (so);
-      p->endaddr += (CORE_ADDR) LM_ADDR (so);
-      so->lmend = (CORE_ADDR) max (p->endaddr, so->lmend);
+      p->addr += LM_ADDR (so);
+      p->endaddr += LM_ADDR (so);
+      so->lmend = max (p->endaddr, so->lmend);
       if (STREQ (p->the_bfd_section->name, ".text"))
        {
          so->textsection = p;
@@ -398,7 +426,7 @@ allocate_rt_common_objfile ()
 
 static void
 solib_add_common_symbols (rtc_symp)
-     struct rtc_symb *rtc_symp;
+     CORE_ADDR rtc_symp;
 {
   struct rtc_symb inferior_rtc_symb;
   struct nlist inferior_rtc_nlist;
@@ -421,10 +449,10 @@ solib_add_common_symbols (rtc_symp)
 
   while (rtc_symp)
     {
-      read_memory ((CORE_ADDR) rtc_symp,
+      read_memory (rtc_symp,
                   (char *) &inferior_rtc_symb,
                   sizeof (inferior_rtc_symb));
-      read_memory ((CORE_ADDR) inferior_rtc_symb.rtc_sp,
+      read_memory (SOLIB_EXTRACT_ADDRESS (inferior_rtc_symb.rtc_sp),
                   (char *) &inferior_rtc_nlist,
                   sizeof (inferior_rtc_nlist));
       if (inferior_rtc_nlist.n_type == N_COMM)
@@ -435,7 +463,8 @@ solib_add_common_symbols (rtc_symp)
          len = inferior_rtc_nlist.n_value - inferior_rtc_nlist.n_un.n_strx;
 
          name = xmalloc (len);
-         read_memory ((CORE_ADDR) inferior_rtc_nlist.n_un.n_name, name, len);
+         read_memory (SOLIB_EXTRACT_ADDRESS (inferior_rtc_nlist.n_un.n_name),
+                      name, len);
 
          /* Allocate the runtime common objfile if necessary. */
          if (rt_common_objfile == NULL)
@@ -445,7 +474,7 @@ solib_add_common_symbols (rtc_symp)
                                      mst_bss, rt_common_objfile);
          free (name);
        }
-      rtc_symp = inferior_rtc_symb.rtc_next;
+      rtc_symp = SOLIB_EXTRACT_ADDRESS (inferior_rtc_symb.rtc_next);
     }
 
   /* Install any minimal symbols that have been collected as the current
@@ -848,19 +877,19 @@ locate_base ()
 
    SYNOPSIS
 
-   static struct link_map *first_link_map_member (void)
+   static CORE_ADDR first_link_map_member (void)
 
    DESCRIPTION
 
    Find the first element in the inferior's dynamic link map, and
    return its address in the inferior.  This function doesn't copy the
-   link map entry itself into our address space; find_solib actually
+   link map entry itself into our address space; current_sos actually
    does the reading.  */
 
-static struct link_map *
+static CORE_ADDR
 first_link_map_member ()
 {
-  struct link_map *lm = NULL;
+  CORE_ADDR lm = 0;
 
 #ifndef SVR4_SHARED_LIBS
 
@@ -869,9 +898,9 @@ first_link_map_member ()
     {
       /* It is a version that we can deal with, so read in the secondary
          structure and find the address of the link map list from it. */
-      read_memory ((CORE_ADDR) dynamic_copy.ld_un.ld_2, (char *) &ld_2_copy,
-                  sizeof (struct link_dynamic_2));
-      lm = ld_2_copy.ld_loaded;
+      read_memory (SOLIB_EXTRACT_ADDRESS (dynamic_copy.ld_un.ld_2),
+                  (char *) &ld_2_copy, sizeof (struct link_dynamic_2));
+      lm = SOLIB_EXTRACT_ADDRESS (ld_2_copy.ld_loaded);
     }
 
 #else /* SVR4_SHARED_LIBS */
@@ -880,7 +909,7 @@ first_link_map_member ()
   /* FIXME:  Perhaps we should validate the info somehow, perhaps by
      checking r_version for a known version number, or r_state for
      RT_CONSISTENT. */
-  lm = debug_copy.r_map;
+  lm = SOLIB_EXTRACT_ADDRESS (debug_copy.r_map);
 
 #endif /* !SVR4_SHARED_LIBS */
 
@@ -912,7 +941,8 @@ open_symbol_file_object (arg)
      PTR arg;
 {
   int from_tty = (int) arg;    /* sneak past catch_errors */
-  struct link_map *lm, lmcopy;
+  CORE_ADDR lm;
+  struct link_map lmcopy;
   char *filename;
   int errcode;
 
@@ -924,17 +954,17 @@ open_symbol_file_object (arg)
     return 0;  /* failed somehow... */
 
   /* First link map member should be the executable.  */
-  if ((lm = first_link_map_member ()) == NULL)
+  if ((lm = first_link_map_member ()) == 0)
     return 0;  /* failed somehow... */
 
   /* Read from target memory to GDB.  */
-  read_memory ((CORE_ADDR) lm, (void *) &lmcopy, sizeof (lmcopy));
+  read_memory (lm, (void *) &lmcopy, sizeof (lmcopy));
 
   if (lmcopy.l_name == 0)
     return 0;  /* no filename.  */
 
   /* Now fetch the filename from target memory.  */
-  target_read_string ((CORE_ADDR) lmcopy.l_name, &filename, 
+  target_read_string (SOLIB_EXTRACT_ADDRESS (lmcopy.l_name), &filename, 
                      MAX_PATH_SIZE - 1, &errcode);
   if (errcode)
     {
@@ -951,141 +981,188 @@ open_symbol_file_object (arg)
 }
 #endif /* SVR4_SHARED_LIBS */
 
-/*
 
-   LOCAL FUNCTION
+/* LOCAL FUNCTION
 
-   find_solib -- step through list of shared objects
+   free_so --- free a `struct so_list' object
 
    SYNOPSIS
 
-   struct so_list *find_solib (struct so_list *so_list_ptr)
+   void free_so (struct so_list *so)
 
    DESCRIPTION
 
-   This module contains the routine which finds the names of any
-   loaded "images" in the current process. The argument in must be
-   NULL on the first call, and then the returned value must be passed
-   in on subsequent calls. This provides the capability to "step" down
-   the list of loaded objects. On the last object, a NULL value is
-   returned.
+   Free the storage associated with the `struct so_list' object SO.
+   If we have opened a BFD for SO, close it.  
 
-   The arg and return value are "struct link_map" pointers, as defined
-   in <link.h>.
- */
+   The caller is responsible for removing SO from whatever list it is
+   a member of.  If we have placed SO's sections in some target's
+   section table, the caller is responsible for removing them.
 
-static struct so_list *
-find_solib (so_list_ptr)
-     struct so_list *so_list_ptr;      /* Last lm or NULL for first one */
+   This function doesn't mess with objfiles at all.  If there is an
+   objfile associated with SO that needs to be removed, the caller is
+   responsible for taking care of that.  */
+
+static void
+free_so (struct so_list *so)
 {
-  struct so_list *so_list_next = NULL;
-  struct link_map *lm = NULL;
-  struct so_list *new;
+  char *bfd_filename = 0;
 
-  if (so_list_ptr == NULL)
+  if (so->sections)
+    free (so->sections);
+      
+  if (so->abfd)
     {
-      /* We are setting up for a new scan through the loaded images. */
-      if ((so_list_next = so_list_head) == NULL)
-       {
-         /* We have not already read in the dynamic linking structures
-            from the inferior, lookup the address of the base structure. */
-         debug_base = locate_base ();
-         if (debug_base != 0)
-           {
-             /* Read the base structure in and find the address of the first
-                link map list member. */
-             lm = first_link_map_member ();
-           }
-       }
+      bfd_filename = bfd_get_filename (so->abfd);
+      if (! bfd_close (so->abfd))
+       warning ("cannot close \"%s\": %s",
+                bfd_filename, bfd_errmsg (bfd_get_error ()));
     }
-  else
+
+  if (bfd_filename)
+    free (bfd_filename);
+
+  free (so);
+}
+
+
+/* On some systems, the only way to recognize the link map entry for
+   the main executable file is by looking at its name.  Return
+   non-zero iff SONAME matches one of the known main executable names.  */
+
+static int
+match_main (soname)
+     char *soname;
+{
+  char **mainp;
+
+  for (mainp = main_name_list; *mainp != NULL; mainp++)
     {
-      /* We have been called before, and are in the process of walking
-         the shared library list.  Advance to the next shared object. */
-      if ((lm = LM_NEXT (so_list_ptr)) == NULL)
-       {
-         /* We have hit the end of the list, so check to see if any were
-            added, but be quiet if we can't read from the target any more. */
-         int status = target_read_memory ((CORE_ADDR) so_list_ptr->lmaddr,
-                                          (char *) &(so_list_ptr->lm),
-                                          sizeof (struct link_map));
-         if (status == 0)
-           {
-             lm = LM_NEXT (so_list_ptr);
-           }
-         else
-           {
-             lm = NULL;
-           }
-       }
-      so_list_next = so_list_ptr->next;
+      if (strcmp (soname, *mainp) == 0)
+       return (1);
     }
-  if ((so_list_next == NULL) && (lm != NULL))
+
+  return (0);
+}
+
+
+/* LOCAL FUNCTION
+
+   current_sos -- build a list of currently loaded shared objects
+
+   SYNOPSIS
+
+   struct so_list *current_sos ()
+
+   DESCRIPTION
+
+   Build a list of `struct so_list' objects describing the shared
+   objects currently loaded in the inferior.  This list does not
+   include an entry for the main executable file.
+
+   Note that we only gather information directly available from the
+   inferior --- we don't examine any of the shared library files
+   themselves.  The declaration of `struct so_list' says which fields
+   we provide values for.  */
+
+static struct so_list *
+current_sos ()
+{
+  CORE_ADDR lm;
+  struct so_list *head = 0;
+  struct so_list **link_ptr = &head;
+
+  /* Make sure we've looked up the inferior's dynamic linker's base
+     structure.  */
+  if (! debug_base)
     {
-      /* Get next link map structure from inferior image and build a local
-         abbreviated load_map structure */
-      new = (struct so_list *) xmalloc (sizeof (struct so_list));
-      memset ((char *) new, 0, sizeof (struct so_list));
+      debug_base = locate_base ();
+
+      /* If we can't find the dynamic linker's base structure, this
+        must not be a dynamically linked executable.  Hmm.  */
+      if (! debug_base)
+       return 0;
+    }
+
+  /* Walk the inferior's link map list, and build our list of
+     `struct so_list' nodes.  */
+  lm = first_link_map_member ();  
+  while (lm)
+    {
+      struct so_list *new
+       = (struct so_list *) xmalloc (sizeof (struct so_list));
+      memset (new, 0, sizeof (*new));
+
       new->lmaddr = lm;
-      /* Add the new node as the next node in the list, or as the root
-         node if this is the first one. */
-      if (so_list_ptr != NULL)
-       {
-         so_list_ptr->next = new;
-       }
-      else
-       {
-         so_list_head = new;
+      read_memory (lm, (char *) &(new->lm), sizeof (struct link_map));
 
-         if (!solib_cleanup_queued)
-           {
-             make_run_cleanup (do_clear_solib, NULL);
-             solib_cleanup_queued = 1;
-           }
+      lm = LM_NEXT (new);
 
-       }
-      so_list_next = new;
-      read_memory ((CORE_ADDR) lm, (char *) &(new->lm),
-                  sizeof (struct link_map));
       /* For SVR4 versions, the first entry in the link map is for the
          inferior executable, so we must ignore it.  For some versions of
          SVR4, it has no name.  For others (Solaris 2.3 for example), it
          does have a name, so we can no longer use a missing name to
          decide when to ignore it. */
-      if (!IGNORE_FIRST_LINK_MAP_ENTRY (new->lm))
+      if (IGNORE_FIRST_LINK_MAP_ENTRY (new))
+       {
+         free_so (new);
+       }
+      else
        {
          int errcode;
          char *buffer;
-         target_read_string ((CORE_ADDR) LM_NAME (new), &buffer,
+
+         /* Extract this shared object's name.  */
+         target_read_string (LM_NAME (new), &buffer,
                              MAX_PATH_SIZE - 1, &errcode);
          if (errcode != 0)
            {
-             warning ("find_solib: Can't read pathname for load map: %s\n",
+             warning ("current_sos: Can't read pathname for load map: %s\n",
                       safe_strerror (errcode));
-             return (so_list_next);
            }
-         strncpy (new->so_name, buffer, MAX_PATH_SIZE - 1);
-         new->so_name[MAX_PATH_SIZE - 1] = '\0';
-         free (buffer);
-         catch_errors (solib_map_sections, new,
-                       "Error while mapping shared library sections:\n",
-                       RETURN_MASK_ALL);
+         else
+           {
+             strncpy (new->so_name, buffer, MAX_PATH_SIZE - 1);
+             new->so_name[MAX_PATH_SIZE - 1] = '\0';
+             free (buffer);
+             strcpy (new->so_original_name, new->so_name);
+           }
+
+         /* If this entry has no name, or its name matches the name
+            for the main executable, don't include it in the list.  */
+         if (! new->so_name[0]
+             || match_main (new->so_name))
+           free_so (new);
+         else
+           {
+             new->next = 0;
+             *link_ptr = new;
+             link_ptr = &new->next;
+           }
        }
     }
-  return (so_list_next);
+
+  return head;
 }
 
+
 /* A small stub to get us past the arg-passing pinhole of catch_errors.  */
 
 static int
 symbol_add_stub (arg)
      PTR arg;
 {
-  register struct so_list *so = (struct so_list *) arg;                /* catch_errs bogon */
+  register struct so_list *so = (struct so_list *) arg;  /* catch_errs bogon */
   CORE_ADDR text_addr = 0;
-  struct section_addr_info section_addrs;
 
-  memset (&section_addrs, 0, sizeof (section_addrs));
+  /* Have we already loaded this shared object?  */
+  ALL_OBJFILES (so->objfile)
+    {
+      if (strcmp (so->objfile->name, so->so_name) == 0)
+       return 1;
+    }
+
+  /* Find the shared object's text segment.  */
   if (so->textsection)
     text_addr = so->textsection->addr;
   else if (so->abfd != NULL)
@@ -1094,75 +1171,73 @@ symbol_add_stub (arg)
 
       /* If we didn't find a mapped non zero sized .text section, set up
          text_addr so that the relocation in symbol_file_add does no harm.  */
-
       lowest_sect = bfd_get_section_by_name (so->abfd, ".text");
       if (lowest_sect == NULL)
        bfd_map_over_sections (so->abfd, find_lowest_section,
                               (PTR) &lowest_sect);
       if (lowest_sect)
        text_addr = bfd_section_vma (so->abfd, lowest_sect)
-         + (CORE_ADDR) LM_ADDR (so);
+         + LM_ADDR (so);
     }
 
-  ALL_OBJFILES (so->objfile)
   {
-    if (strcmp (so->objfile->name, so->so_name) == 0)
-      return 1;
-  }
-  section_addrs.text_addr = text_addr;
-  so->objfile =
-    symbol_file_add (so->so_name, so->from_tty,
-                    &section_addrs, 0, OBJF_SHARED);
-  return (1);
-}
+    struct section_addr_info section_addrs;
 
-/* This function will check the so name to see if matches the main list.
-   In some system the main object is in the list, which we want to exclude */
+    memset (&section_addrs, 0, sizeof (section_addrs));
+    section_addrs.text_addr = text_addr;
 
-static int
-match_main (soname)
-     char *soname;
-{
-  char **mainp;
-
-  for (mainp = main_name_list; *mainp != NULL; mainp++)
-    {
-      if (strcmp (soname, *mainp) == 0)
-       return (1);
-    }
+    so->objfile = symbol_file_add (so->so_name, so->from_tty,
+                                  &section_addrs, 0, OBJF_SHARED);
+  }
 
-  return (0);
+  return (1);
 }
 
-/*
 
-   GLOBAL FUNCTION
+/* LOCAL FUNCTION
 
-   solib_add -- add a shared library file to the symtab and section list
+   solib_add -- synchronize GDB's shared object list with the inferior's
 
    SYNOPSIS
 
-   void solib_add (char *arg_string, int from_tty,
-   struct target_ops *target)
+   void solib_add (char *pattern, int from_tty, struct target_ops *TARGET)
 
    DESCRIPTION
 
- */
+   Extract the list of currently loaded shared objects from the
+   inferior, and compare it with the list of shared objects for which
+   GDB has currently loaded symbolic information.  If new shared
+   objects have been loaded, or old shared objects have disappeared,
+   make the appropriate changes to GDB's tables.
 
-void
-solib_add (arg_string, from_tty, target)
-     char *arg_string;
-     int from_tty;
-     struct target_ops *target;
-{
-  register struct so_list *so = NULL;  /* link map state variable */
+   If PATTERN is non-null, read symbols only for shared objects
+   whose names match PATTERN.
+
+   If FROM_TTY is non-null, feel free to print messages about what
+   we're doing.
 
-  /* Last shared library that we read.  */
-  struct so_list *so_last = NULL;
+   If TARGET is non-null, add the sections of all new shared objects
+   to TARGET's section table.  Note that this doesn't remove any
+   sections for shared objects that have been unloaded, and it
+   doesn't check to see if the new shared objects are already present in
+   the section table.  But we only use this for core files and
+   processes we've just attached to, so that's okay.  */
 
-  char *re_err;
-  int count;
-  int old;
+void
+solib_add (char *pattern, int from_tty, struct target_ops *target)
+{
+  struct so_list *inferior = current_sos ();
+  struct so_list *gdb, **gdb_link;
+
+  /* #define JIMB_DEBUG */
+#ifdef JIMB_DEBUG
+  printf ("GDB's shared library list:\n");
+  for (gdb = so_list_head; gdb; gdb = gdb->next)
+    printf ("  %s\n", gdb->so_original_name);
+  printf ("inferior's shared library list:\n");
+  for (gdb = inferior; gdb; gdb = gdb->next)
+    printf ("  %s\n", gdb->so_original_name);
+#endif
 
 #ifdef SVR4_SHARED_LIBS
   /* If we are attaching to a running process for which we 
@@ -1176,79 +1251,188 @@ solib_add (arg_string, from_tty, target)
 
 #endif SVR4_SHARED_LIBS
 
-  if ((re_err = re_comp (arg_string? arg_string : ".")) != NULL)
+  if (pattern)
     {
-      error ("Invalid regexp: %s", re_err);
+      char *re_err = re_comp (pattern);
+
+      if (re_err)
+       error ("Invalid regexp: %s", re_err);
+    }
+
+  /* Since this function might actually add some elements to the
+     so_list_head list, arrange for it to be cleaned up when
+     appropriate.  */
+  if (!solib_cleanup_queued)
+    {
+      make_run_cleanup (do_clear_solib, NULL);
+      solib_cleanup_queued = 1;
     }
 
-  /* Add the shared library sections to the section table of the
-     specified target, if any.  */
-  if (target)
+  /* GDB and the inferior's dynamic linker each maintain their own
+     list of currently loaded shared objects; we want to bring the
+     former in sync with the latter.  Scan both lists, seeing which
+     shared objects appear where.  There are three cases:
+
+     - A shared object appears on both lists.  This means that GDB
+       knows about it already, and it's still loaded in the inferior.
+       Nothing needs to happen.
+
+     - A shared object appears only on GDB's list.  This means that
+       the inferior has unloaded it.  We should remove the shared
+       object from GDB's tables.
+
+     - A shared object appears only on the inferior's list.  This
+       means that it's just been loaded.  We should add it to GDB's
+       tables.
+
+     So we walk GDB's list, checking each entry to see if it appears
+     in the inferior's list too.  If it does, no action is needed, and
+     we remove it from the inferior's list.  If it doesn't, the
+     inferior has unloaded it, and we remove it from GDB's list.  By
+     the time we're done walking GDB's list, the inferior's list
+     contains only the new shared objects, which we then add.  */
+
+  gdb = so_list_head;
+  gdb_link = &so_list_head;
+  while (gdb)
     {
-      /* Count how many new section_table entries there are.  */
-      so = NULL;
-      count = 0;
-      while ((so = find_solib (so)) != NULL)
+      struct so_list *i = inferior;
+      struct so_list **i_link = &inferior;
+
+      /* Check to see whether the shared object *gdb also appears in
+        the inferior's current list.  */
+      while (i)
        {
-         if (so->so_name[0] && !match_main (so->so_name))
-           {
-             count += so->sections_end - so->sections;
-           }
+         if (! strcmp (gdb->so_original_name, i->so_original_name))
+           break;
+
+         i_link = &i->next;
+         i = *i_link;
        }
 
-      if (count)
+      /* If the shared object appears on the inferior's list too, then
+         it's still loaded, so we don't need to do anything.  Delete
+         it from the inferior's list, and leave it on GDB's list.  */
+      if (i)
        {
-         
-         /* Add these section table entries to the target's table.  */
-         old = target_resize_to_sections (target, count);
-         while ((so = find_solib (so)) != NULL)
-           {
-             if (so->so_name[0])
-               {
-                 count = so->sections_end - so->sections;
-                 memcpy ((char *) (target->to_sections + old),
-                         so->sections,
-                         (sizeof (struct section_table)) * count);
-                 old += count;
-               }
-           }
+         *i_link = i->next;
+#ifdef JIMB_DEBUG
+         printf ("unchanged: %s\n", i->so_name);
+#endif
+         free_so (i);
+         gdb_link = &gdb->next;
+         gdb = *gdb_link;
+       }
+
+      /* If it's not on the inferior's list, remove it from GDB's tables.  */
+      else
+       {
+         *gdb_link = gdb->next;
+#ifdef JIMB_DEBUG
+         printf ("removed:   %s\n", gdb->so_name);
+#endif
+
+         /* Unless the user loaded it explicitly, free SO's objfile.  */
+         if (! (gdb->objfile->flags & OBJF_USERLOADED))
+           free_objfile (gdb->objfile);
+
+         /* Some targets' section tables might be referring to
+            sections from so->abfd; remove them.  */
+         remove_target_sections (gdb->abfd);
+
+         free_so (gdb);
+         gdb = *gdb_link;
        }
     }
 
-  /* Now add the symbol files.  */
-  while ((so = find_solib (so)) != NULL)
+  /* Now the inferior's list contains only shared objects that don't
+     appear in GDB's list --- those that are newly loaded.  Add them
+     to GDB's shared object list, and read in their symbols, if
+     appropriate.  */
+  if (inferior)
     {
-      if (so->so_name[0] && re_exec (so->so_name) &&
-         !match_main (so->so_name))
+      struct so_list *i;
+
+      /* Add the new shared objects to GDB's list.  */
+      *gdb_link = inferior;
+
+      /* Fill in the rest of each of the `struct so_list' nodes, and
+        read symbols for those files whose names match PATTERN.  */
+      for (i = inferior; i; i = i->next)
        {
-         so->from_tty = from_tty;
-         if (so->symbols_loaded)
+         i->from_tty = from_tty;
+
+         /* Fill in the rest of the `struct so_list' node.  */
+         catch_errors (solib_map_sections, i,
+                       "Error while mapping shared library sections:\n",
+                       RETURN_MASK_ALL);
+
+         if (! pattern || re_exec (i->so_name))
            {
-             if (from_tty)
+             if (i->symbols_loaded)
                {
-                 printf_unfiltered ("Symbols already loaded for %s\n", so->so_name);
+                 if (from_tty)
+                   printf_unfiltered ("Symbols already loaded for %s\n",
+                                      i->so_name);
+               }
+             else
+               {
+#ifdef JIMB_DEBUG
+                 printf ("added:     %s\n", i->so_name);
+#endif
+                 if (catch_errors
+                     (symbol_add_stub, i,
+                      "Error while reading shared library symbols:\n",
+                      RETURN_MASK_ALL))
+                   {
+                     if (from_tty)
+                       printf_unfiltered ("Loaded symbols for %s\n",
+                                          i->so_name);
+                     i->symbols_loaded = 1;
+                   }
                }
            }
-         else if (catch_errors
-                  (symbol_add_stub, so,
-                   "Error while reading shared library symbols:\n",
-                   RETURN_MASK_ALL))
+       }
+
+      /* If requested, add the shared objects' sections to the the
+        TARGET's section table.  */
+      if (target)
+       {
+         int new_sections;
+
+         /* Figure out how many sections we'll need to add in total.  */
+         new_sections = 0;
+         for (i = inferior; i; i = i->next)
+           new_sections += (i->sections_end - i->sections);
+
+         if (new_sections > 0)
            {
-             so_last = so;
-             so->symbols_loaded = 1;
+             int space = target_resize_to_sections (target, new_sections);
+
+             for (i = inferior; i; i = i->next)
+               {
+                 int count = (i->sections_end - i->sections);
+                 memcpy (target->to_sections + space,
+                         i->sections,
+                         count * sizeof (i->sections[0]));
+                 space += count;
+               }
            }
        }
-    }
 
-  /* Getting new symbols may change our opinion about what is
-     frameless.  */
-  if (so_last)
-    reinit_frame_cache ();
+      /* Getting new symbols may change our opinion about what is
+         frameless.  */
+      reinit_frame_cache ();
 
-  if (so_last)
-    special_symbol_handling (so_last);
+      special_symbol_handling ();
+    }
+
+#ifdef JIMB_DEBUG
+  putchar ('\n');
+#endif
 }
 
+
 /*
 
    LOCAL FUNCTION
@@ -1289,7 +1473,9 @@ info_sharedlibrary_command (ignore, from_tty)
   addr_fmt = "016l";
 #endif
 
-  while ((so = find_solib (so)) != NULL)
+  solib_add (0, 0, 0);
+
+  for (so = so_list_head; so; so = so->next)
     {
       if (so->so_name[0])
        {
@@ -1347,15 +1533,12 @@ solib_address (address)
 {
   register struct so_list *so = 0;     /* link map state variable */
 
-  while ((so = find_solib (so)) != NULL)
+  for (so = so_list_head; so; so = so->next)
     {
-      if (so->so_name[0])
-       {
-         if ((address >= (CORE_ADDR) LM_ADDR (so)) &&
-             (address < (CORE_ADDR) so->lmend))
-           return (so->so_name);
-       }
+      if (LM_ADDR (so) <= address && address < so->lmend)
+       return (so->so_name);
     }
+
   return (0);
 }
 
@@ -1364,9 +1547,6 @@ solib_address (address)
 void
 clear_solib ()
 {
-  struct so_list *next;
-  char *bfd_filename;
-
   /* This function is expected to handle ELF shared libraries.  It is
      also used on Solaris, which can run either ELF or a.out binaries
      (for compatibility with SunOS 4), both of which can use shared
@@ -1393,27 +1573,11 @@ clear_solib ()
 
   while (so_list_head)
     {
-      if (so_list_head->sections)
-       {
-         free ((PTR) so_list_head->sections);
-       }
-      if (so_list_head->abfd)
-       {
-         bfd_filename = bfd_get_filename (so_list_head->abfd);
-         if (!bfd_close (so_list_head->abfd))
-           warning ("cannot close \"%s\": %s",
-                    bfd_filename, bfd_errmsg (bfd_get_error ()));
-       }
-      else
-       /* This happens for the executable on SVR4.  */
-       bfd_filename = NULL;
-
-      next = so_list_head->next;
-      if (bfd_filename)
-       free ((PTR) bfd_filename);
-      free ((PTR) so_list_head);
-      so_list_head = next;
+      struct so_list *so = so_list_head;
+      so_list_head = so->next;
+      free_so (so);
     }
+
   debug_base = 0;
 }
 
@@ -1484,7 +1648,7 @@ disable_break ()
 
   write_memory (flag_addr, (char *) &in_debugger, sizeof (in_debugger));
 
-  breakpoint_addr = (CORE_ADDR) debug_copy.ldd_bp_addr;
+  breakpoint_addr = SOLIB_EXTRACT_ADDRESS (debug_copy.ldd_bp_addr);
   write_memory (breakpoint_addr, (char *) &debug_copy.ldd_bp_inst,
                sizeof (debug_copy.ldd_bp_inst));
 
@@ -1579,7 +1743,7 @@ enable_break ()
 
   /* Calc address of debugger interface structure */
 
-  debug_addr = (CORE_ADDR) dynamic_copy.ldd;
+  debug_addr = SOLIB_EXTRACT_ADDRESS (dynamic_copy.ldd);
 
   /* Calc address of `in_debugger' member of debugger interface structure */
 
@@ -1775,7 +1939,7 @@ solib_create_inferior_hook ()
 {
   /* If we are using the BKPT_AT_SYMBOL code, then we don't need the base
      yet.  In fact, in the case of a SunOS4 executable being run on
-     Solaris, we can't get it yet.  find_solib will get it when it needs
+     Solaris, we can't get it yet.  current_sos will get it when it needs
      it.  */
 #if !(defined (SVR4_SHARED_LIBS) && defined (BKPT_AT_SYMBOL))
   if ((debug_base = locate_base ()) == 0)
@@ -1843,7 +2007,7 @@ solib_create_inferior_hook ()
 
    SYNOPSIS
 
-   void special_symbol_handling (struct so_list *so)
+   void special_symbol_handling ()
 
    DESCRIPTION
 
@@ -1859,8 +2023,7 @@ solib_create_inferior_hook ()
  */
 
 static void
-special_symbol_handling (so)
-     struct so_list *so;
+special_symbol_handling ()
 {
 #ifndef SVR4_SHARED_LIBS
   int j;
@@ -1881,7 +2044,7 @@ special_symbol_handling (so)
       /* FIXME, this needs work for cross-debugging of core files
          (byteorder, size, alignment, etc).  */
 
-      debug_addr = (CORE_ADDR) dynamic_copy.ldd;
+      debug_addr = SOLIB_EXTRACT_ADDRESS (dynamic_copy.ldd);
     }
 
   /* Read the debugger structure from the inferior, just to make sure
@@ -1896,7 +2059,7 @@ special_symbol_handling (so)
 
   if (debug_copy.ldd_cp)
     {
-      solib_add_common_symbols (debug_copy.ldd_cp);
+      solib_add_common_symbols (SOLIB_EXTRACT_ADDRESS (debug_copy.ldd_cp));
     }
 
 #endif /* !SVR4_SHARED_LIBS */
index fa9cf88..7bb4886 100644 (file)
@@ -1325,6 +1325,39 @@ target_resize_to_sections (struct target_ops *target, int num_added)
 
 }
 
+/* Remove all target sections taken from ABFD.
+
+   Scan the current target stack for targets whose section tables
+   refer to sections from BFD, and remove those sections.  We use this
+   when we notice that the inferior has unloaded a shared object, for
+   example.  */
+void
+remove_target_sections (bfd *abfd)
+{
+  struct target_ops **t;
+
+  for (t = target_structs; t < target_structs + target_struct_size; t++)
+    {
+      struct section_table *src, *dest;
+
+      dest = (*t)->to_sections;
+      for (src = (*t)->to_sections; src < (*t)->to_sections_end; src++)
+       if (src->bfd != abfd)
+         {
+           /* Keep this section.  */
+           if (dest < src) *dest = *src;
+           dest++;
+         }
+
+      /* If we've dropped any sections, resize the section table.  */
+      if (dest < src)
+       target_resize_to_sections (*t, dest - src);
+    }
+}
+
+
+
+
 /* Find a single runnable target in the stack and return it.  If for
    some reason there is more than one, return NULL.  */
 
index 21c3bab..07a94c0 100644 (file)
@@ -1,22 +1,23 @@
 /* Interface between GDB and target environments, including files and processes
-   Copyright 1990, 91, 92, 93, 94, 1999 Free Software Foundation, Inc.
+   Copyright 1990-1994, 1999, 2000 Free Software Foundation, Inc.
    Contributed by Cygnus Support.  Written by John Gilmore.
 
-This file is part of GDB.
+   This file is part of GDB.
 
-This program is free software; you can redistribute it and/or modify
-it under the terms of the GNU General Public License as published by
-the Free Software Foundation; either version 2 of the License, or
-(at your option) any later version.
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 2 of the License, or
+   (at your option) any later version.
 
-This program is distributed in the hope that it will be useful,
-but WITHOUT ANY WARRANTY; without even the implied warranty of
-MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-GNU General Public License for more details.
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
 
-You should have received a copy of the GNU General Public License
-along with this program; if not, write to the Free Software
-Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
+   You should have received a copy of the GNU General Public License
+   along with this program; if not, write to the Free Software
+   Foundation, Inc., 59 Temple Place - Suite 330,
+   Boston, MA 02111-1307, USA.  */
 
 #if !defined (TARGET_H)
 #define TARGET_H
@@ -43,72 +44,95 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 #include "bfd.h"
 #include "symtab.h"
 
-enum strata {
-       dummy_stratum,          /* The lowest of the low */
-       file_stratum,           /* Executable files, etc */
-       core_stratum,           /* Core dump files */
-       download_stratum,       /* Downloading of remote targets */
-       process_stratum         /* Executing processes */
-};
+enum strata
+  {
+    dummy_stratum,             /* The lowest of the low */
+    file_stratum,              /* Executable files, etc */
+    core_stratum,              /* Core dump files */
+    download_stratum,          /* Downloading of remote targets */
+    process_stratum,           /* Executing processes */
+    thread_stratum             /* Executing threads */
+  };
 
-enum thread_control_capabilities {
-       tc_none = 0,            /* Default: can't control thread execution. */
-       tc_schedlock = 1,       /* Can lock the thread scheduler. */
-       tc_switch = 2           /* Can switch the running thread on demand. */
-};
+enum thread_control_capabilities
+  {
+    tc_none = 0,               /* Default: can't control thread execution.  */
+    tc_schedlock = 1,          /* Can lock the thread scheduler.  */
+    tc_switch = 2              /* Can switch the running thread on demand.  */
+  };
 
 /* Stuff for target_wait.  */
 
 /* Generally, what has the program done?  */
-enum target_waitkind {
-  /* The program has exited.  The exit status is in value.integer.  */
-  TARGET_WAITKIND_EXITED,
+enum target_waitkind
+  {
+    /* The program has exited.  The exit status is in value.integer.  */
+    TARGET_WAITKIND_EXITED,
 
-  /* The program has stopped with a signal.  Which signal is in value.sig.  */
-  TARGET_WAITKIND_STOPPED,
+    /* The program has stopped with a signal.  Which signal is in
+       value.sig.  */
+    TARGET_WAITKIND_STOPPED,
 
-  /* The program has terminated with a signal.  Which signal is in
-     value.sig.  */
-  TARGET_WAITKIND_SIGNALLED,
+    /* The program has terminated with a signal.  Which signal is in
+       value.sig.  */
+    TARGET_WAITKIND_SIGNALLED,
 
-  /* The program is letting us know that it dynamically loaded something
-     (e.g. it called load(2) on AIX).  */
-  TARGET_WAITKIND_LOADED,
+    /* The program is letting us know that it dynamically loaded something
+       (e.g. it called load(2) on AIX).  */
+    TARGET_WAITKIND_LOADED,
 
-  /* The program has forked.  A "related" process' ID is in value.related_pid.
-     I.e., if the child forks, value.related_pid is the parent's ID.
-     */
-  TARGET_WAITKIND_FORKED,
+    /* The program has forked.  A "related" process' ID is in
+       value.related_pid.  I.e., if the child forks, value.related_pid
+       is the parent's ID.  */
 
-  /* The program has vforked.  A "related" process's ID is in value.related_pid.
-     */
-  TARGET_WAITKIND_VFORKED,
+    TARGET_WAITKIND_FORKED,
 
-  /* The program has exec'ed a new executable file.  The new file's pathname
-     is pointed to by value.execd_pathname.
-     */
-  TARGET_WAITKIND_EXECD,
+    /* The program has vforked.  A "related" process's ID is in
+       value.related_pid.  */
 
-  /* The program has entered or returned from a system call.  On HP-UX, this
-     is used in the hardware watchpoint implementation.  The syscall's unique
-     integer ID number is in value.syscall_id;
-     */
-  TARGET_WAITKIND_SYSCALL_ENTRY,
-  TARGET_WAITKIND_SYSCALL_RETURN,
+    TARGET_WAITKIND_VFORKED,
 
-  /* Nothing happened, but we stopped anyway.  This perhaps should be handled
-     within target_wait, but I'm not sure target_wait should be resuming the
-     inferior.  */
-  TARGET_WAITKIND_SPURIOUS
+    /* The program has exec'ed a new executable file.  The new file's
+       pathname is pointed to by value.execd_pathname.  */
+
+    TARGET_WAITKIND_EXECD,
+
+    /* The program has entered or returned from a system call.  On
+       HP-UX, this is used in the hardware watchpoint implementation.
+       The syscall's unique integer ID number is in value.syscall_id */
+
+    TARGET_WAITKIND_SYSCALL_ENTRY,
+    TARGET_WAITKIND_SYSCALL_RETURN,
+
+    /* Nothing happened, but we stopped anyway.  This perhaps should be handled
+       within target_wait, but I'm not sure target_wait should be resuming the
+       inferior.  */
+    TARGET_WAITKIND_SPURIOUS,
+
+    /* This is used for target async and extended-async
+       only. Remote_async_wait() returns this when there is an event
+       on the inferior, but the rest of the world is not interested in
+       it. The inferior has not stopped, but has just sent some output
+       to the console, for instance. In this case, we want to go back
+       to the event loop and wait there for another event from the
+       inferior, rather than being stuck in the remote_async_wait()
+       function. This way the event loop is responsive to other events,
+       like for instance the user typing.  */
+    TARGET_WAITKIND_IGNORE
   };
 
 /* The numbering of these signals is chosen to match traditional unix
    signals (insofar as various unices use the same numbers, anyway).
    It is also the numbering of the GDB remote protocol.  Other remote
    protocols, if they use a different numbering, should make sure to
-   translate appropriately.  */
+   translate appropriately.
+
+   Since these numbers have actually made it out into other software
+   (stubs, etc.), you mustn't disturb the assigned numbering.  If you
+   need to add new signals here, add them to the end of the explicitly
+   numbered signals.
 
-/* This is based strongly on Unix/POSIX signals for several reasons:
+   This is based strongly on Unix/POSIX signals for several reasons:
    (1) This set of signals represents a widely-accepted attempt to
    represent events of this sort in a portable fashion, (2) we want a
    signal to make it from wait to child_wait to the user intact, (3) many
@@ -124,119 +148,157 @@ enum target_waitkind {
 /* For an explanation of what each signal means, see
    target_signal_to_string.  */
 
-enum target_signal {
-  /* Used some places (e.g. stop_signal) to record the concept that
-     there is no signal.  */
-  TARGET_SIGNAL_0 = 0,
-  TARGET_SIGNAL_FIRST = 0,
-  TARGET_SIGNAL_HUP = 1,
-  TARGET_SIGNAL_INT = 2,
-  TARGET_SIGNAL_QUIT = 3,
-  TARGET_SIGNAL_ILL = 4,
-  TARGET_SIGNAL_TRAP = 5,
-  TARGET_SIGNAL_ABRT = 6,
-  TARGET_SIGNAL_EMT = 7,
-  TARGET_SIGNAL_FPE = 8,
-  TARGET_SIGNAL_KILL = 9,
-  TARGET_SIGNAL_BUS = 10,
-  TARGET_SIGNAL_SEGV = 11,
-  TARGET_SIGNAL_SYS = 12,
-  TARGET_SIGNAL_PIPE = 13,
-  TARGET_SIGNAL_ALRM = 14,
-  TARGET_SIGNAL_TERM = 15,
-  TARGET_SIGNAL_URG = 16,
-  TARGET_SIGNAL_STOP = 17,
-  TARGET_SIGNAL_TSTP = 18,
-  TARGET_SIGNAL_CONT = 19,
-  TARGET_SIGNAL_CHLD = 20,
-  TARGET_SIGNAL_TTIN = 21,
-  TARGET_SIGNAL_TTOU = 22,
-  TARGET_SIGNAL_IO = 23,
-  TARGET_SIGNAL_XCPU = 24,
-  TARGET_SIGNAL_XFSZ = 25,
-  TARGET_SIGNAL_VTALRM = 26,
-  TARGET_SIGNAL_PROF = 27,
-  TARGET_SIGNAL_WINCH = 28,
-  TARGET_SIGNAL_LOST = 29,
-  TARGET_SIGNAL_USR1 = 30,
-  TARGET_SIGNAL_USR2 = 31,
-  TARGET_SIGNAL_PWR = 32,
-  /* Similar to SIGIO.  Perhaps they should have the same number.  */
-  TARGET_SIGNAL_POLL = 33,
-  TARGET_SIGNAL_WIND = 34,
-  TARGET_SIGNAL_PHONE = 35,
-  TARGET_SIGNAL_WAITING = 36,
-  TARGET_SIGNAL_LWP = 37,
-  TARGET_SIGNAL_DANGER = 38,
-  TARGET_SIGNAL_GRANT = 39,
-  TARGET_SIGNAL_RETRACT = 40,
-  TARGET_SIGNAL_MSG = 41,
-  TARGET_SIGNAL_SOUND = 42,
-  TARGET_SIGNAL_SAK = 43,
-  TARGET_SIGNAL_PRIO = 44,
-  TARGET_SIGNAL_REALTIME_33 = 45,
-  TARGET_SIGNAL_REALTIME_34 = 46,
-  TARGET_SIGNAL_REALTIME_35 = 47,
-  TARGET_SIGNAL_REALTIME_36 = 48,
-  TARGET_SIGNAL_REALTIME_37 = 49,
-  TARGET_SIGNAL_REALTIME_38 = 50,
-  TARGET_SIGNAL_REALTIME_39 = 51,
-  TARGET_SIGNAL_REALTIME_40 = 52,
-  TARGET_SIGNAL_REALTIME_41 = 53,
-  TARGET_SIGNAL_REALTIME_42 = 54,
-  TARGET_SIGNAL_REALTIME_43 = 55,
-  TARGET_SIGNAL_REALTIME_44 = 56,
-  TARGET_SIGNAL_REALTIME_45 = 57,
-  TARGET_SIGNAL_REALTIME_46 = 58,
-  TARGET_SIGNAL_REALTIME_47 = 59,
-  TARGET_SIGNAL_REALTIME_48 = 60,
-  TARGET_SIGNAL_REALTIME_49 = 61,
-  TARGET_SIGNAL_REALTIME_50 = 62,
-  TARGET_SIGNAL_REALTIME_51 = 63,
-  TARGET_SIGNAL_REALTIME_52 = 64,
-  TARGET_SIGNAL_REALTIME_53 = 65,
-  TARGET_SIGNAL_REALTIME_54 = 66,
-  TARGET_SIGNAL_REALTIME_55 = 67,
-  TARGET_SIGNAL_REALTIME_56 = 68,
-  TARGET_SIGNAL_REALTIME_57 = 69,
-  TARGET_SIGNAL_REALTIME_58 = 70,
-  TARGET_SIGNAL_REALTIME_59 = 71,
-  TARGET_SIGNAL_REALTIME_60 = 72,
-  TARGET_SIGNAL_REALTIME_61 = 73,
-  TARGET_SIGNAL_REALTIME_62 = 74,
-  TARGET_SIGNAL_REALTIME_63 = 75,
+enum target_signal
+  {
+    /* Used some places (e.g. stop_signal) to record the concept that
+       there is no signal.  */
+    TARGET_SIGNAL_0 = 0,
+    TARGET_SIGNAL_FIRST = 0,
+    TARGET_SIGNAL_HUP = 1,
+    TARGET_SIGNAL_INT = 2,
+    TARGET_SIGNAL_QUIT = 3,
+    TARGET_SIGNAL_ILL = 4,
+    TARGET_SIGNAL_TRAP = 5,
+    TARGET_SIGNAL_ABRT = 6,
+    TARGET_SIGNAL_EMT = 7,
+    TARGET_SIGNAL_FPE = 8,
+    TARGET_SIGNAL_KILL = 9,
+    TARGET_SIGNAL_BUS = 10,
+    TARGET_SIGNAL_SEGV = 11,
+    TARGET_SIGNAL_SYS = 12,
+    TARGET_SIGNAL_PIPE = 13,
+    TARGET_SIGNAL_ALRM = 14,
+    TARGET_SIGNAL_TERM = 15,
+    TARGET_SIGNAL_URG = 16,
+    TARGET_SIGNAL_STOP = 17,
+    TARGET_SIGNAL_TSTP = 18,
+    TARGET_SIGNAL_CONT = 19,
+    TARGET_SIGNAL_CHLD = 20,
+    TARGET_SIGNAL_TTIN = 21,
+    TARGET_SIGNAL_TTOU = 22,
+    TARGET_SIGNAL_IO = 23,
+    TARGET_SIGNAL_XCPU = 24,
+    TARGET_SIGNAL_XFSZ = 25,
+    TARGET_SIGNAL_VTALRM = 26,
+    TARGET_SIGNAL_PROF = 27,
+    TARGET_SIGNAL_WINCH = 28,
+    TARGET_SIGNAL_LOST = 29,
+    TARGET_SIGNAL_USR1 = 30,
+    TARGET_SIGNAL_USR2 = 31,
+    TARGET_SIGNAL_PWR = 32,
+    /* Similar to SIGIO.  Perhaps they should have the same number.  */
+    TARGET_SIGNAL_POLL = 33,
+    TARGET_SIGNAL_WIND = 34,
+    TARGET_SIGNAL_PHONE = 35,
+    TARGET_SIGNAL_WAITING = 36,
+    TARGET_SIGNAL_LWP = 37,
+    TARGET_SIGNAL_DANGER = 38,
+    TARGET_SIGNAL_GRANT = 39,
+    TARGET_SIGNAL_RETRACT = 40,
+    TARGET_SIGNAL_MSG = 41,
+    TARGET_SIGNAL_SOUND = 42,
+    TARGET_SIGNAL_SAK = 43,
+    TARGET_SIGNAL_PRIO = 44,
+    TARGET_SIGNAL_REALTIME_33 = 45,
+    TARGET_SIGNAL_REALTIME_34 = 46,
+    TARGET_SIGNAL_REALTIME_35 = 47,
+    TARGET_SIGNAL_REALTIME_36 = 48,
+    TARGET_SIGNAL_REALTIME_37 = 49,
+    TARGET_SIGNAL_REALTIME_38 = 50,
+    TARGET_SIGNAL_REALTIME_39 = 51,
+    TARGET_SIGNAL_REALTIME_40 = 52,
+    TARGET_SIGNAL_REALTIME_41 = 53,
+    TARGET_SIGNAL_REALTIME_42 = 54,
+    TARGET_SIGNAL_REALTIME_43 = 55,
+    TARGET_SIGNAL_REALTIME_44 = 56,
+    TARGET_SIGNAL_REALTIME_45 = 57,
+    TARGET_SIGNAL_REALTIME_46 = 58,
+    TARGET_SIGNAL_REALTIME_47 = 59,
+    TARGET_SIGNAL_REALTIME_48 = 60,
+    TARGET_SIGNAL_REALTIME_49 = 61,
+    TARGET_SIGNAL_REALTIME_50 = 62,
+    TARGET_SIGNAL_REALTIME_51 = 63,
+    TARGET_SIGNAL_REALTIME_52 = 64,
+    TARGET_SIGNAL_REALTIME_53 = 65,
+    TARGET_SIGNAL_REALTIME_54 = 66,
+    TARGET_SIGNAL_REALTIME_55 = 67,
+    TARGET_SIGNAL_REALTIME_56 = 68,
+    TARGET_SIGNAL_REALTIME_57 = 69,
+    TARGET_SIGNAL_REALTIME_58 = 70,
+    TARGET_SIGNAL_REALTIME_59 = 71,
+    TARGET_SIGNAL_REALTIME_60 = 72,
+    TARGET_SIGNAL_REALTIME_61 = 73,
+    TARGET_SIGNAL_REALTIME_62 = 74,
+    TARGET_SIGNAL_REALTIME_63 = 75,
+
+    /* Used internally by Solaris threads.  See signal(5) on Solaris.  */
+    TARGET_SIGNAL_CANCEL = 76,
+
+    /* Yes, this pains me, too.  But LynxOS didn't have SIG32, and now
+       Linux does, and we can't disturb the numbering, since it's part
+       of the protocol.  Note that in some GDB's TARGET_SIGNAL_REALTIME_32
+       is number 76.  */
+    TARGET_SIGNAL_REALTIME_32,
+
 #if defined(MACH) || defined(__MACH__)
-  /* Mach exceptions */
-  TARGET_EXC_BAD_ACCESS = 76,
-  TARGET_EXC_BAD_INSTRUCTION = 77,
-  TARGET_EXC_ARITHMETIC = 78,
-  TARGET_EXC_EMULATION = 79,
-  TARGET_EXC_SOFTWARE = 80,
-  TARGET_EXC_BREAKPOINT = 81,
+    /* Mach exceptions */
+    TARGET_EXC_BAD_ACCESS,
+    TARGET_EXC_BAD_INSTRUCTION,
+    TARGET_EXC_ARITHMETIC,
+    TARGET_EXC_EMULATION,
+    TARGET_EXC_SOFTWARE,
+    TARGET_EXC_BREAKPOINT,
 #endif
-  /* Some signal we don't know about.  */
-  TARGET_SIGNAL_UNKNOWN,
-
-  /* Use whatever signal we use when one is not specifically specified
-     (for passing to proceed and so on).  */
-  TARGET_SIGNAL_DEFAULT,
-
-  /* Last and unused enum value, for sizing arrays, etc.  */
-  TARGET_SIGNAL_LAST
-};
-
-struct target_waitstatus {
-  enum target_waitkind kind;
-
-  /* Forked child pid, execd pathname, exit status or signal number.  */
-  union {
-    int integer;
-    enum target_signal sig;
-    int  related_pid;
-    char *  execd_pathname;
-    int  syscall_id;
-  } value;
-};
+    TARGET_SIGNAL_INFO,
+
+    /* Some signal we don't know about.  */
+    TARGET_SIGNAL_UNKNOWN,
+
+    /* Use whatever signal we use when one is not specifically specified
+       (for passing to proceed and so on).  */
+    TARGET_SIGNAL_DEFAULT,
+
+    /* Last and unused enum value, for sizing arrays, etc.  */
+    TARGET_SIGNAL_LAST
+  };
+
+struct target_waitstatus
+  {
+    enum target_waitkind kind;
+
+    /* Forked child pid, execd pathname, exit status or signal number.  */
+    union
+      {
+       int integer;
+       enum target_signal sig;
+       int related_pid;
+       char *execd_pathname;
+       int syscall_id;
+      }
+    value;
+  };
+
+/* Possible types of events that the inferior handler will have to
+   deal with.  */
+enum inferior_event_type
+  {
+    /* There is a request to quit the inferior, abandon it.  */
+    INF_QUIT_REQ,
+    /* Process a normal inferior event which will result in target_wait
+       being called.  */
+    INF_REG_EVENT, 
+    /* Deal with an error on the inferior.  */
+    INF_ERROR,
+    /* We are called because a timer went off.  */
+    INF_TIMER,
+    /* We are called to do stuff after the inferior stops.  */
+    INF_EXEC_COMPLETE,
+    /* We are called to do some stuff after the inferior stops, but we
+       are expected to reenter the proceed() and
+       handle_inferior_event() functions. This is used only in case of
+       'step n' like commands.  */
+    INF_EXEC_CONTINUE
+  };
 
 /* Return the string for a signal.  */
 extern char *target_signal_to_string PARAMS ((enum target_signal));
@@ -246,136 +308,151 @@ extern char *target_signal_to_name PARAMS ((enum target_signal));
 
 /* Given a name (SIGHUP, etc.), return its signal.  */
 enum target_signal target_signal_from_name PARAMS ((char *));
-
 \f
+
 /* If certain kinds of activity happen, target_wait should perform
    callbacks.  */
 /* Right now we just call (*TARGET_ACTIVITY_FUNCTION) if I/O is possible
-   on TARGET_ACTIVITY_FD.   */
+   on TARGET_ACTIVITY_FD.  */
 extern int target_activity_fd;
 /* Returns zero to leave the inferior alone, one to interrupt it.  */
 extern int (*target_activity_function) PARAMS ((void));
 \f
+struct thread_info;            /* fwd decl for parameter list below: */
+
 struct target_ops
-{
-  char        *to_shortname;   /* Name this target type */
-  char        *to_longname;    /* Name for printing */
-  char                *to_doc;         /* Documentation.  Does not include trailing
+  {
+    char *to_shortname;                /* Name this target type */
+    char *to_longname;         /* Name for printing */
+    char *to_doc;              /* Documentation.  Does not include trailing
                                   newline, and starts with a one-line descrip-
-                                  tion (probably similar to to_longname). */
-  void               (*to_open) PARAMS ((char *, int));
-  void               (*to_close) PARAMS ((int));
-  void               (*to_attach) PARAMS ((char *, int));
-  void        (*to_post_attach) PARAMS ((int));
-  void               (*to_require_attach) PARAMS ((char *, int));
-  void               (*to_detach) PARAMS ((char *, int));
-  void               (*to_require_detach) PARAMS ((int, char *, int));
-  void               (*to_resume) PARAMS ((int, int, enum target_signal));
-  int                (*to_wait) PARAMS ((int, struct target_waitstatus *));
-  void        (*to_post_wait) PARAMS ((int, int));
-  void               (*to_fetch_registers) PARAMS ((int));
-  void               (*to_store_registers) PARAMS ((int));
-  void               (*to_prepare_to_store) PARAMS ((void));
-
-  /* Transfer LEN bytes of memory between GDB address MYADDR and
-     target address MEMADDR.  If WRITE, transfer them to the target, else
-     transfer them from the target.  TARGET is the target from which we
-     get this function.
-
-     Return value, N, is one of the following:
-
-     0 means that we can't handle this.  If errno has been set, it is the
-     error which prevented us from doing it (FIXME: What about bfd_error?).
-
-     positive (call it N) means that we have transferred N bytes
-     starting at MEMADDR.  We might be able to handle more bytes
-     beyond this length, but no promises.
-
-     negative (call its absolute value N) means that we cannot
-     transfer right at MEMADDR, but we could transfer at least
-     something at MEMADDR + N.  */
-
-  int                (*to_xfer_memory) PARAMS ((CORE_ADDR memaddr, char *myaddr,
-                                        int len, int write,
-                                        struct target_ops * target));
+                                  tion (probably similar to to_longname).  */
+    void (*to_open) PARAMS ((char *, int));
+    void (*to_close) PARAMS ((int));
+    void (*to_attach) PARAMS ((char *, int));
+    void (*to_post_attach) PARAMS ((int));
+    void (*to_require_attach) PARAMS ((char *, int));
+    void (*to_detach) PARAMS ((char *, int));
+    void (*to_require_detach) PARAMS ((int, char *, int));
+    void (*to_resume) PARAMS ((int, int, enum target_signal));
+    int (*to_wait) PARAMS ((int, struct target_waitstatus *));
+    void (*to_post_wait) PARAMS ((int, int));
+    void (*to_fetch_registers) PARAMS ((int));
+    void (*to_store_registers) PARAMS ((int));
+    void (*to_prepare_to_store) PARAMS ((void));
+
+    /* Transfer LEN bytes of memory between GDB address MYADDR and
+       target address MEMADDR.  If WRITE, transfer them to the target, else
+       transfer them from the target.  TARGET is the target from which we
+       get this function.
+
+       Return value, N, is one of the following:
+
+       0 means that we can't handle this.  If errno has been set, it is the
+       error which prevented us from doing it (FIXME: What about bfd_error?).
+
+       positive (call it N) means that we have transferred N bytes
+       starting at MEMADDR.  We might be able to handle more bytes
+       beyond this length, but no promises.
+
+       negative (call its absolute value N) means that we cannot
+       transfer right at MEMADDR, but we could transfer at least
+       something at MEMADDR + N.  */
+
+    int (*to_xfer_memory) PARAMS ((CORE_ADDR memaddr, char *myaddr,
+                                  int len, int write,
+                                  struct target_ops * target));
 
 #if 0
-  /* Enable this after 4.12.  */
+    /* Enable this after 4.12.  */
+
+    /* Search target memory.  Start at STARTADDR and take LEN bytes of
+       target memory, and them with MASK, and compare to DATA.  If they
+       match, set *ADDR_FOUND to the address we found it at, store the data
+       we found at LEN bytes starting at DATA_FOUND, and return.  If
+       not, add INCREMENT to the search address and keep trying until
+       the search address is outside of the range [LORANGE,HIRANGE).
 
-  /* Search target memory.  Start at STARTADDR and take LEN bytes of
-     target memory, and them with MASK, and compare to DATA.  If they
-     match, set *ADDR_FOUND to the address we found it at, store the data
-     we found at LEN bytes starting at DATA_FOUND, and return.  If
-     not, add INCREMENT to the search address and keep trying until
-     the search address is outside of the range [LORANGE,HIRANGE).
+       If we don't find anything, set *ADDR_FOUND to (CORE_ADDR)0 and
+       return.  */
 
-     If we don't find anything, set *ADDR_FOUND to (CORE_ADDR)0 and return.  */
-  void (*to_search) PARAMS ((int len, char *data, char *mask,
-                            CORE_ADDR startaddr, int increment,
-                            CORE_ADDR lorange, CORE_ADDR hirange,
-                            CORE_ADDR *addr_found, char *data_found));
+    void (*to_search) PARAMS ((int len, char *data, char *mask,
+                              CORE_ADDR startaddr, int increment,
+                              CORE_ADDR lorange, CORE_ADDR hirange,
+                              CORE_ADDR * addr_found, char *data_found));
 
 #define        target_search(len, data, mask, startaddr, increment, lorange, hirange, addr_found, data_found)  \
-  (*current_target.to_search) (len, data, mask, startaddr, increment, \
-                               lorange, hirange, addr_found, data_found)
-#endif /* 0 */
-
-  void               (*to_files_info) PARAMS ((struct target_ops *));
-  int                (*to_insert_breakpoint) PARAMS ((CORE_ADDR, char *));
-  int        (*to_remove_breakpoint) PARAMS ((CORE_ADDR, char *));
-  void               (*to_terminal_init) PARAMS ((void));
-  void               (*to_terminal_inferior) PARAMS ((void));
-  void               (*to_terminal_ours_for_output) PARAMS ((void));
-  void               (*to_terminal_ours) PARAMS ((void));
-  void               (*to_terminal_info) PARAMS ((char *, int));
-  void               (*to_kill) PARAMS ((void));
-  void               (*to_load) PARAMS ((char *, int));
-  int        (*to_lookup_symbol) PARAMS ((char *, CORE_ADDR *));
-  void               (*to_create_inferior) PARAMS ((char *, char *, char **));
-  void        (*to_post_startup_inferior) PARAMS ((int));
-  void        (*to_acknowledge_created_inferior) PARAMS ((int));
-  void        (*to_clone_and_follow_inferior) PARAMS ((int, int *));
-  void        (*to_post_follow_inferior_by_clone) PARAMS ((void));
-  int         (*to_insert_fork_catchpoint) PARAMS ((int));
-  int         (*to_remove_fork_catchpoint) PARAMS ((int));
-  int         (*to_insert_vfork_catchpoint) PARAMS ((int));
-  int         (*to_remove_vfork_catchpoint) PARAMS ((int));
-  int         (*to_has_forked) PARAMS ((int, int *));
-  int         (*to_has_vforked) PARAMS ((int, int *));
-  int         (*to_can_follow_vfork_prior_to_exec) PARAMS ((void));
-  void        (*to_post_follow_vfork) PARAMS ((int, int, int, int));
-  int         (*to_insert_exec_catchpoint) PARAMS ((int));
-  int         (*to_remove_exec_catchpoint) PARAMS ((int));
-  int         (*to_has_execd) PARAMS ((int, char **));
-  int         (*to_reported_exec_events_per_exec_call) PARAMS ((void));
-  int         (*to_has_syscall_event) PARAMS ((int, enum target_waitkind *, int *));
-  int         (*to_has_exited) PARAMS ((int, int, int *));
-  void               (*to_mourn_inferior) PARAMS ((void));
-  int        (*to_can_run) PARAMS ((void));
-  void       (*to_notice_signals) PARAMS ((int pid));
-  int        (*to_thread_alive) PARAMS ((int pid));
-  void       (*to_stop) PARAMS ((void));
-  int        (*to_query) PARAMS ((int/*char*/, char *, char *, int *));
-  struct symtab_and_line * (*to_enable_exception_callback) PARAMS ((enum exception_event_kind, int));
-  struct exception_event_record * (*to_get_current_exception_event) PARAMS ((void));
-  char *      (*to_pid_to_exec_file) PARAMS ((int pid));
-  char *      (*to_core_file_to_sym_file) PARAMS ((char *));
-  enum strata   to_stratum;
-  struct target_ops
-               *DONT_USE;      /* formerly to_next */
-  int          to_has_all_memory;
-  int          to_has_memory;
-  int          to_has_stack;
-  int          to_has_registers;
-  int          to_has_execution;
-  int          to_has_thread_control;  /* control thread execution */
-  struct section_table
-              *to_sections;
-  struct section_table
-              *to_sections_end;
-  int          to_magic;
-  /* Need sub-structure for target machine related rather than comm related? */
-};
+    (*current_target.to_search) (len, data, mask, startaddr, increment, \
+                                lorange, hirange, addr_found, data_found)
+#endif                         /* 0 */
+
+    void (*to_files_info) PARAMS ((struct target_ops *));
+    int (*to_insert_breakpoint) PARAMS ((CORE_ADDR, char *));
+    int (*to_remove_breakpoint) PARAMS ((CORE_ADDR, char *));
+    void (*to_terminal_init) PARAMS ((void));
+    void (*to_terminal_inferior) PARAMS ((void));
+    void (*to_terminal_ours_for_output) PARAMS ((void));
+    void (*to_terminal_ours) PARAMS ((void));
+    void (*to_terminal_info) PARAMS ((char *, int));
+    void (*to_kill) PARAMS ((void));
+    void (*to_load) PARAMS ((char *, int));
+    int (*to_lookup_symbol) PARAMS ((char *, CORE_ADDR *));
+    void (*to_create_inferior) PARAMS ((char *, char *, char **));
+    void (*to_post_startup_inferior) PARAMS ((int));
+    void (*to_acknowledge_created_inferior) PARAMS ((int));
+    void (*to_clone_and_follow_inferior) PARAMS ((int, int *));
+    void (*to_post_follow_inferior_by_clone) PARAMS ((void));
+    int (*to_insert_fork_catchpoint) PARAMS ((int));
+    int (*to_remove_fork_catchpoint) PARAMS ((int));
+    int (*to_insert_vfork_catchpoint) PARAMS ((int));
+    int (*to_remove_vfork_catchpoint) PARAMS ((int));
+    int (*to_has_forked) PARAMS ((int, int *));
+    int (*to_has_vforked) PARAMS ((int, int *));
+    int (*to_can_follow_vfork_prior_to_exec) PARAMS ((void));
+    void (*to_post_follow_vfork) PARAMS ((int, int, int, int));
+    int (*to_insert_exec_catchpoint) PARAMS ((int));
+    int (*to_remove_exec_catchpoint) PARAMS ((int));
+    int (*to_has_execd) PARAMS ((int, char **));
+    int (*to_reported_exec_events_per_exec_call) PARAMS ((void));
+    int (*to_has_syscall_event) PARAMS ((int, enum target_waitkind *, int *));
+    int (*to_has_exited) PARAMS ((int, int, int *));
+    void (*to_mourn_inferior) PARAMS ((void));
+    int (*to_can_run) PARAMS ((void));
+    void (*to_notice_signals) PARAMS ((int pid));
+    int (*to_thread_alive) PARAMS ((int pid));
+    void (*to_find_new_threads) PARAMS ((void));
+    char *(*to_pid_to_str) PARAMS ((int));
+    char *(*to_extra_thread_info) PARAMS ((struct thread_info *));
+    void (*to_stop) PARAMS ((void));
+    int (*to_query) PARAMS ((int /*char */ , char *, char *, int *));
+    void (*to_rcmd) (char *command, struct ui_file *output);
+    struct symtab_and_line *(*to_enable_exception_callback) PARAMS ((enum exception_event_kind, int));
+    struct exception_event_record *(*to_get_current_exception_event) PARAMS ((void));
+    char *(*to_pid_to_exec_file) PARAMS ((int pid));
+    char *(*to_core_file_to_sym_file) PARAMS ((char *));
+    enum strata to_stratum;
+    struct target_ops
+     *DONT_USE;                        /* formerly to_next */
+    int to_has_all_memory;
+    int to_has_memory;
+    int to_has_stack;
+    int to_has_registers;
+    int to_has_execution;
+    int to_has_thread_control; /* control thread execution */
+    struct section_table
+     *to_sections;
+    struct section_table
+     *to_sections_end;
+    /* ASYNC target controls */
+    int (*to_can_async_p) (void);
+    int (*to_is_async_p) (void);
+    void (*to_async) (void (*cb) (enum inferior_event_type, void *context),
+                     void *context);
+    int to_async_mask_value;
+    int to_magic;
+    /* Need sub-structure for target machine related rather than comm related?
+     */
+  };
 
 /* Magic number for checking ops size.  If a struct doesn't end with this
    number, somebody changed the declaration but didn't change all the
@@ -386,15 +463,15 @@ struct target_ops
 /* The ops structure for our "current" target process.  This should
    never be NULL.  If there is no target, it points to the dummy_target.  */
 
-extern struct target_ops       current_target;
+extern struct target_ops current_target;
 
 /* An item on the target stack.  */
 
 struct target_stack_item
-{
-  struct target_stack_item *next;
-  struct target_ops *target_ops;
-};
+  {
+    struct target_stack_item *next;
+    struct target_ops *target_ops;
+  };
 
 /* The target stack.  */
 
@@ -408,8 +485,9 @@ extern struct target_stack_item *target_stack;
 /* The open routine takes the rest of the parameters from the command,
    and (if successful) pushes a new target onto the stack.
    Targets should supply this routine, if only to provide an error message.  */
+
 #define        target_open(name, from_tty)     \
-       (*current_target.to_open) (name, from_tty)
+     (*current_target.to_open) (name, from_tty)
 
 /* Does whatever cleanup is required for a target that we are no longer
    going to be calling.  Argument says whether we are quitting gdb and
@@ -420,7 +498,7 @@ extern struct target_stack_item *target_stack;
    do.  */
 
 #define        target_close(quitting)  \
-       (*current_target.to_close) (quitting)
+     (*current_target.to_close) (quitting)
 
 /* Attaches to a process on the target side.  Arguments are as passed
    to the `attach' command by the user.  This routine can be called
@@ -431,16 +509,15 @@ extern struct target_stack_item *target_stack;
    (without waiting) to an upcoming target_wait call.  */
 
 #define        target_attach(args, from_tty)   \
-       (*current_target.to_attach) (args, from_tty)
+     (*current_target.to_attach) (args, from_tty)
 
 /* The target_attach operation places a process under debugger control,
    and stops the process.
 
    This operation provides a target-specific hook that allows the
-   necessary bookkeeping to be performed after an attach completes.
-   */
+   necessary bookkeeping to be performed after an attach completes.  */
 #define target_post_attach(pid) \
-        (*current_target.to_post_attach) (pid)
+     (*current_target.to_post_attach) (pid)
 
 /* Attaches to a process on the target side, if not already attached.
    (If already attached, takes no action.)
@@ -451,7 +528,7 @@ extern struct target_stack_item *target_stack;
    actual attach operation.  */
 
 #define        target_require_attach(args, from_tty)   \
-       (*current_target.to_require_attach) (args, from_tty)
+     (*current_target.to_require_attach) (args, from_tty)
 
 /* Takes a program previously attached to and detaches it.
    The program may resume execution (some targets do, some don't) and will
@@ -475,8 +552,8 @@ target_detach PARAMS ((char *, int));
    ARGS is arguments typed by the user (e.g. a signal to send the process).
    FROM_TTY says whether to be verbose or not.  */
 
-#define target_require_detach(pid, args, from_tty) \
-       (*current_target.to_require_detach) (pid, args, from_tty)
+#define target_require_detach(pid, args, from_tty)     \
+     (*current_target.to_require_detach) (pid, args, from_tty)
 
 /* Resume execution of the target process PID.  STEP says whether to
    single-step or to run free; SIGGNAL is the signal to be given to
@@ -484,7 +561,7 @@ target_detach PARAMS ((char *, int));
    pass TARGET_SIGNAL_DEFAULT.  */
 
 #define        target_resume(pid, step, siggnal)       \
-       (*current_target.to_resume) (pid, step, siggnal)
+     (*current_target.to_resume) (pid, step, siggnal)
 
 /* Wait for process pid to do something.  Pid = -1 to wait for any pid
    to do something.  Return pid of child, or -1 in case of error;
@@ -495,7 +572,7 @@ target_detach PARAMS ((char *, int));
    stop_pc, etc., set up.  */
 
 #define        target_wait(pid, status)                \
-       (*current_target.to_wait) (pid, status)
+     (*current_target.to_wait) (pid, status)
 
 /* The target_wait operation waits for a process event to occur, and
    thereby stop the process.
@@ -505,23 +582,22 @@ target_detach PARAMS ((char *, int));
    knowledge of what earlier events in the sequence have been seen.
 
    This operation provides a target-specific hook that allows the
-   necessary bookkeeping to be performed to track such sequences.
-   */
+   necessary bookkeeping to be performed to track such sequences.  */
 
 #define target_post_wait(pid, status) \
-        (*current_target.to_post_wait) (pid, status)
+     (*current_target.to_post_wait) (pid, status)
 
 /* Fetch register REGNO, or all regs if regno == -1.  No result.  */
 
 #define        target_fetch_registers(regno)   \
-       (*current_target.to_fetch_registers) (regno)
+     (*current_target.to_fetch_registers) (regno)
 
 /* Store at least register REGNO, or all regs if REGNO == -1.
    It can store as many registers as it wants to, so target_prepare_to_store
    must have been previously called.  Calls error() if there are problems.  */
 
 #define        target_store_registers(regs)    \
-       (*current_target.to_store_registers) (regs)
+     (*current_target.to_store_registers) (regs)
 
 /* Get ready to modify the registers array.  On machines which store
    individual registers, this doesn't need to do anything.  On machines
@@ -530,19 +606,17 @@ target_detach PARAMS ((char *, int));
    debugged.  */
 
 #define        target_prepare_to_store()       \
-       (*current_target.to_prepare_to_store) ()
+     (*current_target.to_prepare_to_store) ()
 
-extern int target_read_string PARAMS ((CORE_ADDR, char **, int, int *));
+extern int 
+target_read_string PARAMS ((CORE_ADDR, char **, int, int *));
 
 extern int
 target_read_memory PARAMS ((CORE_ADDR memaddr, char *myaddr, int len));
 
 extern int
 target_read_memory_section PARAMS ((CORE_ADDR memaddr, char *myaddr, int len,
-                                   asection *bfd_section));
-
-extern int
-target_read_memory_partial PARAMS ((CORE_ADDR, char *, int, int *));
+                                   asection * bfd_section));
 
 extern int
 target_write_memory PARAMS ((CORE_ADDR, char *, int));
@@ -553,6 +627,18 @@ xfer_memory PARAMS ((CORE_ADDR, char *, int, int, struct target_ops *));
 extern int
 child_xfer_memory PARAMS ((CORE_ADDR, char *, int, int, struct target_ops *));
 
+/* Make a single attempt at transfering LEN bytes.  On a successful
+   transfer, the number of bytes actually transfered is returned and
+   ERR is set to 0.  When a transfer fails, -1 is returned (the number
+   of bytes actually transfered is not defined) and ERR is set to a
+   non-zero error indication.  */
+
+extern int 
+target_read_memory_partial (CORE_ADDR addr, char *buf, int len, int *err);
+
+extern int 
+target_write_memory_partial (CORE_ADDR addr, char *buf, int len, int *err);
+
 extern char *
 child_pid_to_exec_file PARAMS ((int));
 
@@ -635,7 +721,7 @@ print_section_info PARAMS ((struct target_ops *, bfd *));
 /* Print a line about the current target.  */
 
 #define        target_files_info()     \
-       (*current_target.to_files_info) (&current_target)
+     (*current_target.to_files_info) (&current_target)
 
 /* Insert a breakpoint at address ADDR in the target machine.
    SAVE is a pointer to memory allocated for saving the
@@ -644,7 +730,7 @@ print_section_info PARAMS ((struct target_ops *, bfd *));
    an errno value.  */
 
 #define        target_insert_breakpoint(addr, save)    \
-       (*current_target.to_insert_breakpoint) (addr, save)
+     (*current_target.to_insert_breakpoint) (addr, save)
 
 /* Remove a breakpoint at address ADDR in the target machine.
    SAVE is a pointer to the same save area 
@@ -652,19 +738,19 @@ print_section_info PARAMS ((struct target_ops *, bfd *));
    Result is 0 for success, or an errno value.  */
 
 #define        target_remove_breakpoint(addr, save)    \
-       (*current_target.to_remove_breakpoint) (addr, save)
+     (*current_target.to_remove_breakpoint) (addr, save)
 
 /* Initialize the terminal settings we record for the inferior,
    before we actually run the inferior.  */
 
 #define target_terminal_init() \
-       (*current_target.to_terminal_init) ()
+     (*current_target.to_terminal_init) ()
 
 /* Put the inferior's terminal settings into effect.
    This is preparation for starting or resuming the inferior.  */
 
 #define target_terminal_inferior() \
-       (*current_target.to_terminal_inferior) ()
+     (*current_target.to_terminal_inferior) ()
 
 /* Put some of our terminal settings into effect,
    enough to get proper results from our output,
@@ -675,52 +761,51 @@ print_section_info PARAMS ((struct target_ops *, bfd *));
    should be called to get back to a normal state of affairs.  */
 
 #define target_terminal_ours_for_output() \
-       (*current_target.to_terminal_ours_for_output) ()
+     (*current_target.to_terminal_ours_for_output) ()
 
 /* Put our terminal settings into effect.
    First record the inferior's terminal settings
    so they can be restored properly later.  */
 
 #define target_terminal_ours() \
-       (*current_target.to_terminal_ours) ()
+     (*current_target.to_terminal_ours) ()
 
 /* Print useful information about our terminal status, if such a thing
    exists.  */
 
 #define target_terminal_info(arg, from_tty) \
-       (*current_target.to_terminal_info) (arg, from_tty)
+     (*current_target.to_terminal_info) (arg, from_tty)
 
 /* Kill the inferior process.   Make it go away.  */
 
 #define target_kill() \
-       (*current_target.to_kill) ()
+     (*current_target.to_kill) ()
 
-/* Load an executable file into the target process.  This is expected to
-   not only bring new code into the target process, but also to update
-   GDB's symbol tables to match.  */
+/* Load an executable file into the target process.  This is expected
+   to not only bring new code into the target process, but also to
+   update GDB's symbol tables to match.  */
 
-#define target_load(arg, from_tty) \
-       (*current_target.to_load) (arg, from_tty)
+extern void target_load (char *arg, int from_tty);
 
 /* Look up a symbol in the target's symbol table.  NAME is the symbol
-   name.  ADDRP is a CORE_ADDR * pointing to where the value of the symbol
-   should be returned.  The result is 0 if successful, nonzero if the
-   symbol does not exist in the target environment.  This function should
-   not call error() if communication with the target is interrupted, since
-   it is called from symbol reading, but should return nonzero, possibly
-   doing a complain().  */
+   name.  ADDRP is a CORE_ADDR * pointing to where the value of the
+   symbol should be returned.  The result is 0 if successful, nonzero
+   if the symbol does not exist in the target environment.  This
+   function should not call error() if communication with the target
+   is interrupted, since it is called from symbol reading, but should
+   return nonzero, possibly doing a complain().  */
 
-#define target_lookup_symbol(name, addrp)      \
-  (*current_target.to_lookup_symbol) (name, addrp)
+#define target_lookup_symbol(name, addrp) \
+     (*current_target.to_lookup_symbol) (name, addrp)
 
 /* Start an inferior process and set inferior_pid to its pid.
    EXEC_FILE is the file to run.
    ALLARGS is a string containing the arguments to the program.
    ENV is the environment vector to pass.  Errors reported with error().
    On VxWorks and various standalone systems, we ignore exec_file.  */
+
 #define        target_create_inferior(exec_file, args, env)    \
-       (*current_target.to_create_inferior) (exec_file, args, env)
+     (*current_target.to_create_inferior) (exec_file, args, env)
 
 
 /* Some targets (such as ttrace-based HPUX) don't allow us to request
@@ -730,17 +815,17 @@ print_section_info PARAMS ((struct target_ops *, bfd *));
    if the shell init file has commands in it, the shell will fork and
    exec for each of those commands, and we will see each such fork
    event.  Very bad.)
-   
-   Such targets will supply an appropriate definition for this function.
-   */
+
+   Such targets will supply an appropriate definition for this function.  */
+
 #define target_post_startup_inferior(pid) \
-        (*current_target.to_post_startup_inferior) (pid)
+     (*current_target.to_post_startup_inferior) (pid)
 
 /* On some targets, the sequence of starting up an inferior requires
-   some synchronization between gdb and the new inferior process, PID.
-   */
+   some synchronization between gdb and the new inferior process, PID.  */
+
 #define target_acknowledge_created_inferior(pid) \
-        (*current_target.to_acknowledge_created_inferior) (pid)
+     (*current_target.to_acknowledge_created_inferior) (pid)
 
 /* An inferior process has been created via a fork() or similar
    system call.  This function will clone the debugger, then ensure
@@ -751,10 +836,10 @@ print_section_info PARAMS ((struct target_ops *, bfd *));
    to determine which they are, if need be.)
 
    (This is not a terribly useful feature without a GUI to prevent
-   the two debuggers from competing for shell input.)
-   */
+   the two debuggers from competing for shell input.)  */
+
 #define target_clone_and_follow_inferior(child_pid,followed_child) \
-        (*current_target.to_clone_and_follow_inferior) (child_pid, followed_child)
+     (*current_target.to_clone_and_follow_inferior) (child_pid, followed_child)
 
 /* This operation is intended to be used as the last in a sequence of
    steps taken when following both parent and child of a fork.  This
@@ -764,40 +849,40 @@ print_section_info PARAMS ((struct target_ops *, bfd *));
    clone has attached to it.
 
    On some targets, this requires a bit of cleanup to make it work
-   correctly.
-   */
+   correctly.  */
+
 #define target_post_follow_inferior_by_clone() \
-        (*current_target.to_post_follow_inferior_by_clone) ()
+     (*current_target.to_post_follow_inferior_by_clone) ()
+
+/* On some targets, we can catch an inferior fork or vfork event when
+   it occurs.  These functions insert/remove an already-created
+   catchpoint for such events.  */
 
-/* On some targets, we can catch an inferior fork or vfork event when it
-   occurs.  These functions insert/remove an already-created catchpoint for
-   such events.
-   */
 #define target_insert_fork_catchpoint(pid) \
-        (*current_target.to_insert_fork_catchpoint) (pid)
+     (*current_target.to_insert_fork_catchpoint) (pid)
 
 #define target_remove_fork_catchpoint(pid) \
-        (*current_target.to_remove_fork_catchpoint) (pid)
+     (*current_target.to_remove_fork_catchpoint) (pid)
 
 #define target_insert_vfork_catchpoint(pid) \
-        (*current_target.to_insert_vfork_catchpoint) (pid)
+     (*current_target.to_insert_vfork_catchpoint) (pid)
 
 #define target_remove_vfork_catchpoint(pid) \
-        (*current_target.to_remove_vfork_catchpoint) (pid)
+     (*current_target.to_remove_vfork_catchpoint) (pid)
 
 /* Returns TRUE if PID has invoked the fork() system call.  And,
    also sets CHILD_PID to the process id of the other ("child")
-   inferior process that was created by that call.
-   */
+   inferior process that was created by that call.  */
+
 #define target_has_forked(pid,child_pid) \
-        (*current_target.to_has_forked) (pid,child_pid)
+     (*current_target.to_has_forked) (pid,child_pid)
+
+/* Returns TRUE if PID has invoked the vfork() system call.  And, 
+   also sets CHILD_PID to the process id of the other ("child") 
+   inferior process that was created by that call.  */
 
-/* Returns TRUE if PID has invoked the vfork() system call.  And,
-   also sets CHILD_PID to the process id of the other ("child")
-   inferior process that was created by that call.
-   */
 #define target_has_vforked(pid,child_pid) \
-        (*current_target.to_has_vforked) (pid,child_pid)
+     (*current_target.to_has_vforked) (pid,child_pid)
 
 /* Some platforms (such as pre-10.20 HP-UX) don't allow us to do
    anything to a vforked child before it subsequently calls exec().
@@ -806,10 +891,10 @@ print_section_info PARAMS ((struct target_ops *, bfd *));
 
    This function should be defined to return 1 by those targets
    which can allow the debugger to immediately follow a vforked
-   child, and 0 if they cannot.
-   */
+   child, and 0 if they cannot.  */
+
 #define target_can_follow_vfork_prior_to_exec() \
-        (*current_target.to_can_follow_vfork_prior_to_exec) ()
+     (*current_target.to_can_follow_vfork_prior_to_exec) ()
 
 /* An inferior process has been created via a vfork() system call.
    The debugger has followed the parent, the child, or both.  The
@@ -817,77 +902,81 @@ print_section_info PARAMS ((struct target_ops *, bfd *));
    target-specific trickery to track the sequence of reported events.
    If so, this function should be defined by those targets that
    require the debugger to perform cleanup or initialization after
-   the vfork follow.
-   */
+   the vfork follow.  */
+
 #define target_post_follow_vfork(parent_pid,followed_parent,child_pid,followed_child) \
-        (*current_target.to_post_follow_vfork) (parent_pid,followed_parent,child_pid,followed_child)
+     (*current_target.to_post_follow_vfork) (parent_pid,followed_parent,child_pid,followed_child)
 
 /* On some targets, we can catch an inferior exec event when it
-   occurs.  These functions insert/remove an already-created catchpoint
-   for such events.
-   */
+   occurs.  These functions insert/remove an already-created
+   catchpoint for such events.  */
+
 #define target_insert_exec_catchpoint(pid) \
-        (*current_target.to_insert_exec_catchpoint) (pid)
+     (*current_target.to_insert_exec_catchpoint) (pid)
+
 #define target_remove_exec_catchpoint(pid) \
-        (*current_target.to_remove_exec_catchpoint) (pid)
+     (*current_target.to_remove_exec_catchpoint) (pid)
 
 /* Returns TRUE if PID has invoked a flavor of the exec() system call.
-   And, also sets EXECD_PATHNAME to the pathname of the executable file
-   that was passed to exec(), and is now being executed.
-   */
+   And, also sets EXECD_PATHNAME to the pathname of the executable
+   file that was passed to exec(), and is now being executed.  */
+
 #define target_has_execd(pid,execd_pathname) \
-        (*current_target.to_has_execd) (pid,execd_pathname)
+     (*current_target.to_has_execd) (pid,execd_pathname)
 
 /* Returns the number of exec events that are reported when a process
    invokes a flavor of the exec() system call on this target, if exec
-   events are being reported.
-   */
+   events are being reported.  */
+
 #define target_reported_exec_events_per_exec_call() \
-        (*current_target.to_reported_exec_events_per_exec_call) ()
+     (*current_target.to_reported_exec_events_per_exec_call) ()
 
 /* Returns TRUE if PID has reported a syscall event.  And, also sets
    KIND to the appropriate TARGET_WAITKIND_, and sets SYSCALL_ID to
-   the unique integer ID of the syscall.
-   */
+   the unique integer ID of the syscall.  */
+
 #define target_has_syscall_event(pid,kind,syscall_id) \
-  (*current_target.to_has_syscall_event) (pid,kind,syscall_id)
+     (*current_target.to_has_syscall_event) (pid,kind,syscall_id)
 
 /* Returns TRUE if PID has exited.  And, also sets EXIT_STATUS to the
-   exit code of PID, if any.
-   */
+   exit code of PID, if any.  */
+
 #define target_has_exited(pid,wait_status,exit_status) \
-        (*current_target.to_has_exited) (pid,wait_status,exit_status)
+     (*current_target.to_has_exited) (pid,wait_status,exit_status)
 
 /* The debugger has completed a blocking wait() call.  There is now
-   some process event that must be processed.  This function should
+   some process event that must be processed.  This function should 
    be defined by those targets that require the debugger to perform
-   cleanup or internal state changes in response to the process event.
-   */
+   cleanup or internal state changes in response to the process event.  */
 
 /* The inferior process has died.  Do what is right.  */
 
 #define        target_mourn_inferior() \
-       (*current_target.to_mourn_inferior) ()
+     (*current_target.to_mourn_inferior) ()
 
 /* Does target have enough data to do a run or attach command? */
 
 #define target_can_run(t) \
-       ((t)->to_can_run) ()
+     ((t)->to_can_run) ()
 
 /* post process changes to signal handling in the inferior.  */
 
 #define target_notice_signals(pid) \
-       (*current_target.to_notice_signals) (pid)
+     (*current_target.to_notice_signals) (pid)
 
 /* Check to see if a thread is still alive.  */
 
 #define target_thread_alive(pid) \
-       (*current_target.to_thread_alive) (pid)
+     (*current_target.to_thread_alive) (pid)
+
+/* Query for new threads and add them to the thread list.  */
+
+#define target_find_new_threads() \
+     (*current_target.to_find_new_threads) (); \
 
-/* Make target stop in a continuable fashion.  (For instance, under Unix, this
-   should act like SIGSTOP).  This function is normally used by GUIs to
-   implement a stop button.  */
+/* Make target stop in a continuable fashion.  (For instance, under
+   Unix, this should act like SIGSTOP).  This function is normally
+   used by GUIs to implement a stop button.  */
 
 #define target_stop current_target.to_stop
 
@@ -896,53 +985,61 @@ print_section_info PARAMS ((struct target_ops *, bfd *));
    should process it.  The second argument is a string that specifies which 
    information is desired and the third is a buffer that carries back the 
    response from the target side. The fourth parameter is the size of the
-   output buffer supplied. */
+   output buffer supplied.  */
+
 #define        target_query(query_type, query, resp_buffer, bufffer_size)      \
-       (*current_target.to_query) (query_type, query, resp_buffer, bufffer_size)
+     (*current_target.to_query) (query_type, query, resp_buffer, bufffer_size)
+
+/* Send the specified COMMAND to the target's monitor
+   (shell,interpreter) for execution.  The result of the query is
+   placed in OUTBUF.  */
+
+#define target_rcmd(command, outbuf) \
+     (*current_target.to_rcmd) (command, outbuf)
+
 
 /* Get the symbol information for a breakpointable routine called when
    an exception event occurs. 
    Intended mainly for C++, and for those
    platforms/implementations where such a callback mechanism is available,
    e.g. HP-UX with ANSI C++ (aCC).  Some compilers (e.g. g++) support
-   different mechanisms for debugging exceptions. */
+   different mechanisms for debugging exceptions.  */
 
 #define target_enable_exception_callback(kind, enable) \
-        (*current_target.to_enable_exception_callback) (kind, enable)
+     (*current_target.to_enable_exception_callback) (kind, enable)
+
+/* Get the current exception event kind -- throw or catch, etc.  */
 
-/* Get the current exception event kind -- throw or catch, etc. */
-   
 #define target_get_current_exception_event() \
-        (*current_target.to_get_current_exception_event) ()
+     (*current_target.to_get_current_exception_event) ()
 
 /* Pointer to next target in the chain, e.g. a core file and an exec file.  */
 
 #define        target_next \
-       (current_target.to_next)
+     (current_target.to_next)
 
 /* Does the target include all of memory, or only part of it?  This
    determines whether we look up the target chain for other parts of
    memory if this target can't satisfy a request.  */
 
 #define        target_has_all_memory   \
-       (current_target.to_has_all_memory)
+     (current_target.to_has_all_memory)
 
 /* Does the target include memory?  (Dummy targets don't.)  */
 
 #define        target_has_memory       \
-       (current_target.to_has_memory)
+     (current_target.to_has_memory)
 
 /* Does the target have a stack?  (Exec files don't, VxWorks doesn't, until
    we start a process.)  */
-   
+
 #define        target_has_stack        \
-       (current_target.to_has_stack)
+     (current_target.to_has_stack)
 
 /* Does the target have registers?  (Exec files don't.)  */
 
 #define        target_has_registers    \
-       (current_target.to_has_registers)
+     (current_target.to_has_registers)
 
 /* Does the target have execution?  Can we make it jump (through
    hoops), or pop its stack a few times?  FIXME: If this is to work that
@@ -952,17 +1049,46 @@ print_section_info PARAMS ((struct target_ops *, bfd *));
    this just tells us whether this target is *capable* of execution.  */
 
 #define        target_has_execution    \
-       (current_target.to_has_execution)
+     (current_target.to_has_execution)
 
 /* Can the target support the debugger control of thread execution?
    a) Can it lock the thread scheduler?
    b) Can it switch the currently running thread?  */
 
 #define target_can_lock_scheduler \
-       (current_target.to_has_thread_control & tc_schedlock)
+     (current_target.to_has_thread_control & tc_schedlock)
 
 #define target_can_switch_threads \
-       (current_target.to_has_thread_control & tc_switch)
+     (current_target.to_has_thread_control & tc_switch)
+
+/* Can the target support asynchronous execution? */
+#define target_can_async_p() (current_target.to_can_async_p ())
+
+/* Is the target in asynchronous execution mode? */
+#define target_is_async_p() (current_target.to_is_async_p())
+
+/* Put the target in async mode with the specified callback function. */
+#define target_async(CALLBACK,CONTEXT) \
+     (current_target.to_async((CALLBACK), (CONTEXT)))
+
+/* This is to be used ONLY within run_stack_dummy(). It
+   provides a workaround, to have inferior function calls done in
+   sychronous mode, even though the target is asynchronous. After
+   target_async_mask(0) is called, calls to target_can_async_p() will
+   return FALSE , so that target_resume() will not try to start the
+   target asynchronously. After the inferior stops, we IMMEDIATELY
+   restore the previous nature of the target, by calling
+   target_async_mask(1). After that, target_can_async_p() will return
+   TRUE. ANY OTHER USE OF THIS FEATURE IS DEPRECATED. 
+
+   FIXME ezannoni 1999-12-13: we won't need this once we move
+   the turning async on and off to the single execution commands,
+   from where it is done currently, in remote_resume().  */
+
+#define        target_async_mask_value \
+     (current_target.to_async_mask_value)
+
+extern int target_async_mask (int mask);     
 
 extern void target_link PARAMS ((char *, CORE_ADDR *));
 
@@ -970,45 +1096,68 @@ extern void target_link PARAMS ((char *, CORE_ADDR *));
    `process xyz', but on some systems it may contain
    `process xyz thread abc'.  */
 
-#ifndef target_pid_to_str
-#define target_pid_to_str(PID) \
-       normal_pid_to_str (PID)
-extern char *normal_pid_to_str PARAMS ((int pid));
-#endif
+#undef target_pid_to_str
+#define target_pid_to_str(PID) current_target.to_pid_to_str (PID)
 
 #ifndef target_tid_to_str
 #define target_tid_to_str(PID) \
-        normal_pid_to_str (PID)
+     target_pid_to_str (PID)
 extern char *normal_pid_to_str PARAMS ((int pid));
 #endif
 
-#ifndef target_new_objfile
-#define target_new_objfile(OBJFILE)
-#endif
+/* Return a short string describing extra information about PID,
+   e.g. "sleeping", "runnable", "running on LWP 3".  Null return value
+   is okay.  */
+
+#define target_extra_thread_info(TP) \
+     (current_target.to_extra_thread_info (TP))
+
+/*
+ * New Objfile Event Hook:
+ *
+ * Sometimes a GDB component wants to get notified whenever a new
+ * objfile is loaded.  Mainly this is used by thread-debugging 
+ * implementations that need to know when symbols for the target
+ * thread implemenation are available.
+ *
+ * The old way of doing this is to define a macro 'target_new_objfile'
+ * that points to the function that you want to be called on every
+ * objfile/shlib load.
+ *
+ * The new way is to grab the function pointer, 'target_new_objfile_hook',
+ * and point it to the function that you want to be called on every
+ * objfile/shlib load.
+ *
+ * If multiple clients are willing to be cooperative, they can each
+ * save a pointer to the previous value of target_new_objfile_hook
+ * before modifying it, and arrange for their function to call the
+ * previous function in the chain.  In that way, multiple clients
+ * can receive this notification (something like with signal handlers).
+ */
+
+extern void (*target_new_objfile_hook) PARAMS ((struct objfile *));
 
 #ifndef target_pid_or_tid_to_str
 #define target_pid_or_tid_to_str(ID) \
-       normal_pid_to_str (ID)
+     target_pid_to_str (ID)
 #endif
 
 /* Attempts to find the pathname of the executable file
    that was run to create a specified process.
 
    The process PID must be stopped when this operation is used.
-   
+
    If the executable file cannot be determined, NULL is returned.
 
    Else, a pointer to a character string containing the pathname
    is returned.  This string should be copied into a buffer by
    the client if the string will not be immediately used, or if
-   it must persist.
-   */
+   it must persist.  */
 
 #define target_pid_to_exec_file(pid) \
-       (current_target.to_pid_to_exec_file) (pid)
+     (current_target.to_pid_to_exec_file) (pid)
 
-/* Hook to call target-dependant code after reading in a new symbol table. */
+/* Hook to call target-dependant code after reading in a new symbol table.  */
 
 #ifndef TARGET_SYMFILE_POSTREAD
 #define TARGET_SYMFILE_POSTREAD(OBJFILE)
@@ -1032,8 +1181,8 @@ extern char *normal_pid_to_str PARAMS ((int pid));
 
 /* HP-UX supplies these operations, which respectively disable and enable
    the memory page-protections that are used to implement hardware watchpoints
-   on that platform.  See wait_for_inferior's use of these.
-   */
+   on that platform.  See wait_for_inferior's use of these.  */
+
 #if !defined(TARGET_DISABLE_HW_WATCHPOINTS)
 #define TARGET_DISABLE_HW_WATCHPOINTS(pid)
 #endif
@@ -1042,7 +1191,7 @@ extern char *normal_pid_to_str PARAMS ((int pid));
 #define TARGET_ENABLE_HW_WATCHPOINTS(pid)
 #endif
 
-/* Provide defaults for systems that don't support hardware watchpoints. */
+/* Provide defaults for systems that don't support hardware watchpoints.  */
 
 #ifndef TARGET_HAS_HARDWARE_WATCHPOINTS
 
@@ -1055,7 +1204,7 @@ extern char *normal_pid_to_str PARAMS ((int pid));
 
 #if !defined(TARGET_REGION_SIZE_OK_FOR_HW_WATCHPOINT)
 #define TARGET_REGION_SIZE_OK_FOR_HW_WATCHPOINT(byte_count) \
-        (LONGEST)(byte_count) <= REGISTER_SIZE
+     (LONGEST)(byte_count) <= REGISTER_SIZE
 #endif
 
 /* However, some addresses may not be profitable to use hardware to watch,
@@ -1063,8 +1212,8 @@ extern char *normal_pid_to_str PARAMS ((int pid));
    scope, and hence should be unwatched.  On some targets, this may have
    severe performance penalties, such that we might as well use regular
    watchpoints, and save (possibly precious) hardware watchpoints for other
-   locations.
-   */
+   locations.  */
+
 #if !defined(TARGET_RANGE_PROFITABLE_FOR_HW_WATCHPOINT)
 #define TARGET_RANGE_PROFITABLE_FOR_HW_WATCHPOINT(pid,start,len) 0
 #endif
@@ -1098,8 +1247,8 @@ extern char *normal_pid_to_str PARAMS ((int pid));
 /* Sometimes gdb may pick up what appears to be a valid target address
    from a minimal symbol, but the value really means, essentially,
    "This is an index into a table which is populated when the inferior
-   is run.  Therefore, do not attempt to use this as a PC."
-   */
+   is run.  Therefore, do not attempt to use this as a PC."  */
+
 #if !defined(PC_REQUIRES_RUN_BEFORE_USE)
 #define PC_REQUIRES_RUN_BEFORE_USE(pc) (0)
 #endif
@@ -1110,8 +1259,7 @@ extern char *normal_pid_to_str PARAMS ((int pid));
    On some targets (such as HP-UX 10.20 and earlier), resuming a newly vforked
    child process after it has exec'd, causes the parent process to resume as
    well.  To prevent the parent from running spontaneously, such targets should
-   define this to a function that prevents that from happening.
-   */
+   define this to a function that prevents that from happening.  */
 #if !defined(ENSURE_VFORKING_PARENT_REMAINS_STOPPED)
 #define ENSURE_VFORKING_PARENT_REMAINS_STOPPED(PID) (0)
 #endif
@@ -1121,8 +1269,8 @@ extern char *normal_pid_to_str PARAMS ((int pid));
 
    On some targets (such as HP-UX 10.20 and earlier), a newly vforked child
    process must be resumed when it delivers its exec event, before the parent
-   vfork event will be delivered to us.
-   */
+   vfork event will be delivered to us.  */
+
 #if !defined(RESUME_EXECD_VFORKING_CHILD_TO_GET_PARENT_VFORK)
 #define RESUME_EXECD_VFORKING_CHILD_TO_GET_PARENT_VFORK() (0)
 #endif
@@ -1132,15 +1280,15 @@ extern char *normal_pid_to_str PARAMS ((int pid));
    add_target:   Add a target to the list of all possible targets.
 
    push_target:  Make this target the top of the stack of currently used
-                targets, within its particular stratum of the stack.  Result
-                is 0 if now atop the stack, nonzero if not on top (maybe
-                should warn user).
+   targets, within its particular stratum of the stack.  Result
+   is 0 if now atop the stack, nonzero if not on top (maybe
+   should warn user).
 
    unpush_target: Remove this from the stack of currently used targets,
-                no matter where it is on the list.  Returns 0 if no
-                change, 1 if removed from stack.
+   no matter where it is on the list.  Returns 0 if no
+   change, 1 if removed from stack.
 
-   pop_target:  Remove the top thing on the stack of current targets.  */
+   pop_target:   Remove the top thing on the stack of current targets.  */
 
 extern void
 add_target PARAMS ((struct target_ops *));
@@ -1161,14 +1309,15 @@ pop_target PARAMS ((void));
    mostly used with BFD files, but can be used without (e.g. for handling
    raw disks, or files not in formats handled by BFD).  */
 
-struct section_table {
-  CORE_ADDR addr;              /* Lowest address in section */
-  CORE_ADDR endaddr;           /* 1+highest address in section */
+struct section_table
+  {
+    CORE_ADDR addr;            /* Lowest address in section */
+    CORE_ADDR endaddr;         /* 1+highest address in section */
 
-  sec_ptr the_bfd_section;
+    sec_ptr the_bfd_section;
 
-  bfd     *bfd;                /* BFD file pointer */
-};
+    bfd *bfd;                  /* BFD file pointer */
+  };
 
 /* Builds a section table, given args BFD, SECTABLE_PTR, SECEND_PTR.
    Returns 0 if OK, 1 on error.  */
@@ -1183,9 +1332,14 @@ extern int memory_remove_breakpoint PARAMS ((CORE_ADDR, char *));
 
 extern int memory_insert_breakpoint PARAMS ((CORE_ADDR, char *));
 
+extern int default_memory_remove_breakpoint PARAMS ((CORE_ADDR, char *));
+
+extern int default_memory_insert_breakpoint PARAMS ((CORE_ADDR, char *));
+
 extern breakpoint_from_pc_fn memory_breakpoint_from_pc;
 #ifndef BREAKPOINT_FROM_PC
-#define BREAKPOINT_FROM_PC(pcptr, lenptr) memory_breakpoint_from_pc (pcptr, lenptr)
+#define BREAKPOINT_FROM_PC(pcptr, lenptr) \
+     memory_breakpoint_from_pc (pcptr, lenptr)
 #endif
 
 
@@ -1200,20 +1354,32 @@ noprocess PARAMS ((void));
 extern void
 find_default_attach PARAMS ((char *, int));
 
-void
+extern void
 find_default_require_attach PARAMS ((char *, int));
 
-void
+extern void
 find_default_require_detach PARAMS ((int, char *, int));
 
 extern void
 find_default_create_inferior PARAMS ((char *, char *, char **));
 
-void
+extern void
 find_default_clone_and_follow_inferior PARAMS ((int, int *));
 
 extern struct target_ops *
+find_run_target PARAMS ((void));
+
+extern struct target_ops *
 find_core_target PARAMS ((void));
+
+extern struct target_ops *
+find_target_beneath PARAMS ((struct target_ops *));
+
+extern int
+target_resize_to_sections PARAMS ((struct target_ops *target, int num_added));
+
+extern void remove_target_sections (bfd *abfd);
+
 \f
 /* Stuff that should be shared among the various remote targets.  */
 
@@ -1233,7 +1399,25 @@ extern asection *target_memory_bfd_section;
 /* This is for native targets which use a unix/POSIX-style waitstatus.  */
 extern void store_waitstatus PARAMS ((struct target_waitstatus *, int));
 
-/* Convert between host signal numbers and enum target_signal's.  */
+/* Predicate to target_signal_to_host(). Return non-zero if the enum
+   targ_signal SIGNO has an equivalent ``host'' representation.  */
+/* FIXME: cagney/1999-11-22: The name below was chosen in preference
+   to the shorter target_signal_p() because it is far less ambigious.
+   In this context ``target_signal'' refers to GDB's internal
+   representation of the target's set of signals while ``host signal''
+   refers to the target operating system's signal.  Confused?  */
+
+extern int target_signal_to_host_p (enum target_signal signo);
+
+/* Convert between host signal numbers and enum target_signal's.
+   target_signal_to_host() returns 0 and prints a warning() on GDB's
+   console if SIGNO has no equivalent host representation.  */
+/* FIXME: cagney/1999-11-22: Here ``host'' is used incorrectly, it is
+   refering to the target operating system's signal numbering.
+   Similarly, ``enum target_signal'' is named incorrectly, ``enum
+   gdb_signal'' would probably be better as it is refering to GDB's
+   internal representation of a target operating system's signal.  */
+
 extern enum target_signal target_signal_from_host PARAMS ((int));
 extern int target_signal_to_host PARAMS ((enum target_signal));
 
@@ -1247,7 +1431,8 @@ extern void push_remote_target PARAMS ((char *name, int from_tty));
 
 #ifndef SOFTWARE_SINGLE_STEP_P
 #define SOFTWARE_SINGLE_STEP_P 0
-#define SOFTWARE_SINGLE_STEP(sig,bp_p) abort ()
+#define SOFTWARE_SINGLE_STEP(sig,bp_p) \
+     (internal_error ("SOFTWARE_SINGLE_STEP"), 0)
 #endif /* SOFTWARE_SINGLE_STEP_P */
 
 /* Blank target vector entries are initialized to target_ignore. */
@@ -1261,4 +1446,4 @@ void target_ignore PARAMS ((void));
    do { *(REGP) = FP_REGNUM; *(OFFP) =  0; } while (0)
 #endif /* TARGET_VIRTUAL_FRAME_POINTER */
 
-#endif /* !defined (TARGET_H) */
+#endif /* !defined (TARGET_H) */