]> git.ipfire.org Git - thirdparty/binutils-gdb.git/blobdiff - gdb/procfs.c
* layout.cc (Layout::add_target_dynamic_tags): If
[thirdparty/binutils-gdb.git] / gdb / procfs.c
index 19316b9a5acb92cc62640e97264f3846c2bb5f1b..4409e5bfb0c0095241acdcbb78726deda3f84b97 100644 (file)
@@ -1,7 +1,6 @@
 /* Machine independent support for SVR4 /proc (process file system) for GDB.
 
-   Copyright (C) 1999, 2000, 2001, 2002, 2003, 2006, 2007, 2008, 2009, 2010
-   Free Software Foundation, Inc.
+   Copyright (C) 1999-2003, 2006-2012 Free Software Foundation, Inc.
 
    Written by Michael Snyder at Cygnus Solutions.
    Based on work by Fred Fish, Stu Grossman, Geoff Noer, and others.
@@ -32,7 +31,7 @@
 #include "inf-child.h"
 
 #if defined (NEW_PROC_API)
-#define _STRUCTURED_PROC 1     /* Should be done by configure script. */
+#define _STRUCTURED_PROC 1     /* Should be done by configure script.  */
 #endif
 
 #include <sys/procfs.h>
 #include "procfs.h"
 #include "observer.h"
 
-/*
- * PROCFS.C
- *
- * This module provides the interface between GDB and the
- * /proc file system, which is used on many versions of Unix
- * as a means for debuggers to control other processes.
- * Examples of the systems that use this interface are:
- *   Irix
- *   Solaris
- *   OSF
- *   Unixware
- *   AIX5
- *
- * /proc works by imitating a file system: you open a simulated file
- * that represents the process you wish to interact with, and
- * perform operations on that "file" in order to examine or change
- * the state of the other process.
- *
- * The most important thing to know about /proc and this module
- * is that there are two very different interfaces to /proc:
- *   One that uses the ioctl system call, and
- *   another that uses read and write system calls.
- * This module has to support both /proc interfaces.  This means
- * that there are two different ways of doing every basic operation.
- *
- * In order to keep most of the code simple and clean, I have
- * defined an interface "layer" which hides all these system calls.
- * An ifdef (NEW_PROC_API) determines which interface we are using,
- * and most or all occurrances of this ifdef should be confined to
- * this interface layer.
- */
-
-
-/* Determine which /proc API we are using:
-   The ioctl API defines PIOCSTATUS, while
-   the read/write (multiple fd) API never does.  */
+/* This module provides the interface between GDB and the
+   /proc file system, which is used on many versions of Unix
+   as a means for debuggers to control other processes.
+
+   Examples of the systems that use this interface are:
+
+     Irix
+     Solaris
+     OSF
+     Unixware
+     AIX5
+
+   /proc works by imitating a file system: you open a simulated file
+   that represents the process you wish to interact with, and perform
+   operations on that "file" in order to examine or change the state
+   of the other process.
+
+   The most important thing to know about /proc and this module is
+   that there are two very different interfaces to /proc:
+
+     One that uses the ioctl system call, and another that uses read
+     and write system calls.
+
+   This module has to support both /proc interfaces.  This means that
+   there are two different ways of doing every basic operation.
+
+   In order to keep most of the code simple and clean, I have defined
+   an interface "layer" which hides all these system calls.  An ifdef
+   (NEW_PROC_API) determines which interface we are using, and most or
+   all occurrances of this ifdef should be confined to this interface
+   layer.  */
+
+/* Determine which /proc API we are using: The ioctl API defines
+   PIOCSTATUS, while the read/write (multiple fd) API never does.  */
 
 #ifdef NEW_PROC_API
 #include <sys/types.h>
 
 #include "proc-utils.h"
 
-/* Prototypes for supply_gregset etc. */
+/* Prototypes for supply_gregset etc.  */
 #include "gregset.h"
 
 /* =================== TARGET_OPS "MODULE" =================== */
 
-/*
- * This module defines the GDB target vector and its methods.
- */
+/* This module defines the GDB target vector and its methods.  */
 
 static void procfs_attach (struct target_ops *, char *, int);
 static void procfs_detach (struct target_ops *, char *, int);
 static void procfs_resume (struct target_ops *,
-                          ptid_t, int, enum target_signal);
+                          ptid_t, int, enum gdb_signal);
 static void procfs_stop (ptid_t);
 static void procfs_files_info (struct target_ops *);
 static void procfs_fetch_registers (struct target_ops *,
                                    struct regcache *, int);
 static void procfs_store_registers (struct target_ops *,
                                    struct regcache *, int);
-static void procfs_notice_signals (ptid_t);
+static void procfs_pass_signals (int, unsigned char *);
 static void procfs_kill_inferior (struct target_ops *ops);
 static void procfs_mourn_inferior (struct target_ops *ops);
-static void procfs_create_inferior (struct target_ops *, char *, 
+static void procfs_create_inferior (struct target_ops *, char *,
                                    char *, char **, int);
 static ptid_t procfs_wait (struct target_ops *,
                           ptid_t, struct target_waitstatus *, int);
@@ -137,13 +132,14 @@ static int procfs_xfer_memory (CORE_ADDR, gdb_byte *, int, int,
 static LONGEST procfs_xfer_partial (struct target_ops *ops,
                                    enum target_object object,
                                    const char *annex,
-                                   gdb_byte *readbuf, const gdb_byte *writebuf,
+                                   gdb_byte *readbuf,
+                                   const gdb_byte *writebuf,
                                    ULONGEST offset, LONGEST len);
 
 static int procfs_thread_alive (struct target_ops *ops, ptid_t);
 
-void procfs_find_new_threads (struct target_ops *ops);
-char *procfs_pid_to_str (struct target_ops *, ptid_t);
+static void procfs_find_new_threads (struct target_ops *ops);
+static char *procfs_pid_to_str (struct target_ops *, ptid_t);
 
 static int proc_find_memory_regions (int (*) (CORE_ADDR,
                                              unsigned long,
@@ -155,20 +151,23 @@ static char * procfs_make_note_section (bfd *, int *);
 
 static int procfs_can_use_hw_breakpoint (int, int, int);
 
+static void procfs_info_proc (struct target_ops *, char *,
+                             enum info_proc_what);
+
 #if defined (PR_MODEL_NATIVE) && (PR_MODEL_NATIVE == PR_MODEL_LP64)
-/* When GDB is built as 64-bit application on Solaris, the auxv data is
-   presented in 64-bit format.  We need to provide a custom parser to handle 
-   that.  */
+/* When GDB is built as 64-bit application on Solaris, the auxv data
+   is presented in 64-bit format.  We need to provide a custom parser
+   to handle that.  */
 static int
 procfs_auxv_parse (struct target_ops *ops, gdb_byte **readptr,
-                  gdb_byte *endptr, CORE_ADDR *typep, CORE_ADDR *valp)
+                  gdb_byte *endptr, CORE_ADDR *typep, CORE_ADDR *valp)
 {
   enum bfd_endian byte_order = gdbarch_byte_order (target_gdbarch);
   gdb_byte *ptr = *readptr;
 
   if (endptr == ptr)
     return 0;
-  
+
   if (endptr - ptr < 8 * 2)
     return -1;
 
@@ -189,50 +188,49 @@ procfs_target (void)
 {
   struct target_ops *t = inf_child_target ();
 
-  t->to_shortname           = "procfs";
-  t->to_longname            = "Unix /proc child process";
-  t->to_doc                 =
+  t->to_shortname = "procfs";
+  t->to_longname = "Unix /proc child process";
+  t->to_doc =
     "Unix /proc child process (started by the \"run\" command).";
-  t->to_create_inferior     = procfs_create_inferior;
-  t->to_kill                = procfs_kill_inferior;
-  t->to_mourn_inferior      = procfs_mourn_inferior;
-  t->to_attach              = procfs_attach;
-  t->to_detach              = procfs_detach;
-  t->to_wait                = procfs_wait;
-  t->to_resume              = procfs_resume;
-  t->to_fetch_registers     = procfs_fetch_registers;
-  t->to_store_registers     = procfs_store_registers;
-  t->to_xfer_partial        = procfs_xfer_partial;
+  t->to_create_inferior = procfs_create_inferior;
+  t->to_kill = procfs_kill_inferior;
+  t->to_mourn_inferior = procfs_mourn_inferior;
+  t->to_attach = procfs_attach;
+  t->to_detach = procfs_detach;
+  t->to_wait = procfs_wait;
+  t->to_resume = procfs_resume;
+  t->to_fetch_registers = procfs_fetch_registers;
+  t->to_store_registers = procfs_store_registers;
+  t->to_xfer_partial = procfs_xfer_partial;
   t->deprecated_xfer_memory = procfs_xfer_memory;
-  t->to_notice_signals      = procfs_notice_signals;
-  t->to_files_info          = procfs_files_info;
-  t->to_stop                = procfs_stop;
+  t->to_pass_signals = procfs_pass_signals;
+  t->to_files_info = procfs_files_info;
+  t->to_stop = procfs_stop;
 
-  t->to_find_new_threads    = procfs_find_new_threads;
-  t->to_thread_alive        = procfs_thread_alive;
-  t->to_pid_to_str          = procfs_pid_to_str;
+  t->to_find_new_threads = procfs_find_new_threads;
+  t->to_thread_alive = procfs_thread_alive;
+  t->to_pid_to_str = procfs_pid_to_str;
 
-  t->to_has_thread_control  = tc_schedlock;
+  t->to_has_thread_control = tc_schedlock;
   t->to_find_memory_regions = proc_find_memory_regions;
   t->to_make_corefile_notes = procfs_make_note_section;
+  t->to_info_proc = procfs_info_proc;
 
 #if defined(PR_MODEL_NATIVE) && (PR_MODEL_NATIVE == PR_MODEL_LP64)
   t->to_auxv_parse = procfs_auxv_parse;
 #endif
 
-  t->to_magic               = OPS_MAGIC;
+  t->to_magic = OPS_MAGIC;
 
   return t;
 }
 
 /* =================== END, TARGET_OPS "MODULE" =================== */
 
-/*
- * World Unification:
- *
- * Put any typedefs, defines etc. here that are required for
- * the unification of code that handles different versions of /proc.
- */
+/* World Unification:
+
+   Put any typedefs, defines etc. here that are required for the
+   unification of code that handles different versions of /proc.  */
 
 #ifdef NEW_PROC_API            /* Solaris 7 && 8 method for watchpoints */
 #ifdef WA_READ
@@ -268,7 +266,7 @@ typedef struct sigaction gdb_sigaction_t;
 #ifdef HAVE_PR_SIGINFO64_T
 typedef pr_siginfo64_t gdb_siginfo_t;
 #else
-typedef struct siginfo gdb_siginfo_t;
+typedef siginfo_t gdb_siginfo_t;
 #endif
 
 /* On mips-irix, praddset and prdelset are defined in such a way that
@@ -317,8 +315,7 @@ typedef struct siginfo gdb_siginfo_t;
    syscall numbers.  Instead, they're stored in /proc/PID/sysent
    for each process.  We are at least guaranteed that they won't
    change over the lifetime of the process.  But each process could
-   (in theory) have different syscall numbers.
-*/
+   (in theory) have different syscall numbers.  */
 #ifdef HAVE_PRSYSENT_T
 #define DYNAMIC_SYSCALLS
 #endif
@@ -344,7 +341,7 @@ typedef struct siginfo gdb_siginfo_t;
    There will be a separate procinfo structure for use by the (not yet
    implemented) "info proc" command, so that we can print useful
    information about any random process without interfering with the
-   inferior's procinfo information. */
+   inferior's procinfo information.  */
 
 #ifdef NEW_PROC_API
 /* format strings for /proc paths */
@@ -369,7 +366,7 @@ typedef lwpstatus_t gdb_lwpstatus_t;
 #  define STATUS_PROC_NAME_FMT "/proc/%05d"
 #  define MAX_PROC_NAME_SIZE sizeof("/proc/ttttppppp")
 # endif
-/* the name of the proc status struct depends on the implementation */
+/* The name of the proc status struct depends on the implementation.  */
 typedef prstatus_t gdb_prstatus_t;
 typedef prstatus_t gdb_lwpstatus_t;
 #endif /* NEW_PROC_API */
@@ -388,13 +385,13 @@ typedef struct procinfo {
      (old ioctl or new read/write).  */
 
   int ctl_fd;                  /* File descriptor for /proc control file */
-  /*
-   * The next three file descriptors are actually only needed in the
-   * read/write, multiple-file-descriptor implemenation (NEW_PROC_API).
-   * However, to avoid a bunch of #ifdefs in the code, we will use
-   * them uniformly by (in the case of the ioctl single-file-descriptor
-   * implementation) filling them with copies of the control fd.
-   */
+
+  /* The next three file descriptors are actually only needed in the
+     read/write, multiple-file-descriptor implemenation
+     (NEW_PROC_API).  However, to avoid a bunch of #ifdefs in the
+     code, we will use them uniformly by (in the case of the ioctl
+     single-file-descriptor implementation) filling them with copies
+     of the control fd.  */
   int status_fd;               /* File descriptor for /proc status file */
   int as_fd;                   /* File descriptor for /proc as file */
 
@@ -445,29 +442,16 @@ static void free_syscalls (procinfo *pi);
 static int find_syscall (procinfo *pi, char *name);
 #endif /* DYNAMIC_SYSCALLS */
 
-/* A function type used as a callback back iterate_over_mappings.  */
-typedef int (iterate_over_mappings_cb_ftype)
-  (CORE_ADDR vaddr, unsigned long size, int read, int write, int execute,
-   void *data);
-
 static int iterate_over_mappings
-  (procinfo *pi,
-   iterate_over_mappings_cb_ftype *child_func,
-   void *data,
-   int (*func) (struct prmap *map,
-                iterate_over_mappings_cb_ftype *child_func,
-                void *data));
+  (procinfo *pi, find_memory_region_ftype child_func, void *data,
+   int (*func) (struct prmap *map, find_memory_region_ftype child_func,
+               void *data));
 
 /* The head of the procinfo list: */
 static procinfo * procinfo_list;
 
-/*
- * Function: find_procinfo
- *
- * Search the procinfo list.
- *
- * Returns: pointer to procinfo, or NULL if not found.
- */
+/* Search the procinfo list.  Return a pointer to procinfo, or NULL if
+   not found.  */
 
 static procinfo *
 find_procinfo (int pid, int tid)
@@ -486,7 +470,7 @@ find_procinfo (int pid, int tid)
           here.  This means that in general it is the caller's
           responsibility to check threads_valid and update before
           calling find_procinfo, if the caller wants to find a new
-          thread. */
+          thread.  */
 
        for (pi = pi->thread_list; pi; pi = pi->next)
          if (pi->tid == tid)
@@ -496,11 +480,7 @@ find_procinfo (int pid, int tid)
   return pi;
 }
 
-/*
- * Function: find_procinfo_or_die
- *
- * Calls find_procinfo, but errors on failure.
- */
+/* Calls find_procinfo, but errors on failure.  */
 
 static procinfo *
 find_procinfo_or_die (int pid, int tid)
@@ -510,7 +490,8 @@ find_procinfo_or_die (int pid, int tid)
   if (pi == NULL)
     {
       if (tid)
-       error (_("procfs: couldn't find pid %d (kernel thread %d) in procinfo list."),
+       error (_("procfs: couldn't find pid %d "
+                "(kernel thread %d) in procinfo list."),
               pid, tid);
       else
        error (_("procfs: couldn't find pid %d in procinfo list."), pid);
@@ -518,16 +499,15 @@ find_procinfo_or_die (int pid, int tid)
   return pi;
 }
 
-/* open_with_retry() is a wrapper for open().  The appropriate
-   open() call is attempted; if unsuccessful, it will be retried as
-   many times as needed for the EAGAIN and EINTR conditions.
+/* Wrapper for `open'.  The appropriate open call is attempted; if
+   unsuccessful, it will be retried as many times as needed for the
+   EAGAIN and EINTR conditions.
 
-   For other conditions, open_with_retry() will retry the open() a
-   limited number of times.  In addition, a short sleep is imposed
-   prior to retrying the open().  The reason for this sleep is to give
-   the kernel a chance to catch up and create the file in question in
-   the event that GDB "wins" the race to open a file before the kernel
-   has created it.  */
+   For other conditions, retry the open a limited number of times.  In
+   addition, a short sleep is imposed prior to retrying the open.  The
+   reason for this sleep is to give the kernel a chance to catch up
+   and create the file in question in the event that GDB "wins" the
+   race to open a file before the kernel has created it.  */
 
 static int
 open_with_retry (const char *pathname, int flags)
@@ -552,18 +532,12 @@ open_with_retry (const char *pathname, int flags)
   return status;
 }
 
-/*
- * Function: open_procinfo_files
- *
- * Open the file descriptor for the process or LWP.
- * ifdef NEW_PROC_API, we only open the control file descriptor;
- * the others are opened lazily as needed.
- * else (if not NEW_PROC_API), there is only one real
- * file descriptor, but we keep multiple copies of it so that
- * the code that uses them does not have to be #ifdef'd.
- *
- * Return: file descriptor, or zero for failure.
- */
+/* Open the file descriptor for the process or LWP.  If NEW_PROC_API
+   is defined, we only open the control file descriptor; the others
+   are opened lazily as needed.  Otherwise (if not NEW_PROC_API),
+   there is only one real file descriptor, but we keep multiple copies
+   of it so that the code that uses them does not have to be #ifdef'd.
+   Returns the file descriptor, or zero for failure.  */
 
 enum { FD_CTL, FD_STATUS, FD_AS };
 
@@ -575,79 +549,75 @@ open_procinfo_files (procinfo *pi, int which)
 #endif
   int  fd;
 
-  /*
-   * This function is getting ALMOST long enough to break up into several.
-   * Here is some rationale:
-   *
-   * NEW_PROC_API (Solaris 2.6, Solaris 2.7, Unixware):
-   *   There are several file descriptors that may need to be open
-   *   for any given process or LWP.  The ones we're intereted in are:
-   *     - control      (ctl)    write-only    change the state
-   *     - status       (status) read-only     query the state
-   *     - address space (as)     read/write   access memory
-   *     - map           (map)    read-only     virtual addr map
-   *   Most of these are opened lazily as they are needed.
-   *   The pathnames for the 'files' for an LWP look slightly
-   *   different from those of a first-class process:
-   *     Pathnames for a process (<proc-id>):
-   *       /proc/<proc-id>/ctl
-   *       /proc/<proc-id>/status
-   *       /proc/<proc-id>/as
-   *       /proc/<proc-id>/map
-   *     Pathnames for an LWP (lwp-id):
-   *       /proc/<proc-id>/lwp/<lwp-id>/lwpctl
-   *       /proc/<proc-id>/lwp/<lwp-id>/lwpstatus
-   *   An LWP has no map or address space file descriptor, since
-   *   the memory map and address space are shared by all LWPs.
-   *
-   * Everyone else (Solaris 2.5, Irix, OSF)
-   *   There is only one file descriptor for each process or LWP.
-   *   For convenience, we copy the same file descriptor into all
-   *   three fields of the procinfo struct (ctl_fd, status_fd, and
-   *   as_fd, see NEW_PROC_API above) so that code that uses them
-   *   doesn't need any #ifdef's.
-   *     Pathname for all:
-   *       /proc/<proc-id>
-   *
-   *   Solaris 2.5 LWP's:
-   *     Each LWP has an independent file descriptor, but these
-   *     are not obtained via the 'open' system call like the rest:
-   *     instead, they're obtained thru an ioctl call (PIOCOPENLWP)
-   *     to the file descriptor of the parent process.
-   *
-   *   OSF threads:
-   *     These do not even have their own independent file descriptor.
-   *     All operations are carried out on the file descriptor of the
-   *     parent process.  Therefore we just call open again for each
-   *     thread, getting a new handle for the same 'file'.
-   */
+  /* This function is getting ALMOST long enough to break up into
+     several.  Here is some rationale:
+
+     NEW_PROC_API (Solaris 2.6, Solaris 2.7, Unixware):
+     There are several file descriptors that may need to be open
+       for any given process or LWP.  The ones we're intereted in are:
+        - control       (ctl)    write-only    change the state
+        - status        (status) read-only     query the state
+        - address space (as)     read/write    access memory
+        - map           (map)    read-only     virtual addr map
+       Most of these are opened lazily as they are needed.
+       The pathnames for the 'files' for an LWP look slightly
+       different from those of a first-class process:
+        Pathnames for a process (<proc-id>):
+          /proc/<proc-id>/ctl
+          /proc/<proc-id>/status
+          /proc/<proc-id>/as
+          /proc/<proc-id>/map
+        Pathnames for an LWP (lwp-id):
+          /proc/<proc-id>/lwp/<lwp-id>/lwpctl
+          /proc/<proc-id>/lwp/<lwp-id>/lwpstatus
+       An LWP has no map or address space file descriptor, since
+       the memory map and address space are shared by all LWPs.
+
+     Everyone else (Solaris 2.5, Irix, OSF)
+       There is only one file descriptor for each process or LWP.
+       For convenience, we copy the same file descriptor into all
+       three fields of the procinfo struct (ctl_fd, status_fd, and
+       as_fd, see NEW_PROC_API above) so that code that uses them
+       doesn't need any #ifdef's.
+        Pathname for all:
+          /proc/<proc-id>
+
+       Solaris 2.5 LWP's:
+        Each LWP has an independent file descriptor, but these
+        are not obtained via the 'open' system call like the rest:
+        instead, they're obtained thru an ioctl call (PIOCOPENLWP)
+        to the file descriptor of the parent process.
+
+       OSF threads:
+        These do not even have their own independent file descriptor.
+        All operations are carried out on the file descriptor of the
+        parent process.  Therefore we just call open again for each
+        thread, getting a new handle for the same 'file'.  */
 
 #ifdef NEW_PROC_API
-  /*
-   * In this case, there are several different file descriptors that
-   * we might be asked to open.  The control file descriptor will be
-   * opened early, but the others will be opened lazily as they are
-   * needed.
-   */
+  /* In this case, there are several different file descriptors that
+     we might be asked to open.  The control file descriptor will be
+     opened early, but the others will be opened lazily as they are
+     needed.  */
 
   strcpy (tmp, pi->pathname);
-  switch (which) {     /* which file descriptor to open? */
+  switch (which) {     /* Which file descriptor to open?  */
   case FD_CTL:
     if (pi->tid)
       strcat (tmp, "/lwpctl");
     else
       strcat (tmp, "/ctl");
     fd = open_with_retry (tmp, O_WRONLY);
-    if (fd <= 0)
+    if (fd < 0)
       return 0;                /* fail */
     pi->ctl_fd = fd;
     break;
   case FD_AS:
     if (pi->tid)
-      return 0;                /* there is no 'as' file descriptor for an lwp */
+      return 0;                /* There is no 'as' file descriptor for an lwp.  */
     strcat (tmp, "/as");
     fd = open_with_retry (tmp, O_RDWR);
-    if (fd <= 0)
+    if (fd < 0)
       return 0;                /* fail */
     pi->as_fd = fd;
     break;
@@ -657,7 +627,7 @@ open_procinfo_files (procinfo *pi, int which)
     else
       strcat (tmp, "/status");
     fd = open_with_retry (tmp, O_RDONLY);
-    if (fd <= 0)
+    if (fd < 0)
       return 0;                /* fail */
     pi->status_fd = fd;
     break;
@@ -665,29 +635,27 @@ open_procinfo_files (procinfo *pi, int which)
     return 0;          /* unknown file descriptor */
   }
 #else  /* not NEW_PROC_API */
-  /*
-   * In this case, there is only one file descriptor for each procinfo
-   * (ie. each process or LWP).  In fact, only the file descriptor for
-   * the process can actually be opened by an 'open' system call.
-   * The ones for the LWPs have to be obtained thru an IOCTL call
-   * on the process's file descriptor.
-   *
-   * For convenience, we copy each procinfo's single file descriptor
-   * into all of the fields occupied by the several file descriptors
-   * of the NEW_PROC_API implementation.  That way, the code that uses
-   * them can be written without ifdefs.
-   */
+  /* In this case, there is only one file descriptor for each procinfo
+     (ie. each process or LWP).  In fact, only the file descriptor for
+     the process can actually be opened by an 'open' system call.  The
+     ones for the LWPs have to be obtained thru an IOCTL call on the
+     process's file descriptor.
+
+     For convenience, we copy each procinfo's single file descriptor
+     into all of the fields occupied by the several file descriptors
+     of the NEW_PROC_API implementation.  That way, the code that uses
+     them can be written without ifdefs.  */
 
 
 #ifdef PIOCTSTATUS     /* OSF */
-  /* Only one FD; just open it. */
-  if ((fd = open_with_retry (pi->pathname, O_RDWR)) == 0)
+  /* Only one FD; just open it.  */
+  if ((fd = open_with_retry (pi->pathname, O_RDWR)) < 0)
     return 0;
-#else                  /* Sol 2.5, Irix, other? */
+#else                  /* Sol 2.5, Irix, other?  */
   if (pi->tid == 0)    /* Master procinfo for the process */
     {
       fd = open_with_retry (pi->pathname, O_RDWR);
-      if (fd <= 0)
+      if (fd < 0)
        return 0;       /* fail */
     }
   else                 /* LWP thread procinfo */
@@ -696,15 +664,15 @@ open_procinfo_files (procinfo *pi, int which)
       procinfo *process;
       int lwpid = pi->tid;
 
-      /* Find the procinfo for the entire process. */
+      /* Find the procinfo for the entire process.  */
       if ((process = find_procinfo (pi->pid, 0)) == NULL)
        return 0;       /* fail */
 
-      /* Now obtain the file descriptor for the LWP. */
-      if ((fd = ioctl (process->ctl_fd, PIOCOPENLWP, &lwpid)) <= 0)
+      /* Now obtain the file descriptor for the LWP.  */
+      if ((fd = ioctl (process->ctl_fd, PIOCOPENLWP, &lwpid)) < 0)
        return 0;       /* fail */
-#else                  /* Irix, other? */
-      return 0;                /* Don't know how to open threads */
+#else                  /* Irix, other?  */
+      return 0;                /* Don't know how to open threads */
 #endif /* Sol 2.5 PIOCOPENLWP */
     }
 #endif /* OSF     PIOCTSTATUS */
@@ -714,14 +682,9 @@ open_procinfo_files (procinfo *pi, int which)
   return 1;            /* success */
 }
 
-/*
- * Function: create_procinfo
- *
- * Allocate a data structure and link it into the procinfo list.
- * (First tries to find a pre-existing one (FIXME: why?)
- *
- * Return: pointer to new procinfo struct.
- */
+/* Allocate a data structure and link it into the procinfo list.
+   First tries to find a pre-existing one (FIXME: why?).  Returns the
+   pointer to new procinfo struct.  */
 
 static procinfo *
 create_procinfo (int pid, int tid)
@@ -729,13 +692,13 @@ create_procinfo (int pid, int tid)
   procinfo *pi, *parent = NULL;
 
   if ((pi = find_procinfo (pid, tid)))
-    return pi;                 /* Already exists, nothing to do. */
+    return pi;                 /* Already exists, nothing to do.  */
 
-  /* find parent before doing malloc, to save having to cleanup */
+  /* Find parent before doing malloc, to save having to cleanup.  */
   if (tid != 0)
     parent = find_procinfo_or_die (pid, 0);    /* FIXME: should I
                                                   create it if it
-                                                  doesn't exist yet? */
+                                                  doesn't exist yet?  */
 
   pi = (procinfo *) xmalloc (sizeof (procinfo));
   memset (pi, 0, sizeof (procinfo));
@@ -769,11 +732,7 @@ create_procinfo (int pid, int tid)
   return pi;
 }
 
-/*
- * Function: close_procinfo_files
- *
- * Close all file descriptors associated with the procinfo
- */
+/* Close all file descriptors associated with the procinfo.  */
 
 static void
 close_procinfo_files (procinfo *pi)
@@ -789,18 +748,14 @@ close_procinfo_files (procinfo *pi)
   pi->ctl_fd = pi->as_fd = pi->status_fd = 0;
 }
 
-/*
- * Function: destroy_procinfo
- *
- * Destructor function.  Close, unlink and deallocate the object.
- */
+/* Destructor function.  Close, unlink and deallocate the object.  */
 
 static void
 destroy_one_procinfo (procinfo **list, procinfo *pi)
 {
   procinfo *ptr;
 
-  /* Step one: unlink the procinfo from its list */
+  /* Step one: unlink the procinfo from its list */
   if (pi == *list)
     *list = pi->next;
   else
@@ -811,10 +766,10 @@ destroy_one_procinfo (procinfo **list, procinfo *pi)
          break;
        }
 
-  /* Step two: close any open file descriptors */
+  /* Step two: close any open file descriptors */
   close_procinfo_files (pi);
 
-  /* Step three: free the memory. */
+  /* Step three: free the memory.  */
 #ifdef DYNAMIC_SYSCALLS
   free_syscalls (pi);
 #endif
@@ -828,12 +783,12 @@ destroy_procinfo (procinfo *pi)
 {
   procinfo *tmp;
 
-  if (pi->tid != 0)    /* destroy a thread procinfo */
+  if (pi->tid != 0)    /* Destroy a thread procinfo.  */
     {
-      tmp = find_procinfo (pi->pid, 0);        /* find the parent process */
+      tmp = find_procinfo (pi->pid, 0);        /* Find the parent process.  */
       destroy_one_procinfo (&tmp->thread_list, pi);
     }
-  else                 /* destroy a process procinfo and all its threads */
+  else                 /* Destroy a process procinfo and all its threads.  */
     {
       /* First destroy the children, if any; */
       while (pi->thread_list != NULL)
@@ -851,13 +806,9 @@ do_destroy_procinfo_cleanup (void *pi)
 
 enum { NOKILL, KILL };
 
-/*
- * Function: dead_procinfo
- *
- * To be called on a non_recoverable error for a procinfo.
- * Prints error messages, optionally sends a SIGKILL to the process,
- * then destroys the data structure.
- */
+/* To be called on a non_recoverable error for a procinfo.  Prints
+   error messages, optionally sends a SIGKILL to the process, then
+   destroys the data structure.  */
 
 static void
 dead_procinfo (procinfo *pi, char *msg, int kill_p)
@@ -880,13 +831,9 @@ dead_procinfo (procinfo *pi, char *msg, int kill_p)
   error ("%s", msg);
 }
 
-/*
- * Function: sysset_t_size
- *
- * Returns the (complete) size of a sysset_t struct.  Normally, this
- * is just sizeof (syset_t), but in the case of Monterey/64, the actual
- * size of sysset_t isn't known until runtime.
- */
+/* Returns the (complete) size of a sysset_t struct.  Normally, this
+   is just sizeof (sysset_t), but in the case of Monterey/64, the
+   actual size of sysset_t isn't known until runtime.  */
 
 static int
 sysset_t_size (procinfo * pi)
@@ -900,31 +847,28 @@ sysset_t_size (procinfo * pi)
 #endif
 }
 
-/* Function: sysset_t_alloc
-
-   Allocate and (partially) initialize a sysset_t struct.  */
+/* Allocate and (partially) initialize a sysset_t struct.  */
 
 static sysset_t *
 sysset_t_alloc (procinfo * pi)
 {
   sysset_t *ret;
   int size = sysset_t_size (pi);
+
   ret = xmalloc (size);
 #ifdef DYNAMIC_SYSCALLS
-  ret->pr_size = (pi->num_syscalls + (8 * sizeof (uint64_t) - 1))
-                 / (8 * sizeof (uint64_t));
+  ret->pr_size = ((pi->num_syscalls + (8 * sizeof (uint64_t) - 1))
+                 / (8 * sizeof (uint64_t)));
 #endif
   return ret;
 }
 
 #ifdef DYNAMIC_SYSCALLS
 
-/* Function: load_syscalls
-
-   Extract syscall numbers and names from /proc/<pid>/sysent.  Initialize
+/* Extract syscall numbers and names from /proc/<pid>/sysent.  Initialize
    pi->num_syscalls with the number of syscalls and pi->syscall_names
    with the names.  (Certain numbers may be skipped in which case the
-   names for these numbers will be left as NULL.) */
+   names for these numbers will be left as NULL.)  */
 
 #define MAX_SYSCALL_NAME_LENGTH 256
 #define MAX_SYSCALLS 65536
@@ -937,17 +881,19 @@ load_syscalls (procinfo *pi)
   prsysent_t header;
   prsyscall_t *syscalls;
   int i, size, maxcall;
+  struct cleanup *cleanups;
 
   pi->num_syscalls = 0;
   pi->syscall_names = 0;
 
-  /* Open the file descriptor for the sysent file */
+  /* Open the file descriptor for the sysent file */
   sprintf (pathname, "/proc/%d/sysent", pi->pid);
   sysent_fd = open_with_retry (pathname, O_RDONLY);
   if (sysent_fd < 0)
     {
       error (_("load_syscalls: Can't open /proc/%d/sysent"), pi->pid);
     }
+  cleanups = make_cleanup_close (sysent_fd);
 
   size = sizeof header - sizeof (prsyscall_t);
   if (read (sysent_fd, &header, size) != size)
@@ -957,28 +903,27 @@ load_syscalls (procinfo *pi)
 
   if (header.pr_nsyscalls == 0)
     {
-      error (_("load_syscalls: /proc/%d/sysent contains no syscalls!"), pi->pid);
+      error (_("load_syscalls: /proc/%d/sysent contains no syscalls!"),
+            pi->pid);
     }
 
   size = header.pr_nsyscalls * sizeof (prsyscall_t);
   syscalls = xmalloc (size);
+  make_cleanup (free_current_contents, &syscalls);
 
   if (read (sysent_fd, syscalls, size) != size)
-    {
-      xfree (syscalls);
-      error (_("load_syscalls: Error reading /proc/%d/sysent"), pi->pid);
-    }
+    error (_("load_syscalls: Error reading /proc/%d/sysent"), pi->pid);
 
   /* Find maximum syscall number.  This may not be the same as
      pr_nsyscalls since that value refers to the number of entries
      in the table.  (Also, the docs indicate that some system
-     call numbers may be skipped.) */
+     call numbers may be skipped.)  */
 
   maxcall = syscalls[0].pr_number;
 
   for (i = 1; i <  header.pr_nsyscalls; i++)
     if (syscalls[i].pr_number > maxcall
-        && syscalls[i].pr_nameoff > 0
+       && syscalls[i].pr_nameoff > 0
        && syscalls[i].pr_number < MAX_SYSCALLS)
       maxcall = syscalls[i].pr_number;
 
@@ -988,7 +933,7 @@ load_syscalls (procinfo *pi)
   for (i = 0; i < pi->num_syscalls; i++)
     pi->syscall_names[i] = NULL;
 
-  /* Read the syscall names in */
+  /* Read the syscall names in */
   for (i = 0; i < header.pr_nsyscalls; i++)
     {
       char namebuf[MAX_SYSCALL_NAME_LENGTH];
@@ -996,10 +941,10 @@ load_syscalls (procinfo *pi)
       int callnum;
 
       if (syscalls[i].pr_number >= MAX_SYSCALLS
-          || syscalls[i].pr_number < 0
+         || syscalls[i].pr_number < 0
          || syscalls[i].pr_nameoff <= 0
          || (lseek (sysent_fd, (off_t) syscalls[i].pr_nameoff, SEEK_SET)
-                                       != (off_t) syscalls[i].pr_nameoff))
+                                      != (off_t) syscalls[i].pr_nameoff))
        continue;
 
       nread = read (sysent_fd, namebuf, sizeof namebuf);
@@ -1010,7 +955,7 @@ load_syscalls (procinfo *pi)
 
       if (pi->syscall_names[callnum] != NULL)
        {
-         /* FIXME: Generate warning */
+         /* FIXME: Generate warning */
          continue;
        }
 
@@ -1021,13 +966,10 @@ load_syscalls (procinfo *pi)
       pi->syscall_names[callnum][size-1] = '\0';
     }
 
-  close (sysent_fd);
-  xfree (syscalls);
+  do_cleanups (cleanups);
 }
 
-/* Function: free_syscalls
-
-   Free the space allocated for the syscall names from the procinfo
+/* Free the space allocated for the syscall names from the procinfo
    structure.  */
 
 static void
@@ -1046,15 +988,14 @@ free_syscalls (procinfo *pi)
     }
 }
 
-/* Function: find_syscall
-
-   Given a name, look up (and return) the corresponding syscall number.
+/* Given a name, look up (and return) the corresponding syscall number.
    If no match is found, return -1.  */
 
 static int
 find_syscall (procinfo *pi, char *name)
 {
   int i;
+
   for (i = 0; i < pi->num_syscalls; i++)
     {
       if (pi->syscall_names[i] && strcmp (name, pi->syscall_names[i]) == 0)
@@ -1068,96 +1009,50 @@ find_syscall (procinfo *pi, char *name)
 
 /* ===================  /proc  "MODULE" =================== */
 
-/*
- * This "module" is the interface layer between the /proc system API
- * and the gdb target vector functions.  This layer consists of
- * access functions that encapsulate each of the basic operations
- * that we need to use from the /proc API.
- *
- * The main motivation for this layer is to hide the fact that
- * there are two very different implementations of the /proc API.
- * Rather than have a bunch of #ifdefs all thru the gdb target vector
- * functions, we do our best to hide them all in here.
- */
-
-int proc_get_status (procinfo * pi);
-long proc_flags (procinfo * pi);
-int proc_why (procinfo * pi);
-int proc_what (procinfo * pi);
-int proc_set_run_on_last_close (procinfo * pi);
-int proc_unset_run_on_last_close (procinfo * pi);
-int proc_set_inherit_on_fork (procinfo * pi);
-int proc_unset_inherit_on_fork (procinfo * pi);
-int proc_set_async (procinfo * pi);
-int proc_unset_async (procinfo * pi);
-int proc_stop_process (procinfo * pi);
-int proc_trace_signal (procinfo * pi, int signo);
-int proc_ignore_signal (procinfo * pi, int signo);
-int proc_clear_current_fault (procinfo * pi);
-int proc_set_current_signal (procinfo * pi, int signo);
-int proc_clear_current_signal (procinfo * pi);
-int proc_set_gregs (procinfo * pi);
-int proc_set_fpregs (procinfo * pi);
-int proc_wait_for_stop (procinfo * pi);
-int proc_run_process (procinfo * pi, int step, int signo);
-int proc_kill (procinfo * pi, int signo);
-int proc_parent_pid (procinfo * pi);
-int proc_get_nthreads (procinfo * pi);
-int proc_get_current_thread (procinfo * pi);
-int proc_set_held_signals (procinfo * pi, gdb_sigset_t * sighold);
-int proc_set_traced_sysexit (procinfo * pi, sysset_t * sysset);
-int proc_set_traced_sysentry (procinfo * pi, sysset_t * sysset);
-int proc_set_traced_faults (procinfo * pi, fltset_t * fltset);
-int proc_set_traced_signals (procinfo * pi, gdb_sigset_t * sigset);
-
-int proc_update_threads (procinfo * pi);
-int proc_iterate_over_threads (procinfo * pi,
-                              int (*func) (procinfo *, procinfo *, void *),
-                              void *ptr);
-
-gdb_gregset_t *proc_get_gregs (procinfo * pi);
-gdb_fpregset_t *proc_get_fpregs (procinfo * pi);
-sysset_t *proc_get_traced_sysexit (procinfo * pi, sysset_t * save);
-sysset_t *proc_get_traced_sysentry (procinfo * pi, sysset_t * save);
-fltset_t *proc_get_traced_faults (procinfo * pi, fltset_t * save);
-gdb_sigset_t *proc_get_traced_signals (procinfo * pi, gdb_sigset_t * save);
-gdb_sigset_t *proc_get_held_signals (procinfo * pi, gdb_sigset_t * save);
-gdb_sigset_t *proc_get_pending_signals (procinfo * pi, gdb_sigset_t * save);
-gdb_sigaction_t *proc_get_signal_actions (procinfo * pi, gdb_sigaction_t *save);
-
-void proc_warn (procinfo * pi, char *func, int line);
-void proc_error (procinfo * pi, char *func, int line);
+/* This "module" is the interface layer between the /proc system API
+   and the gdb target vector functions.  This layer consists of access
+   functions that encapsulate each of the basic operations that we
+   need to use from the /proc API.
+
+   The main motivation for this layer is to hide the fact that there
+   are two very different implementations of the /proc API.  Rather
+   than have a bunch of #ifdefs all thru the gdb target vector
+   functions, we do our best to hide them all in here.  */
+
+static long proc_flags (procinfo * pi);
+static int proc_why (procinfo * pi);
+static int proc_what (procinfo * pi);
+static int proc_set_current_signal (procinfo * pi, int signo);
+static int proc_get_current_thread (procinfo * pi);
+static int proc_iterate_over_threads
+  (procinfo * pi,
+   int (*func) (procinfo *, procinfo *, void *),
+   void *ptr);
 
-void
+static void
 proc_warn (procinfo *pi, char *func, int line)
 {
   sprintf (errmsg, "procfs: %s line %d, %s", func, line, pi->pathname);
   print_sys_errmsg (errmsg, errno);
 }
 
-void
+static void
 proc_error (procinfo *pi, char *func, int line)
 {
   sprintf (errmsg, "procfs: %s line %d, %s", func, line, pi->pathname);
   perror_with_name (errmsg);
 }
 
-/*
- * Function: proc_get_status
- *
- * Updates the status struct in the procinfo.
- * There is a 'valid' flag, to let other functions know when
- * this function needs to be called (so the status is only
- * read when it is needed).  The status file descriptor is
- * also only opened when it is needed.
- *
- * Return: non-zero for success, zero for failure.
- */
+/* Updates the status struct in the procinfo.  There is a 'valid'
+   flag, to let other functions know when this function needs to be
+   called (so the status is only read when it is needed).  The status
+   file descriptor is also only opened when it is needed.  Returns
+   non-zero for success, zero for failure.  */
 
-int
+static int
 proc_get_status (procinfo *pi)
 {
-  /* Status file descriptor is opened "lazily" */
+  /* Status file descriptor is opened "lazily" */
   if (pi->status_fd == 0 &&
       open_procinfo_files (pi, FD_STATUS) == 0)
     {
@@ -1171,7 +1066,7 @@ proc_get_status (procinfo *pi)
   else
     {
       /* Sigh... I have to read a different data structure,
-        depending on whether this is a main process or an LWP. */
+        depending on whether this is a main process or an LWP.  */
       if (pi->tid)
        pi->status_valid = (read (pi->status_fd,
                                  (char *) &pi->prstatus.pr_lwp,
@@ -1187,7 +1082,7 @@ proc_get_status (procinfo *pi)
          if (pi->status_valid &&
              (pi->prstatus.pr_lwp.pr_flags & PR_ISTOP) &&
              pi->prstatus.pr_lwp.pr_why == PR_REQUESTED)
-           /* Unixware peculiarity -- read the damn thing again! */
+           /* Unixware peculiarity -- read the damn thing again!  */
            pi->status_valid = (read (pi->status_fd,
                                      (char *) &pi->prstatus,
                                      sizeof (gdb_prstatus_t))
@@ -1199,7 +1094,7 @@ proc_get_status (procinfo *pi)
 #ifdef PIOCTSTATUS     /* osf */
   if (pi->tid == 0)    /* main process */
     {
-      /* Just read the danged status.  Now isn't that simple? */
+      /* Just read the danged status.  Now isn't that simple?  */
       pi->status_valid =
        (ioctl (pi->status_fd, PIOCSTATUS, &pi->prstatus) >= 0);
     }
@@ -1223,7 +1118,7 @@ proc_get_status (procinfo *pi)
        }
     }
 #else
-  /* Just read the danged status.  Now isn't that simple? */
+  /* Just read the danged status.  Now isn't that simple?  */
   pi->status_valid = (ioctl (pi->status_fd, PIOCSTATUS, &pi->prstatus) >= 0);
 #endif
 #endif
@@ -1236,34 +1131,30 @@ proc_get_status (procinfo *pi)
                                proc_get_current_thread (pi));
     }
 
-  /* The status struct includes general regs, so mark them valid too */
+  /* The status struct includes general regs, so mark them valid too */
   pi->gregs_valid  = pi->status_valid;
 #ifdef NEW_PROC_API
-  /* In the read/write multiple-fd model,
-     the status struct includes the fp regs too, so mark them valid too */
+  /* In the read/write multiple-fd model, the status struct includes
+     the fp regs too, so mark them valid too.  */
   pi->fpregs_valid = pi->status_valid;
 #endif
-  return pi->status_valid;     /* True if success, false if failure. */
+  return pi->status_valid;     /* True if success, false if failure.  */
 }
 
-/*
- * Function: proc_flags
- *
- * returns the process flags (pr_flags field).
- */
+/* Returns the process flags (pr_flags field).  */
 
-long
+static long
 proc_flags (procinfo *pi)
 {
   if (!pi->status_valid)
     if (!proc_get_status (pi))
-      return 0;        /* FIXME: not a good failure value (but what is?) */
+      return 0;        /* FIXME: not a good failure value (but what is?)  */
 
 #ifdef NEW_PROC_API
 # ifdef UNIXWARE
   /* UnixWare 7.1 puts process status flags, e.g. PR_ASYNC, in
      pstatus_t and LWP status flags, e.g. PR_STOPPED, in lwpstatus_t.
-     The two sets of flags don't overlap. */
+     The two sets of flags don't overlap.  */
   return pi->prstatus.pr_flags | pi->prstatus.pr_lwp.pr_flags;
 # else
   return pi->prstatus.pr_lwp.pr_flags;
@@ -1273,18 +1164,14 @@ proc_flags (procinfo *pi)
 #endif
 }
 
-/*
- * Function: proc_why
- *
- * returns the pr_why field (why the process stopped).
- */
+/* Returns the pr_why field (why the process stopped).  */
 
-int
+static int
 proc_why (procinfo *pi)
 {
   if (!pi->status_valid)
     if (!proc_get_status (pi))
-      return 0;        /* FIXME: not a good failure value (but what is?) */
+      return 0;        /* FIXME: not a good failure value (but what is?)  */
 
 #ifdef NEW_PROC_API
   return pi->prstatus.pr_lwp.pr_why;
@@ -1293,18 +1180,14 @@ proc_why (procinfo *pi)
 #endif
 }
 
-/*
- * Function: proc_what
- *
- * returns the pr_what field (details of why the process stopped).
- */
+/* Returns the pr_what field (details of why the process stopped).  */
 
-int
+static int
 proc_what (procinfo *pi)
 {
   if (!pi->status_valid)
     if (!proc_get_status (pi))
-      return 0;        /* FIXME: not a good failure value (but what is?) */
+      return 0;        /* FIXME: not a good failure value (but what is?)  */
 
 #ifdef NEW_PROC_API
   return pi->prstatus.pr_lwp.pr_what;
@@ -1313,14 +1196,10 @@ proc_what (procinfo *pi)
 #endif
 }
 
-/*
- * Function: proc_watchpoint_address
- *
- *   This function is only called when PI is stopped by a watchpoint.
- *   Assuming OS supports it, write to *ADDR the data address which
- * triggered it and return 1.
- * Return 0 if it is not possible to know the address.
- */
+/* This function is only called when PI is stopped by a watchpoint.
+   Assuming the OS supports it, write to *ADDR the data address which
+   triggered it and return 1.  Return 0 if it is not possible to know
+   the address.  */
 
 static int
 proc_watchpoint_address (procinfo *pi, CORE_ADDR *addr)
@@ -1342,13 +1221,11 @@ proc_watchpoint_address (procinfo *pi, CORE_ADDR *addr)
 }
 
 #ifndef PIOCSSPCACT    /* The following is not supported on OSF.  */
-/*
- * Function: proc_nsysarg
- *
- * returns the pr_nsysarg field (number of args to the current syscall).
- */
 
-int
+/* Returns the pr_nsysarg field (number of args to the current
+   syscall).  */
+
+static int
 proc_nsysarg (procinfo *pi)
 {
   if (!pi->status_valid)
@@ -1362,13 +1239,10 @@ proc_nsysarg (procinfo *pi)
 #endif
 }
 
-/*
- * Function: proc_sysargs
- *
- * returns the pr_sysarg field (pointer to the arguments of current syscall).
- */
+/* Returns the pr_sysarg field (pointer to the arguments of current
+   syscall).  */
 
-long *
+static long *
 proc_sysargs (procinfo *pi)
 {
   if (!pi->status_valid)
@@ -1381,40 +1255,17 @@ proc_sysargs (procinfo *pi)
   return (long *) &pi->prstatus.pr_sysarg;
 #endif
 }
-
-/*
- * Function: proc_syscall
- *
- * returns the pr_syscall field (id of current syscall if we are in one).
- */
-
-int
-proc_syscall (procinfo *pi)
-{
-  if (!pi->status_valid)
-    if (!proc_get_status (pi))
-      return 0;
-
-#ifdef NEW_PROC_API
-  return pi->prstatus.pr_lwp.pr_syscall;
-#else
-  return pi->prstatus.pr_syscall;
-#endif
-}
 #endif /* PIOCSSPCACT */
 
-/*
- * Function: proc_cursig:
- *
- * returns the pr_cursig field (current signal).
- */
+#ifdef PROCFS_DONT_PIOCSSIG_CURSIG
+/* Returns the pr_cursig field (current signal).  */
 
-long
+static long
 proc_cursig (struct procinfo *pi)
 {
   if (!pi->status_valid)
     if (!proc_get_status (pi))
-      return 0;        /* FIXME: not a good failure value (but what is?) */
+      return 0;        /* FIXME: not a good failure value (but what is?)  */
 
 #ifdef NEW_PROC_API
   return pi->prstatus.pr_lwp.pr_cursig;
@@ -1422,39 +1273,36 @@ proc_cursig (struct procinfo *pi)
   return pi->prstatus.pr_cursig;
 #endif
 }
+#endif /* PROCFS_DONT_PIOCSSIG_CURSIG */
+
+/* === I appologize for the messiness of this function.
+   === This is an area where the different versions of
+   === /proc are more inconsistent than usual.
+
+   Set or reset any of the following process flags:
+      PR_FORK  -- forked child will inherit trace flags
+      PR_RLC   -- traced process runs when last /proc file closed.
+      PR_KLC    -- traced process is killed when last /proc file closed.
+      PR_ASYNC -- LWP's get to run/stop independently.
 
-/*
- * Function: proc_modify_flag
- *
- *  === I appologize for the messiness of this function.
- *  === This is an area where the different versions of
- *  === /proc are more inconsistent than usual.     MVS
- *
- * Set or reset any of the following process flags:
- *    PR_FORK  -- forked child will inherit trace flags
- *    PR_RLC   -- traced process runs when last /proc file closed.
- *    PR_KLC    -- traced process is killed when last /proc file closed.
- *    PR_ASYNC -- LWP's get to run/stop independently.
- *
- * There are three methods for doing this function:
- * 1) Newest: read/write [PCSET/PCRESET/PCUNSET]
- *    [Sol6, Sol7, UW]
- * 2) Middle: PIOCSET/PIOCRESET
- *    [Irix, Sol5]
- * 3) Oldest: PIOCSFORK/PIOCRFORK/PIOCSRLC/PIOCRRLC
- *    [OSF, Sol5]
- *
- * Note: Irix does not define PR_ASYNC.
- * Note: OSF  does not define PR_KLC.
- * Note: OSF  is the only one that can ONLY use the oldest method.
- *
- * Arguments:
- *    pi   -- the procinfo
- *    flag -- one of PR_FORK, PR_RLC, or PR_ASYNC
- *    mode -- 1 for set, 0 for reset.
- *
- * Returns non-zero for success, zero for failure.
- */
+   There are three methods for doing this function:
+   1) Newest: read/write [PCSET/PCRESET/PCUNSET]
+      [Sol6, Sol7, UW]
+   2) Middle: PIOCSET/PIOCRESET
+      [Irix, Sol5]
+   3) Oldest: PIOCSFORK/PIOCRFORK/PIOCSRLC/PIOCRRLC
+      [OSF, Sol5]
+
+   Note: Irix does not define PR_ASYNC.
+   Note: OSF  does not define PR_KLC.
+   Note: OSF  is the only one that can ONLY use the oldest method.
+
+   Arguments:
+      pi   -- the procinfo
+      flag -- one of PR_FORK, PR_RLC, or PR_ASYNC
+      mode -- 1 for set, 0 for reset.
+
+   Returns non-zero for success, zero for failure.  */
 
 enum { FLAG_RESET, FLAG_SET };
 
@@ -1463,19 +1311,16 @@ proc_modify_flag (procinfo *pi, long flag, long mode)
 {
   long win = 0;                /* default to fail */
 
-  /*
-   * These operations affect the process as a whole, and applying
-   * them to an individual LWP has the same meaning as applying them
-   * to the main process.  Therefore, if we're ever called with a
-   * pointer to an LWP's procinfo, let's substitute the process's
-   * procinfo and avoid opening the LWP's file descriptor
-   * unnecessarily.
-   */
+  /* These operations affect the process as a whole, and applying them
+     to an individual LWP has the same meaning as applying them to the
+     main process.  Therefore, if we're ever called with a pointer to
+     an LWP's procinfo, let's substitute the process's procinfo and
+     avoid opening the LWP's file descriptor unnecessarily.  */
 
   if (pi->pid != 0)
     pi = find_procinfo_or_die (pi->pid, 0);
 
-#ifdef NEW_PROC_API    /* Newest method: UnixWare and newer Solarii */
+#ifdef NEW_PROC_API    /* Newest method: UnixWare and newer Solarii */
   /* First normalize the PCUNSET/PCRESET command opcode
      (which for no obvious reason has a different definition
      from one operating system to the next...)  */
@@ -1489,9 +1334,9 @@ proc_modify_flag (procinfo *pi, long flag, long mode)
   {
     procfs_ctl_t arg[2];
 
-    if (mode == FLAG_SET)      /* Set the flag (RLC, FORK, or ASYNC) */
+    if (mode == FLAG_SET)      /* Set the flag (RLC, FORK, or ASYNC) */
       arg[0] = PCSET;
-    else                       /* Reset the flag */
+    else                       /* Reset the flag */
       arg[0] = GDBRESET;
 
     arg[1] = flag;
@@ -1499,11 +1344,11 @@ proc_modify_flag (procinfo *pi, long flag, long mode)
   }
 #else
 #ifdef PIOCSET         /* Irix/Sol5 method */
-  if (mode == FLAG_SET)        /* Set the flag (hopefully RLC, FORK, or ASYNC) */
+  if (mode == FLAG_SET)        /* Set the flag (hopefully RLC, FORK, or ASYNC) */
     {
       win = (ioctl (pi->ctl_fd, PIOCSET, &flag)   >= 0);
     }
-  else                 /* Reset the flag */
+  else                 /* Reset the flag */
     {
       win = (ioctl (pi->ctl_fd, PIOCRESET, &flag) >= 0);
     }
@@ -1532,14 +1377,15 @@ proc_modify_flag (procinfo *pi, long flag, long mode)
       }
     break;
   default:
-    win = 0;           /* fail -- unknown flag (can't do PR_ASYNC) */
+    win = 0;           /* Fail -- unknown flag (can't do PR_ASYNC).  */
     break;
   }
 #endif
 #endif
 #endif
 #undef GDBRESET
-  /* The above operation renders the procinfo's cached pstatus obsolete. */
+  /* The above operation renders the procinfo's cached pstatus
+     obsolete.  */
   pi->status_valid = 0;
 
   if (!win)
@@ -1558,154 +1404,69 @@ proc_modify_flag (procinfo *pi, long flag, long mode)
   return win;
 }
 
-/*
- * Function: proc_set_run_on_last_close
- *
- * Set the run_on_last_close flag.
- * Process with all threads will become runnable
- * when debugger closes all /proc fds.
- *
- * Returns non-zero for success, zero for failure.
- */
+/* Set the run_on_last_close flag.  Process with all threads will
+   become runnable when debugger closes all /proc fds.  Returns
+   non-zero for success, zero for failure.  */
 
-int
+static int
 proc_set_run_on_last_close (procinfo *pi)
 {
   return proc_modify_flag (pi, PR_RLC, FLAG_SET);
 }
 
-/*
- * Function: proc_unset_run_on_last_close
- *
- * Reset the run_on_last_close flag.
- * Process will NOT become runnable
- * when debugger closes its file handles.
- *
- * Returns non-zero for success, zero for failure.
- */
+/* Reset the run_on_last_close flag.  The process will NOT become
+   runnable when debugger closes its file handles.  Returns non-zero
+   for success, zero for failure.  */
 
-int
+static int
 proc_unset_run_on_last_close (procinfo *pi)
 {
   return proc_modify_flag (pi, PR_RLC, FLAG_RESET);
 }
 
-#ifdef PR_KLC
-/*
- * Function: proc_set_kill_on_last_close
- *
- * Set the kill_on_last_close flag.
- * Process with all threads will be killed when debugger
- * closes all /proc fds (or debugger exits or dies).
- *
- * Returns non-zero for success, zero for failure.
- */
-
-int
-proc_set_kill_on_last_close (procinfo *pi)
-{
-  return proc_modify_flag (pi, PR_KLC, FLAG_SET);
-}
-
-/*
- * Function: proc_unset_kill_on_last_close
- *
- * Reset the kill_on_last_close flag.
- * Process will NOT be killed when debugger
- * closes its file handles (or exits or dies).
- *
- * Returns non-zero for success, zero for failure.
- */
-
-int
-proc_unset_kill_on_last_close (procinfo *pi)
-{
-  return proc_modify_flag (pi, PR_KLC, FLAG_RESET);
-}
-#endif /* PR_KLC */
-
-/*
- * Function: proc_set_inherit_on_fork
- *
- * Set inherit_on_fork flag.
- * If the process forks a child while we are registered for events
- * in the parent, then we will also recieve events from the child.
- *
- * Returns non-zero for success, zero for failure.
- */
-
-int
-proc_set_inherit_on_fork (procinfo *pi)
-{
-  return proc_modify_flag (pi, PR_FORK, FLAG_SET);
-}
-
-/*
- * Function: proc_unset_inherit_on_fork
- *
- * Reset inherit_on_fork flag.
- * If the process forks a child while we are registered for events
- * in the parent, then we will NOT recieve events from the child.
- *
- * Returns non-zero for success, zero for failure.
- */
-
-int
+/* Reset inherit_on_fork flag.  If the process forks a child while we
+   are registered for events in the parent, then we will NOT recieve
+   events from the child.  Returns non-zero for success, zero for
+   failure.  */
+
+static int
 proc_unset_inherit_on_fork (procinfo *pi)
 {
   return proc_modify_flag (pi, PR_FORK, FLAG_RESET);
 }
 
 #ifdef PR_ASYNC
-/*
- * Function: proc_set_async
- *
- * Set PR_ASYNC flag.
- * If one LWP stops because of a debug event (signal etc.),
- * the remaining LWPs will continue to run.
- *
- * Returns non-zero for success, zero for failure.
- */
-
-int
+/* Set PR_ASYNC flag.  If one LWP stops because of a debug event
+   (signal etc.), the remaining LWPs will continue to run.  Returns
+   non-zero for success, zero for failure.  */
+
+static int
 proc_set_async (procinfo *pi)
 {
   return proc_modify_flag (pi, PR_ASYNC, FLAG_SET);
 }
 
-/*
- * Function: proc_unset_async
- *
- * Reset PR_ASYNC flag.
- * If one LWP stops because of a debug event (signal etc.),
- * then all other LWPs will stop as well.
- *
- * Returns non-zero for success, zero for failure.
- */
+/* Reset PR_ASYNC flag.  If one LWP stops because of a debug event
+   (signal etc.), then all other LWPs will stop as well.  Returns
+   non-zero for success, zero for failure.  */
 
-int
+static int
 proc_unset_async (procinfo *pi)
 {
   return proc_modify_flag (pi, PR_ASYNC, FLAG_RESET);
 }
 #endif /* PR_ASYNC */
 
-/*
- * Function: proc_stop_process
- *
- * Request the process/LWP to stop.  Does not wait.
- * Returns non-zero for success, zero for failure.
- */
+/* Request the process/LWP to stop.  Does not wait.  Returns non-zero
+   for success, zero for failure.  */
 
-int
+static int
 proc_stop_process (procinfo *pi)
 {
   int win;
 
-  /*
-   * We might conceivably apply this operation to an LWP, and
-   * the LWP's ctl file descriptor might not be open.
-   */
+  /* We might conceivably apply this operation to an LWP, and the
+     LWP's ctl file descriptor might not be open.  */
 
   if (pi->ctl_fd == 0 &&
       open_procinfo_files (pi, FD_CTL) == 0)
@@ -1714,6 +1475,7 @@ proc_stop_process (procinfo *pi)
     {
 #ifdef NEW_PROC_API
       procfs_ctl_t cmd = PCSTOP;
+
       win = (write (pi->ctl_fd, (char *) &cmd, sizeof (cmd)) == sizeof (cmd));
 #else  /* ioctl method */
       win = (ioctl (pi->ctl_fd, PIOCSTOP, &pi->prstatus) >= 0);
@@ -1732,24 +1494,18 @@ proc_stop_process (procinfo *pi)
   return win;
 }
 
-/*
- * Function: proc_wait_for_stop
- *
- * Wait for the process or LWP to stop (block until it does).
- * Returns non-zero for success, zero for failure.
- */
+/* Wait for the process or LWP to stop (block until it does).  Returns
+   non-zero for success, zero for failure.  */
 
-int
+static int
 proc_wait_for_stop (procinfo *pi)
 {
   int win;
 
-  /*
-   * We should never have to apply this operation to any procinfo
-   * except the one for the main process.  If that ever changes
-   * for any reason, then take out the following clause and
-   * replace it with one that makes sure the ctl_fd is open.
-   */
+  /* We should never have to apply this operation to any procinfo
+     except the one for the main process.  If that ever changes for
+     any reason, then take out the following clause and replace it
+     with one that makes sure the ctl_fd is open.  */
 
   if (pi->tid != 0)
     pi = find_procinfo_or_die (pi->pid, 0);
@@ -1757,6 +1513,7 @@ proc_wait_for_stop (procinfo *pi)
 #ifdef NEW_PROC_API
   {
     procfs_ctl_t cmd = PCWSTOP;
+
     win = (write (pi->ctl_fd, (char *) &cmd, sizeof (cmd)) == sizeof (cmd));
     /* We been runnin' and we stopped -- need to update status.  */
     pi->status_valid = 0;
@@ -1777,42 +1534,33 @@ proc_wait_for_stop (procinfo *pi)
   return win;
 }
 
-/*
- * Function: proc_run_process
- *
- * Make the process or LWP runnable.
- * Options (not all are implemented):
- *   - single-step
- *   - clear current fault
- *   - clear current signal
- *   - abort the current system call
- *   - stop as soon as finished with system call
- *   - (ioctl): set traced signal set
- *   - (ioctl): set held   signal set
- *   - (ioctl): set traced fault  set
- *   - (ioctl): set start pc (vaddr)
- * Always clear the current fault.
- * Clear the current signal if 'signo' is zero.
- *
- * Arguments:
- *   pi                the process or LWP to operate on.
- *   step      if true, set the process or LWP to trap after one instr.
- *   signo     if zero, clear the current signal if any.
- *             if non-zero, set the current signal to this one.
- *
- * Returns non-zero for success, zero for failure.
- */
-
-int
+/* Make the process or LWP runnable.
+
+   Options (not all are implemented):
+     - single-step
+     - clear current fault
+     - clear current signal
+     - abort the current system call
+     - stop as soon as finished with system call
+     - (ioctl): set traced signal set
+     - (ioctl): set held   signal set
+     - (ioctl): set traced fault  set
+     - (ioctl): set start pc (vaddr)
+
+   Always clears the current fault.  PI is the process or LWP to
+   operate on.  If STEP is true, set the process or LWP to trap after
+   one instruction.  If SIGNO is zero, clear the current signal if
+   any; if non-zero, set the current signal to this one.  Returns
+   non-zero for success, zero for failure.  */
+
+static int
 proc_run_process (procinfo *pi, int step, int signo)
 {
   int win;
   int runflags;
 
-  /*
-   * We will probably have to apply this operation to individual threads,
-   * so make sure the control file descriptor is open.
-   */
+  /* We will probably have to apply this operation to individual
+     threads, so make sure the control file descriptor is open.  */
 
   if (pi->ctl_fd == 0 &&
       open_procinfo_files (pi, FD_CTL) == 0)
@@ -1820,12 +1568,12 @@ proc_run_process (procinfo *pi, int step, int signo)
       return 0;
     }
 
-  runflags    = PRCFAULT;      /* always clear current fault  */
+  runflags    = PRCFAULT;      /* Always clear current fault.  */
   if (step)
     runflags |= PRSTEP;
   if (signo == 0)
     runflags |= PRCSIG;
-  else if (signo != -1)                /* -1 means do nothing W.R.T. signals */
+  else if (signo != -1)                /* -1 means do nothing W.R.T. signals */
     proc_set_current_signal (pi, signo);
 
 #ifdef NEW_PROC_API
@@ -1849,24 +1597,18 @@ proc_run_process (procinfo *pi, int step, int signo)
   return win;
 }
 
-/*
- * Function: proc_set_traced_signals
- *
- * Register to trace signals in the process or LWP.
- * Returns non-zero for success, zero for failure.
- */
+/* Register to trace signals in the process or LWP.  Returns non-zero
+   for success, zero for failure.  */
 
-int
+static int
 proc_set_traced_signals (procinfo *pi, gdb_sigset_t *sigset)
 {
   int win;
 
-  /*
-   * We should never have to apply this operation to any procinfo
-   * except the one for the main process.  If that ever changes
-   * for any reason, then take out the following clause and
-   * replace it with one that makes sure the ctl_fd is open.
-   */
+  /* We should never have to apply this operation to any procinfo
+     except the one for the main process.  If that ever changes for
+     any reason, then take out the following clause and replace it
+     with one that makes sure the ctl_fd is open.  */
 
   if (pi->tid != 0)
     pi = find_procinfo_or_die (pi->pid, 0);
@@ -1887,7 +1629,7 @@ proc_set_traced_signals (procinfo *pi, gdb_sigset_t *sigset)
 #else  /* ioctl method */
   win = (ioctl (pi->ctl_fd, PIOCSTRACE, sigset) >= 0);
 #endif
-  /* The above operation renders the procinfo's cached pstatus obsolete. */
+  /* The above operation renders the procinfo's cached pstatus obsolete.  */
   pi->status_valid = 0;
 
   if (!win)
@@ -1895,24 +1637,18 @@ proc_set_traced_signals (procinfo *pi, gdb_sigset_t *sigset)
   return win;
 }
 
-/*
- * Function: proc_set_traced_faults
- *
- * Register to trace hardware faults in the process or LWP.
- * Returns non-zero for success, zero for failure.
- */
+/* Register to trace hardware faults in the process or LWP.  Returns
+   non-zero for success, zero for failure.  */
 
-int
+static int
 proc_set_traced_faults (procinfo *pi, fltset_t *fltset)
 {
   int win;
 
-  /*
-   * We should never have to apply this operation to any procinfo
-   * except the one for the main process.  If that ever changes
-   * for any reason, then take out the following clause and
-   * replace it with one that makes sure the ctl_fd is open.
-   */
+  /* We should never have to apply this operation to any procinfo
+     except the one for the main process.  If that ever changes for
+     any reason, then take out the following clause and replace it
+     with one that makes sure the ctl_fd is open.  */
 
   if (pi->tid != 0)
     pi = find_procinfo_or_die (pi->pid, 0);
@@ -1933,30 +1669,24 @@ proc_set_traced_faults (procinfo *pi, fltset_t *fltset)
 #else  /* ioctl method */
   win = (ioctl (pi->ctl_fd, PIOCSFAULT, fltset) >= 0);
 #endif
-  /* The above operation renders the procinfo's cached pstatus obsolete. */
+  /* The above operation renders the procinfo's cached pstatus obsolete.  */
   pi->status_valid = 0;
 
   return win;
 }
 
-/*
- * Function: proc_set_traced_sysentry
- *
- * Register to trace entry to system calls in the process or LWP.
- * Returns non-zero for success, zero for failure.
- */
+/* Register to trace entry to system calls in the process or LWP.
+   Returns non-zero for success, zero for failure.  */
 
-int
+static int
 proc_set_traced_sysentry (procinfo *pi, sysset_t *sysset)
 {
   int win;
 
-  /*
-   * We should never have to apply this operation to any procinfo
-   * except the one for the main process.  If that ever changes
-   * for any reason, then take out the following clause and
-   * replace it with one that makes sure the ctl_fd is open.
-   */
+  /* We should never have to apply this operation to any procinfo
+     except the one for the main process.  If that ever changes for
+     any reason, then take out the following clause and replace it
+     with one that makes sure the ctl_fd is open.  */
 
   if (pi->tid != 0)
     pi = find_procinfo_or_die (pi->pid, 0);
@@ -1969,7 +1699,7 @@ proc_set_traced_sysentry (procinfo *pi, sysset_t *sysset)
       char sysset[sizeof (sysset_t)];
     } *argp;
     int argp_size = sizeof (struct gdb_proc_ctl_pcsentry)
-                  - sizeof (sysset_t)
+                 - sizeof (sysset_t)
                  + sysset_t_size (pi);
 
     argp = xmalloc (argp_size);
@@ -1983,30 +1713,25 @@ proc_set_traced_sysentry (procinfo *pi, sysset_t *sysset)
 #else  /* ioctl method */
   win = (ioctl (pi->ctl_fd, PIOCSENTRY, sysset) >= 0);
 #endif
-  /* The above operation renders the procinfo's cached pstatus obsolete. */
+  /* The above operation renders the procinfo's cached pstatus
+     obsolete.  */
   pi->status_valid = 0;
 
   return win;
 }
 
-/*
- * Function: proc_set_traced_sysexit
- *
- * Register to trace exit from system calls in the process or LWP.
- * Returns non-zero for success, zero for failure.
- */
+/* Register to trace exit from system calls in the process or LWP.
+   Returns non-zero for success, zero for failure.  */
 
-int
+static int
 proc_set_traced_sysexit (procinfo *pi, sysset_t *sysset)
 {
   int win;
 
-  /*
-   * We should never have to apply this operation to any procinfo
-   * except the one for the main process.  If that ever changes
-   * for any reason, then take out the following clause and
-   * replace it with one that makes sure the ctl_fd is open.
-   */
+  /* We should never have to apply this operation to any procinfo
+     except the one for the main process.  If that ever changes for
+     any reason, then take out the following clause and replace it
+     with one that makes sure the ctl_fd is open.  */
 
   if (pi->tid != 0)
     pi = find_procinfo_or_die (pi->pid, 0);
@@ -2019,7 +1744,7 @@ proc_set_traced_sysexit (procinfo *pi, sysset_t *sysset)
       char sysset[sizeof (sysset_t)];
     } *argp;
     int argp_size = sizeof (struct gdb_proc_ctl_pcsexit)
-                  - sizeof (sysset_t)
+                 - sizeof (sysset_t)
                  + sysset_t_size (pi);
 
     argp = xmalloc (argp_size);
@@ -2033,30 +1758,25 @@ proc_set_traced_sysexit (procinfo *pi, sysset_t *sysset)
 #else  /* ioctl method */
   win = (ioctl (pi->ctl_fd, PIOCSEXIT, sysset) >= 0);
 #endif
-  /* The above operation renders the procinfo's cached pstatus obsolete. */
+  /* The above operation renders the procinfo's cached pstatus
+     obsolete.  */
   pi->status_valid = 0;
 
   return win;
 }
 
-/*
- * Function: proc_set_held_signals
- *
- * Specify the set of blocked / held signals in the process or LWP.
- * Returns non-zero for success, zero for failure.
- */
+/* Specify the set of blocked / held signals in the process or LWP.
+   Returns non-zero for success, zero for failure.  */
 
-int
+static int
 proc_set_held_signals (procinfo *pi, gdb_sigset_t *sighold)
 {
   int win;
 
-  /*
-   * We should never have to apply this operation to any procinfo
-   * except the one for the main process.  If that ever changes
-   * for any reason, then take out the following clause and
-   * replace it with one that makes sure the ctl_fd is open.
-   */
+  /* We should never have to apply this operation to any procinfo
+     except the one for the main process.  If that ever changes for
+     any reason, then take out the following clause and replace it
+     with one that makes sure the ctl_fd is open.  */
 
   if (pi->tid != 0)
     pi = find_procinfo_or_die (pi->pid, 0);
@@ -2076,104 +1796,25 @@ proc_set_held_signals (procinfo *pi, gdb_sigset_t *sighold)
 #else
   win = (ioctl (pi->ctl_fd, PIOCSHOLD, sighold) >= 0);
 #endif
-  /* The above operation renders the procinfo's cached pstatus obsolete. */
+  /* The above operation renders the procinfo's cached pstatus
+     obsolete.  */
   pi->status_valid = 0;
 
   return win;
 }
 
-/*
- * Function: proc_get_pending_signals
- *
- * returns the set of signals that are pending in the process or LWP.
- * Will also copy the sigset if 'save' is non-zero.
- */
+/* Returns the set of signals that are held / blocked.  Will also copy
+   the sigset if SAVE is non-zero.  */
 
-gdb_sigset_t *
-proc_get_pending_signals (procinfo *pi, gdb_sigset_t *save)
-{
-  gdb_sigset_t *ret = NULL;
-
-  /*
-   * We should never have to apply this operation to any procinfo
-   * except the one for the main process.  If that ever changes
-   * for any reason, then take out the following clause and
-   * replace it with one that makes sure the ctl_fd is open.
-   */
-
-  if (pi->tid != 0)
-    pi = find_procinfo_or_die (pi->pid, 0);
-
-  if (!pi->status_valid)
-    if (!proc_get_status (pi))
-      return NULL;
-
-#ifdef NEW_PROC_API
-  ret = &pi->prstatus.pr_lwp.pr_lwppend;
-#else
-  ret = &pi->prstatus.pr_sigpend;
-#endif
-  if (save && ret)
-    memcpy (save, ret, sizeof (gdb_sigset_t));
-
-  return ret;
-}
-
-/*
- * Function: proc_get_signal_actions
- *
- * returns the set of signal actions.
- * Will also copy the sigactionset if 'save' is non-zero.
- */
-
-gdb_sigaction_t *
-proc_get_signal_actions (procinfo *pi, gdb_sigaction_t *save)
-{
-  gdb_sigaction_t *ret = NULL;
-
-  /*
-   * We should never have to apply this operation to any procinfo
-   * except the one for the main process.  If that ever changes
-   * for any reason, then take out the following clause and
-   * replace it with one that makes sure the ctl_fd is open.
-   */
-
-  if (pi->tid != 0)
-    pi = find_procinfo_or_die (pi->pid, 0);
-
-  if (!pi->status_valid)
-    if (!proc_get_status (pi))
-      return NULL;
-
-#ifdef NEW_PROC_API
-  ret = &pi->prstatus.pr_lwp.pr_action;
-#else
-  ret = &pi->prstatus.pr_action;
-#endif
-  if (save && ret)
-    memcpy (save, ret, sizeof (gdb_sigaction_t));
-
-  return ret;
-}
-
-/*
- * Function: proc_get_held_signals
- *
- * returns the set of signals that are held / blocked.
- * Will also copy the sigset if 'save' is non-zero.
- */
-
-gdb_sigset_t *
+static gdb_sigset_t *
 proc_get_held_signals (procinfo *pi, gdb_sigset_t *save)
 {
   gdb_sigset_t *ret = NULL;
 
-  /*
-   * We should never have to apply this operation to any procinfo
-   * except the one for the main process.  If that ever changes
-   * for any reason, then take out the following clause and
-   * replace it with one that makes sure the ctl_fd is open.
-   */
+  /* We should never have to apply this operation to any procinfo
+     except the one for the main process.  If that ever changes for
+     any reason, then take out the following clause and replace it
+     with one that makes sure the ctl_fd is open.  */
 
   if (pi->tid != 0)
     pi = find_procinfo_or_die (pi->pid, 0);
@@ -2202,24 +1843,18 @@ proc_get_held_signals (procinfo *pi, gdb_sigset_t *save)
   return ret;
 }
 
-/*
- * Function: proc_get_traced_signals
- *
- * returns the set of signals that are traced / debugged.
- * Will also copy the sigset if 'save' is non-zero.
- */
+/* Returns the set of signals that are traced / debugged.  Will also
+   copy the sigset if SAVE is non-zero.  */
 
-gdb_sigset_t *
+static gdb_sigset_t *
 proc_get_traced_signals (procinfo *pi, gdb_sigset_t *save)
 {
   gdb_sigset_t *ret = NULL;
 
-  /*
-   * We should never have to apply this operation to any procinfo
-   * except the one for the main process.  If that ever changes
-   * for any reason, then take out the following clause and
-   * replace it with one that makes sure the ctl_fd is open.
-   */
+  /* We should never have to apply this operation to any procinfo
+     except the one for the main process.  If that ever changes for
+     any reason, then take out the following clause and replace it
+     with one that makes sure the ctl_fd is open.  */
 
   if (pi->tid != 0)
     pi = find_procinfo_or_die (pi->pid, 0);
@@ -2244,92 +1879,18 @@ proc_get_traced_signals (procinfo *pi, gdb_sigset_t *save)
   return ret;
 }
 
-/*
- * Function: proc_trace_signal
- *
- * Add 'signo' to the set of signals that are traced.
- * Returns non-zero for success, zero for failure.
- */
-
-int
-proc_trace_signal (procinfo *pi, int signo)
-{
-  gdb_sigset_t temp;
-
-  /*
-   * We should never have to apply this operation to any procinfo
-   * except the one for the main process.  If that ever changes
-   * for any reason, then take out the following clause and
-   * replace it with one that makes sure the ctl_fd is open.
-   */
+/* Returns the set of hardware faults that are traced /debugged.  Will
+   also copy the faultset if SAVE is non-zero.  */
 
-  if (pi->tid != 0)
-    pi = find_procinfo_or_die (pi->pid, 0);
-
-  if (pi)
-    {
-      if (proc_get_traced_signals (pi, &temp))
-       {
-         gdb_praddset (&temp, signo);
-         return proc_set_traced_signals (pi, &temp);
-       }
-    }
-
-  return 0;    /* failure */
-}
-
-/*
- * Function: proc_ignore_signal
- *
- * Remove 'signo' from the set of signals that are traced.
- * Returns non-zero for success, zero for failure.
- */
-
-int
-proc_ignore_signal (procinfo *pi, int signo)
-{
-  gdb_sigset_t temp;
-
-  /*
-   * We should never have to apply this operation to any procinfo
-   * except the one for the main process.  If that ever changes
-   * for any reason, then take out the following clause and
-   * replace it with one that makes sure the ctl_fd is open.
-   */
-
-  if (pi->tid != 0)
-    pi = find_procinfo_or_die (pi->pid, 0);
-
-  if (pi)
-    {
-      if (proc_get_traced_signals (pi, &temp))
-       {
-         gdb_prdelset (&temp, signo);
-         return proc_set_traced_signals (pi, &temp);
-       }
-    }
-
-  return 0;    /* failure */
-}
-
-/*
- * Function: proc_get_traced_faults
- *
- * returns the set of hardware faults that are traced /debugged.
- * Will also copy the faultset if 'save' is non-zero.
- */
-
-fltset_t *
+static fltset_t *
 proc_get_traced_faults (procinfo *pi, fltset_t *save)
 {
   fltset_t *ret = NULL;
 
-  /*
-   * We should never have to apply this operation to any procinfo
-   * except the one for the main process.  If that ever changes
-   * for any reason, then take out the following clause and
-   * replace it with one that makes sure the ctl_fd is open.
-   */
+  /* We should never have to apply this operation to any procinfo
+     except the one for the main process.  If that ever changes for
+     any reason, then take out the following clause and replace it
+     with one that makes sure the ctl_fd is open.  */
 
   if (pi->tid != 0)
     pi = find_procinfo_or_die (pi->pid, 0);
@@ -2354,24 +1915,18 @@ proc_get_traced_faults (procinfo *pi, fltset_t *save)
   return ret;
 }
 
-/*
- * Function: proc_get_traced_sysentry
- *
- * returns the set of syscalls that are traced /debugged on entry.
- * Will also copy the syscall set if 'save' is non-zero.
- */
+/* Returns the set of syscalls that are traced /debugged on entry.
+   Will also copy the syscall set if SAVE is non-zero.  */
 
-sysset_t *
+static sysset_t *
 proc_get_traced_sysentry (procinfo *pi, sysset_t *save)
 {
   sysset_t *ret = NULL;
 
-  /*
-   * We should never have to apply this operation to any procinfo
-   * except the one for the main process.  If that ever changes
-   * for any reason, then take out the following clause and
-   * replace it with one that makes sure the ctl_fd is open.
-   */
+  /* We should never have to apply this operation to any procinfo
+     except the one for the main process.  If that ever changes for
+     any reason, then take out the following clause and replace it
+     with one that makes sure the ctl_fd is open.  */
 
   if (pi->tid != 0)
     pi = find_procinfo_or_die (pi->pid, 0);
@@ -2402,7 +1957,7 @@ proc_get_traced_sysentry (procinfo *pi, sysset_t *save)
        int rsize;
 
        if (lseek (pi->status_fd, (off_t) pi->prstatus.pr_sysentry_offset,
-                  SEEK_SET)
+                  SEEK_SET)
            != (off_t) pi->prstatus.pr_sysentry_offset)
          return NULL;
        size = sysset_t_size (pi);
@@ -2427,24 +1982,18 @@ proc_get_traced_sysentry (procinfo *pi, sysset_t *save)
   return ret;
 }
 
-/*
- * Function: proc_get_traced_sysexit
- *
- * returns the set of syscalls that are traced /debugged on exit.
- * Will also copy the syscall set if 'save' is non-zero.
- */
+/* Returns the set of syscalls that are traced /debugged on exit.
+   Will also copy the syscall set if SAVE is non-zero.  */
 
-sysset_t *
+static sysset_t *
 proc_get_traced_sysexit (procinfo *pi, sysset_t *save)
 {
   sysset_t * ret = NULL;
 
-  /*
-   * We should never have to apply this operation to any procinfo
-   * except the one for the main process.  If that ever changes
-   * for any reason, then take out the following clause and
-   * replace it with one that makes sure the ctl_fd is open.
-   */
+  /* We should never have to apply this operation to any procinfo
+     except the one for the main process.  If that ever changes for
+     any reason, then take out the following clause and replace it
+     with one that makes sure the ctl_fd is open.  */
 
   if (pi->tid != 0)
     pi = find_procinfo_or_die (pi->pid, 0);
@@ -2474,7 +2023,8 @@ proc_get_traced_sysexit (procinfo *pi, sysset_t *save)
       {
        int rsize;
 
-       if (lseek (pi->status_fd, (off_t) pi->prstatus.pr_sysexit_offset, SEEK_SET)
+       if (lseek (pi->status_fd, (off_t) pi->prstatus.pr_sysexit_offset,
+                  SEEK_SET)
            != (off_t) pi->prstatus.pr_sysexit_offset)
          return NULL;
        size = sysset_t_size (pi);
@@ -2499,25 +2049,19 @@ proc_get_traced_sysexit (procinfo *pi, sysset_t *save)
   return ret;
 }
 
-/*
- * Function: proc_clear_current_fault
- *
- * The current fault (if any) is cleared; the associated signal
- * will not be sent to the process or LWP when it resumes.
- * Returns non-zero for success,  zero for failure.
- */
+/* The current fault (if any) is cleared; the associated signal will
+   not be sent to the process or LWP when it resumes.  Returns
+   non-zero for success, zero for failure.  */
 
-int
+static int
 proc_clear_current_fault (procinfo *pi)
 {
   int win;
 
-  /*
-   * We should never have to apply this operation to any procinfo
-   * except the one for the main process.  If that ever changes
-   * for any reason, then take out the following clause and
-   * replace it with one that makes sure the ctl_fd is open.
-   */
+  /* We should never have to apply this operation to any procinfo
+     except the one for the main process.  If that ever changes for
+     any reason, then take out the following clause and replace it
+     with one that makes sure the ctl_fd is open.  */
 
   if (pi->tid != 0)
     pi = find_procinfo_or_die (pi->pid, 0);
@@ -2525,6 +2069,7 @@ proc_clear_current_fault (procinfo *pi)
 #ifdef NEW_PROC_API
   {
     procfs_ctl_t cmd = PCCFAULT;
+
     win = (write (pi->ctl_fd, (void *) &cmd, sizeof (cmd)) == sizeof (cmd));
   }
 #else
@@ -2534,20 +2079,15 @@ proc_clear_current_fault (procinfo *pi)
   return win;
 }
 
-/*
- * Function: proc_set_current_signal
- *
- * Set the "current signal" that will be delivered next to the process.
- * NOTE: semantics are different from those of KILL.
- * This signal will be delivered to the process or LWP
- * immediately when it is resumed (even if the signal is held/blocked);
- * it will NOT immediately cause another event of interest, and will NOT
- * first trap back to the debugger.
- *
- * Returns non-zero for success,  zero for failure.
- */
-
-int
+/* Set the "current signal" that will be delivered next to the
+   process.  NOTE: semantics are different from those of KILL.  This
+   signal will be delivered to the process or LWP immediately when it
+   is resumed (even if the signal is held/blocked); it will NOT
+   immediately cause another event of interest, and will NOT first
+   trap back to the debugger.  Returns non-zero for success, zero for
+   failure.  */
+
+static int
 proc_set_current_signal (procinfo *pi, int signo)
 {
   int win;
@@ -2560,31 +2100,29 @@ proc_set_current_signal (procinfo *pi, int signo)
   ptid_t wait_ptid;
   struct target_waitstatus wait_status;
 
-  /*
-   * We should never have to apply this operation to any procinfo
-   * except the one for the main process.  If that ever changes
-   * for any reason, then take out the following clause and
-   * replace it with one that makes sure the ctl_fd is open.
-   */
+  /* We should never have to apply this operation to any procinfo
+     except the one for the main process.  If that ever changes for
+     any reason, then take out the following clause and replace it
+     with one that makes sure the ctl_fd is open.  */
 
   if (pi->tid != 0)
     pi = find_procinfo_or_die (pi->pid, 0);
 
 #ifdef PROCFS_DONT_PIOCSSIG_CURSIG
   /* With Alpha OSF/1 procfs, the kernel gets really confused if it
-   * receives a PIOCSSIG with a signal identical to the current signal,
-   * it messes up the current signal. Work around the kernel bug.
-   */
+     receives a PIOCSSIG with a signal identical to the current
+     signal, it messes up the current signal.  Work around the kernel
+     bug.  */
   if (signo > 0 &&
       signo == proc_cursig (pi))
-    return 1;           /* I assume this is a success? */
+    return 1;           /* I assume this is a success?  */
 #endif
 
   /* The pointer is just a type alias.  */
   get_last_target_status (&wait_ptid, &wait_status);
   if (ptid_equal (wait_ptid, inferior_ptid)
       && wait_status.kind == TARGET_WAITKIND_STOPPED
-      && wait_status.value.sig == target_signal_from_host (signo)
+      && wait_status.value.sig == gdb_signal_from_host (signo)
       && proc_get_status (pi)
 #ifdef NEW_PROC_API
       && pi->prstatus.pr_lwp.pr_info.si_signo == signo
@@ -2618,25 +2156,19 @@ proc_set_current_signal (procinfo *pi, int signo)
   return win;
 }
 
-/*
- * Function: proc_clear_current_signal
- *
- * The current signal (if any) is cleared, and
- * is not sent to the process or LWP when it resumes.
- * Returns non-zero for success,  zero for failure.
- */
+/* The current signal (if any) is cleared, and is not sent to the
+   process or LWP when it resumes.  Returns non-zero for success, zero
+   for failure.  */
 
-int
+static int
 proc_clear_current_signal (procinfo *pi)
 {
   int win;
 
-  /*
-   * We should never have to apply this operation to any procinfo
-   * except the one for the main process.  If that ever changes
-   * for any reason, then take out the following clause and
-   * replace it with one that makes sure the ctl_fd is open.
-   */
+  /* We should never have to apply this operation to any procinfo
+     except the one for the main process.  If that ever changes for
+     any reason, then take out the following clause and replace it
+     with one that makes sure the ctl_fd is open.  */
 
   if (pi->tid != 0)
     pi = find_procinfo_or_die (pi->pid, 0);
@@ -2671,7 +2203,7 @@ proc_clear_current_signal (procinfo *pi)
 /* Return the general-purpose registers for the process or LWP
    corresponding to PI.  Upon failure, return NULL.  */
 
-gdb_gregset_t *
+static gdb_gregset_t *
 proc_get_gregs (procinfo *pi)
 {
   if (!pi->status_valid || !pi->gregs_valid)
@@ -2695,7 +2227,7 @@ proc_get_gregs (procinfo *pi)
 /* Return the general-purpose registers for the process or LWP
    corresponding to PI.  Upon failure, return NULL.  */
 
-gdb_fpregset_t *
+static gdb_fpregset_t *
 proc_get_fpregs (procinfo *pi)
 {
 #ifdef NEW_PROC_API
@@ -2768,7 +2300,7 @@ proc_get_fpregs (procinfo *pi)
    corresponding to PI.  Return non-zero for success, zero for
    failure.  */
 
-int
+static int
 proc_set_gregs (procinfo *pi)
 {
   gdb_gregset_t *gregs;
@@ -2808,7 +2340,7 @@ proc_set_gregs (procinfo *pi)
    corresponding to PI.  Return non-zero for success, zero for
    failure.  */
 
-int
+static int
 proc_set_fpregs (procinfo *pi)
 {
   gdb_fpregset_t *fpregs;
@@ -2863,22 +2395,16 @@ proc_set_fpregs (procinfo *pi)
   return win;
 }
 
-/*
- * Function: proc_kill
- *
- * Send a signal to the proc or lwp with the semantics of "kill()".
- * Returns non-zero for success,  zero for failure.
- */
+/* Send a signal to the proc or lwp with the semantics of "kill()".
+   Returns non-zero for success, zero for failure.  */
 
-int
+static int
 proc_kill (procinfo *pi, int signo)
 {
   int win;
 
-  /*
-   * We might conceivably apply this operation to an LWP, and
-   * the LWP's ctl file descriptor might not be open.
-   */
+  /* We might conceivably apply this operation to an LWP, and the
+     LWP's ctl file descriptor might not be open.  */
 
   if (pi->ctl_fd == 0 &&
       open_procinfo_files (pi, FD_CTL) == 0)
@@ -2895,7 +2421,7 @@ proc_kill (procinfo *pi, int signo)
       win = (write (pi->ctl_fd, (char *) &cmd, sizeof (cmd)) == sizeof (cmd));
 #else   /* ioctl method */
       /* FIXME: do I need the Alpha OSF fixups present in
-        procfs.c/unconditionally_kill_inferior?  Perhaps only for SIGKILL? */
+        procfs.c/unconditionally_kill_inferior?  Perhaps only for SIGKILL?  */
       win = (ioctl (pi->ctl_fd, PIOCKILL, &signo) >= 0);
 #endif
   }
@@ -2903,22 +2429,16 @@ proc_kill (procinfo *pi, int signo)
   return win;
 }
 
-/*
- * Function: proc_parent_pid
- *
- * Find the pid of the process that started this one.
- * Returns the parent process pid, or zero.
- */
+/* Find the pid of the process that started this one.  Returns the
+   parent process pid, or zero.  */
 
-int
+static int
 proc_parent_pid (procinfo *pi)
 {
-  /*
-   * We should never have to apply this operation to any procinfo
-   * except the one for the main process.  If that ever changes
-   * for any reason, then take out the following clause and
-   * replace it with one that makes sure the ctl_fd is open.
-   */
+  /* We should never have to apply this operation to any procinfo
+     except the one for the main process.  If that ever changes for
+     any reason, then take out the following clause and replace it
+     with one that makes sure the ctl_fd is open.  */
 
   if (pi->tid != 0)
     pi = find_procinfo_or_die (pi->pid, 0);
@@ -2948,12 +2468,7 @@ procfs_address_to_host_pointer (CORE_ADDR addr)
 }
 #endif
 
-/*
- * Function: proc_set_watchpoint
- *
- */
-
-int
+static int
 proc_set_watchpoint (procinfo *pi, CORE_ADDR addr, int len, int wflags)
 {
 #if !defined (PCWATCH) && !defined (PIOCSWATCH)
@@ -2962,8 +2477,8 @@ proc_set_watchpoint (procinfo *pi, CORE_ADDR addr, int len, int wflags)
      systems.  */
   return 0;
 #else
-/* Horrible hack!  Detect Solaris 2.5, because this doesn't work on 2.5 */
-#if defined (PIOCOPENLWP) || defined (UNIXWARE)        /* Solaris 2.5: bail out */
+/* Horrible hack!  Detect Solaris 2.5, because this doesn't work on 2.5 */
+#if defined (PIOCOPENLWP) || defined (UNIXWARE)        /* Solaris 2.5: bail out */
   return 0;
 #else
   struct {
@@ -3001,18 +2516,9 @@ proc_set_watchpoint (procinfo *pi, CORE_ADDR addr, int len, int wflags)
 
 #include <sys/sysi86.h>
 
-/*
- * Function: proc_get_LDT_entry
- *
- * Inputs:
- *   procinfo *pi;
- *   int key;
- *
- * The 'key' is actually the value of the lower 16 bits of
- * the GS register for the LWP that we're interested in.
- *
- * Return: matching ssh struct (LDT entry).
- */
+/* The KEY is actually the value of the lower 16 bits of the GS
+   register for the LWP that we're interested in.  Returns the
+   matching ssh struct (LDT entry).  */
 
 struct ssd *
 proc_get_LDT_entry (procinfo *pi, int key)
@@ -3035,7 +2541,7 @@ proc_get_LDT_entry (procinfo *pi, int key)
       proc_warn (pi, "proc_get_LDT_entry (open)", __LINE__);
       return NULL;
     }
-  /* Make sure it gets closed again! */
+  /* Make sure it gets closed again!  */
   old_chain = make_cleanup_close (fd);
 
   /* Now 'read' thru the table, find a match and return it.  */
@@ -3046,11 +2552,11 @@ proc_get_LDT_entry (procinfo *pi, int key)
          ldt_entry->acc1 == 0 &&
          ldt_entry->acc2 == 0)
        break;  /* end of table */
-      /* If key matches, return this entry. */
+      /* If key matches, return this entry.  */
       if (ldt_entry->sel == key)
        return ldt_entry;
     }
-  /* Loop ended, match not found. */
+  /* Loop ended, match not found.  */
   return NULL;
 #else
   int nldt, i;
@@ -3063,8 +2569,8 @@ proc_get_LDT_entry (procinfo *pi, int key)
       return NULL;
     }
 
-  /* Allocate space for the number of LDT entries. */
-  /* This alloc has to persist, 'cause we return a pointer to it. */
+  /* Allocate space for the number of LDT entries.  */
+  /* This alloc has to persist, 'cause we return a pointer to it.  */
   if (nldt > nalloc)
     {
       ldt_entry = (struct ssd *)
@@ -3079,25 +2585,17 @@ proc_get_LDT_entry (procinfo *pi, int key)
       return NULL;
     }
 
-  /* Search the table and return the (first) entry matching 'key'. */
+  /* Search the table and return the (first) entry matching 'key'.  */
   for (i = 0; i < nldt; i++)
     if (ldt_entry[i].sel == key)
       return &ldt_entry[i];
 
-  /* Loop ended, match not found. */
+  /* Loop ended, match not found.  */
   return NULL;
 #endif
 }
 
-/*
- * Function: procfs_find_LDT_entry
- *
- * Input:
- *   ptid_t ptid;      // The GDB-style pid-plus-LWP.
- *
- * Return:
- *   pointer to the corresponding LDT entry.
- */
+/* Returns the pointer to the LDT entry of PTID.  */
 
 struct ssd *
 procfs_find_LDT_entry (ptid_t ptid)
@@ -3106,24 +2604,24 @@ procfs_find_LDT_entry (ptid_t ptid)
   int            key;
   procinfo      *pi;
 
-  /* Find procinfo for the lwp. */
+  /* Find procinfo for the lwp.  */
   if ((pi = find_procinfo (PIDGET (ptid), TIDGET (ptid))) == NULL)
     {
       warning (_("procfs_find_LDT_entry: could not find procinfo for %d:%ld."),
               PIDGET (ptid), TIDGET (ptid));
       return NULL;
     }
-  /* get its general registers. */
+  /* get its general registers.  */
   if ((gregs = proc_get_gregs (pi)) == NULL)
     {
       warning (_("procfs_find_LDT_entry: could not read gregs for %d:%ld."),
               PIDGET (ptid), TIDGET (ptid));
       return NULL;
     }
-  /* Now extract the GS register's lower 16 bits. */
+  /* Now extract the GS register's lower 16 bits.  */
   key = (*gregs)[GS] & 0xffff;
 
-  /* Find the matching entry and return it. */
+  /* Find the matching entry and return it.  */
   return proc_get_LDT_entry (pi, key);
 }
 
@@ -3136,17 +2634,11 @@ procfs_find_LDT_entry (ptid_t ptid)
 /* NOTE: you'll see more ifdefs and duplication of functions here,
    since there is a different way to do threads on every OS.  */
 
-/*
- * Function: proc_get_nthreads
- *
- * Return the number of threads for the process
- */
+/* Returns the number of threads for the process.  */
 
 #if defined (PIOCNTHR) && defined (PIOCTLIST)
-/*
- * OSF version
- */
-int
+/* OSF version */
+static int
 proc_get_nthreads (procinfo *pi)
 {
   int nthreads = 0;
@@ -3159,32 +2651,26 @@ proc_get_nthreads (procinfo *pi)
 
 #else
 #if defined (SYS_lwpcreate) || defined (SYS_lwp_create) /* FIXME: multiple */
-/*
- * Solaris and Unixware version
- */
-int
+/* Solaris and Unixware version */
+static int
 proc_get_nthreads (procinfo *pi)
 {
   if (!pi->status_valid)
     if (!proc_get_status (pi))
       return 0;
 
-  /*
-   * NEW_PROC_API: only works for the process procinfo,
-   * because the LWP procinfos do not get prstatus filled in.
-   */
+  /* NEW_PROC_API: only works for the process procinfo, because the
+     LWP procinfos do not get prstatus filled in.  */
 #ifdef NEW_PROC_API
-  if (pi->tid != 0)    /* find the parent process procinfo */
+  if (pi->tid != 0)    /* Find the parent process procinfo.  */
     pi = find_procinfo_or_die (pi->pid, 0);
 #endif
   return pi->prstatus.pr_nlwp;
 }
 
 #else
-/*
- * Default version
- */
-int
+/* Default version */
+static int
 proc_get_nthreads (procinfo *pi)
 {
   return 0;
@@ -3192,28 +2678,22 @@ proc_get_nthreads (procinfo *pi)
 #endif
 #endif
 
-/*
- * Function: proc_get_current_thread (LWP version)
- *
- * Return the ID of the thread that had an event of interest.
- * (ie. the one that hit a breakpoint or other traced event).
- * All other things being equal, this should be the ID of a
- * thread that is currently executing.
- */
+/* LWP version.
+
+   Return the ID of the thread that had an event of interest.
+   (ie. the one that hit a breakpoint or other traced event).  All
+   other things being equal, this should be the ID of a thread that is
+   currently executing.  */
 
 #if defined (SYS_lwpcreate) || defined (SYS_lwp_create) /* FIXME: multiple */
-/*
- * Solaris and Unixware version
- */
-int
+/* Solaris and Unixware version */
+static int
 proc_get_current_thread (procinfo *pi)
 {
-  /*
-   * Note: this should be applied to the root procinfo for the process,
-   * not to the procinfo for an LWP.  If applied to the procinfo for
-   * an LWP, it will simply return that LWP's ID.  In that case,
-   * find the parent process procinfo.
-   */
+  /* Note: this should be applied to the root procinfo for the
+     process, not to the procinfo for an LWP.  If applied to the
+     procinfo for an LWP, it will simply return that LWP's ID.  In
+     that case, find the parent process procinfo.  */
 
   if (pi->tid != 0)
     pi = find_procinfo_or_die (pi->pid, 0);
@@ -3231,13 +2711,11 @@ proc_get_current_thread (procinfo *pi)
 
 #else
 #if defined (PIOCNTHR) && defined (PIOCTLIST)
-/*
- * OSF version
- */
-int
+/* OSF version */
+static int
 proc_get_current_thread (procinfo *pi)
 {
-#if 0  /* FIXME: not ready for prime time? */
+#if 0  /* FIXME: not ready for prime time?  */
   return pi->prstatus.pr_tid;
 #else
   return 0;
@@ -3245,10 +2723,8 @@ proc_get_current_thread (procinfo *pi)
 }
 
 #else
-/*
- * Default version
- */
-int
+/* Default version */
+static int
 proc_get_current_thread (procinfo *pi)
 {
   return 0;
@@ -3257,18 +2733,12 @@ proc_get_current_thread (procinfo *pi)
 #endif
 #endif
 
-/*
- * Function: proc_update_threads
- *
- * Discover the IDs of all the threads within the process, and
- * create a procinfo for each of them (chained to the parent).
- *
- * This unfortunately requires a different method on every OS.
- *
- * Return: non-zero for success, zero for failure.
- */
+/* Discover the IDs of all the threads within the process, and create
+   a procinfo for each of them (chained to the parent).  This
+   unfortunately requires a different method on every OS.  Returns
+   non-zero for success, zero for failure.  */
 
-int
+static int
 proc_delete_dead_threads (procinfo *parent, procinfo *thread, void *ignore)
 {
   if (thread && parent)        /* sanity */
@@ -3281,10 +2751,8 @@ proc_delete_dead_threads (procinfo *parent, procinfo *thread, void *ignore)
 }
 
 #if defined (PIOCLSTATUS)
-/*
- * Solaris 2.5 (ioctl) version
- */
-int
+/* Solaris 2.5 (ioctl) version */
+static int
 proc_update_threads (procinfo *pi)
 {
   gdb_prstatus_t *prstatus;
@@ -3292,12 +2760,10 @@ proc_update_threads (procinfo *pi)
   procinfo *thread;
   int nlwp, i;
 
-  /*
-   * We should never have to apply this operation to any procinfo
-   * except the one for the main process.  If that ever changes
-   * for any reason, then take out the following clause and
-   * replace it with one that makes sure the ctl_fd is open.
-   */
+  /* We should never have to apply this operation to any procinfo
+     except the one for the main process.  If that ever changes for
+     any reason, then take out the following clause and replace it
+     with one that makes sure the ctl_fd is open.  */
 
   if (pi->tid != 0)
     pi = find_procinfo_or_die (pi->pid, 0);
@@ -3313,7 +2779,7 @@ proc_update_threads (procinfo *pi)
   if (ioctl (pi->ctl_fd, PIOCLSTATUS, prstatus) < 0)
     proc_error (pi, "update_threads (PIOCLSTATUS)", __LINE__);
 
-  /* Skip element zero, which represents the process as a whole. */
+  /* Skip element zero, which represents the process as a whole.  */
   for (i = 1; i < nlwp + 1; i++)
     {
       if ((thread = create_procinfo (pi->pid, prstatus[i].pr_who)) == NULL)
@@ -3328,16 +2794,14 @@ proc_update_threads (procinfo *pi)
 }
 #else
 #ifdef NEW_PROC_API
-/*
- * Unixware and Solaris 6 (and later) version
- */
+/* Unixware and Solaris 6 (and later) version.  */
 static void
 do_closedir_cleanup (void *dir)
 {
   closedir (dir);
 }
 
-int
+static int
 proc_update_threads (procinfo *pi)
 {
   char pathname[MAX_PROC_NAME_SIZE + 16];
@@ -3347,27 +2811,23 @@ proc_update_threads (procinfo *pi)
   DIR *dirp;
   int lwpid;
 
-  /*
-   * We should never have to apply this operation to any procinfo
-   * except the one for the main process.  If that ever changes
-   * for any reason, then take out the following clause and
-   * replace it with one that makes sure the ctl_fd is open.
-   */
+  /* We should never have to apply this operation to any procinfo
+     except the one for the main process.  If that ever changes for
+     any reason, then take out the following clause and replace it
+     with one that makes sure the ctl_fd is open.  */
 
   if (pi->tid != 0)
     pi = find_procinfo_or_die (pi->pid, 0);
 
   proc_iterate_over_threads (pi, proc_delete_dead_threads, NULL);
 
-  /*
-   * Unixware
-   *
-   * Note: this brute-force method is the only way I know of
-   * to accomplish this task on Unixware.  This method will
-   * also work on Solaris 2.6 and 2.7.  There is a much simpler
-   * and more elegant way to do this on Solaris, but the margins
-   * of this manuscript are too small to write it here...  ;-)
-   */
+  /* Unixware
+
+     Note: this brute-force method is the only way I know of to
+     accomplish this task on Unixware.  This method will also work on
+     Solaris 2.6 and 2.7.  There is a much simpler and more elegant
+     way to do this on Solaris, but the margins of this manuscript are
+     too small to write it here...  ;-)  */
 
   strcpy (pathname, pi->pathname);
   strcat (pathname, "/lwp");
@@ -3388,21 +2848,17 @@ proc_update_threads (procinfo *pi)
 }
 #else
 #ifdef PIOCTLIST
-/*
- * OSF version
- */
-int
+/* OSF version */
+static int
 proc_update_threads (procinfo *pi)
 {
   int nthreads, i;
   tid_t *threads;
 
-  /*
-   * We should never have to apply this operation to any procinfo
-   * except the one for the main process.  If that ever changes
-   * for any reason, then take out the following clause and
-   * replace it with one that makes sure the ctl_fd is open.
-   */
+  /* We should never have to apply this operation to any procinfo
+     except the one for the main process.  If that ever changes for
+     any reason, then take out the following clause and replace it
+     with one that makes sure the ctl_fd is open.  */
 
   if (pi->tid != 0)
     pi = find_procinfo_or_die (pi->pid, 0);
@@ -3411,7 +2867,7 @@ proc_update_threads (procinfo *pi)
 
   nthreads = proc_get_nthreads (pi);
   if (nthreads < 2)
-    return 0;          /* nothing to do for 1 or fewer threads */
+    return 0;          /* Nothing to do for 1 or fewer threads.  */
 
   threads = xmalloc (nthreads * sizeof (tid_t));
 
@@ -3428,10 +2884,8 @@ proc_update_threads (procinfo *pi)
   return 1;
 }
 #else
-/*
- * Default version
- */
-int
+/* Default version */
+static int
 proc_update_threads (procinfo *pi)
 {
   return 0;
@@ -3440,30 +2894,20 @@ proc_update_threads (procinfo *pi)
 #endif  /* NEW_PROC_API   */
 #endif  /* SOL 2.5 PIOCLSTATUS */
 
-/*
- * Function: proc_iterate_over_threads
- *
- * Description:
- *   Given a pointer to a function, call that function once
- *   for each lwp in the procinfo list, until the function
- *   returns non-zero, in which event return the value
- *   returned by the function.
- *
- * Note: this function does NOT call update_threads.
- * If you want to discover new threads first, you must
- * call that function explicitly.  This function just makes
- * a quick pass over the currently-known procinfos.
- *
- * Arguments:
- *   pi                - parent process procinfo
- *   func      - per-thread function
- *   ptr       - opaque parameter for function.
- *
- * Return:
- *   First non-zero return value from the callee, or zero.
- */
-
-int
+/* Given a pointer to a function, call that function once for each lwp
+   in the procinfo list, until the function returns non-zero, in which
+   event return the value returned by the function.
+
+   Note: this function does NOT call update_threads.  If you want to
+   discover new threads first, you must call that function explicitly.
+   This function just makes a quick pass over the currently-known
+   procinfos.
+
+   PI is the parent process procinfo.  FUNC is the per-thread
+   function.  PTR is an opaque parameter for function.  Returns the
+   first non-zero return value from the callee, or zero.  */
+
+static int
 proc_iterate_over_threads (procinfo *pi,
                           int (*func) (procinfo *, procinfo *, void *),
                           void *ptr)
@@ -3471,19 +2915,17 @@ proc_iterate_over_threads (procinfo *pi,
   procinfo *thread, *next;
   int retval = 0;
 
-  /*
-   * We should never have to apply this operation to any procinfo
-   * except the one for the main process.  If that ever changes
-   * for any reason, then take out the following clause and
-   * replace it with one that makes sure the ctl_fd is open.
-   */
+  /* We should never have to apply this operation to any procinfo
+     except the one for the main process.  If that ever changes for
+     any reason, then take out the following clause and replace it
+     with one that makes sure the ctl_fd is open.  */
 
   if (pi->tid != 0)
     pi = find_procinfo_or_die (pi->pid, 0);
 
   for (thread = pi->thread_list; thread != NULL; thread = next)
     {
-      next = thread->next;     /* in case thread is destroyed */
+      next = thread->next;     /* In case thread is destroyed.  */
       if ((retval = (*func) (pi, thread, ptr)) != 0)
        break;
     }
@@ -3497,15 +2939,13 @@ proc_iterate_over_threads (procinfo *pi,
 
 /* ===================  GDB  "MODULE" =================== */
 
-/*
- * Here are all of the gdb target vector functions and their friends.
- */
+/* Here are all of the gdb target vector functions and their
+   friends.  */
 
 static ptid_t do_attach (ptid_t ptid);
 static void do_detach (int signo);
-static int register_gdb_signals (procinfo *, gdb_sigset_t *);
 static void proc_trace_syscalls_1 (procinfo *pi, int syscallnum,
-                                   int entry_or_exit, int mode, int from_tty);
+                                  int entry_or_exit, int mode, int from_tty);
 
 /* On mips-irix, we need to insert a breakpoint at __dbx_link during
    the startup phase.  The following two variables are used to record
@@ -3514,16 +2954,11 @@ static void proc_trace_syscalls_1 (procinfo *pi, int syscallnum,
 static int dbx_link_bpt_addr = 0;
 static void *dbx_link_bpt;
 
-/*
- * Function: procfs_debug_inferior
- *
- * Sets up the inferior to be debugged.
- * Registers to trace signals, hardware faults, and syscalls.
- * Note: does not set RLC flag: caller may want to customize that.
- *
- * Returns: zero for success (note! unlike most functions in this module)
- *   On failure, returns the LINE NUMBER where it failed!
- */
+/* Sets up the inferior to be debugged.  Registers to trace signals,
+   hardware faults, and syscalls.  Note: does not set RLC flag: caller
+   may want to customize that.  Returns zero for success (note!
+   unlike most functions in this module); on failure, returns the LINE
+   NUMBER where it failed!  */
 
 static int
 procfs_debug_inferior (procinfo *pi)
@@ -3537,19 +2972,19 @@ procfs_debug_inferior (procinfo *pi)
 #ifdef PROCFS_DONT_TRACE_FAULTS
   /* On some systems (OSF), we don't trace hardware faults.
      Apparently it's enough that we catch them as signals.
-     Wonder why we don't just do that in general? */
-  premptyset (&traced_faults);         /* don't trace faults. */
+     Wonder why we don't just do that in general?  */
+  premptyset (&traced_faults);         /* don't trace faults.  */
 #else
-  /* Register to trace hardware faults in the child. */
-  prfillset (&traced_faults);          /* trace all faults... */
+  /* Register to trace hardware faults in the child.  */
+  prfillset (&traced_faults);          /* trace all faults...  */
   gdb_prdelset  (&traced_faults, FLTPAGE);     /* except page fault.  */
 #endif
   if (!proc_set_traced_faults  (pi, &traced_faults))
     return __LINE__;
 
-  /* Register to trace selected signals in the child. */
-  premptyset (&traced_signals);
-  if (!register_gdb_signals (pi, &traced_signals))
+  /* Initially, register to trace all signals in the child.  */
+  prfillset (&traced_signals);
+  if (!proc_set_traced_signals (pi, &traced_signals))
     return __LINE__;
 
 
@@ -3560,7 +2995,7 @@ procfs_debug_inferior (procinfo *pi)
   gdb_praddsysset (traced_syscall_entries, SYS_exit);
 #endif
 #ifdef SYS_lwpexit
-  gdb_praddsysset (traced_syscall_entries, SYS_lwpexit);       /* And _lwp_exit... */
+  gdb_praddsysset (traced_syscall_entries, SYS_lwpexit);/* And _lwp_exit...  */
 #endif
 #ifdef SYS_lwp_exit
   gdb_praddsysset (traced_syscall_entries, SYS_lwp_exit);
@@ -3568,6 +3003,7 @@ procfs_debug_inferior (procinfo *pi)
 #ifdef DYNAMIC_SYSCALLS
   {
     int callnum = find_syscall (pi, "_exit");
+
     if (callnum >= 0)
       gdb_praddsysset (traced_syscall_entries, callnum);
   }
@@ -3582,7 +3018,7 @@ procfs_debug_inferior (procinfo *pi)
   /* OSF method for tracing exec syscalls.  Quoting:
      Under Alpha OSF/1 we have to use a PIOCSSPCACT ioctl to trace
      exits from exec system calls because of the user level loader.  */
-  /* FIXME: make nice and maybe move into an access function. */
+  /* FIXME: make nice and maybe move into an access function.  */
   {
     int prfs_flags;
 
@@ -3595,11 +3031,11 @@ procfs_debug_inferior (procinfo *pi)
       return __LINE__;
   }
 #else /* not PRFS_STOPEXEC */
-  /* Everyone else's (except OSF) method for tracing exec syscalls */
+  /* Everyone else's (except OSF) method for tracing exec syscalls */
   /* GW: Rationale...
      Not all systems with /proc have all the exec* syscalls with the same
      names.  On the SGI, for example, there is no SYS_exec, but there
-     *is* a SYS_execv.  So, we try to account for that. */
+     *is* a SYS_execv.  So, we try to account for that.  */
 
   traced_syscall_exits = sysset_t_alloc (pi);
   gdb_premptysysset (traced_syscall_exits);
@@ -3618,7 +3054,7 @@ procfs_debug_inferior (procinfo *pi)
   gdb_praddsysset (traced_syscall_exits, SYS_lwpexit);
 #endif
 
-#ifdef SYS_lwp_create  /* FIXME: once only, please */
+#ifdef SYS_lwp_create  /* FIXME: once only, please */
   gdb_praddsysset (traced_syscall_exits, SYS_lwp_create);
   gdb_praddsysset (traced_syscall_exits, SYS_lwp_exit);
 #endif
@@ -3626,6 +3062,7 @@ procfs_debug_inferior (procinfo *pi)
 #ifdef DYNAMIC_SYSCALLS
   {
     int callnum = find_syscall (pi, "execve");
+
     if (callnum >= 0)
       gdb_praddsysset (traced_syscall_exits, callnum);
     callnum = find_syscall (pi, "ra_execve");
@@ -3663,7 +3100,7 @@ procfs_attach (struct target_ops *ops, char *args, int from_tty)
                         exec_file, target_pid_to_str (pid_to_ptid (pid)));
       else
        printf_filtered (_("Attaching to %s\n"),
-                        target_pid_to_str (pid_to_ptid (pid)));
+                        target_pid_to_str (pid_to_ptid (pid)));
 
       fflush (stdout);
     }
@@ -3732,7 +3169,7 @@ do_attach (ptid_t ptid)
       if (!proc_set_run_on_last_close (pi))
        dead_procinfo (pi, "do_attach: couldn't set RLC.", NOKILL);
 
-      /* Now stop the process. */
+      /* Now stop the process.  */
       if (!proc_stop_process (pi))
        dead_procinfo (pi, "do_attach: couldn't stop the process.", NOKILL);
       pi->ignore_next_sigstop = 1;
@@ -3775,7 +3212,7 @@ do_detach (int signo)
 {
   procinfo *pi;
 
-  /* Find procinfo for the main process */
+  /* Find procinfo for the main process */
   pi = find_procinfo_or_die (PIDGET (inferior_ptid), 0); /* FIXME: threads */
   if (signo)
     if (!proc_set_current_signal (pi, signo))
@@ -3873,7 +3310,7 @@ procfs_fetch_registers (struct target_ops *ops,
    NOTE: Since the /proc interface will not read individual registers,
    we will cache these requests until the process is resumed, and only
    then write them back to the inferior process.
+
    FIXME: is that a really bad idea?  Have to think about cases where
    writing one register might affect the value of others, etc.  */
 
@@ -3923,7 +3360,6 @@ procfs_store_registers (struct target_ops *ops,
 static int
 syscall_is_lwp_exit (procinfo *pi, int scall)
 {
-
 #ifdef SYS_lwp_exit
   if (scall == SYS_lwp_exit)
     return 1;
@@ -4031,8 +3467,8 @@ dbx_link_addr (bfd *abfd)
       asymbol *sym = symbol_table[i];
 
       if ((sym->flags & BSF_GLOBAL)
-          && sym->name != NULL && strcmp (sym->name, "__dbx_link") == 0)
-        return (sym->value + sym->section->vma);
+         && sym->name != NULL && strcmp (sym->name, "__dbx_link") == 0)
+       return (sym->value + sym->section->vma);
     }
 
   /* Symbol not found, return NULL.  */
@@ -4040,7 +3476,7 @@ dbx_link_addr (bfd *abfd)
 }
 
 /* Search the symbol table of the file referenced by FD for a symbol
-   named __dbx_link(). If found, then insert a breakpoint at this location,
+   named __dbx_link().  If found, then insert a breakpoint at this location,
    and return nonzero.  Return zero otherwise.  */
 
 static int
@@ -4050,7 +3486,7 @@ insert_dbx_link_bpt_in_file (int fd, CORE_ADDR ignored)
   long storage_needed;
   CORE_ADDR sym_addr;
 
-  abfd = bfd_fdopenr ("unamed", 0, fd);
+  abfd = gdb_bfd_fdopenr ("unamed", 0, fd);
   if (abfd == NULL)
     {
       warning (_("Failed to create a bfd: %s."), bfd_errmsg (bfd_get_error ()));
@@ -4060,8 +3496,8 @@ insert_dbx_link_bpt_in_file (int fd, CORE_ADDR ignored)
   if (!bfd_check_format (abfd, bfd_object))
     {
       /* Not the correct format, so we can not possibly find the dbx_link
-         symbol in it.  */
-      bfd_close (abfd);
+        symbol in it.  */
+      gdb_bfd_unref (abfd);
       return 0;
     }
 
@@ -4073,28 +3509,26 @@ insert_dbx_link_bpt_in_file (int fd, CORE_ADDR ignored)
       dbx_link_bpt = deprecated_insert_raw_breakpoint (target_gdbarch, NULL,
                                                       sym_addr);
       if (dbx_link_bpt == NULL)
-        {
-          warning (_("Failed to insert dbx_link breakpoint."));
-          bfd_close (abfd);
-          return 0;
-        }
-      bfd_close (abfd);
+       {
+         warning (_("Failed to insert dbx_link breakpoint."));
+         gdb_bfd_unref (abfd);
+         return 0;
+       }
+      gdb_bfd_unref (abfd);
       return 1;
     }
 
-  bfd_close (abfd);
+  gdb_bfd_unref (abfd);
   return 0;
 }
 
 /* Calls the supplied callback function once for each mapped address
-   space in the process.  The callback function  receives an open
-   file descriptor for the file corresponding to that mapped
-   address space (if there is one), and the base address of the
-   mapped space.  Quit when the callback function returns a
-   nonzero value, or at teh end of the mappings.
-
-   Returns: the first non-zero return value of the callback function,
-   or zero.  */
+   space in the process.  The callback function receives an open file
+   descriptor for the file corresponding to that mapped address space
+   (if there is one), and the base address of the mapped space.  Quit
+   when the callback function returns a nonzero value, or at teh end
+   of the mappings.  Returns the first non-zero return value of the
+   callback function, or zero.  */
 
 static int
 solib_mappings_callback (struct prmap *map, int (*func) (int, CORE_ADDR),
@@ -4139,8 +3573,8 @@ solib_mappings_callback (struct prmap *map, int (*func) (int, CORE_ADDR),
 
 static int
 insert_dbx_link_bpt_in_region (struct prmap *map,
-                               iterate_over_mappings_cb_ftype *child_func,
-                               void *data)
+                              find_memory_region_ftype child_func,
+                              void *data)
 {
   procinfo *pi = (procinfo *) data;
 
@@ -4163,23 +3597,17 @@ insert_dbx_link_breakpoint (procinfo *pi)
 }
 #endif
 
-/*
- * Function: target_wait
- *
- * Retrieve the next stop event from the child process.
- * If child has not stopped yet, wait for it to stop.
- * Translate /proc eventcodes (or possibly wait eventcodes)
- * into gdb internal event codes.
- *
- * Return: id of process (and possibly thread) that incurred the event.
- *         event codes are returned thru a pointer parameter.
- */
+/* Retrieve the next stop event from the child process.  If child has
+   not stopped yet, wait for it to stop.  Translate /proc eventcodes
+   (or possibly wait eventcodes) into gdb internal event codes.
+   Returns the id of process (and possibly thread) that incurred the
+   event.  Event codes are returned through a pointer parameter.  */
 
 static ptid_t
 procfs_wait (struct target_ops *ops,
             ptid_t ptid, struct target_waitstatus *status, int options)
 {
-  /* First cut: loosely based on original version 2.1 */
+  /* First cut: loosely based on original version 2.1 */
   procinfo *pi;
   int       wstat;
   int       temp_tid;
@@ -4193,45 +3621,46 @@ wait_again:
   wstat    = 0;
   retval   = pid_to_ptid (-1);
 
-  /* Find procinfo for main process */
+  /* Find procinfo for main process */
   pi = find_procinfo_or_die (PIDGET (inferior_ptid), 0);
   if (pi)
     {
-      /* We must assume that the status is stale now... */
+      /* We must assume that the status is stale now...  */
       pi->status_valid = 0;
       pi->gregs_valid  = 0;
       pi->fpregs_valid = 0;
 
-#if 0  /* just try this out... */
+#if 0  /* just try this out...  */
       flags = proc_flags (pi);
       why   = proc_why (pi);
       if ((flags & PR_STOPPED) && (why == PR_REQUESTED))
-       pi->status_valid = 0;   /* re-read again, IMMEDIATELY... */
+       pi->status_valid = 0;   /* re-read again, IMMEDIATELY...  */
 #endif
       /* If child is not stopped, wait for it to stop.  */
       if (!(proc_flags (pi) & (PR_STOPPED | PR_ISTOP)) &&
          !proc_wait_for_stop (pi))
        {
-         /* wait_for_stop failed: has the child terminated? */
+         /* wait_for_stop failed: has the child terminated?  */
          if (errno == ENOENT)
            {
              int wait_retval;
 
-             /* /proc file not found; presumably child has terminated. */
-             wait_retval = wait (&wstat); /* "wait" for the child's exit  */
+             /* /proc file not found; presumably child has terminated.  */
+             wait_retval = wait (&wstat); /* "wait" for the child's exit.  */
 
-             if (wait_retval != PIDGET (inferior_ptid)) /* wrong child? */
-               error (_("procfs: couldn't stop process %d: wait returned %d."),
+             if (wait_retval != PIDGET (inferior_ptid)) /* wrong child?  */
+               error (_("procfs: couldn't stop "
+                        "process %d: wait returned %d."),
                       PIDGET (inferior_ptid), wait_retval);
              /* FIXME: might I not just use waitpid?
-                Or try find_procinfo to see if I know about this child? */
+                Or try find_procinfo to see if I know about this child?  */
              retval = pid_to_ptid (wait_retval);
            }
          else if (errno == EINTR)
            goto wait_again;
          else
            {
-             /* Unknown error from wait_for_stop. */
+             /* Unknown error from wait_for_stop.  */
              proc_error (pi, "target_wait (wait_for_stop)", __LINE__);
            }
        }
@@ -4286,7 +3715,7 @@ wait_again:
                  {
                    struct inferior *inf;
 
-                   /* Handle SYS_exit call only */
+                   /* Handle SYS_exit call only */
                    /* Stopped at entry to SYS_exit.
                       Make it runnable, resume it, then use
                       the wait system call to get its exit code.
@@ -4320,7 +3749,8 @@ wait_again:
                           somehow, I could get its termination event
                           here.  */
 
-                       /* If wait returns -1, that's what we return to GDB. */
+                       /* If wait returns -1, that's what we return
+                          to GDB.  */
                        if (temp < 0)
                          retval = pid_to_ptid (temp);
                      }
@@ -4337,7 +3767,8 @@ wait_again:
                      if ((nsysargs = proc_nsysarg (pi)) > 0 &&
                          (sysargs  = proc_sysargs (pi)) != NULL)
                        {
-                         printf_filtered (_("%ld syscall arguments:\n"), nsysargs);
+                         printf_filtered (_("%ld syscall arguments:\n"),
+                                          nsysargs);
                          for (i = 0; i < nsysargs; i++)
                            printf_filtered ("#%ld: 0x%08lx\n",
                                             i, sysargs[i]);
@@ -4347,14 +3778,15 @@ wait_again:
 #endif
                    if (status)
                      {
-                       /* How to exit gracefully, returning "unknown event" */
+                       /* How to exit gracefully, returning "unknown
+                          event".  */
                        status->kind = TARGET_WAITKIND_SPURIOUS;
                        return inferior_ptid;
                      }
                    else
                      {
                        /* How to keep going without returning to wfi: */
-                       target_resume (ptid, 0, TARGET_SIGNAL_0);
+                       target_resume (ptid, 0, GDB_SIGNAL_0);
                        goto wait_again;
                      }
                  }
@@ -4365,35 +3797,34 @@ wait_again:
                    /* Hopefully this is our own "fork-child" execing
                       the real child.  Hoax this event into a trap, and
                       GDB will see the child about to execute its start
-                      address. */
+                      address.  */
                    wstat = (SIGTRAP << 8) | 0177;
                  }
 #ifdef SYS_syssgi
-                else if (what == SYS_syssgi)
-                  {
-                    /* see if we can break on dbx_link().  If yes, then
-                       we no longer need the SYS_syssgi notifications.  */
-                    if (insert_dbx_link_breakpoint (pi))
-                      proc_trace_syscalls_1 (pi, SYS_syssgi, PR_SYSEXIT,
-                                             FLAG_RESET, 0);
-
-                    /* This is an internal event and should be transparent
-                       to wfi, so resume the execution and wait again.  See
-                       comment in procfs_init_inferior() for more details.  */
-                    target_resume (ptid, 0, TARGET_SIGNAL_0);
-                    goto wait_again;
-                  }
+               else if (what == SYS_syssgi)
+                 {
+                   /* see if we can break on dbx_link().  If yes, then
+                      we no longer need the SYS_syssgi notifications.  */
+                   if (insert_dbx_link_breakpoint (pi))
+                     proc_trace_syscalls_1 (pi, SYS_syssgi, PR_SYSEXIT,
+                                            FLAG_RESET, 0);
+
+                   /* This is an internal event and should be transparent
+                      to wfi, so resume the execution and wait again.  See
+                      comment in procfs_init_inferior() for more details.  */
+                   target_resume (ptid, 0, GDB_SIGNAL_0);
+                   goto wait_again;
+                 }
 #endif
                else if (syscall_is_lwp_create (pi, what))
                  {
-                   /*
-                    * This syscall is somewhat like fork/exec.
-                    * We will get the event twice: once for the parent LWP,
-                    * and once for the child.  We should already know about
-                    * the parent LWP, but the child will be new to us.  So,
-                    * whenever we get this event, if it represents a new
-                    * thread, simply add the thread to the list.
-                    */
+                   /* This syscall is somewhat like fork/exec.  We
+                      will get the event twice: once for the parent
+                      LWP, and once for the child.  We should already
+                      know about the parent LWP, but the child will
+                      be new to us.  So, whenever we get this event,
+                      if it represents a new thread, simply add the
+                      thread to the list.  */
 
                    /* If not in procinfo list, add it.  */
                    temp_tid = proc_get_current_thread (pi);
@@ -4405,7 +3836,7 @@ wait_again:
                    if (!in_thread_list (temp_ptid))
                      add_thread (temp_ptid);
 
-                   /* Return to WFI, but tell it to immediately resume. */
+                   /* Return to WFI, but tell it to immediately resume.  */
                    status->kind = TARGET_WAITKIND_SPURIOUS;
                    return inferior_ptid;
                  }
@@ -4439,7 +3870,8 @@ wait_again:
                      if ((nsysargs = proc_nsysarg (pi)) > 0 &&
                          (sysargs  = proc_sysargs (pi)) != NULL)
                        {
-                         printf_filtered (_("%ld syscall arguments:\n"), nsysargs);
+                         printf_filtered (_("%ld syscall arguments:\n"),
+                                          nsysargs);
                          for (i = 0; i < nsysargs; i++)
                            printf_filtered ("#%ld: 0x%08lx\n",
                                             i, sysargs[i]);
@@ -4493,43 +3925,44 @@ wait_again:
                  wstat = (SIGTRAP << 8) | 0177;
                  break;
 #endif
-                 /* FIXME: use si_signo where possible. */
+                 /* FIXME: use si_signo where possible.  */
                case FLTPRIV:
-#if (FLTILL != FLTPRIV)                /* avoid "duplicate case" error */
+#if (FLTILL != FLTPRIV)                /* Avoid "duplicate case" error.  */
                case FLTILL:
 #endif
                  wstat = (SIGILL << 8) | 0177;
                  break;
                case FLTBPT:
-#if (FLTTRACE != FLTBPT)       /* avoid "duplicate case" error */
+#if (FLTTRACE != FLTBPT)       /* Avoid "duplicate case" error.  */
                case FLTTRACE:
 #endif
-                  /* If we hit our __dbx_link() internal breakpoint,
-                     then remove it.  See comments in procfs_init_inferior()
-                     for more details.  */
-                  if (dbx_link_bpt_addr != 0
-                      && dbx_link_bpt_addr
+                 /* If we hit our __dbx_link() internal breakpoint,
+                    then remove it.  See comments in procfs_init_inferior()
+                    for more details.  */
+                 if (dbx_link_bpt_addr != 0
+                     && dbx_link_bpt_addr
                         == regcache_read_pc (get_current_regcache ()))
-                    remove_dbx_link_breakpoint ();
+                   remove_dbx_link_breakpoint ();
 
                  wstat = (SIGTRAP << 8) | 0177;
                  break;
                case FLTSTACK:
                case FLTACCESS:
-#if (FLTBOUNDS != FLTSTACK)    /* avoid "duplicate case" error */
+#if (FLTBOUNDS != FLTSTACK)    /* Avoid "duplicate case" error.  */
                case FLTBOUNDS:
 #endif
                  wstat = (SIGSEGV << 8) | 0177;
                  break;
                case FLTIOVF:
                case FLTIZDIV:
-#if (FLTFPE != FLTIOVF)                /* avoid "duplicate case" error */
+#if (FLTFPE != FLTIOVF)                /* Avoid "duplicate case" error.  */
                case FLTFPE:
 #endif
                  wstat = (SIGFPE << 8) | 0177;
                  break;
-               case FLTPAGE:           /* Recoverable page fault */
-               default:         /* FIXME: use si_signo if possible for fault */
+               case FLTPAGE:   /* Recoverable page fault */
+               default:        /* FIXME: use si_signo if possible for
+                                  fault.  */
                  retval = pid_to_ptid (-1);
                  printf_filtered ("procfs:%d -- ", __LINE__);
                  printf_filtered (_("child stopped for unknown reason:\n"));
@@ -4545,28 +3978,23 @@ wait_again:
                error (_("... giving up..."));
                break;
              }
-             /*
-              * Got this far without error:
-              * If retval isn't in the threads database, add it.
-              */
+             /* Got this far without error: If retval isn't in the
+                threads database, add it.  */
              if (PIDGET (retval) > 0 &&
                  !ptid_equal (retval, inferior_ptid) &&
                  !in_thread_list (retval))
                {
-                 /*
-                  * We have a new thread.
-                  * We need to add it both to GDB's list and to our own.
-                  * If we don't create a procinfo, resume may be unhappy
-                  * later.
-                  */
+                 /* We have a new thread.  We need to add it both to
+                    GDB's list and to our own.  If we don't create a
+                    procinfo, resume may be unhappy later.  */
                  add_thread (retval);
                  if (find_procinfo (PIDGET (retval), TIDGET (retval)) == NULL)
                    create_procinfo (PIDGET (retval), TIDGET (retval));
                }
            }
-         else  /* flags do not indicate STOPPED */
+         else  /* Flags do not indicate STOPPED.  */
            {
-             /* surely this can't happen... */
+             /* surely this can't happen...  */
              printf_filtered ("procfs:%d -- process not stopped.\n",
                               __LINE__);
              proc_prettyprint_flags (flags, 1);
@@ -4623,7 +4051,7 @@ procfs_xfer_partial (struct target_ops *ops, enum target_object object,
    transferred.  Otherwise, it will be a positive value which
    indicates the number of bytes transferred between gdb and the
    target.  (Note that the interface also makes provisions for
-   negative values, but this capability isn't implemented here.) */
+   negative values, but this capability isn't implemented here.)  */
 
 static int
 procfs_xfer_memory (CORE_ADDR memaddr, gdb_byte *myaddr, int len, int dowrite,
@@ -4632,7 +4060,7 @@ procfs_xfer_memory (CORE_ADDR memaddr, gdb_byte *myaddr, int len, int dowrite,
   procinfo *pi;
   int nbytes = 0;
 
-  /* Find procinfo for main process */
+  /* Find procinfo for main process */
   pi = find_procinfo_or_die (PIDGET (inferior_ptid), 0);
   if (pi->as_fd == 0 &&
       open_procinfo_files (pi, FD_AS) == 0)
@@ -4646,15 +4074,15 @@ procfs_xfer_memory (CORE_ADDR memaddr, gdb_byte *myaddr, int len, int dowrite,
       if (dowrite)
        {
 #ifdef NEW_PROC_API
-         PROCFS_NOTE ("write memory: ");
+         PROCFS_NOTE ("write memory:\n");
 #else
-         PROCFS_NOTE ("write memory: \n");
+         PROCFS_NOTE ("write memory:\n");
 #endif
          nbytes = write (pi->as_fd, myaddr, len);
        }
       else
        {
-         PROCFS_NOTE ("read  memory: \n");
+         PROCFS_NOTE ("read  memory:\n");
          nbytes = read (pi->as_fd, myaddr, len);
        }
       if (nbytes < 0)
@@ -4665,33 +4093,26 @@ procfs_xfer_memory (CORE_ADDR memaddr, gdb_byte *myaddr, int len, int dowrite,
   return nbytes;
 }
 
-/*
- * Function: invalidate_cache
- *
- * Called by target_resume before making child runnable.
- * Mark cached registers and status's invalid.
- * If there are "dirty" caches that need to be written back
- * to the child process, do that.
- *
- * File descriptors are also cached.
- * As they are a limited resource, we cannot hold onto them indefinitely.
- * However, as they are expensive to open, we don't want to throw them
- * away indescriminately either.  As a compromise, we will keep the
- * file descriptors for the parent process, but discard any file
- * descriptors we may have accumulated for the threads.
- *
- * Return value:
- * As this function is called by iterate_over_threads, it always
- * returns zero (so that iterate_over_threads will keep iterating).
- */
+/* Called by target_resume before making child runnable.  Mark cached
+   registers and status's invalid.  If there are "dirty" caches that
+   need to be written back to the child process, do that.
 
+   File descriptors are also cached.  As they are a limited resource,
+   we cannot hold onto them indefinitely.  However, as they are
+   expensive to open, we don't want to throw them away
+   indescriminately either.  As a compromise, we will keep the file
+   descriptors for the parent process, but discard any file
+   descriptors we may have accumulated for the threads.
+
+   As this function is called by iterate_over_threads, it always
+   returns zero (so that iterate_over_threads will keep
+   iterating).  */
 
 static int
 invalidate_cache (procinfo *parent, procinfo *pi, void *ptr)
 {
-  /*
-   * About to run the child; invalidate caches and do any other cleanup.
-   */
+  /* About to run the child; invalidate caches and do any other
+     cleanup.  */
 
 #if 0
   if (pi->gregs_dirty)
@@ -4730,13 +4151,9 @@ invalidate_cache (procinfo *parent, procinfo *pi, void *ptr)
 }
 
 #if 0
-/*
- * Function: make_signal_thread_runnable
- *
- * A callback function for iterate_over_threads.
- * Find the asynchronous signal thread, and make it runnable.
- * See if that helps matters any.
- */
+/* A callback function for iterate_over_threads.  Find the
+   asynchronous signal thread, and make it runnable.  See if that
+   helps matters any.  */
 
 static int
 make_signal_thread_runnable (procinfo *process, procinfo *pi, void *ptr)
@@ -4753,26 +4170,19 @@ make_signal_thread_runnable (procinfo *process, procinfo *pi, void *ptr)
 }
 #endif
 
-/*
- * Function: target_resume
- *
- * Make the child process runnable.  Normally we will then call
- * procfs_wait and wait for it to stop again (unles gdb is async).
- *
- * Arguments:
- *  step:  if true, then arrange for the child to stop again
- *         after executing a single instruction.
- *  signo: if zero, then cancel any pending signal.
- *         If non-zero, then arrange for the indicated signal
- *         to be delivered to the child when it runs.
- *  pid:   if -1, then allow any child thread to run.
- *         if non-zero, then allow only the indicated thread to run.
- *******   (not implemented yet)
- */
+/* Make the child process runnable.  Normally we will then call
+   procfs_wait and wait for it to stop again (unless gdb is async).
+
+   If STEP is true, then arrange for the child to stop again after
+   executing a single instruction.  If SIGNO is zero, then cancel any
+   pending signal; if non-zero, then arrange for the indicated signal
+   to be delivered to the child when it runs.  If PID is -1, then
+   allow any child thread to run; if non-zero, then allow only the
+   indicated thread to run.  (not implemented yet).  */
 
 static void
 procfs_resume (struct target_ops *ops,
-              ptid_t ptid, int step, enum target_signal signo)
+              ptid_t ptid, int step, enum gdb_signal signo)
 {
   procinfo *pi, *thread;
   int native_signo;
@@ -4785,46 +4195,46 @@ procfs_resume (struct target_ops *ops,
      prrun.prflags |= PRCFAULT;    clear current fault.
 
      PRSTRACE and PRSFAULT can be done by other means
-       (proc_trace_signals, proc_trace_faults)
+       (proc_trace_signals, proc_trace_faults)
      PRSVADDR is unnecessary.
      PRCFAULT may be replaced by a PIOCCFAULT call (proc_clear_current_fault)
      This basically leaves PRSTEP and PRCSIG.
      PRCSIG is like PIOCSSIG (proc_clear_current_signal).
      So basically PR_STEP is the sole argument that must be passed
-     to proc_run_process (for use in the prrun struct by ioctl). */
+     to proc_run_process (for use in the prrun struct by ioctl).  */
 
-  /* Find procinfo for main process */
+  /* Find procinfo for main process */
   pi = find_procinfo_or_die (PIDGET (inferior_ptid), 0);
 
-  /* First cut: ignore pid argument */
+  /* First cut: ignore pid argument */
   errno = 0;
 
   /* Convert signal to host numbering.  */
   if (signo == 0 ||
-      (signo == TARGET_SIGNAL_STOP && pi->ignore_next_sigstop))
+      (signo == GDB_SIGNAL_STOP && pi->ignore_next_sigstop))
     native_signo = 0;
   else
-    native_signo = target_signal_to_host (signo);
+    native_signo = gdb_signal_to_host (signo);
 
   pi->ignore_next_sigstop = 0;
 
-  /* Running the process voids all cached registers and status. */
-  /* Void the threads' caches first */
+  /* Running the process voids all cached registers and status.  */
+  /* Void the threads' caches first */
   proc_iterate_over_threads (pi, invalidate_cache, NULL);
   /* Void the process procinfo's caches.  */
   invalidate_cache (NULL, pi, NULL);
 
   if (PIDGET (ptid) != -1)
     {
-      /* Resume a specific thread, presumably suppressing the others. */
+      /* Resume a specific thread, presumably suppressing the
+        others.  */
       thread = find_procinfo (PIDGET (ptid), TIDGET (ptid));
       if (thread != NULL)
        {
          if (thread->tid != 0)
            {
-             /* We're to resume a specific thread, and not the others.
-              * Set the child process's PR_ASYNC flag.
-              */
+             /* We're to resume a specific thread, and not the
+                others.  Set the child process's PR_ASYNC flag.  */
 #ifdef PR_ASYNC
              if (!proc_set_async (pi))
                proc_error (pi, "target_resume, set_async", __LINE__);
@@ -4834,7 +4244,8 @@ procfs_resume (struct target_ops *ops,
                                         make_signal_thread_runnable,
                                         NULL);
 #endif
-             pi = thread;      /* substitute the thread's procinfo for run */
+             pi = thread;      /* Substitute the thread's procinfo
+                                  for run.  */
            }
        }
     }
@@ -4842,80 +4253,50 @@ procfs_resume (struct target_ops *ops,
   if (!proc_run_process (pi, step, native_signo))
     {
       if (errno == EBUSY)
-       warning (_("resume: target already running.  Pretend to resume, and hope for the best!"));
+       warning (_("resume: target already running.  "
+                  "Pretend to resume, and hope for the best!"));
       else
        proc_error (pi, "target_resume", __LINE__);
     }
 }
 
-/*
- * Function: register_gdb_signals
- *
- * Traverse the list of signals that GDB knows about
- * (see "handle" command), and arrange for the target
- * to be stopped or not, according to these settings.
- *
- * Returns non-zero for success, zero for failure.
- */
+/* Set up to trace signals in the child process.  */
 
-static int
-register_gdb_signals (procinfo *pi, gdb_sigset_t *signals)
+static void
+procfs_pass_signals (int numsigs, unsigned char *pass_signals)
 {
+  gdb_sigset_t signals;
+  procinfo *pi = find_procinfo_or_die (PIDGET (inferior_ptid), 0);
   int signo;
 
-  for (signo = 0; signo < NSIG; signo ++)
-    if (signal_stop_state  (target_signal_from_host (signo)) == 0 &&
-       signal_print_state (target_signal_from_host (signo)) == 0 &&
-       signal_pass_state  (target_signal_from_host (signo)) == 1)
-      gdb_prdelset (signals, signo);
-    else
-      gdb_praddset (signals, signo);
-
-  return proc_set_traced_signals (pi, signals);
-}
-
-/*
- * Function: target_notice_signals
- *
- * Set up to trace signals in the child process.
- */
+  prfillset (&signals);
 
-static void
-procfs_notice_signals (ptid_t ptid)
-{
-  gdb_sigset_t signals;
-  procinfo *pi = find_procinfo_or_die (PIDGET (ptid), 0);
+  for (signo = 0; signo < NSIG; signo++)
+    {
+      int target_signo = gdb_signal_from_host (signo);
+      if (target_signo < numsigs && pass_signals[target_signo])
+       gdb_prdelset (&signals, signo);
+    }
 
-  if (proc_get_traced_signals (pi, &signals) &&
-      register_gdb_signals    (pi, &signals))
-    return;
-  else
-    proc_error (pi, "notice_signals", __LINE__);
+  if (!proc_set_traced_signals (pi, &signals))
+    proc_error (pi, "pass_signals", __LINE__);
 }
 
-/*
- * Function: target_files_info
- *
- * Print status information about the child process.
- */
+/* Print status information about the child process.  */
 
 static void
 procfs_files_info (struct target_ops *ignore)
 {
   struct inferior *inf = current_inferior ();
+
   printf_filtered (_("\tUsing the running image of %s %s via /proc.\n"),
                   inf->attach_flag? "attached": "child",
                   target_pid_to_str (inferior_ptid));
 }
 
-/*
- * Function: target_stop
- *
- * Stop the child process asynchronously, as when the
- * gdb user types control-c or presses a "stop" button.
- *
- * Works by sending kill(SIGINT) to the child's process group.
- */
+/* Stop the child process asynchronously, as when the gdb user types
+   control-c or presses a "stop" button.  Works by sending
+   kill(SIGINT) to the child's process group.  */
 
 static void
 procfs_stop (ptid_t ptid)
@@ -4923,14 +4304,10 @@ procfs_stop (ptid_t ptid)
   kill (-inferior_process_group (), SIGINT);
 }
 
-/*
- * Function: unconditionally_kill_inferior
- *
- * Make it die.  Wait for it to die.  Clean up after it.
- * Note: this should only be applied to the real process,
- * not to an LWP, because of the check for parent-process.
- * If we need this to work for an LWP, it needs some more logic.
- */
+/* Make it die.  Wait for it to die.  Clean up after it.  Note: this
+   should only be applied to the real process, not to an LWP, because
+   of the check for parent-process.  If we need this to work for an
+   LWP, it needs some more logic.  */
 
 static void
 unconditionally_kill_inferior (procinfo *pi)
@@ -4939,7 +4316,7 @@ unconditionally_kill_inferior (procinfo *pi)
 
   parent_pid = proc_parent_pid (pi);
 #ifdef PROCFS_NEED_CLEAR_CURSIG_FOR_KILL
-  /* FIXME: use access functions */
+  /* FIXME: use access functions */
   /* Alpha OSF/1-3.x procfs needs a clear of the current signal
      before the PIOCKILL, otherwise it might generate a corrupted core
      file for the inferior.  */
@@ -4963,7 +4340,7 @@ unconditionally_kill_inferior (procinfo *pi)
     newsiginfo.si_errno = 0;
     newsiginfo.si_pid = getpid ();
     newsiginfo.si_uid = getuid ();
-    /* FIXME: use proc_set_current_signal */
+    /* FIXME: use proc_set_current_signal */
     ioctl (pi->ctl_fd, PIOCSSIG, &newsiginfo);
   }
 #else /* PROCFS_NEED_PIOCSSIG_FOR_KILL */
@@ -4987,19 +4364,15 @@ unconditionally_kill_inferior (procinfo *pi)
     }
 }
 
-/*
- * Function: target_kill_inferior
- *
- * We're done debugging it, and we want it to go away.
- * Then we want GDB to forget all about it.
- */
+/* We're done debugging it, and we want it to go away.  Then we want
+   GDB to forget all about it.  */
 
 static void
 procfs_kill_inferior (struct target_ops *ops)
 {
   if (!ptid_equal (inferior_ptid, null_ptid)) /* ? */
     {
-      /* Find procinfo for main process */
+      /* Find procinfo for main process */
       procinfo *pi = find_procinfo (PIDGET (inferior_ptid), 0);
 
       if (pi)
@@ -5008,11 +4381,7 @@ procfs_kill_inferior (struct target_ops *ops)
     }
 }
 
-/*
- * Function: target_mourn_inferior
- *
- * Forget we ever debugged this thing!
- */
+/* Forget we ever debugged this thing!  */
 
 static void
 procfs_mourn_inferior (struct target_ops *ops)
@@ -5021,7 +4390,7 @@ procfs_mourn_inferior (struct target_ops *ops)
 
   if (!ptid_equal (inferior_ptid, null_ptid))
     {
-      /* Find procinfo for main process */
+      /* Find procinfo for main process */
       pi = find_procinfo (PIDGET (inferior_ptid), 0);
       if (pi)
        destroy_procinfo (pi);
@@ -5038,14 +4407,10 @@ procfs_mourn_inferior (struct target_ops *ops)
   generic_mourn_inferior ();
 }
 
-/*
- * Function: init_inferior
- *
- * When GDB forks to create a runnable inferior process,
- * this function is called on the parent side of the fork.
- * It's job is to do whatever is necessary to make the child
- * ready to be debugged, and then wait for the child to synchronize.
- */
+/* When GDB forks to create a runnable inferior process, this function
+   is called on the parent side of the fork.  It's job is to do
+   whatever is necessary to make the child ready to be debugged, and
+   then wait for the child to synchronize.  */
 
 static void
 procfs_init_inferior (struct target_ops *ops, int pid)
@@ -5060,7 +4425,7 @@ procfs_init_inferior (struct target_ops *ops, int pid)
   push_target (ops);
 
   if ((pi = create_procinfo (pid, 0)) == NULL)
-    perror ("procfs: out of memory in 'init_inferior'");
+    perror (_("procfs: out of memory in 'init_inferior'"));
 
   if (!open_procinfo_files (pi, FD_CTL))
     proc_error (pi, "init_inferior, open_proc_files", __LINE__);
@@ -5077,14 +4442,14 @@ procfs_init_inferior (struct target_ops *ops, int pid)
     PIOCSFAULT
     */
 
-  /* If not stopped yet, wait for it to stop. */
+  /* If not stopped yet, wait for it to stop.  */
   if (!(proc_flags (pi) & PR_STOPPED) &&
       !(proc_wait_for_stop (pi)))
     dead_procinfo (pi, "init_inferior: wait_for_stop failed", KILL);
 
   /* Save some of the /proc state to be restored if we detach.  */
   /* FIXME: Why?  In case another debugger was debugging it?
-     We're it's parent, for Ghu's sake! */
+     We're it's parent, for Ghu's sake!  */
   if (!proc_get_traced_signals  (pi, &pi->saved_sigset))
     proc_error (pi, "init_inferior, get_traced_signals", __LINE__);
   if (!proc_get_held_signals    (pi, &pi->saved_sighold))
@@ -5096,11 +4461,6 @@ procfs_init_inferior (struct target_ops *ops, int pid)
   if (!proc_get_traced_sysexit  (pi, pi->saved_exitset))
     proc_error (pi, "init_inferior, get_traced_sysexit", __LINE__);
 
-  /* Register to trace selected signals in the child. */
-  prfillset (&signals);
-  if (!register_gdb_signals (pi, &signals))
-    proc_error (pi, "init_inferior, register_signals", __LINE__);
-
   if ((fail = procfs_debug_inferior (pi)) != 0)
     proc_error (pi, "init_inferior (procfs_debug_inferior)", fail);
 
@@ -5140,10 +4500,10 @@ procfs_init_inferior (struct target_ops *ops, int pid)
      So what we need to do is to insert a breakpoint in the runtime
      loader (rld), more precisely in __dbx_link().  This procedure is
      called by rld once all shared libraries have been mapped, but before
-     the -init code is executed. Unfortuantely, this is not straightforward,
+     the -init code is executed.  Unfortuantely, this is not straightforward,
      as rld is not part of the executable we are running, and thus we need
      the inferior to run until rld itself has been mapped in memory.
-     
+
      For this, we trace all syssgi() syscall exit events.  Each time
      we detect such an event, we iterate over each text memory maps,
      get its associated fd, and scan the symbol table for __dbx_link().
@@ -5155,17 +4515,13 @@ procfs_init_inferior (struct target_ops *ops, int pid)
 #endif
 }
 
-/*
- * Function: set_exec_trap
- *
- * When GDB forks to create a new process, this function is called
- * on the child side of the fork before GDB exec's the user program.
- * Its job is to make the child minimally debuggable, so that the
- * parent GDB process can connect to the child and take over.
- * This function should do only the minimum to make that possible,
- * and to synchronize with the parent process.  The parent process
- * should take care of the details.
- */
+/* When GDB forks to create a new process, this function is called on
+   the child side of the fork before GDB exec's the user program.  Its
+   job is to make the child minimally debuggable, so that the parent
+   GDB process can connect to the child and take over.  This function
+   should do only the minimum to make that possible, and to
+   synchronize with the parent process.  The parent process should
+   take care of the details.  */
 
 static void
 procfs_set_exec_trap (void)
@@ -5184,7 +4540,8 @@ procfs_set_exec_trap (void)
     {
       proc_warn (pi, "set_exec_trap, open_proc_files", __LINE__);
       gdb_flush (gdb_stderr);
-      /* no need to call "dead_procinfo", because we're going to exit. */
+      /* No need to call "dead_procinfo", because we're going to
+        exit.  */
       _exit (127);
     }
 
@@ -5192,7 +4549,7 @@ procfs_set_exec_trap (void)
   /* OSF method for tracing exec syscalls.  Quoting:
      Under Alpha OSF/1 we have to use a PIOCSSPCACT ioctl to trace
      exits from exec system calls because of the user level loader.  */
-  /* FIXME: make nice and maybe move into an access function. */
+  /* FIXME: make nice and maybe move into an access function.  */
   {
     int prfs_flags;
 
@@ -5212,11 +4569,11 @@ procfs_set_exec_trap (void)
       }
   }
 #else /* not PRFS_STOPEXEC */
-  /* Everyone else's (except OSF) method for tracing exec syscalls */
+  /* Everyone else's (except OSF) method for tracing exec syscalls */
   /* GW: Rationale...
      Not all systems with /proc have all the exec* syscalls with the same
      names.  On the SGI, for example, there is no SYS_exec, but there
-     *is* a SYS_execv.  So, we try to account for that. */
+     *is* a SYS_execv.  So, we try to account for that.  */
 
   exitset = sysset_t_alloc (pi);
   gdb_premptysysset (exitset);
@@ -5250,7 +4607,7 @@ procfs_set_exec_trap (void)
     }
 #endif /* PRFS_STOPEXEC */
 
-  /* FIXME: should this be done in the parent instead? */
+  /* FIXME: should this be done in the parent instead?  */
   /* Turn off inherit on fork flag so that all grand-children
      of gdb start with tracing flags cleared.  */
   if (!proc_unset_inherit_on_fork (pi))
@@ -5263,24 +4620,20 @@ procfs_set_exec_trap (void)
     proc_warn (pi, "set_exec_trap, unset_RLC", __LINE__);
 
   /* FIXME: No need to destroy the procinfo --
-     we have our own address space, and we're about to do an exec! */
+     we have our own address space, and we're about to do an exec!  */
   /*destroy_procinfo (pi);*/
 }
 
-/*
- * Function: create_inferior
- *
- * This function is called BEFORE gdb forks the inferior process.
- * Its only real responsibility is to set things up for the fork,
- * and tell GDB which two functions to call after the fork (one
- * for the parent, and one for the child).
- *
- * This function does a complicated search for a unix shell program,
- * which it then uses to parse arguments and environment variables
- * to be sent to the child.  I wonder whether this code could not
- * be abstracted out and shared with other unix targets such as
- * infptrace?
- */
+/* This function is called BEFORE gdb forks the inferior process.  Its
+   only real responsibility is to set things up for the fork, and tell
+   GDB which two functions to call after the fork (one for the parent,
+   and one for the child).
+
+   This function does a complicated search for a unix shell program,
+   which it then uses to parse arguments and environment variables to
+   be sent to the child.  I wonder whether this code could not be
+   abstracted out and shared with other unix targets such as
+   inf-ptrace?  */
 
 static void
 procfs_create_inferior (struct target_ops *ops, char *exec_file,
@@ -5359,7 +4712,7 @@ procfs_create_inferior (struct target_ops *ops, char *exec_file,
     }
 
   pid = fork_inferior (exec_file, allargs, env, procfs_set_exec_trap,
-                      NULL, NULL, shell_file);
+                      NULL, NULL, shell_file, NULL);
 
   procfs_init_inferior (ops, pid);
 }
@@ -5370,7 +4723,7 @@ static void
 procfs_inferior_created (struct target_ops *ops, int from_tty)
 {
 #ifdef SYS_syssgi
-  /* Make sure to cancel the syssgi() syscall-exit notifications.  
+  /* Make sure to cancel the syssgi() syscall-exit notifications.
      They should normally have been removed by now, but they may still
      be activated if the inferior doesn't use shared libraries, or if
      we didn't locate __dbx_link, or if we never stopped in __dbx_link.
@@ -5384,16 +4737,11 @@ procfs_inferior_created (struct target_ops *ops, int from_tty)
     return;
 
   proc_trace_syscalls_1 (find_procinfo_or_die (PIDGET (inferior_ptid), 0),
-                         SYS_syssgi, PR_SYSEXIT, FLAG_RESET, 0);
+                        SYS_syssgi, PR_SYSEXIT, FLAG_RESET, 0);
 #endif
 }
 
-/*
- * Function: notice_thread
- *
- * Callback for find_new_threads.
- * Calls "add_thread".
- */
+/* Callback for find_new_threads.  Calls "add_thread".  */
 
 static int
 procfs_notice_thread (procinfo *pi, procinfo *thread, void *ptr)
@@ -5406,32 +4754,23 @@ procfs_notice_thread (procinfo *pi, procinfo *thread, void *ptr)
   return 0;
 }
 
-/*
- * Function: target_find_new_threads
- *
- * Query all the threads that the target knows about,
- * and give them back to GDB to add to its list.
- */
+/* Query all the threads that the target knows about, and give them
+   back to GDB to add to its list.  */
 
-void
+static void
 procfs_find_new_threads (struct target_ops *ops)
 {
   procinfo *pi;
 
-  /* Find procinfo for main process */
+  /* Find procinfo for main process */
   pi = find_procinfo_or_die (PIDGET (inferior_ptid), 0);
   proc_update_threads (pi);
   proc_iterate_over_threads (pi, procfs_notice_thread, NULL);
 }
 
-/*
- * Function: target_thread_alive
- *
- * Return true if the thread is still 'alive'.
- *
- * This guy doesn't really seem to be doing his job.
- * Got to investigate how to tell when a thread is really gone.
- */
+/* Return true if the thread is still 'alive'.  This guy doesn't
+   really seem to be doing his job.  Got to investigate how to tell
+   when a thread is really gone.  */
 
 static int
 procfs_thread_alive (struct target_ops *ops, ptid_t ptid)
@@ -5441,7 +4780,7 @@ procfs_thread_alive (struct target_ops *ops, ptid_t ptid)
 
   proc    = PIDGET (ptid);
   thread  = TIDGET (ptid);
-  /* If I don't know it, it ain't alive! */
+  /* If I don't know it, it ain't alive!  */
   if ((pi = find_procinfo (proc, thread)) == NULL)
     return 0;
 
@@ -5452,13 +4791,15 @@ procfs_thread_alive (struct target_ops *ops, ptid_t ptid)
       destroy_procinfo (pi);
       return 0;
     }
-  /* I couldn't have got its status if it weren't alive, so it's alive.  */
+  /* I couldn't have got its status if it weren't alive, so it's
+     alive.  */
   return 1;
 }
 
-/* Convert PTID to a string.  Returns the string in a static buffer.  */
+/* Convert PTID to a string.  Returns the string in a static
+   buffer.  */
 
-char *
+static char *
 procfs_pid_to_str (struct target_ops *ops, ptid_t ptid)
 {
   static char buf[80];
@@ -5471,14 +4812,11 @@ procfs_pid_to_str (struct target_ops *ops, ptid_t ptid)
   return buf;
 }
 
-/*
- * Function: procfs_set_watchpoint
- * Insert a watchpoint
- */
+/* Insert a watchpoint.  */
 
-int
+static int
 procfs_set_watchpoint (ptid_t ptid, CORE_ADDR addr, int len, int rwflag,
-                       int after)
+                      int after)
 {
 #ifndef UNIXWARE
 #ifndef AIX5
@@ -5488,10 +4826,10 @@ procfs_set_watchpoint (ptid_t ptid, CORE_ADDR addr, int len, int rwflag,
   pi = find_procinfo_or_die (PIDGET (ptid) == -1 ?
                             PIDGET (inferior_ptid) : PIDGET (ptid), 0);
 
-  /* Translate from GDB's flags to /proc's */
-  if (len > 0) /* len == 0 means delete watchpoint */
+  /* Translate from GDB's flags to /proc's */
+  if (len > 0) /* len == 0 means delete watchpoint */
     {
-      switch (rwflag) {                /* FIXME: need an enum! */
+      switch (rwflag) {                /* FIXME: need an enum!  */
       case hw_write:           /* default watchpoint (write) */
        pflags = WRITE_WATCHFLAG;
        break;
@@ -5504,16 +4842,16 @@ procfs_set_watchpoint (ptid_t ptid, CORE_ADDR addr, int len, int rwflag,
       case hw_execute:         /* execution HW breakpoint */
        pflags = EXEC_WATCHFLAG;
        break;
-      default:                 /* Something weird.  Return error. */
+      default:                 /* Something weird.  Return error.  */
        return -1;
       }
-      if (after)               /* Stop after r/w access is completed. */
+      if (after)               /* Stop after r/w access is completed.  */
        pflags |= AFTER_WATCHFLAG;
     }
 
   if (!proc_set_watchpoint (pi, addr, len, pflags))
     {
-      if (errno == E2BIG)      /* Typical error for no resources */
+      if (errno == E2BIG)      /* Typical error for no resources */
        return -1;              /* fail */
       /* GDB may try to remove the same watchpoint twice.
         If a remove request returns no match, don't error.  */
@@ -5547,6 +4885,7 @@ procfs_can_use_hw_breakpoint (int type, int cnt, int othertype)
      will be generated when the host and target pointer sizes are
      different.  */
   struct type *ptr_type = builtin_type (target_gdbarch)->builtin_data_ptr;
+
   if (sizeof (void *) != TYPE_LENGTH (ptr_type))
     return 0;
 
@@ -5555,12 +4894,8 @@ procfs_can_use_hw_breakpoint (int type, int cnt, int othertype)
   return 1;
 }
 
-/*
- * Function: stopped_by_watchpoint
- *
- * Returns non-zero if process is stopped on a hardware watchpoint fault,
- * else returns zero.
- */
+/* Returns non-zero if process is stopped on a hardware watchpoint
+   fault, else returns zero.  */
 
 static int
 procfs_stopped_by_watchpoint (void)
@@ -5586,16 +4921,11 @@ procfs_stopped_by_watchpoint (void)
   return 0;
 }
 
-/*
- * Function procfs_stopped_data_address
- * 
- * Returns 1 if we the OS knows the position of the triggered
- * watchpoint.  Sets *ADDR to that address.
- * Returns 0 if OS cannot report that address.
- * This function is only called if procfs_stopped_by_watchpoint
- * returned 1, thus no further checks are done.
- * The function also assumes that ADDR is not NULL.
- */
+/* Returns 1 if the OS knows the position of the triggered watchpoint,
+   and sets *ADDR to that address.  Returns 0 if OS cannot report that
+   address.  This function is only called if
+   procfs_stopped_by_watchpoint returned 1, thus no further checks are
+   done.  The function also assumes that ADDR is not NULL.  */
 
 static int
 procfs_stopped_data_address (struct target_ops *targ, CORE_ADDR *addr)
@@ -5607,7 +4937,8 @@ procfs_stopped_data_address (struct target_ops *targ, CORE_ADDR *addr)
 }
 
 static int
-procfs_insert_watchpoint (CORE_ADDR addr, int len, int type)
+procfs_insert_watchpoint (CORE_ADDR addr, int len, int type,
+                         struct expression *cond)
 {
   if (!target_have_steppable_watchpoint
       && !gdbarch_have_nonsteppable_watchpoint (target_gdbarch))
@@ -5628,7 +4959,8 @@ procfs_insert_watchpoint (CORE_ADDR addr, int len, int type)
 }
 
 static int
-procfs_remove_watchpoint (CORE_ADDR addr, int len, int type)
+procfs_remove_watchpoint (CORE_ADDR addr, int len, int type,
+                         struct expression *cond)
 {
   return procfs_set_watchpoint (inferior_ptid, addr, 0, 0, 0);
 }
@@ -5655,34 +4987,25 @@ procfs_use_watchpoints (struct target_ops *t)
   t->to_stopped_data_address = procfs_stopped_data_address;
 }
 
-/*
- * Memory Mappings Functions:
- */
-
-/*
- * Function: iterate_over_mappings
- *
- * Call a callback function once for each mapping, passing it the mapping,
- * an optional secondary callback function, and some optional opaque data.
- * Quit and return the first non-zero value returned from the callback.
- *
- * Arguments:
- *   pi   -- procinfo struct for the process to be mapped.
- *   func -- callback function to be called by this iterator.
- *   data -- optional opaque data to be passed to the callback function.
- *   child_func -- optional secondary function pointer to be passed
- *                 to the child function.
- *
- * Return: First non-zero return value from the callback function,
- *         or zero.
- */
+/* Memory Mappings Functions: */
+
+/* Call a callback function once for each mapping, passing it the
+   mapping, an optional secondary callback function, and some optional
+   opaque data.  Quit and return the first non-zero value returned
+   from the callback.
+
+   PI is the procinfo struct for the process to be mapped.  FUNC is
+   the callback function to be called by this iterator.  DATA is the
+   optional opaque data to be passed to the callback function.
+   CHILD_FUNC is the optional secondary function pointer to be passed
+   to the child function.  Returns the first non-zero return value
+   from the callback function, or zero.  */
 
 static int
-iterate_over_mappings (procinfo *pi,
-                      iterate_over_mappings_cb_ftype *child_func,
+iterate_over_mappings (procinfo *pi, find_memory_region_ftype child_func,
                       void *data,
                       int (*func) (struct prmap *map,
-                                   iterate_over_mappings_cb_ftype *child_func,
+                                   find_memory_region_ftype child_func,
                                    void *data))
 {
   char pathname[MAX_PROC_NAME_SIZE];
@@ -5691,6 +5014,7 @@ iterate_over_mappings (procinfo *pi,
   int funcstat;
   int map_fd;
   int nmap;
+  struct cleanup *cleanups = make_cleanup (null_cleanup, NULL);
 #ifdef NEW_PROC_API
   struct stat sbuf;
 #endif
@@ -5698,12 +5022,12 @@ iterate_over_mappings (procinfo *pi,
   /* Get the number of mappings, allocate space,
      and read the mappings into prmaps.  */
 #ifdef NEW_PROC_API
-  /* Open map fd. */
+  /* Open map fd.  */
   sprintf (pathname, "/proc/%d/map", pi->pid);
   if ((map_fd = open (pathname, O_RDONLY)) < 0)
     proc_error (pi, "iterate_over_mappings (open)", __LINE__);
 
-  /* Make sure it gets closed again. */
+  /* Make sure it gets closed again.  */
   make_cleanup_close (map_fd);
 
   /* Use stat to determine the file size, and compute
@@ -5728,33 +5052,22 @@ iterate_over_mappings (procinfo *pi,
 
   for (prmap = prmaps; nmap > 0; prmap++, nmap--)
     if ((funcstat = (*func) (prmap, child_func, data)) != 0)
-      return funcstat;
+      {
+       do_cleanups (cleanups);
+        return funcstat;
+      }
 
+  do_cleanups (cleanups);
   return 0;
 }
 
-/*
- * Function: find_memory_regions_callback
- *
- * Implements the to_find_memory_regions method.
- * Calls an external function for each memory region.
- * External function will have the signiture:
- *
- *   int callback (CORE_ADDR vaddr,
- *                 unsigned long size,
- *                 int read, int write, int execute,
- *                 void *data);
- *
- * Returns the integer value returned by the callback.
- */
+/* Implements the to_find_memory_regions method.  Calls an external
+   function for each memory region.
+   Returns the integer value returned by the callback.  */
 
 static int
 find_memory_regions_callback (struct prmap *map,
-                             int (*func) (CORE_ADDR,
-                                          unsigned long,
-                                          int, int, int,
-                                          void *),
-                             void *data)
+                             find_memory_region_ftype func, void *data)
 {
   return (*func) ((CORE_ADDR) map->pr_vaddr,
                  map->pr_size,
@@ -5764,27 +5077,20 @@ find_memory_regions_callback (struct prmap *map,
                  data);
 }
 
-/*
- * Function: proc_find_memory_regions
- *
- * External interface.  Calls a callback function once for each
- * mapped memory region in the child process, passing as arguments
- *     CORE_ADDR virtual_address,
- *     unsigned long size,
- *     int read,       TRUE if region is readable by the child
- *     int write,      TRUE if region is writable by the child
- *     int execute     TRUE if region is executable by the child.
- *
- * Stops iterating and returns the first non-zero value
- * returned by the callback.
- */
+/* External interface.  Calls a callback function once for each
+   mapped memory region in the child process, passing as arguments:
+
+       CORE_ADDR virtual_address,
+       unsigned long size,
+       int read,       TRUE if region is readable by the child
+       int write,      TRUE if region is writable by the child
+       int execute     TRUE if region is executable by the child.
+
+   Stops iterating and returns the first non-zero value returned by
+   the callback.  */
 
 static int
-proc_find_memory_regions (int (*func) (CORE_ADDR,
-                                      unsigned long,
-                                      int, int, int,
-                                      void *),
-                         void *data)
+proc_find_memory_regions (find_memory_region_ftype func, void *data)
 {
   procinfo *pi = find_procinfo_or_die (PIDGET (inferior_ptid), 0);
 
@@ -5792,11 +5098,7 @@ proc_find_memory_regions (int (*func) (CORE_ADDR,
                                find_memory_regions_callback);
 }
 
-/*
- * Function: mappingflags
- *
- * Returns an ascii representation of a memory mapping's flags.
- */
+/* Returns an ascii representation of a memory mapping's flags.  */
 
 static char *
 mappingflags (long flags)
@@ -5823,15 +5125,11 @@ mappingflags (long flags)
   return (asciiflags);
 }
 
-/*
- * Function: info_mappings_callback
- *
- * Callback function, does the actual work for 'info proc mappings'.
- */
+/* Callback function, does the actual work for 'info proc
+   mappings'.  */
 
 static int
-info_mappings_callback (struct prmap *map,
-                       iterate_over_mappings_cb_ftype *ignore,
+info_mappings_callback (struct prmap *map, find_memory_region_ftype ignore,
                        void *unused)
 {
   unsigned int pr_off;
@@ -5860,17 +5158,13 @@ info_mappings_callback (struct prmap *map,
   return 0;
 }
 
-/*
- * Function: info_proc_mappings
- *
- * Implement the "info proc mappings" subcommand.
- */
+/* Implement the "info proc mappings" subcommand.  */
 
 static void
 info_proc_mappings (procinfo *pi, int summary)
 {
   if (summary)
-    return;    /* No output for summary mode. */
+    return;    /* No output for summary mode.  */
 
   printf_filtered (_("Mapped address spaces:\n\n"));
   if (gdbarch_ptr_bit (target_gdbarch) == 32)
@@ -5892,14 +5186,11 @@ info_proc_mappings (procinfo *pi, int summary)
   printf_filtered ("\n");
 }
 
-/*
- * Function: info_proc_cmd
- *
- * Implement the "info proc" command.
- */
+/* Implement the "info proc" command.  */
 
 static void
-info_proc_cmd (char *args, int from_tty)
+procfs_info_proc (struct target_ops *ops, char *args,
+                 enum info_proc_what what)
 {
   struct cleanup *old_chain;
   procinfo *process  = NULL;
@@ -5910,6 +5201,20 @@ info_proc_cmd (char *args, int from_tty)
   int       tid      = 0;
   int       mappings = 0;
 
+  switch (what)
+    {
+    case IP_MINIMAL:
+      break;
+
+    case IP_MAPPINGS:
+    case IP_ALL:
+      mappings = 1;
+      break;
+
+    default:
+      error (_("Not supported on this target."));
+    }
+
   old_chain = make_cleanup (null_cleanup, 0);
   if (args)
     {
@@ -5928,14 +5233,6 @@ info_proc_cmd (char *args, int from_tty)
        {
          tid = strtoul (argv[0] + 1, NULL, 10);
        }
-      else if (strncmp (argv[0], "mappings", strlen (argv[0])) == 0)
-       {
-         mappings = 1;
-       }
-      else
-       {
-         /* [...] */
-       }
       argv++;
     }
   if (pid == 0)
@@ -5945,7 +5242,7 @@ info_proc_cmd (char *args, int from_tty)
   else
     {
       /* Have pid, will travel.
-        First see if it's a process we're already debugging. */
+        First see if it's a process we're already debugging.  */
       process = find_procinfo (pid, 0);
        if (process == NULL)
         {
@@ -5990,17 +5287,17 @@ info_proc_cmd (char *args, int from_tty)
    the set of syscalls that are currently traced/debugged.
 
    If ENTRY_OR_EXIT is set to PR_SYSENTRY, then the entry syscalls set
-   will be updated. Otherwise, the exit syscalls set will be updated.
+   will be updated.  Otherwise, the exit syscalls set will be updated.
 
-   If MODE is FLAG_SET, then traces will be enabled. Otherwise, they
+   If MODE is FLAG_SET, then traces will be enabled.  Otherwise, they
    will be disabled.  */
 
 static void
 proc_trace_syscalls_1 (procinfo *pi, int syscallnum, int entry_or_exit,
-                      int mode, int from_tty)
+                      int mode, int from_tty)
 {
   sysset_t *sysset;
-  
+
   if (entry_or_exit == PR_SYSENTRY)
     sysset = proc_get_traced_sysentry (pi, NULL);
   else
@@ -6017,12 +5314,12 @@ proc_trace_syscalls_1 (procinfo *pi, int syscallnum, int entry_or_exit,
   if (entry_or_exit == PR_SYSENTRY)
     {
       if (!proc_set_traced_sysentry (pi, sysset))
-        proc_error (pi, "proc-trace, set_traced_sysentry", __LINE__);
+       proc_error (pi, "proc-trace, set_traced_sysentry", __LINE__);
     }
   else
     {
       if (!proc_set_traced_sysexit (pi, sysset))
-        proc_error (pi, "proc-trace, set_traced_sysexit", __LINE__);
+       proc_error (pi, "proc-trace, set_traced_sysexit", __LINE__);
     }
 }
 
@@ -6071,15 +5368,14 @@ proc_untrace_sysexit_cmd (char *args, int from_tty)
 }
 
 
+/* Provide a prototype to silence -Wmissing-prototypes.  */
+extern void _initialize_procfs (void);
+
 void
 _initialize_procfs (void)
 {
   observer_attach_inferior_created (procfs_inferior_created);
 
-  add_info ("proc", info_proc_cmd, _("\
-Show /proc process information about any running process.\n\
-Specify process id, or use the program being debugged by default.\n\
-Specify keyword 'mappings' for detailed info on memory mappings."));
   add_com ("proc-trace-entry", no_class, proc_trace_sysentry_cmd,
           _("Give a trace of entries into the syscall."));
   add_com ("proc-trace-exit", no_class, proc_trace_sysexit_cmd,
@@ -6094,15 +5390,13 @@ Specify keyword 'mappings' for detailed info on memory mappings."));
 
 
 
-/* miscellaneous stubs:                                             */
-/* The following satisfy a few random symbols mostly created by    */
-/* the solaris threads implementation, which I will chase down     */
-/* later.        */
+/* miscellaneous stubs: */
 
-/*
- * Return a pid for which we guarantee
- * we will be able to find a 'live' procinfo.
- */
+/* The following satisfy a few random symbols mostly created by the
+   solaris threads implementation, which I will chase down later.  */
+
+/* Return a pid for which we guarantee we will be able to find a
+   'live' procinfo.  */
 
 ptid_t
 procfs_first_available (void)
@@ -6117,7 +5411,7 @@ procfs_first_available (void)
 static char *
 procfs_do_thread_registers (bfd *obfd, ptid_t ptid,
                            char *note_data, int *note_size,
-                           enum target_signal stop_signal)
+                           enum gdb_signal stop_signal)
 {
   struct regcache *regcache = get_thread_regcache (ptid);
   gdb_gregset_t gregs;
@@ -6130,7 +5424,7 @@ procfs_do_thread_registers (bfd *obfd, ptid_t ptid,
   /* This part is the old method for fetching registers.
      It should be replaced by the newer one using regsets
      once it is implemented in this platform:
-     gdbarch_regset_from_core_section() and regset->collect_regset(). */
+     gdbarch_regset_from_core_section() and regset->collect_regset().  */
 
   old_chain = save_inferior_ptid ();
   inferior_ptid = ptid;
@@ -6168,7 +5462,7 @@ struct procfs_corefile_thread_data {
   bfd *obfd;
   char *note_data;
   int *note_size;
-  enum target_signal stop_signal;
+  enum gdb_signal stop_signal;
 };
 
 static int
@@ -6179,6 +5473,7 @@ procfs_corefile_thread_callback (procinfo *pi, procinfo *thread, void *data)
   if (pi != NULL)
     {
       ptid_t ptid = MERGEPID (pi->pid, thread->tid);
+
       args->note_data = procfs_do_thread_registers (args->obfd, ptid,
                                                    args->note_data,
                                                    args->note_size,
@@ -6190,23 +5485,23 @@ procfs_corefile_thread_callback (procinfo *pi, procinfo *thread, void *data)
 static int
 find_signalled_thread (struct thread_info *info, void *data)
 {
-  if (info->stop_signal != TARGET_SIGNAL_0
+  if (info->suspend.stop_signal != GDB_SIGNAL_0
       && ptid_get_pid (info->ptid) == ptid_get_pid (inferior_ptid))
     return 1;
 
   return 0;
 }
 
-static enum target_signal
+static enum gdb_signal
 find_stop_signal (void)
 {
   struct thread_info *info =
     iterate_over_threads (find_signalled_thread, NULL);
 
   if (info)
-    return info->stop_signal;
+    return info->suspend.stop_signal;
   else
-    return TARGET_SIGNAL_0;
+    return GDB_SIGNAL_0;
 }
 
 static char *
@@ -6223,13 +5518,14 @@ procfs_make_note_section (bfd *obfd, int *note_size)
   struct procfs_corefile_thread_data thread_args;
   gdb_byte *auxv;
   int auxv_len;
-  enum target_signal stop_signal;
+  enum gdb_signal stop_signal;
 
   if (get_exec_file (0))
     {
-      strncpy (fname, strrchr (get_exec_file (0), '/') + 1, sizeof (fname));
-      strncpy (psargs, get_exec_file (0),
-              sizeof (psargs));
+      strncpy (fname, lbasename (get_exec_file (0)), sizeof (fname));
+      fname[sizeof (fname) - 1] = 0;
+      strncpy (psargs, get_exec_file (0), sizeof (psargs));
+      psargs[sizeof (psargs) - 1] = 0;
 
       inf_args = get_inferior_args ();
       if (inf_args && *inf_args &&
@@ -6261,7 +5557,8 @@ procfs_make_note_section (bfd *obfd, int *note_size)
   thread_args.note_data = note_data;
   thread_args.note_size = note_size;
   thread_args.stop_signal = stop_signal;
-  proc_iterate_over_threads (pi, procfs_corefile_thread_callback, &thread_args);
+  proc_iterate_over_threads (pi, procfs_corefile_thread_callback,
+                            &thread_args);
 
   /* There should be always at least one thread.  */
   gdb_assert (thread_args.note_data != note_data);