Thu Feb 5 11:57:06 1998 Michael Snyder (msnyder@cleaver.cygnus.com)
[deliverable/binutils-gdb.git] / gdb / infrun.c
index d08a1bf09420f6a2c95f074f37788248676f9a51..2e1dc95f63a1e05c87ff566d70cd3790971df4f1 100644 (file)
-/* Start and stop the inferior process, for GDB.
-   Copyright (C) 1986, 1987, 1988 Free Software Foundation, Inc.
-
-GDB is distributed in the hope that it will be useful, but WITHOUT ANY
-WARRANTY.  No author or distributor accepts responsibility to anyone
-for the consequences of using it or for whether it serves any
-particular purpose or works at all, unless he says so in writing.
-Refer to the GDB General Public License for full details.
-
-Everyone is granted permission to copy, modify and redistribute GDB,
-but only under the conditions described in the GDB General Public
-License.  A copy of this license is supposed to have been given to you
-along with GDB so you can know your rights and responsibilities.  It
-should be in a file named COPYING.  Among other things, the copyright
-notice and this notice must be preserved on all copies.
-
-In other words, go ahead and share GDB, but don't try to stop
-anyone else from sharing it farther.  Help stamp out software hoarding!
-*/
-
-/* Notes on the algorithm used in wait_for_inferior to determine if we
-   just did a subroutine call when stepping.  We have the following
-   information at that point:
-
-                  Current and previous (just before this step) pc.
-                 Current and previous sp.
-                 Current and previous start of current function.
-
-   If the start's of the functions don't match, then
-
-       a) We did a subroutine call.
-
-   In this case, the pc will be at the beginning of a function.
-
-       b) We did a subroutine return.
-
-   Otherwise.
-
-       c) We did a longjmp.
-
-   If we did a longjump, we were doing "nexti", since a next would
-   have attempted to skip over the assembly language routine in which
-   the longjmp is coded and would have simply been the equivalent of a
-   continue.  I consider this ok behaivior.  We'd like one of two
-   things to happen if we are doing a nexti through the longjmp()
-   routine: 1) It behaves as a stepi, or 2) It acts like a continue as
-   above.  Given that this is a special case, and that anybody who
-   thinks that the concept of sub calls is meaningful in the context
-   of a longjmp, I'll take either one.  Let's see what happens.  
-
-   Acts like a subroutine return.  I can handle that with no problem
-   at all.
-
-   -->So: If the current and previous beginnings of the current
-   function don't match, *and* the pc is at the start of a function,
-   we've done a subroutine call.  If the pc is not at the start of a
-   function, we *didn't* do a subroutine call.  
-
-   -->If the beginnings of the current and previous function do match,
-   either: 
-
-       a) We just did a recursive call.
-
-          In this case, we would be at the very beginning of a
-          function and 1) it will have a prologue (don't jump to
-          before prologue, or 2) (we assume here that it doesn't have
-          a prologue) there will have been a change in the stack
-          pointer over the last instruction.  (Ie. it's got to put
-          the saved pc somewhere.  The stack is the usual place.  In
-          a recursive call a register is only an option if there's a
-          prologue to do something with it.  This is even true on
-          register window machines; the prologue sets up the new
-          window.  It might not be true on a register window machine
-          where the call instruction moved the register window
-          itself.  Hmmm.  One would hope that the stack pointer would
-          also change.  If it doesn't, somebody send me a note, and
-          I'll work out a more general theory.
-          randy@wheaties.ai.mit.edu).  This is true (albeit slipperly
-          so) on all machines I'm aware of:
-
-             m68k:     Call changes stack pointer.  Regular jumps don't.
-
-             sparc:    Recursive calls must have frames and therefor,
-                       prologues.
-
-             vax:      All calls have frames and hence change the
-                       stack pointer.
-
-       b) We did a return from a recursive call.  I don't see that we
-          have either the ability or the need to distinguish this
-          from an ordinary jump.  The stack frame will be printed
-          when and if the frame pointer changes; if we are in a
-          function without a frame pointer, it's the users own
-          lookout.
-
-       c) We did a jump within a function.  We assume that this is
-          true if we didn't do a recursive call.
-
-       d) We are in no-man's land ("I see no symbols here").  We
-          don't worry about this; it will make calls look like simple
-          jumps (and the stack frames will be printed when the frame
-          pointer moves), which is a reasonably non-violent response.
+/* Target-struct-independent code to start (run) and stop an inferior process.
+   Copyright 1986, 1987, 1988, 1989, 1991, 1992, 1993, 1994, 1995, 1996
+   Free Software Foundation, Inc.
 
-#if 0
-    We skip this; it causes more problems than it's worth.
-#ifdef SUN4_COMPILER_FEATURE
-    We do a special ifdef for the sun 4, forcing it to single step
-  into calls which don't have prologues.  This means that we can't
-  nexti over leaf nodes, we can probably next over them (since they
-  won't have debugging symbols, usually), and we can next out of
-  functions returning structures (with a "call .stret4" at the end).
-#endif
-#endif
-*/
-   
+This file is part of GDB.
+
+This program is free software; you can redistribute it and/or modify
+it under the terms of the GNU General Public License as published by
+the Free Software Foundation; either version 2 of the License, or
+(at your option) any later version.
 
-   
-   
+This program is distributed in the hope that it will be useful,
+but WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+GNU General Public License for more details.
+
+You should have received a copy of the GNU General Public License
+along with this program; if not, write to the Free Software
+Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 
 #include "defs.h"
-#include "param.h"
+#include "gdb_string.h"
+#include <ctype.h>
 #include "symtab.h"
 #include "frame.h"
 #include "inferior.h"
+#include "breakpoint.h"
 #include "wait.h"
+#include "gdbcore.h"
+#include "gdbcmd.h"
+#include "target.h"
+#include "gdbthread.h"
+#include "annotate.h"
 
-#include <stdio.h>
 #include <signal.h>
 
-/* unistd.h is needed to #define X_OK */
-#ifdef USG
-#include <unistd.h>
-#else
-#include <sys/file.h>
+/* Prototypes for local functions */
+
+static void signals_info PARAMS ((char *, int));
+
+static void handle_command PARAMS ((char *, int));
+
+static void sig_print_info PARAMS ((enum target_signal));
+
+static void sig_print_header PARAMS ((void));
+
+static void resume_cleanups PARAMS ((int));
+
+static int hook_stop_stub PARAMS ((char *));
+
+static void delete_breakpoint_current_contents PARAMS ((PTR));
+
+/* GET_LONGJMP_TARGET returns the PC at which longjmp() will resume the
+   program.  It needs to examine the jmp_buf argument and extract the PC
+   from it.  The return value is non-zero on success, zero otherwise. */
+
+#ifndef GET_LONGJMP_TARGET
+#define GET_LONGJMP_TARGET(PC_ADDR) 0
 #endif
 
-/* The idiots at Apple only define X_OK if POSIX is defined.  Fuck 'em. */
-#ifndef X_OK
-#define        X_OK    1       /* Execute permission for access() */
+
+/* Some machines have trampoline code that sits between function callers
+   and the actual functions themselves.  If this machine doesn't have
+   such things, disable their processing.  */
+
+#ifndef SKIP_TRAMPOLINE_CODE
+#define        SKIP_TRAMPOLINE_CODE(pc)        0
 #endif
 
-#ifdef UMAX_PTRACE
-#include <sys/param.h>
-#include <sys/ptrace.h>
-#endif /* UMAX_PTRACE */
+/* Dynamic function trampolines are similar to solib trampolines in that they
+   are between the caller and the callee.  The difference is that when you
+   enter a dynamic trampoline, you can't determine the callee's address.  Some
+   (usually complex) code needs to run in the dynamic trampoline to figure out
+   the callee's address.  This macro is usually called twice.  First, when we
+   enter the trampoline (looks like a normal function call at that point).  It
+   should return the PC of a point within the trampoline where the callee's
+   address is known.  Second, when we hit the breakpoint, this routine returns
+   the callee's address.  At that point, things proceed as per a step resume
+   breakpoint.  */
+
+#ifndef DYNAMIC_TRAMPOLINE_NEXTPC
+#define DYNAMIC_TRAMPOLINE_NEXTPC(pc) 0
+#endif
 
-extern char *sys_siglist[];
-extern int errno;
+/* For SVR4 shared libraries, each call goes through a small piece of
+   trampoline code in the ".plt" section.  IN_SOLIB_CALL_TRAMPOLINE evaluates
+   to nonzero if we are current stopped in one of these. */
 
-/* Tables of how to react to signals; the user sets them.  */
+#ifndef IN_SOLIB_CALL_TRAMPOLINE
+#define IN_SOLIB_CALL_TRAMPOLINE(pc,name)      0
+#endif
 
-static char signal_stop[NSIG];
-static char signal_print[NSIG];
-static char signal_program[NSIG];
+/* In some shared library schemes, the return path from a shared library
+   call may need to go through a trampoline too.  */
 
-/* Nonzero if breakpoints are now inserted in the inferior.  */
+#ifndef IN_SOLIB_RETURN_TRAMPOLINE
+#define IN_SOLIB_RETURN_TRAMPOLINE(pc,name)    0
+#endif
 
-static int breakpoints_inserted;
+/* On MIPS16, a function that returns a floating point value may call
+   a library helper function to copy the return value to a floating point
+   register.  The IGNORE_HELPER_CALL macro returns non-zero if we
+   should ignore (i.e. step over) this function call.  */
+#ifndef IGNORE_HELPER_CALL
+#define IGNORE_HELPER_CALL(pc) 0
+#endif
 
-/* Function inferior was in as of last step command.  */
+/* On some systems, the PC may be left pointing at an instruction that  won't
+   actually be executed.  This is usually indicated by a bit in the PSW.  If
+   we find ourselves in such a state, then we step the target beyond the
+   nullified instruction before returning control to the user so as to avoid
+   confusion. */
 
-static struct symbol *step_start_function;
+#ifndef INSTRUCTION_NULLIFIED
+#define INSTRUCTION_NULLIFIED 0
+#endif
 
-/* This is the sequence of bytes we insert for a breakpoint.  */
+/* Tables of how to react to signals; the user sets them.  */
 
-static char break_insn[] = BREAKPOINT;
+static unsigned char *signal_stop;
+static unsigned char *signal_print;
+static unsigned char *signal_program;
 
-/* Nonzero => address for special breakpoint for resuming stepping.  */
+#define SET_SIGS(nsigs,sigs,flags) \
+  do { \
+    int signum = (nsigs); \
+    while (signum-- > 0) \
+      if ((sigs)[signum]) \
+       (flags)[signum] = 1; \
+  } while (0)
 
-static CORE_ADDR step_resume_break_address;
+#define UNSET_SIGS(nsigs,sigs,flags) \
+  do { \
+    int signum = (nsigs); \
+    while (signum-- > 0) \
+      if ((sigs)[signum]) \
+       (flags)[signum] = 0; \
+  } while (0)
 
-/* Original contents of the byte where the special breakpoint is.  */
 
-static char step_resume_break_shadow[sizeof break_insn];
+/* Command list pointer for the "stop" placeholder.  */
 
-/* Nonzero means the special breakpoint is a duplicate
-   so it has not itself been inserted.  */
+static struct cmd_list_element *stop_command;
 
-static int step_resume_break_duplicate;
+/* Nonzero if breakpoints are now inserted in the inferior.  */
 
-/* Nonzero if we are expecting a trace trap and should proceed from it.
-   2 means expecting 2 trace traps and should continue both times.
-   That occurs when we tell sh to exec the program: we will get
-   a trap after the exec of sh and a second when the program is exec'd.  */
+static int breakpoints_inserted;
+
+/* Function inferior was in as of last step command.  */
+
+static struct symbol *step_start_function;
+
+/* Nonzero if we are expecting a trace trap and should proceed from it.  */
 
 static int trap_expected;
 
+#ifdef SOLIB_ADD
+/* Nonzero if we want to give control to the user when we're notified
+   of shared library events by the dynamic linker.  */
+static int stop_on_solib_events;
+#endif
+
+#ifdef HP_OS_BUG
 /* Nonzero if the next time we try to continue the inferior, it will
    step one instruction and generate a spurious trace trap.
    This is used to compensate for a bug in HP-UX.  */
 
 static int trap_expected_after_continue;
+#endif
 
 /* Nonzero means expecting a trace trap
    and should stop the inferior and return silently when it happens.  */
 
 int stop_after_trap;
 
-/* Nonzero means expecting a trace trap due to attaching to a process.  */
-
-int stop_after_attach;
-
-/* Nonzero if pc has been changed by the debugger
-   since the inferior stopped.  */
+/* Nonzero means expecting a trap and caller will handle it themselves.
+   It is used after attach, due to attaching to a process;
+   when running in the shell before the child program has been exec'd;
+   and when running some kinds of remote stuff (FIXME?).  */
 
-int pc_changed;
+int stop_soon_quietly;
 
-/* Nonzero if debugging a remote machine via a serial link or ethernet.  */
+/* Nonzero if proceed is being used for a "finish" command or a similar
+   situation when stop_registers should be saved.  */
 
-int remote_debugging;
+int proceed_to_finish;
 
-/* Save register contents here when about to pop a stack dummy frame.  */
+/* Save register contents here when about to pop a stack dummy frame,
+   if-and-only-if proceed_to_finish is set.
+   Thus this contains the return value from the called function (assuming
+   values are returned in a register).  */
 
 char stop_registers[REGISTER_BYTES];
 
@@ -217,23 +196,62 @@ char stop_registers[REGISTER_BYTES];
 
 static int breakpoints_failed;
 
-/* Nonzero if inferior is in sh before our program got exec'd.  */
-
-static int running_in_shell;
-
 /* Nonzero after stop if current stack frame should be printed.  */
 
 static int stop_print_frame;
 
+\f
+/* Things to clean up if we QUIT out of resume ().  */
+/* ARGSUSED */
+static void
+resume_cleanups (arg)
+     int arg;
+{
+  normal_stop ();
+}
+
+/* Resume the inferior, but allow a QUIT.  This is useful if the user
+   wants to interrupt some lengthy single-stepping operation
+   (for child processes, the SIGINT goes to the inferior, and so
+   we get a SIGINT random_signal, but for remote debugging and perhaps
+   other targets, that's not true).
+
+   STEP nonzero if we should step (zero to continue instead).
+   SIG is the signal to give the inferior (zero for none).  */
+void
+resume (step, sig)
+     int step;
+     enum target_signal sig;
+{
+  struct cleanup *old_cleanups = make_cleanup (resume_cleanups, 0);
+  QUIT;
+
+#ifdef CANNOT_STEP_BREAKPOINT
+  /* Most targets can step a breakpoint instruction, thus executing it
+     normally.  But if this one cannot, just continue and we will hit
+     it anyway.  */
+  if (step && breakpoints_inserted && breakpoint_here_p (read_pc ()))
+    step = 0;
+#endif
+
 #ifdef NO_SINGLE_STEP
-extern int one_stepped;                /* From machine dependent code */
-extern void single_step ();    /* Same. */
-#endif /* NO_SINGLE_STEP */
+  if (step) {
+    single_step(sig);  /* Do it the hard way, w/temp breakpoints */
+    step = 0;          /* ...and don't ask hardware to do it.  */
+  }
+#endif
+
+  /* Handle any optimized stores to the inferior NOW...  */
+#ifdef DO_DEFERRED_STORES
+  DO_DEFERRED_STORES;
+#endif
+
+  /* Install inferior's terminal modes.  */
+  target_terminal_inferior ();
 
-static void insert_step_breakpoint ();
-static void remove_step_breakpoint ();
-static void wait_for_inferior ();
-static void normal_stop ();
+  target_resume (-1, step, sig);
+  discard_cleanups (old_cleanups);
+}
 
 \f
 /* Clear out all variables saying what to do when inferior is continued.
@@ -247,18 +265,19 @@ clear_proceed_status ()
   step_range_end = 0;
   step_frame_address = 0;
   step_over_calls = -1;
-  step_resume_break_address = 0;
   stop_after_trap = 0;
-  stop_after_attach = 0;
+  stop_soon_quietly = 0;
+  proceed_to_finish = 0;
+  breakpoint_proceeded = 1;    /* We're about to proceed... */
 
-  /* Discard any remaining commands left by breakpoint we had stopped at.  */
-  clear_breakpoint_commands ();
+  /* Discard any remaining commands or status from previous stop.  */
+  bpstat_clear (&stop_bpstat);
 }
 
 /* Basic routine for continuing the program in various fashions.
 
    ADDR is the address to resume at, or -1 for resume where stopped.
-   SIGNAL is the signal to give it, or 0 for none,
+   SIGGNAL is the signal to give it, or 0 for none,
      or -1 for act according to how it stopped.
    STEP is nonzero if should trap after one instruction.
      -1 means return after that and print nothing.
@@ -268,9 +287,9 @@ clear_proceed_status ()
    You should call clear_proceed_status before calling proceed.  */
 
 void
-proceed (addr, signal, step)
+proceed (addr, siggnal, step)
      CORE_ADDR addr;
-     int signal;
+     enum target_signal siggnal;
      int step;
 {
   int oneproc = 0;
@@ -280,23 +299,46 @@ proceed (addr, signal, step)
   if (step < 0)
     stop_after_trap = 1;
 
-  if (addr == -1)
+  if (addr == (CORE_ADDR)-1)
     {
       /* If there is a breakpoint at the address we will resume at,
         step one instruction before inserting breakpoints
         so that we do not stop right away.  */
 
-      if (!pc_changed && breakpoint_here_p (read_pc ()))
+      if (read_pc () == stop_pc && breakpoint_here_p (read_pc ()))
        oneproc = 1;
+
+#ifdef STEP_SKIPS_DELAY
+      /* Check breakpoint_here_p first, because breakpoint_here_p is fast
+        (it just checks internal GDB data structures) and STEP_SKIPS_DELAY
+        is slow (it needs to read memory from the target).  */
+      if (breakpoint_here_p (read_pc () + 4)
+         && STEP_SKIPS_DELAY (read_pc ()))
+       oneproc = 1;
+#endif /* STEP_SKIPS_DELAY */
     }
   else
-    {
-      write_register (PC_REGNUM, addr);
-#ifdef NPC_REGNUM
-      write_register (NPC_REGNUM, addr + 4);
-#endif
-    }
-
+    write_pc (addr);
+
+#ifdef PREPARE_TO_PROCEED
+  /* In a multi-threaded task we may select another thread and then continue.
+     
+     In this case the thread that stopped at a breakpoint will immediately
+     cause another stop, if it is not stepped over first. On the other hand,
+     if (ADDR != -1) we only want to single step over the breakpoint if we did
+     switch to another thread.
+
+     If we are single stepping, don't do any of the above.
+     (Note that in the current implementation single stepping another
+     thread after a breakpoint and then continuing will cause the original
+     breakpoint to be hit again, but you can always continue, so it's not
+     a big deal.)  */
+
+  if (! step && PREPARE_TO_PROCEED (1) && breakpoint_here_p (read_pc ()))
+    oneproc = 1;
+#endif /* PREPARE_TO_PROCEED */
+
+#ifdef HP_OS_BUG
   if (trap_expected_after_continue)
     {
       /* If (step == 0), a trap will be automatically generated after
@@ -306,6 +348,7 @@ proceed (addr, signal, step)
       oneproc = 1;
       trap_expected_after_continue = 0;
     }
+#endif /* HP_OS_BUG */
 
   if (oneproc)
     /* We will get a trace trap after one instruction.
@@ -323,18 +366,21 @@ The same program may be running in another process.");
       breakpoints_inserted = 1;
     }
 
-  /* Install inferior's terminal modes.  */
-  terminal_inferior ();
-
-  if (signal >= 0)
-    stop_signal = signal;
+  if (siggnal != TARGET_SIGNAL_DEFAULT)
+    stop_signal = siggnal;
   /* If this signal should not be seen by program,
      give it zero.  Used for debugging signals.  */
-  else if (stop_signal < NSIG && !signal_program[stop_signal])
-    stop_signal= 0;
+  else if (!signal_program[stop_signal])
+    stop_signal = TARGET_SIGNAL_0;
+
+  annotate_starting ();
+
+  /* Make sure that output from GDB appears before output from the
+     inferior.  */
+  gdb_flush (gdb_stdout);
 
   /* Resume inferior.  */
-  resume (oneproc || step, stop_signal);
+  resume (oneproc || step || bpstat_should_step (), stop_signal);
 
   /* Wait for it to stop (if not standalone)
      and in any case decode why it stopped, and act accordingly.  */
@@ -343,103 +389,57 @@ The same program may be running in another process.");
   normal_stop ();
 }
 
-/* Writing the inferior pc as a register calls this function
-   to inform infrun that the pc has been set in the debugger.  */
-
-void
-writing_pc (val)
-     CORE_ADDR val;
-{
-  stop_pc = val;
-  pc_changed = 1;
-}
-
-/* Start an inferior process for the first time.
-   Actually it was started by the fork that created it,
-   but it will have stopped one instruction after execing sh.
-   Here we must get it up to actual execution of the real program.  */
-
-void
-start_inferior ()
-{
-  /* We will get a trace trap after one instruction.
-     Continue it automatically.  Eventually (after shell does an exec)
-     it will get another trace trap.  Then insert breakpoints and continue.  */
-
-#ifdef START_INFERIOR_TRAPS_EXPECTED
-  trap_expected = START_INFERIOR_TRAPS_EXPECTED;
-#else
-  trap_expected = 2;
-#endif
-
-  running_in_shell = 0;                /* Set to 1 at first SIGTRAP, 0 at second.  */
-  trap_expected_after_continue = 0;
-  breakpoints_inserted = 0;
-  mark_breakpoints_out ();
-
-  /* Set up the "saved terminal modes" of the inferior
-     based on what modes we are starting it with.  */
-  terminal_init_inferior ();
-
-  /* Install inferior's terminal modes.  */
-  terminal_inferior ();
-
-  if (remote_debugging)
-    {
-      trap_expected = 0;
-      fetch_inferior_registers();
-      set_current_frame (create_new_frame (read_register (FP_REGNUM),
-                                          read_pc ()));
-      stop_frame_address = FRAME_FP (get_current_frame());
-      inferior_pid = 3;
-      if (insert_breakpoints())
-       fatal("Can't insert breakpoints");
-      breakpoints_inserted = 1;
-      proceed(-1, -1, 0);
-    }
-  else
-    {
-      wait_for_inferior ();
-      normal_stop ();
-    }
-}
+/* Record the pc and sp of the program the last time it stopped.
+   These are just used internally by wait_for_inferior, but need
+   to be preserved over calls to it and cleared when the inferior
+   is started.  */
+static CORE_ADDR prev_pc;
+static CORE_ADDR prev_func_start;
+static char *prev_func_name;
 
+\f
 /* Start remote-debugging of a machine over a serial link.  */
 
 void
 start_remote ()
 {
+  init_thread_list ();
+  init_wait_for_inferior ();
   clear_proceed_status ();
-  running_in_shell = 0;
+  stop_soon_quietly = 1;
   trap_expected = 0;
-  inferior_pid = 3;
-  breakpoints_inserted = 0;
-  mark_breakpoints_out ();
   wait_for_inferior ();
-  normal_stop();
+  normal_stop ();
 }
 
-#ifdef ATTACH_DETACH
-
-/* Attach to process PID, then initialize for debugging it
-   and wait for the trace-trap that results from attaching.  */
+/* Initialize static vars when a new inferior begins.  */
 
 void
-attach_program (pid)
-     int pid;
+init_wait_for_inferior ()
 {
-  attach (pid);
-  inferior_pid = pid;
+  /* These are meaningless until the first time through wait_for_inferior.  */
+  prev_pc = 0;
+  prev_func_start = 0;
+  prev_func_name = NULL;
 
-  mark_breakpoints_out ();
-  terminal_init_inferior ();
-  clear_proceed_status ();
-  stop_after_attach = 1;
-  /*proceed (-1, 0, -2);*/
-  wait_for_inferior ();
-  normal_stop ();
+#ifdef HP_OS_BUG
+  trap_expected_after_continue = 0;
+#endif
+  breakpoints_inserted = 0;
+  breakpoint_init_inferior ();
+
+  /* Don't confuse first call to proceed(). */
+  stop_signal = TARGET_SIGNAL_0;
+}
+
+static void
+delete_breakpoint_current_contents (arg)
+     PTR arg;
+{
+  struct breakpoint **breakpointp = (struct breakpoint **)arg;
+  if (*breakpointp != NULL)
+    delete_breakpoint (*breakpointp);
 }
-#endif /* ATTACH_DETACH */
 \f
 /* Wait for control to return from inferior to debugger.
    If inferior gets a signal, we may decide to start it up again
@@ -447,229 +447,457 @@ attach_program (pid)
    When this function actually returns it means the inferior
    should be left stopped and GDB should read more commands.  */
 
-static void
+void
 wait_for_inferior ()
 {
-  register int pid;
-  WAITTYPE w;
-  CORE_ADDR pc;
-  int tem;
+  struct cleanup *old_cleanups;
+  struct target_waitstatus w;
   int another_trap;
-  int random_signal;
-  CORE_ADDR stop_sp, prev_sp;
-  CORE_ADDR prev_func_start, stop_func_start;
-  CORE_ADDR prologue_pc;
-  int stop_step_resume_break;
-  CORE_ADDR step_resume_break_sp;
-  int newmisc;
-  int newfun_pc;
-  struct symbol *newfun;
+  int random_signal = 0;
+  CORE_ADDR stop_func_start;
+  CORE_ADDR stop_func_end;
+  char *stop_func_name;
+#if 0
+  CORE_ADDR prologue_pc = 0;
+#endif
+  CORE_ADDR tmp;
   struct symtab_and_line sal;
-  int prev_pc;
-  extern CORE_ADDR text_end;
-
-  prev_pc = read_pc ();
-  prev_func_start = get_pc_function_start (prev_pc) + FUNCTION_START_OFFSET;
-  prev_sp = read_register (SP_REGNUM);
+  int remove_breakpoints_on_following_step = 0;
+  int current_line;
+  struct symtab *current_symtab;
+  int handling_longjmp = 0;    /* FIXME */
+  struct breakpoint *step_resume_breakpoint = NULL;
+  struct breakpoint *through_sigtramp_breakpoint = NULL;
+  int pid;
+  int update_step_sp = 0;
+
+  old_cleanups = make_cleanup (delete_breakpoint_current_contents,
+                              &step_resume_breakpoint);
+  make_cleanup (delete_breakpoint_current_contents,
+               &through_sigtramp_breakpoint);
+  sal = find_pc_line(prev_pc, 0);
+  current_line = sal.line;
+  current_symtab = sal.symtab;
+
+  /* Are we stepping?  */
+#define CURRENTLY_STEPPING() \
+  ((through_sigtramp_breakpoint == NULL \
+    && !handling_longjmp \
+    && ((step_range_end && step_resume_breakpoint == NULL) \
+       || trap_expected)) \
+   || bpstat_should_step ())
 
   while (1)
     {
-      /* Clean up saved state that will become invalid */
-      pc_changed = 0;
-      flush_cached_frames ();
+      /* We have to invalidate the registers BEFORE calling target_wait because
+        they can be loaded from the target while in target_wait.  This makes
+        remote debugging a bit more efficient for those targets that provide
+        critical registers as part of their normal status mechanism. */
+
+      registers_changed ();
 
-      if (remote_debugging)
-       remote_wait (&w);
+      if (target_wait_hook)
+       pid = target_wait_hook (-1, &w);
       else
+       pid = target_wait (-1, &w);
+
+    /* Gross.
+
+       We goto this label from elsewhere in wait_for_inferior when we want
+       to continue the main loop without calling "wait" and trashing the
+       waitstatus contained in W.  */
+    have_waited:
+
+      flush_cached_frames ();
+
+      /* If it's a new process, add it to the thread database */
+
+      if (pid != inferior_pid
+         && !in_thread_list (pid))
        {
-         pid = wait (&w);
-         if (pid != inferior_pid)
-           continue;
+         fprintf_unfiltered (gdb_stderr, "[New %s]\n", target_pid_to_str (pid));
+         add_thread (pid);
+
+         /* We may want to consider not doing a resume here in order to give
+            the user a chance to play with the new thread.  It might be good
+            to make that a user-settable option.  */
+
+         /* At this point, all threads are stopped (happens automatically in
+            either the OS or the native code).  Therefore we need to continue
+            all threads in order to make progress.  */
+
+         target_resume (-1, 0, TARGET_SIGNAL_0);
+         continue;
        }
 
-      /* See if the process still exists; clean up if it doesn't. */
-      if (WIFEXITED (w))
+      switch (w.kind)
        {
-         terminal_ours_for_output ();
-         if (WRETCODE (w))
-           printf ("\nProgram exited with code 0%o.\n", WRETCODE (w));
+       case TARGET_WAITKIND_LOADED:
+         /* Ignore it gracefully.  */
+         if (breakpoints_inserted)
+           {
+             mark_breakpoints_out ();
+             insert_breakpoints ();
+           }
+         resume (0, TARGET_SIGNAL_0);
+         continue;
+
+       case TARGET_WAITKIND_SPURIOUS:
+         resume (0, TARGET_SIGNAL_0);
+         continue;
+
+       case TARGET_WAITKIND_EXITED:
+         target_terminal_ours ();      /* Must do this before mourn anyway */
+         annotate_exited (w.value.integer);
+         if (w.value.integer)
+           printf_filtered ("\nProgram exited with code 0%o.\n", 
+                            (unsigned int)w.value.integer);
          else
-           printf ("\nProgram exited normally.\n");
-         fflush (stdout);
-         inferior_died ();
+           printf_filtered ("\nProgram exited normally.\n");
+
+         /* Record the exit code in the convenience variable $_exitcode, so
+            that the user can inspect this again later.  */
+         set_internalvar (lookup_internalvar ("_exitcode"),
+                          value_from_longest (builtin_type_int, 
+                                              (LONGEST) w.value.integer));
+         gdb_flush (gdb_stdout);
+         target_mourn_inferior ();
 #ifdef NO_SINGLE_STEP
-         one_stepped = 0;      /* Clear single_step state since proc gone */
-#endif /* NO_SINGLE_STEP */
+         one_stepped = 0;
+#endif
+         stop_print_frame = 0;
+         goto stop_stepping;
+
+       case TARGET_WAITKIND_SIGNALLED:
          stop_print_frame = 0;
+         stop_signal = w.value.sig;
+         target_terminal_ours ();      /* Must do this before mourn anyway */
+         annotate_signalled ();
+
+         /* This looks pretty bogus to me.  Doesn't TARGET_WAITKIND_SIGNALLED
+            mean it is already dead?  This has been here since GDB 2.8, so
+            perhaps it means rms didn't understand unix waitstatuses?
+            For the moment I'm just kludging around this in remote.c
+            rather than trying to change it here --kingdon, 5 Dec 1994.  */
+         target_kill ();               /* kill mourns as well */
+
+         printf_filtered ("\nProgram terminated with signal ");
+         annotate_signal_name ();
+         printf_filtered ("%s", target_signal_to_name (stop_signal));
+         annotate_signal_name_end ();
+         printf_filtered (", ");
+         annotate_signal_string ();
+         printf_filtered ("%s", target_signal_to_string (stop_signal));
+         annotate_signal_string_end ();
+         printf_filtered (".\n");
+
+         printf_filtered ("The program no longer exists.\n");
+         gdb_flush (gdb_stdout);
+#ifdef NO_SINGLE_STEP
+         one_stepped = 0;
+#endif
+         goto stop_stepping;
+
+       case TARGET_WAITKIND_STOPPED:
+         /* This is the only case in which we keep going; the above cases
+            end in a continue or goto.  */
          break;
        }
-      else if (!WIFSTOPPED (w))
+
+      stop_signal = w.value.sig;
+
+      stop_pc = read_pc_pid (pid);
+
+      /* See if a thread hit a thread-specific breakpoint that was meant for
+        another thread.  If so, then step that thread past the breakpoint,
+        and continue it.  */
+
+      if (stop_signal == TARGET_SIGNAL_TRAP)
        {
-         kill_inferior ();
-         stop_print_frame = 0;
-         stop_signal = WTERMSIG (w);
-         terminal_ours_for_output ();
-         printf ("\nProgram terminated with signal %d, %s\n",
-                 stop_signal,
-                 stop_signal < NSIG
-                 ? sys_siglist[stop_signal]
-                 : "(undocumented)");
-         printf ("The inferior process no longer exists.\n");
-         fflush (stdout);
 #ifdef NO_SINGLE_STEP
-         one_stepped = 0;      /* Clear single_step state since proc gone */
-#endif /* NO_SINGLE_STEP */
-         break;
+         if (one_stepped)
+           random_signal = 0;
+         else
+#endif
+           if (breakpoints_inserted
+               && breakpoint_here_p (stop_pc - DECR_PC_AFTER_BREAK))
+             {
+               random_signal = 0;
+               if (!breakpoint_thread_match (stop_pc - DECR_PC_AFTER_BREAK, pid))
+                 {
+                   /* Saw a breakpoint, but it was hit by the wrong thread.  Just continue. */
+                   write_pc_pid (stop_pc - DECR_PC_AFTER_BREAK, pid);
+
+                   remove_breakpoints ();
+                   target_resume (pid, 1, TARGET_SIGNAL_0); /* Single step */
+                   /* FIXME: What if a signal arrives instead of the single-step
+                      happening?  */
+
+                   if (target_wait_hook)
+                     target_wait_hook (pid, &w);
+                   else
+                     target_wait (pid, &w);
+                   insert_breakpoints ();
+
+                   /* We need to restart all the threads now.  */
+                   target_resume (-1, 0, TARGET_SIGNAL_0);
+                   continue;
+                 }
+             }
+       }
+      else
+       random_signal = 1;
+
+      /* See if something interesting happened to the non-current thread.  If
+         so, then switch to that thread, and eventually give control back to
+        the user.  */
+
+      if (pid != inferior_pid)
+       {
+         int printed = 0;
+
+         /* If it's a random signal for a non-current thread, notify user
+            if he's expressed an interest.  */
+
+         if (random_signal
+             && signal_print[stop_signal])
+           {
+             printed = 1;
+             target_terminal_ours_for_output ();
+             printf_filtered ("\nProgram received signal %s, %s.\n",
+                              target_signal_to_name (stop_signal),
+                              target_signal_to_string (stop_signal));
+             gdb_flush (gdb_stdout);
+           }
+
+         /* If it's not SIGTRAP and not a signal we want to stop for, then
+            continue the thread. */
+
+         if (stop_signal != TARGET_SIGNAL_TRAP
+             && !signal_stop[stop_signal])
+           {
+             if (printed)
+               target_terminal_inferior ();
+
+             /* Clear the signal if it should not be passed.  */
+             if (signal_program[stop_signal] == 0)
+               stop_signal = TARGET_SIGNAL_0;
+
+             target_resume (pid, 0, stop_signal);
+             continue;
+           }
+
+         /* It's a SIGTRAP or a signal we're interested in.  Switch threads,
+            and fall into the rest of wait_for_inferior().  */
+
+         /* Save infrun state for the old thread.  */
+         save_infrun_state (inferior_pid, prev_pc,
+                            prev_func_start, prev_func_name,
+                            trap_expected, step_resume_breakpoint,
+                            through_sigtramp_breakpoint,
+                            step_range_start, step_range_end,
+                            step_frame_address, handling_longjmp,
+                            another_trap);
+
+         inferior_pid = pid;
+
+         /* Load infrun state for the new thread.  */
+         load_infrun_state (inferior_pid, &prev_pc,
+                            &prev_func_start, &prev_func_name,
+                            &trap_expected, &step_resume_breakpoint,
+                            &through_sigtramp_breakpoint,
+                            &step_range_start, &step_range_end,
+                            &step_frame_address, &handling_longjmp,
+                            &another_trap);
+         printf_filtered ("[Switching to %s]\n", target_pid_to_str (pid));
+
+         flush_cached_frames ();
        }
 
 #ifdef NO_SINGLE_STEP
       if (one_stepped)
        single_step (0);        /* This actually cleans up the ss */
 #endif /* NO_SINGLE_STEP */
+      
+      /* If PC is pointing at a nullified instruction, then step beyond
+        it so that the user won't be confused when GDB appears to be ready
+        to execute it. */
 
-      fetch_inferior_registers ();
-      stop_pc = read_pc ();
-      set_current_frame ( create_new_frame (read_register (FP_REGNUM),
-                                           read_pc ()));
-#ifdef CONVEX_PTRACE
-      /* pop frame stored by user-mode trap, if present */
-      if (stop_pc == BREAK_TRAP_ADDR)
+      if (INSTRUCTION_NULLIFIED)
        {
-         POP_FRAME;
-         stop_pc = read_pc () - 2;
-         write_register (PC_REGNUM, stop_pc);
-#ifdef NPC_REGNUM
-         write_register (NPC_REGNUM, stop_pc + 4);
-#endif
-         pc_changed = 0;
+         struct target_waitstatus tmpstatus;
+
+         registers_changed ();
+         target_resume (pid, 1, TARGET_SIGNAL_0);
+
+         /* We may have received a signal that we want to pass to
+            the inferior; therefore, we must not clobber the waitstatus
+            in W.  So we call wait ourselves, then continue the loop
+            at the "have_waited" label.  */
+         if (target_wait_hook)
+           target_wait_hook (pid, &tmpstatus);
+         else
+           target_wait (pid, &tmpstatus);
+
+
+         goto have_waited;
        }
-      else if (stop_pc > STACK_END_ADDR)
+
+#ifdef HAVE_STEPPABLE_WATCHPOINT
+      /* It may not be necessary to disable the watchpoint to stop over
+        it.  For example, the PA can (with some kernel cooperation) 
+        single step over a watchpoint without disabling the watchpoint.  */
+      if (STOPPED_BY_WATCHPOINT (w))
+       {
+         resume (1, 0);
+         continue;
+       }
+#endif
+
+#ifdef HAVE_NONSTEPPABLE_WATCHPOINT
+      /* It is far more common to need to disable a watchpoint
+        to step the inferior over it.  FIXME.  What else might
+        a debug register or page protection watchpoint scheme need
+        here?  */
+      if (STOPPED_BY_WATCHPOINT (w))
        {
-         POP_FRAME;
-         stop_pc = read_pc ();
+/* At this point, we are stopped at an instruction which has attempted to write
+   to a piece of memory under control of a watchpoint.  The instruction hasn't
+   actually executed yet.  If we were to evaluate the watchpoint expression
+   now, we would get the old value, and therefore no change would seem to have
+   occurred.
+
+   In order to make watchpoints work `right', we really need to complete the
+   memory write, and then evaluate the watchpoint expression.  The following
+   code does that by removing the watchpoint (actually, all watchpoints and
+   breakpoints), single-stepping the target, re-inserting watchpoints, and then
+   falling through to let normal single-step processing handle proceed.  Since
+   this includes evaluating watchpoints, things will come to a stop in the
+   correct manner.  */
+
+         write_pc (stop_pc - DECR_PC_AFTER_BREAK);
+
+         remove_breakpoints ();
+         target_resume (pid, 1, TARGET_SIGNAL_0); /* Single step */
+
+         if (target_wait_hook)
+           target_wait_hook (pid, &w);
+         else
+           target_wait (pid, &w);
+         insert_breakpoints ();
+         /* FIXME-maybe: is this cleaner than setting a flag?  Does it
+            handle things like signals arriving and other things happening
+            in combination correctly?  */
+         goto have_waited;
        }
-#endif /* CONVEX_PTRACE */
-      stop_frame_address = FRAME_FP (get_current_frame ());
-      stop_sp = read_register (SP_REGNUM);
-      stop_func_start =
-       get_pc_function_start (stop_pc) + FUNCTION_START_OFFSET;
+#endif
+
+#ifdef HAVE_CONTINUABLE_WATCHPOINT
+      /* It may be possible to simply continue after a watchpoint.  */
+      STOPPED_BY_WATCHPOINT (w);
+#endif
+
+      stop_func_start = 0;
+      stop_func_end = 0;
+      stop_func_name = 0;
+      /* Don't care about return value; stop_func_start and stop_func_name
+        will both be 0 if it doesn't work.  */
+      find_pc_partial_function (stop_pc, &stop_func_name, &stop_func_start,
+                               &stop_func_end);
+      stop_func_start += FUNCTION_START_OFFSET;
       another_trap = 0;
-      stop_breakpoint = 0;
+      bpstat_clear (&stop_bpstat);
       stop_step = 0;
       stop_stack_dummy = 0;
       stop_print_frame = 1;
-      stop_step_resume_break = 0;
       random_signal = 0;
       stopped_by_random_signal = 0;
       breakpoints_failed = 0;
-
+      
       /* Look at the cause of the stop, and decide what to do.
         The alternatives are:
         1) break; to really stop and return to the debugger,
         2) drop through to start up again
-          (set another_trap to 1 to single step once)
+        (set another_trap to 1 to single step once)
         3) set random_signal to 1, and the decision between 1 and 2
-          will be made according to the signal handling tables.  */
-
-      stop_signal = WSTOPSIG (w);
-
+        will be made according to the signal handling tables.  */
+      
       /* First, distinguish signals caused by the debugger from signals
         that have to do with the program's own actions.
         Note that breakpoint insns may cause SIGTRAP or SIGILL
         or SIGEMT, depending on the operating system version.
         Here we detect when a SIGILL or SIGEMT is really a breakpoint
         and change it to SIGTRAP.  */
-
-      if (stop_signal == SIGTRAP
-#ifndef CONVEX_PTRACE
+      
+      if (stop_signal == TARGET_SIGNAL_TRAP
          || (breakpoints_inserted &&
-             (stop_signal == SIGILL
-              || stop_signal == SIGEMT))
-#endif /* not CONVEX_PTRACE */
-         || stop_after_attach)
+             (stop_signal == TARGET_SIGNAL_ILL
+              || stop_signal == TARGET_SIGNAL_EMT
+            ))
+         || stop_soon_quietly)
        {
-         if (stop_signal == SIGTRAP && stop_after_trap)
+         if (stop_signal == TARGET_SIGNAL_TRAP && stop_after_trap)
            {
              stop_print_frame = 0;
              break;
            }
-         if (stop_after_attach)
+         if (stop_soon_quietly)
            break;
+
          /* Don't even think about breakpoints
-            if still running the shell that will exec the program
-            or if just proceeded over a breakpoint.  */
-         if (stop_signal == SIGTRAP && trap_expected)
-           stop_breakpoint = 0;
+            if just proceeded over a breakpoint.
+
+            However, if we are trying to proceed over a breakpoint
+            and end up in sigtramp, then through_sigtramp_breakpoint
+            will be set and we should check whether we've hit the
+            step breakpoint.  */
+         if (stop_signal == TARGET_SIGNAL_TRAP && trap_expected
+             && through_sigtramp_breakpoint == NULL)
+           bpstat_clear (&stop_bpstat);
          else
            {
              /* See if there is a breakpoint at the current PC.  */
-#if DECR_PC_AFTER_BREAK
-             /* Notice the case of stepping through a jump
-                that leads just after a breakpoint.
-                Don't confuse that with hitting the breakpoint.
-                What we check for is that 1) stepping is going on
-                and 2) the pc before the last insn does not match
-                the address of the breakpoint before the current pc.  */
-             if (!(prev_pc != stop_pc - DECR_PC_AFTER_BREAK
-                   && step_range_end && !step_resume_break_address))
-#endif /* DECR_PC_AFTER_BREAK not zero */
-               {
-                 /* For condition exprs. */
-                 select_frame (get_current_frame (), 0);
-                 stop_breakpoint =
-                   breakpoint_stop_status (stop_pc, stop_frame_address);
-                 /* Following in case break condition called a
-                    function.  */
-                 stop_print_frame = 1;
-                 if (stop_breakpoint && DECR_PC_AFTER_BREAK)
-                   {
-                     stop_pc -= DECR_PC_AFTER_BREAK;
-                     write_register (PC_REGNUM, stop_pc);
-#ifdef NPC_REGNUM
-                     write_register (NPC_REGNUM, stop_pc + 4);
-#endif
-                     pc_changed = 0;
-                   }
-               }
-             /* See if we stopped at the special breakpoint for
-                stepping over a subroutine call.  */
-             if (stop_pc - DECR_PC_AFTER_BREAK
-                 == step_resume_break_address)
-               {
-                 stop_step_resume_break = 1;
-                 if (DECR_PC_AFTER_BREAK)
-                   {
-                     stop_pc -= DECR_PC_AFTER_BREAK;
-                     write_register (PC_REGNUM, stop_pc);
-                     pc_changed = 0;
-                   }
-               }
+             stop_bpstat = bpstat_stop_status
+               (&stop_pc,
+                (DECR_PC_AFTER_BREAK ?
+                /* Notice the case of stepping through a jump
+                   that lands just after a breakpoint.
+                   Don't confuse that with hitting the breakpoint.
+                   What we check for is that 1) stepping is going on
+                   and 2) the pc before the last insn does not match
+                   the address of the breakpoint before the current pc.  */
+                (prev_pc != stop_pc - DECR_PC_AFTER_BREAK
+                 && CURRENTLY_STEPPING ()) :
+                0)
+                );
+             /* Following in case break condition called a
+                function.  */
+             stop_print_frame = 1;
            }
 
-         if (stop_signal == SIGTRAP)
+         if (stop_signal == TARGET_SIGNAL_TRAP)
            random_signal
-             = !(stop_breakpoint || trap_expected
-                 || stop_step_resume_break
-#ifndef CONVEX_PTRACE
-                 || (stop_sp INNER_THAN stop_pc
-                     && stop_pc INNER_THAN stop_frame_address)
-#else
-                 || stop_pc == text_end - 2
-#endif
-                 || (step_range_end && !step_resume_break_address));
+             = !(bpstat_explains_signal (stop_bpstat)
+                 || trap_expected
+#ifndef CALL_DUMMY_BREAKPOINT_OFFSET
+                 || PC_IN_CALL_DUMMY (stop_pc, read_sp (),
+                                      FRAME_FP (get_current_frame ()))
+#endif /* No CALL_DUMMY_BREAKPOINT_OFFSET.  */
+                 || (step_range_end && step_resume_breakpoint == NULL));
          else
            {
              random_signal
-               = !(stop_breakpoint
-                   || stop_step_resume_break
-#ifdef news800
-                   || (stop_sp INNER_THAN stop_pc
-                       && stop_pc INNER_THAN stop_frame_address)
-#endif
-
+               = !(bpstat_explains_signal (stop_bpstat)
+                   /* End of a stack dummy.  Some systems (e.g. Sony
+                      news) give another signal besides SIGTRAP,
+                      so check here as well as above.  */
+#ifndef CALL_DUMMY_BREAKPOINT_OFFSET
+                   || PC_IN_CALL_DUMMY (stop_pc, read_sp (),
+                                        FRAME_FP (get_current_frame ()))
+#endif /* No CALL_DUMMY_BREAKPOINT_OFFSET.  */
                    );
              if (!random_signal)
-               stop_signal = SIGTRAP;
+               stop_signal = TARGET_SIGNAL_TRAP;
            }
        }
       else
@@ -678,246 +906,661 @@ wait_for_inferior ()
       /* For the program's own signals, act according to
         the signal handling tables.  */
 
-      if (random_signal
-         && !(running_in_shell && stop_signal == SIGSEGV))
+      if (random_signal)
        {
          /* Signal not for debugging purposes.  */
          int printed = 0;
-
+         
          stopped_by_random_signal = 1;
-
-         if (stop_signal >= NSIG
-             || signal_print[stop_signal])
+         
+         if (signal_print[stop_signal])
            {
              printed = 1;
-             terminal_ours_for_output ();
-             printf ("\nProgram received signal %d, %s\n",
-                     stop_signal,
-                     stop_signal < NSIG
-                      ? sys_siglist[stop_signal]
-                      : "(undocumented)");
-             fflush (stdout);
+             target_terminal_ours_for_output ();
+             annotate_signal ();
+             printf_filtered ("\nProgram received signal ");
+             annotate_signal_name ();
+             printf_filtered ("%s", target_signal_to_name (stop_signal));
+             annotate_signal_name_end ();
+             printf_filtered (", ");
+             annotate_signal_string ();
+             printf_filtered ("%s", target_signal_to_string (stop_signal));
+             annotate_signal_string_end ();
+             printf_filtered (".\n");
+             gdb_flush (gdb_stdout);
            }
-         if (stop_signal >= NSIG
-             || signal_stop[stop_signal])
+         if (signal_stop[stop_signal])
            break;
          /* If not going to stop, give terminal back
             if we took it away.  */
          else if (printed)
-           terminal_inferior ();
+           target_terminal_inferior ();
+
+         /* Clear the signal if it should not be passed.  */
+         if (signal_program[stop_signal] == 0)
+           stop_signal = TARGET_SIGNAL_0;
+
+         /* I'm not sure whether this needs to be check_sigtramp2 or
+            whether it could/should be keep_going.  */
+         goto check_sigtramp2;
        }
 
       /* Handle cases caused by hitting a breakpoint.  */
+      {
+       CORE_ADDR jmp_buf_pc;
+       struct bpstat_what what;
 
-      if (!random_signal
-         && (stop_breakpoint || stop_step_resume_break))
-       {
-         /* Does a breakpoint want us to stop?  */
-         if (stop_breakpoint && stop_breakpoint != -1
-             && stop_breakpoint != -0x1000001)
+       what = bpstat_what (stop_bpstat);
+
+       if (what.call_dummy)
+         {
+           stop_stack_dummy = 1;
+#ifdef HP_OS_BUG
+           trap_expected_after_continue = 1;
+#endif
+         }
+
+       switch (what.main_action)
+         {
+         case BPSTAT_WHAT_SET_LONGJMP_RESUME:
+           /* If we hit the breakpoint at longjmp, disable it for the
+              duration of this command.  Then, install a temporary
+              breakpoint at the target of the jmp_buf. */
+           disable_longjmp_breakpoint();
+           remove_breakpoints ();
+           breakpoints_inserted = 0;
+           if (!GET_LONGJMP_TARGET(&jmp_buf_pc)) goto keep_going;
+
+           /* Need to blow away step-resume breakpoint, as it
+              interferes with us */
+           if (step_resume_breakpoint != NULL)
+             {
+               delete_breakpoint (step_resume_breakpoint);
+               step_resume_breakpoint = NULL;
+             }
+           /* Not sure whether we need to blow this away too, but probably
+              it is like the step-resume breakpoint.  */
+           if (through_sigtramp_breakpoint != NULL)
+             {
+               delete_breakpoint (through_sigtramp_breakpoint);
+               through_sigtramp_breakpoint = NULL;
+             }
+
+#if 0
+           /* FIXME - Need to implement nested temporary breakpoints */
+           if (step_over_calls > 0)
+             set_longjmp_resume_breakpoint(jmp_buf_pc,
+                                           get_current_frame());
+           else
+#endif                         /* 0 */
+             set_longjmp_resume_breakpoint(jmp_buf_pc, NULL);
+           handling_longjmp = 1; /* FIXME */
+           goto keep_going;
+
+         case BPSTAT_WHAT_CLEAR_LONGJMP_RESUME:
+         case BPSTAT_WHAT_CLEAR_LONGJMP_RESUME_SINGLE:
+           remove_breakpoints ();
+           breakpoints_inserted = 0;
+#if 0
+           /* FIXME - Need to implement nested temporary breakpoints */
+           if (step_over_calls
+               && (FRAME_FP (get_current_frame ())
+                   INNER_THAN step_frame_address))
+             {
+               another_trap = 1;
+               goto keep_going;
+             }
+#endif                         /* 0 */
+           disable_longjmp_breakpoint();
+           handling_longjmp = 0; /* FIXME */
+           if (what.main_action == BPSTAT_WHAT_CLEAR_LONGJMP_RESUME)
+             break;
+           /* else fallthrough */
+
+         case BPSTAT_WHAT_SINGLE:
+           if (breakpoints_inserted)
+             remove_breakpoints ();
+           breakpoints_inserted = 0;
+           another_trap = 1;
+           /* Still need to check other stuff, at least the case
+              where we are stepping and step out of the right range.  */
+           break;
+
+         case BPSTAT_WHAT_STOP_NOISY:
+           stop_print_frame = 1;
+
+           /* We are about to nuke the step_resume_breakpoint and
+              through_sigtramp_breakpoint via the cleanup chain, so
+              no need to worry about it here.  */
+
+           goto stop_stepping;
+
+         case BPSTAT_WHAT_STOP_SILENT:
+           stop_print_frame = 0;
+
+           /* We are about to nuke the step_resume_breakpoint and
+              through_sigtramp_breakpoint via the cleanup chain, so
+              no need to worry about it here.  */
+
+           goto stop_stepping;
+
+         case BPSTAT_WHAT_STEP_RESUME:
+           delete_breakpoint (step_resume_breakpoint);
+           step_resume_breakpoint = NULL;
+           break;
+
+         case BPSTAT_WHAT_THROUGH_SIGTRAMP:
+           if (through_sigtramp_breakpoint)
+             delete_breakpoint (through_sigtramp_breakpoint);
+           through_sigtramp_breakpoint = NULL;
+
+           /* If were waiting for a trap, hitting the step_resume_break
+              doesn't count as getting it.  */
+           if (trap_expected)
+             another_trap = 1;
+           break;
+
+         case BPSTAT_WHAT_CHECK_SHLIBS:
+#ifdef SOLIB_ADD
            {
-             /* 0x1000000 is set in stop_breakpoint as returned by
-                breakpoint_stop_status to indicate a silent
-                breakpoint.  */
-             if ((stop_breakpoint > 0 ? stop_breakpoint :
-                  -stop_breakpoint)
-                 & 0x1000000)
+             extern int auto_solib_add;
+
+             /* Remove breakpoints, we eventually want to step over the
+                shlib event breakpoint, and SOLIB_ADD might adjust
+                breakpoint addresses via breakpoint_re_set.  */
+             if (breakpoints_inserted)
+               remove_breakpoints ();
+             breakpoints_inserted = 0;
+
+             /* Check for any newly added shared libraries if we're
+                supposed to be adding them automatically.  */
+             if (auto_solib_add)
+               {
+                 /* Switch terminal for any messages produced by
+                    breakpoint_re_set.  */
+                 target_terminal_ours_for_output ();
+                 SOLIB_ADD (NULL, 0, NULL);
+                 target_terminal_inferior ();
+               }
+
+             /* Try to reenable shared library breakpoints, additional
+                code segments in shared libraries might be mapped in now. */
+             re_enable_breakpoints_in_shlibs ();
+
+             /* If requested, stop when the dynamic linker notifies
+                gdb of events.  This allows the user to get control
+                and place breakpoints in initializer routines for
+                dynamically loaded objects (among other things).  */
+             if (stop_on_solib_events)
                {
                  stop_print_frame = 0;
-                 if (stop_breakpoint > 0)
-                   stop_breakpoint -= 0x1000000;
-                 else
-                   stop_breakpoint += 0x1000000;
+                 goto stop_stepping;
+               }
+             else
+               {
+                 /* We want to step over this breakpoint, then keep going.  */
+                 another_trap = 1;
+                 break;
                }
-             break;
            }
-         /* But if we have hit the step-resumption breakpoint,
-            remove it.  It has done its job getting us here.
-            The sp test is to make sure that we don't get hung
-            up in recursive calls in functions without frame
-            pointers.  If the stack pointer isn't outside of
-            where the breakpoint was set (within a routine to be
-            stepped over), we're in the middle of a recursive
-            call. Not true for reg window machines (sparc)
-            because the must change frames to call things and
-            the stack pointer doesn't have to change if it
-            the bp was set in a routine without a frame (pc can
-            be stored in some other window).
-
-            The removal of the sp test is to allow calls to
-            alloca.  Nasty things were happening.  Oh, well,
-            gdb can only handle one level deep of lack of
-            frame pointer. */
-         if (stop_step_resume_break
-             && (step_frame_address == 0
-                 || (stop_frame_address == step_frame_address
-#if 0
-#ifndef HAVE_REGISTER_WINDOWS
-                     && step_resume_break_sp INNER_THAN stop_sp
-#endif
 #endif
-                     )))
-           {
-             remove_step_breakpoint ();
-             step_resume_break_address = 0;
-           }
-         /* Otherwise, must remove breakpoints and single-step
-            to get us past the one we hit.  */
-         else
-           {
-             remove_breakpoints ();
-             remove_step_breakpoint ();
-             breakpoints_inserted = 0;
-             another_trap = 1;
-           }
+         break;
 
-         /* We come here if we hit a breakpoint but should not
-            stop for it.  Possibly we also were stepping
-            and should stop for that.  So fall through and
-            test for stepping.  But, if not stepping,
-            do not stop.  */
-       }
+         case BPSTAT_WHAT_LAST:
+           /* Not a real code, but listed here to shut up gcc -Wall.  */
+
+         case BPSTAT_WHAT_KEEP_CHECKING:
+           break;
+         }
+      }
+
+      /* We come here if we hit a breakpoint but should not
+        stop for it.  Possibly we also were stepping
+        and should stop for that.  So fall through and
+        test for stepping.  But, if not stepping,
+        do not stop.  */
+
+#ifndef CALL_DUMMY_BREAKPOINT_OFFSET
+      /* This is the old way of detecting the end of the stack dummy.
+        An architecture which defines CALL_DUMMY_BREAKPOINT_OFFSET gets
+        handled above.  As soon as we can test it on all of them, all
+        architectures should define it.  */
 
       /* If this is the breakpoint at the end of a stack dummy,
-        just stop silently.  */
-#ifndef CONVEX_PTRACE
-      if (stop_sp INNER_THAN stop_pc
-         && stop_pc INNER_THAN stop_frame_address)
-#else
-         /* "stack" dummy must be in text segment for Convex Unix */
-      if (stop_pc == text_end - 2)
-#endif
+        just stop silently, unless the user was doing an si/ni, in which
+        case she'd better know what she's doing.  */
+
+      if (PC_IN_CALL_DUMMY (stop_pc, read_sp (), FRAME_FP (get_current_frame ()))
+         && !step_range_end)
        {
          stop_print_frame = 0;
          stop_stack_dummy = 1;
-#ifdef HP9K320
+#ifdef HP_OS_BUG
          trap_expected_after_continue = 1;
 #endif
          break;
        }
+#endif /* No CALL_DUMMY_BREAKPOINT_OFFSET.  */
 
-      if (step_resume_break_address)
+      if (step_resume_breakpoint)
        /* Having a step-resume breakpoint overrides anything
           else having to do with stepping commands until
           that breakpoint is reached.  */
-       ;
+       /* I'm not sure whether this needs to be check_sigtramp2 or
+          whether it could/should be keep_going.  */
+       goto check_sigtramp2;
+
+      if (step_range_end == 0)
+       /* Likewise if we aren't even stepping.  */
+       /* I'm not sure whether this needs to be check_sigtramp2 or
+          whether it could/should be keep_going.  */
+       goto check_sigtramp2;
+
       /* If stepping through a line, keep going if still within it.  */
-      else if (!random_signal
-              && step_range_end
-              && stop_pc >= step_range_start
-              && stop_pc < step_range_end
-              /* The step range might include the start of the
-                 function, so if we are at the start of the
-                 step range and either the stack or frame pointers
-                 just changed, we've stepped outside */
-              && !(stop_pc == step_range_start
-                   && stop_frame_address
-                   && (stop_sp != prev_sp
-                       || stop_frame_address != step_frame_address)))
+      if (stop_pc >= step_range_start
+         && stop_pc < step_range_end
+#if 0
+/* I haven't a clue what might trigger this clause, and it seems wrong anyway,
+   so I've disabled it until someone complains.  -Stu 10/24/95 */
+
+         /* The step range might include the start of the
+            function, so if we are at the start of the
+            step range and either the stack or frame pointers
+            just changed, we've stepped outside */
+         && !(stop_pc == step_range_start
+              && FRAME_FP (get_current_frame ())
+              && (read_sp () INNER_THAN step_sp
+                  || FRAME_FP (get_current_frame ()) != step_frame_address))
+#endif
+)
        {
-         /* Don't step through the return from a function
-            unless that is the first instruction stepped through.  */
-         if (ABOUT_TO_RETURN (stop_pc))
-           {
-             stop_step = 1;
-             break;
-           }
+         /* We might be doing a BPSTAT_WHAT_SINGLE and getting a signal.
+            So definately need to check for sigtramp here.  */
+         goto check_sigtramp2;
+       }
+
+      /* We stepped out of the stepping range.  */
+
+      /* We can't update step_sp every time through the loop, because
+        reading the stack pointer would slow down stepping too much.
+        But we can update it every time we leave the step range.  */
+      update_step_sp = 1;
+
+      /* Did we just take a signal?  */
+      if (IN_SIGTRAMP (stop_pc, stop_func_name)
+         && !IN_SIGTRAMP (prev_pc, prev_func_name)
+         && read_sp () INNER_THAN step_sp)
+       {
+         /* We've just taken a signal; go until we are back to
+            the point where we took it and one more.  */
+
+         /* This code is needed at least in the following case:
+            The user types "next" and then a signal arrives (before
+            the "next" is done).  */
+
+         /* Note that if we are stopped at a breakpoint, then we need
+            the step_resume breakpoint to override any breakpoints at
+            the same location, so that we will still step over the
+            breakpoint even though the signal happened.  */
+
+         {
+           struct symtab_and_line sr_sal;
+
+           INIT_SAL (&sr_sal);         /* initialize to zeroes */
+           sr_sal.pc = prev_pc;
+           /* We could probably be setting the frame to
+              step_frame_address; I don't think anyone thought to try it.  */
+           step_resume_breakpoint =
+             set_momentary_breakpoint (sr_sal, NULL, bp_step_resume);
+           if (breakpoints_inserted)
+             insert_breakpoints ();
+         }
+
+         /* If this is stepi or nexti, make sure that the stepping range
+            gets us past that instruction.  */
+         if (step_range_end == 1)
+           /* FIXME: Does this run afoul of the code below which, if
+              we step into the middle of a line, resets the stepping
+              range?  */
+           step_range_end = (step_range_start = prev_pc) + 1;
+
+         remove_breakpoints_on_following_step = 1;
+         goto keep_going;
        }
 
-      /* We stepped out of the stepping range.  See if that was due
-        to a subroutine call that we should proceed to the end of.  */
-      else if (!random_signal && step_range_end)
+#if 0
+      /* I disabled this test because it was too complicated and slow.  The
+        SKIP_PROLOGUE was especially slow, because it caused unnecessary
+        prologue examination on various architectures.  The code in the #else
+        clause has been tested on the Sparc, Mips, PA, and Power
+        architectures, so it's pretty likely to be correct.  -Stu 10/24/95 */
+
+      /* See if we left the step range due to a subroutine call that
+        we should proceed to the end of.  */
+
+      if (stop_func_start)
+       {
+         struct symtab *s;
+
+         /* Do this after the IN_SIGTRAMP check; it might give
+            an error.  */
+         prologue_pc = stop_func_start;
+
+         /* Don't skip the prologue if this is assembly source */
+         s = find_pc_symtab (stop_pc);
+         if (s && s->language != language_asm)
+           SKIP_PROLOGUE (prologue_pc);
+       }
+
+      if (!(step_sp INNER_THAN read_sp ())     /* don't mistake (sig)return as a call */
+         && (/* Might be a non-recursive call.  If the symbols are missing
+                enough that stop_func_start == prev_func_start even though
+                they are really two functions, we will treat some calls as
+                jumps.  */
+             stop_func_start != prev_func_start
+
+             /* Might be a recursive call if either we have a prologue
+                or the call instruction itself saves the PC on the stack.  */
+             || prologue_pc != stop_func_start
+             || read_sp () != step_sp)
+         && (/* PC is completely out of bounds of any known objfiles.  Treat
+                like a subroutine call. */
+             ! stop_func_start
+
+             /* If we do a call, we will be at the start of a function...  */
+             || stop_pc == stop_func_start
+
+             /* ...except on the Alpha with -O (and also Irix 5 and
+                perhaps others), in which we might call the address
+                after the load of gp.  Since prologues don't contain
+                calls, we can't return to within one, and we don't
+                jump back into them, so this check is OK.  */
+
+             || stop_pc < prologue_pc
+
+             /* ...and if it is a leaf function, the prologue might
+                consist of gp loading only, so the call transfers to
+                the first instruction after the prologue.  */
+             || (stop_pc == prologue_pc
+
+                 /* Distinguish this from the case where we jump back
+                    to the first instruction after the prologue,
+                    within a function.  */
+                  && stop_func_start != prev_func_start)
+
+             /* If we end up in certain places, it means we did a subroutine
+                call.  I'm not completely sure this is necessary now that we
+                have the above checks with stop_func_start (and now that
+                find_pc_partial_function is pickier).  */
+             || IN_SOLIB_CALL_TRAMPOLINE (stop_pc, stop_func_name)
+
+             /* If none of the above apply, it is a jump within a function,
+                or a return from a subroutine.  The other case is longjmp,
+                which can no longer happen here as long as the
+                handling_longjmp stuff is working.  */
+             ))
+#else
+       /* This test is a much more streamlined, (but hopefully correct)
+          replacement for the code above.  It's been tested on the Sparc,
+          Mips, PA, and Power architectures with good results.  */
+
+       if (stop_pc == stop_func_start /* Quick test */
+           || in_prologue (stop_pc, stop_func_start)
+           || IN_SOLIB_CALL_TRAMPOLINE (stop_pc, stop_func_name)
+           || stop_func_start == 0)
+#endif
+
        {
-         if (stop_func_start)
+         /* It's a subroutine call.  */
+
+         if (step_over_calls == 0)
            {
-             prologue_pc = stop_func_start;
-             SKIP_PROLOGUE (prologue_pc);
+             /* I presume that step_over_calls is only 0 when we're
+                supposed to be stepping at the assembly language level
+                ("stepi").  Just stop.  */
+             stop_step = 1;
+             break;
            }
 
-         /* ==> See comments at top of file on this algorithm.  <==*/
-
-         if (stop_pc == stop_func_start
-             && (stop_func_start != prev_func_start
-                 || prologue_pc != stop_func_start
-                 || stop_sp != prev_sp))
+         if (step_over_calls > 0 || IGNORE_HELPER_CALL (stop_pc))
+           /* We're doing a "next".  */
+           goto step_over_function;
+
+         /* If we are in a function call trampoline (a stub between
+            the calling routine and the real function), locate the real
+            function.  That's what tells us (a) whether we want to step
+            into it at all, and (b) what prologue we want to run to
+            the end of, if we do step into it.  */
+         tmp = SKIP_TRAMPOLINE_CODE (stop_pc);
+         if (tmp != 0)
+           stop_func_start = tmp;
+         else
            {
-             newfun = find_pc_function (stop_pc);
-             /* It's a subroutine call */
-             if (step_over_calls > 0 || (step_over_calls && newfun == 0))
+             tmp = DYNAMIC_TRAMPOLINE_NEXTPC (stop_pc);
+             if (tmp)
                {
-                 /* A subroutine call has happened.  */
-                 /* Set a special breakpoint after the return */
-                 step_resume_break_address =
-                   SAVED_PC_AFTER_CALL (get_current_frame ());
-                 step_resume_break_duplicate
-                   = breakpoint_here_p (step_resume_break_address);
-                 step_resume_break_sp = stop_sp;
-                 if (breakpoints_inserted)
-                   insert_step_breakpoint ();
+                 struct symtab_and_line xxx;
+                 /* Why isn't this s_a_l called "sr_sal", like all of the
+                    other s_a_l's where this code is duplicated?  */
+                 INIT_SAL (&xxx);      /* initialize to zeroes */
+                 xxx.pc = tmp;
+                 step_resume_breakpoint = 
+                   set_momentary_breakpoint (xxx, NULL, bp_step_resume);
+                 insert_breakpoints ();
+                 goto keep_going;
                }
-             /* Subroutine call with source code we should not step over.
-                Do step to the first line of code in it.  */
-             else if (step_over_calls)
-               {
-                 SKIP_PROLOGUE (stop_func_start);
-                 sal = find_pc_line (stop_func_start, 0);
-                 /* Use the step_resume_break to step until
-                    the end of the prologue, even if that involves jumps
-                    (as it seems to on the vax under 4.2).  */
-                 /* If the prologue ends in the middle of a source line,
-                    continue to the end of that source line.
-                    Otherwise, just go to end of prologue.  */
-#ifdef convex
-                 /* no, don't either.  It skips any code that's
-                    legitimately on the first line.  */
+           }
+
+         /* If we have line number information for the function we
+            are thinking of stepping into, step into it.
+
+            If there are several symtabs at that PC (e.g. with include
+            files), just want to know whether *any* of them have line
+            numbers.  find_pc_line handles this.  */
+         {
+           struct symtab_and_line tmp_sal;
+
+           tmp_sal = find_pc_line (stop_func_start, 0);
+           if (tmp_sal.line != 0)
+             goto step_into_function;
+         }
+
+step_over_function:
+         /* A subroutine call has happened.  */
+         {
+           /* Set a special breakpoint after the return */
+           struct symtab_and_line sr_sal;
+
+           INIT_SAL (&sr_sal);         /* initialize to zeroes */
+           sr_sal.pc = 
+             ADDR_BITS_REMOVE
+               (SAVED_PC_AFTER_CALL (get_current_frame ()));
+           step_resume_breakpoint =
+             set_momentary_breakpoint (sr_sal, get_current_frame (),
+                                       bp_step_resume);
+           step_resume_breakpoint->frame = step_frame_address;
+           if (breakpoints_inserted)
+             insert_breakpoints ();
+         }
+         goto keep_going;
+
+step_into_function:
+         /* Subroutine call with source code we should not step over.
+            Do step to the first line of code in it.  */
+         {
+           struct symtab *s;
+
+           s = find_pc_symtab (stop_pc);
+           if (s && s->language != language_asm)
+             SKIP_PROLOGUE (stop_func_start);
+         }
+         sal = find_pc_line (stop_func_start, 0);
+         /* Use the step_resume_break to step until
+            the end of the prologue, even if that involves jumps
+            (as it seems to on the vax under 4.2).  */
+         /* If the prologue ends in the middle of a source line,
+            continue to the end of that source line (if it is still
+            within the function).  Otherwise, just go to end of prologue.  */
+#ifdef PROLOGUE_FIRSTLINE_OVERLAP
+         /* no, don't either.  It skips any code that's
+            legitimately on the first line.  */
 #else
-                 if (sal.end && sal.pc != stop_func_start)
-                   stop_func_start = sal.end;
+         if (sal.end && sal.pc != stop_func_start && sal.end < stop_func_end)
+           stop_func_start = sal.end;
 #endif
-                 
-                 if (stop_func_start == stop_pc)
-                   {
-                     /* We are already there: stop now.  */
-                     stop_step = 1;
-                     break;
-                   }
-                 else
-                   /* Put the step-breakpoint there and go until there. */
-                   {
-                     step_resume_break_address = stop_func_start;
-                     step_resume_break_sp = stop_sp;
-                     
-                     step_resume_break_duplicate
-                       = breakpoint_here_p (step_resume_break_address);
-                     if (breakpoints_inserted)
-                       insert_step_breakpoint ();
-                     /* Do not specify what the fp should be when we stop
-                        since on some machines the prologue
-                        is where the new fp value is established.  */
-                     step_frame_address = 0;
-                     /* And make sure stepping stops right away then.  */
-                     step_range_end = step_range_start;
-                   }
-               }
-             else
-               {
-                 /* We get here only if step_over_calls is 0 and we
-                    just stepped into a subroutine.  I presume
-                    that step_over_calls is only 0 when we're
-                    supposed to be stepping at the assembly
-                    language level.*/
-                 stop_step = 1;
-                 break;
-               }
-           }
-         /* No subroutince call; stop now.  */
-         else
+
+         if (stop_func_start == stop_pc)
            {
+             /* We are already there: stop now.  */
              stop_step = 1;
              break;
            }
+         else
+           /* Put the step-breakpoint there and go until there. */
+           {
+             struct symtab_and_line sr_sal;
+
+             INIT_SAL (&sr_sal);       /* initialize to zeroes */
+             sr_sal.pc = stop_func_start;
+             /* Do not specify what the fp should be when we stop
+                since on some machines the prologue
+                is where the new fp value is established.  */
+             step_resume_breakpoint =
+               set_momentary_breakpoint (sr_sal, NULL, bp_step_resume);
+             if (breakpoints_inserted)
+               insert_breakpoints ();
+
+             /* And make sure stepping stops right away then.  */
+             step_range_end = step_range_start;
+           }
+         goto keep_going;
+       }
+
+      /* We've wandered out of the step range.  */
+
+      sal = find_pc_line(stop_pc, 0);
+
+      if (step_range_end == 1)
+       {
+         /* It is stepi or nexti.  We always want to stop stepping after
+            one instruction.  */
+         stop_step = 1;
+         break;
        }
 
+      /* If we're in the return path from a shared library trampoline,
+        we want to proceed through the trampoline when stepping.  */
+      if (IN_SOLIB_RETURN_TRAMPOLINE(stop_pc, stop_func_name))
+       {
+         CORE_ADDR tmp;
+
+         /* Determine where this trampoline returns.  */
+         tmp = SKIP_TRAMPOLINE_CODE (stop_pc);
+
+         /* Only proceed through if we know where it's going.  */
+         if (tmp)
+           {
+             /* And put the step-breakpoint there and go until there. */
+             struct symtab_and_line sr_sal;
+
+             INIT_SAL (&sr_sal);       /* initialize to zeroes */
+             sr_sal.pc = tmp;
+             /* Do not specify what the fp should be when we stop
+                since on some machines the prologue
+                is where the new fp value is established.  */
+             step_resume_breakpoint =
+               set_momentary_breakpoint (sr_sal, NULL, bp_step_resume);
+             if (breakpoints_inserted)
+               insert_breakpoints ();
+
+             /* Restart without fiddling with the step ranges or
+                other state.  */
+             goto keep_going;
+           }
+       }
+        
+      if (sal.line == 0)
+       {
+         /* We have no line number information.  That means to stop
+            stepping (does this always happen right after one instruction,
+            when we do "s" in a function with no line numbers,
+            or can this happen as a result of a return or longjmp?).  */
+         stop_step = 1;
+         break;
+       }
+
+      if (stop_pc == sal.pc
+         && (current_line != sal.line || current_symtab != sal.symtab))
+       {
+         /* We are at the start of a different line.  So stop.  Note that
+            we don't stop if we step into the middle of a different line.
+            That is said to make things like for (;;) statements work
+            better.  */
+         stop_step = 1;
+         break;
+       }
+
+      /* We aren't done stepping.
+
+        Optimize by setting the stepping range to the line.
+        (We might not be in the original line, but if we entered a
+        new line in mid-statement, we continue stepping.  This makes 
+        things like for(;;) statements work better.)  */
+
+      if (stop_func_end && sal.end >= stop_func_end)
+       {
+         /* If this is the last line of the function, don't keep stepping
+            (it would probably step us out of the function).
+            This is particularly necessary for a one-line function,
+            in which after skipping the prologue we better stop even though
+            we will be in mid-line.  */
+         stop_step = 1;
+         break;
+       }
+      step_range_start = sal.pc;
+      step_range_end = sal.end;
+      step_frame_address = FRAME_FP (get_current_frame ());
+      current_line = sal.line;
+      current_symtab = sal.symtab;
+      goto keep_going;
+
+    check_sigtramp2:
+      if (trap_expected
+         && IN_SIGTRAMP (stop_pc, stop_func_name)
+         && !IN_SIGTRAMP (prev_pc, prev_func_name)
+         && read_sp () INNER_THAN step_sp)
+       {
+         /* What has happened here is that we have just stepped the inferior
+            with a signal (because it is a signal which shouldn't make
+            us stop), thus stepping into sigtramp.
+
+            So we need to set a step_resume_break_address breakpoint
+            and continue until we hit it, and then step.  FIXME: This should
+            be more enduring than a step_resume breakpoint; we should know
+            that we will later need to keep going rather than re-hitting
+            the breakpoint here (see testsuite/gdb.t06/signals.exp where
+            it says "exceedingly difficult").  */
+         struct symtab_and_line sr_sal;
+
+         INIT_SAL (&sr_sal);           /* initialize to zeroes */
+         sr_sal.pc = prev_pc;
+         /* We perhaps could set the frame if we kept track of what
+            the frame corresponding to prev_pc was.  But we don't,
+            so don't.  */
+         through_sigtramp_breakpoint =
+           set_momentary_breakpoint (sr_sal, NULL, bp_through_sigtramp);
+         if (breakpoints_inserted)
+           insert_breakpoints ();
+
+         remove_breakpoints_on_following_step = 1;
+         another_trap = 1;
+       }
+
+    keep_going:
+      /* Come to this label when you need to resume the inferior.
+        It's really much cleaner to do a goto than a maze of if-else
+        conditions.  */
+
       /* Save the pc before execution, to compare with pc after stop.  */
       prev_pc = read_pc ();    /* Might have been DECR_AFTER_BREAK */
       prev_func_start = stop_func_start; /* Ok, since if DECR_PC_AFTER
@@ -925,38 +1568,48 @@ wait_for_inferior ()
                                          original pc would not have
                                          been at the start of a
                                          function. */
-      prev_sp = stop_sp;
+      prev_func_name = stop_func_name;
+
+      if (update_step_sp)
+       step_sp = read_sp ();
+      update_step_sp = 0;
 
       /* If we did not do break;, it means we should keep
         running the inferior and not return to debugger.  */
 
-      /* If trap_expected is 2, it means continue once more
-        and insert breakpoints at the next trap.
-        If trap_expected is 1 and the signal was SIGSEGV, it means
-        the shell is doing some memory allocation--just resume it
-        with SIGSEGV.
-        Otherwise insert breakpoints now, and possibly single step.  */
-
-      if (trap_expected > 1)
+      if (trap_expected && stop_signal != TARGET_SIGNAL_TRAP)
        {
-         trap_expected--;
-         running_in_shell = 1;
-         resume (0, 0);
-       }
-      else if (running_in_shell && stop_signal == SIGSEGV)
-       {
-         resume (0, SIGSEGV);
+         /* We took a signal (which we are supposed to pass through to
+            the inferior, else we'd have done a break above) and we
+            haven't yet gotten our trap.  Simply continue.  */
+         resume (CURRENTLY_STEPPING (), stop_signal);
        }
       else
        {
-         /* Here, we are not awaiting another exec to get
-            the program we really want to debug.
+         /* Either the trap was not expected, but we are continuing
+            anyway (the user asked that this signal be passed to the
+            child)
+              -- or --
+            The signal was SIGTRAP, e.g. it was our signal, but we
+            decided we should resume from it.
+
+            We're going to run this baby now!
+
             Insert breakpoints now, unless we are trying
             to one-proceed past a breakpoint.  */
-         running_in_shell = 0;
-         if (!breakpoints_inserted && !another_trap)
+         /* If we've just finished a special step resume and we don't
+            want to hit a breakpoint, pull em out.  */
+         if (step_resume_breakpoint == NULL
+             && through_sigtramp_breakpoint == NULL
+             && remove_breakpoints_on_following_step)
+           {
+             remove_breakpoints_on_following_step = 0;
+             remove_breakpoints ();
+             breakpoints_inserted = 0;
+           }
+         else if (!breakpoints_inserted &&
+                  (through_sigtramp_breakpoint != NULL || !another_trap))
            {
-             insert_step_breakpoint ();
              breakpoints_failed = insert_breakpoints ();
              if (breakpoints_failed)
                break;
@@ -965,56 +1618,70 @@ wait_for_inferior ()
 
          trap_expected = another_trap;
 
-         if (stop_signal == SIGTRAP)
-           stop_signal = 0;
+         if (stop_signal == TARGET_SIGNAL_TRAP)
+           stop_signal = TARGET_SIGNAL_0;
+
+#ifdef SHIFT_INST_REGS
+         /* I'm not sure when this following segment applies.  I do know, now,
+            that we shouldn't rewrite the regs when we were stopped by a
+            random signal from the inferior process.  */
+         /* FIXME: Shouldn't this be based on the valid bit of the SXIP?
+            (this is only used on the 88k).  */
+
+          if (!bpstat_explains_signal (stop_bpstat)
+             && (stop_signal != TARGET_SIGNAL_CHLD) 
+              && !stopped_by_random_signal)
+            SHIFT_INST_REGS();
+#endif /* SHIFT_INST_REGS */
 
-         resume ((step_range_end && !step_resume_break_address)
-                 || trap_expected,
-                 stop_signal);
+         resume (CURRENTLY_STEPPING (), stop_signal);
        }
     }
+
+ stop_stepping:
+  if (target_has_execution)
+    {
+      /* Assuming the inferior still exists, set these up for next
+        time, just like we did above if we didn't break out of the
+        loop.  */
+      prev_pc = read_pc ();
+      prev_func_start = stop_func_start;
+      prev_func_name = stop_func_name;
+    }
+  do_cleanups (old_cleanups);
 }
 \f
 /* Here to return control to GDB when the inferior stops for real.
    Print appropriate messages, remove breakpoints, give terminal our modes.
 
-   RUNNING_IN_SHELL nonzero means the shell got a signal before
-   exec'ing the program we wanted to run.
    STOP_PRINT_FRAME nonzero means print the executing frame
    (pc, function, args, file, line number and line text).
    BREAKPOINTS_FAILED nonzero means stop was due to error
    attempting to insert breakpoints.  */
 
-/* FIXME, normal_stop is ALWAYS called immediately after wait_for_inferior.
-   They should probably be merged into a single function, since that
-   would avoid numerous tests (e.g. of inferior_pid).  */
-
-static void
+void
 normal_stop ()
 {
   /* Make sure that the current_frame's pc is correct.  This
      is a correction for setting up the frame info before doing
      DECR_PC_AFTER_BREAK */
-  if (inferior_pid)
+  if (target_has_execution && get_current_frame())
     (get_current_frame ())->pc = read_pc ();
   
   if (breakpoints_failed)
     {
-      terminal_ours_for_output ();
+      target_terminal_ours_for_output ();
       print_sys_errmsg ("ptrace", breakpoints_failed);
-      printf ("Stopped; cannot insert breakpoints.\n\
+      printf_filtered ("Stopped; cannot insert breakpoints.\n\
 The same program may be running in another process.\n");
     }
 
-  if (inferior_pid)
-    remove_step_breakpoint ();
-
-  if (inferior_pid && breakpoints_inserted)
+  if (target_has_execution && breakpoints_inserted)
     if (remove_breakpoints ())
       {
-       terminal_ours_for_output ();
-       printf ("Cannot remove breakpoints because program is no longer writable.\n\
-It must be running in another process.\n\
+       target_terminal_ours_for_output ();
+       printf_filtered ("Cannot remove breakpoints because program is no longer writable.\n\
+It might be running in another process.\n\
 Further execution is probably impossible.\n");
       }
 
@@ -1023,69 +1690,64 @@ Further execution is probably impossible.\n");
   /* Delete the breakpoint we stopped at, if it wants to be deleted.
      Delete any breakpoint that is to be deleted at the next stop.  */
 
-  breakpoint_auto_delete (stop_breakpoint);
+  breakpoint_auto_delete (stop_bpstat);
 
   /* If an auto-display called a function and that got a signal,
      delete that auto-display to avoid an infinite recursion.  */
 
   if (stopped_by_random_signal)
-    delete_current_display ();
+    disable_current_display ();
 
   if (step_multi && stop_step)
-    return;
+    goto done;
 
-  terminal_ours ();
+  target_terminal_ours ();
 
-  if (running_in_shell)
-    {
-      if (stop_signal == SIGSEGV)
-       {
-         char *exec_file = (char *) get_exec_file (1);
+  if (stop_bpstat
+      && stop_bpstat->breakpoint_at
+      && stop_bpstat->breakpoint_at->type == bp_shlib_event)
+    printf_filtered ("Stopped due to shared library event\n");
 
-         if (access (exec_file, X_OK) != 0)
-           printf ("The file \"%s\" is not executable.\n", exec_file);
-         else
-           printf ("\
-You have just encountered a bug in \"sh\".  GDB starts your program\n\
-by running \"sh\" with a command to exec your program.\n\
-This is so that \"sh\" will process wildcards and I/O redirection.\n\
-This time, \"sh\" crashed.\n\
-\n\
-One known bug in \"sh\" bites when the environment takes up a lot of space.\n\
-Try \"info env\" to see the environment; then use \"unset-env\" to kill\n\
-some variables whose values are large; then do \"run\" again.\n\
-\n\
-If that works, you might want to put those \"unset-env\" commands\n\
-into a \".gdbinit\" file in this directory so they will happen every time.\n");
-       }
-      /* Don't confuse user with his program's symbols on sh's data.  */
-      stop_print_frame = 0;
+  /* Look up the hook_stop and run it if it exists.  */
+
+  if (stop_command->hook)
+    {
+      catch_errors (hook_stop_stub, (char *)stop_command->hook,
+                   "Error while running hook_stop:\n", RETURN_MASK_ALL);
     }
 
-  if (inferior_pid == 0)
-    return;
+  if (!target_has_stack)
+    goto done;
 
   /* Select innermost stack frame except on return from a stack dummy routine,
-     or if the program has exited.  */
+     or if the program has exited.  Print it without a level number if
+     we have changed functions or hit a breakpoint.  Print source line
+     if we have one.  */
   if (!stop_stack_dummy)
     {
       select_frame (get_current_frame (), 0);
 
       if (stop_print_frame)
        {
-         if (stop_breakpoint > 0)
-           printf ("\nBpt %d, ", stop_breakpoint);
-         print_sel_frame (stop_step
-                          && step_frame_address == stop_frame_address
-                          && step_start_function == find_pc_function (stop_pc));
+         int source_only;
+
+         source_only = bpstat_print (stop_bpstat);
+         source_only = source_only ||
+               (   stop_step
+                && step_frame_address == FRAME_FP (get_current_frame ())
+                && step_start_function == find_pc_function (stop_pc));
+
+          print_stack_frame (selected_frame, -1, source_only? -1: 1);
+
          /* Display the auto-display expressions.  */
          do_displays ();
        }
     }
 
-  /* Save the function value return registers
+  /* Save the function value return registers, if we care.
      We might be about to restore their previous contents.  */
-  read_register_bytes (0, stop_registers, REGISTER_BYTES);
+  if (proceed_to_finish)
+    read_register_bytes (0, stop_registers, REGISTER_BYTES);
 
   if (stop_stack_dummy)
     {
@@ -1093,30 +1755,63 @@ into a \".gdbinit\" file in this directory so they will happen every time.\n");
          POP_FRAME ends with a setting of the current frame, so we
         can use that next. */
       POP_FRAME;
+      /* Set stop_pc to what it was before we called the function.  Can't rely
+        on restore_inferior_status because that only gets called if we don't
+        stop in the called function.  */
+      stop_pc = read_pc();
       select_frame (get_current_frame (), 0);
     }
+ done:
+  annotate_stopped ();
+}
+
+static int
+hook_stop_stub (cmd)
+     char *cmd;
+{
+  execute_user_command ((struct cmd_list_element *)cmd, 0);
+  return (0);
 }
 \f
+int signal_stop_state (signo)
+     int signo;
+{
+  return signal_stop[signo];
+}
+
+int signal_print_state (signo)
+     int signo;
+{
+  return signal_print[signo];
+}
+
+int signal_pass_state (signo)
+     int signo;
+{
+  return signal_program[signo];
+}
+
 static void
-insert_step_breakpoint ()
+sig_print_header ()
 {
-  if (step_resume_break_address && !step_resume_break_duplicate)
-    {
-      read_memory (step_resume_break_address,
-                  step_resume_break_shadow, sizeof break_insn);
-      write_memory (step_resume_break_address,
-                   break_insn, sizeof break_insn);
-    }
+  printf_filtered ("\
+Signal        Stop\tPrint\tPass to program\tDescription\n");
 }
 
 static void
-remove_step_breakpoint ()
+sig_print_info (oursig)
+     enum target_signal oursig;
 {
-  if (step_resume_break_address && !step_resume_break_duplicate)
-    write_memory (step_resume_break_address, step_resume_break_shadow,
-                 sizeof break_insn);
+  char *name = target_signal_to_name (oursig);
+  printf_filtered ("%s", name);
+  printf_filtered ("%*.*s ", 13 - strlen (name), 13 - strlen (name),
+                  "                 ");
+  printf_filtered ("%s\t", signal_stop[oursig] ? "Yes" : "No");
+  printf_filtered ("%s\t", signal_print[oursig] ? "Yes" : "No");
+  printf_filtered ("%s\t\t", signal_program[oursig] ? "Yes" : "No");
+  printf_filtered ("%s\n", target_signal_to_string (oursig));
 }
-\f
+
 /* Specify how various signals in the inferior should be handled.  */
 
 static void
@@ -1124,142 +1819,237 @@ handle_command (args, from_tty)
      char *args;
      int from_tty;
 {
-  register char *p = args;
-  int signum = 0;
-  register int digits, wordlen;
+  char **argv;
+  int digits, wordlen;
+  int sigfirst, signum, siglast;
+  enum target_signal oursig;
+  int allsigs;
+  int nsigs;
+  unsigned char *sigs;
+  struct cleanup *old_chain;
+
+  if (args == NULL)
+    {
+      error_no_arg ("signal to handle");
+    }
+
+  /* Allocate and zero an array of flags for which signals to handle. */
+
+  nsigs = (int)TARGET_SIGNAL_LAST;
+  sigs = (unsigned char *) alloca (nsigs);
+  memset (sigs, 0, nsigs);
 
-  if (!args)
-    error_no_arg ("signal to handle");
+  /* Break the command line up into args. */
 
-  while (*p)
+  argv = buildargv (args);
+  if (argv == NULL)
     {
-      /* Find the end of the next word in the args.  */
-      for (wordlen = 0; p[wordlen] && p[wordlen] != ' ' && p[wordlen] != '\t';
-          wordlen++);
-      for (digits = 0; p[digits] >= '0' && p[digits] <= '9'; digits++);
+      nomem (0);
+    }
+  old_chain = make_cleanup (freeargv, (char *) argv);
+
+  /* Walk through the args, looking for signal oursigs, signal names, and
+     actions.  Signal numbers and signal names may be interspersed with
+     actions, with the actions being performed for all signals cumulatively
+     specified.  Signal ranges can be specified as <LOW>-<HIGH>. */
+
+  while (*argv != NULL)
+    {
+      wordlen = strlen (*argv);
+      for (digits = 0; isdigit ((*argv)[digits]); digits++) {;}
+      allsigs = 0;
+      sigfirst = siglast = -1;
 
-      /* If it is all digits, it is signal number to operate on.  */
-      if (digits == wordlen)
+      if (wordlen >= 1 && !strncmp (*argv, "all", wordlen))
+       {
+         /* Apply action to all signals except those used by the
+            debugger.  Silently skip those. */
+         allsigs = 1;
+         sigfirst = 0;
+         siglast = nsigs - 1;
+       }
+      else if (wordlen >= 1 && !strncmp (*argv, "stop", wordlen))
+       {
+         SET_SIGS (nsigs, sigs, signal_stop);
+         SET_SIGS (nsigs, sigs, signal_print);
+       }
+      else if (wordlen >= 1 && !strncmp (*argv, "ignore", wordlen))
+       {
+         UNSET_SIGS (nsigs, sigs, signal_program);
+       }
+      else if (wordlen >= 2 && !strncmp (*argv, "print", wordlen))
+       {
+         SET_SIGS (nsigs, sigs, signal_print);
+       }
+      else if (wordlen >= 2 && !strncmp (*argv, "pass", wordlen))
+       {
+         SET_SIGS (nsigs, sigs, signal_program);
+       }
+      else if (wordlen >= 3 && !strncmp (*argv, "nostop", wordlen))
+       {
+         UNSET_SIGS (nsigs, sigs, signal_stop);
+       }
+      else if (wordlen >= 3 && !strncmp (*argv, "noignore", wordlen))
+       {
+         SET_SIGS (nsigs, sigs, signal_program);
+       }
+      else if (wordlen >= 4 && !strncmp (*argv, "noprint", wordlen))
+       {
+         UNSET_SIGS (nsigs, sigs, signal_print);
+         UNSET_SIGS (nsigs, sigs, signal_stop);
+       }
+      else if (wordlen >= 4 && !strncmp (*argv, "nopass", wordlen))
+       {
+         UNSET_SIGS (nsigs, sigs, signal_program);
+       }
+      else if (digits > 0)
        {
-         signum = atoi (p);
-         if (signum <= 0 || signum >= NSIG)
+         /* It is numeric.  The numeric signal refers to our own internal
+            signal numbering from target.h, not to host/target signal number.
+            This is a feature; users really should be using symbolic names
+            anyway, and the common ones like SIGHUP, SIGINT, SIGALRM, etc.
+            will work right anyway.  */
+
+         sigfirst = siglast = (int) target_signal_from_command (atoi (*argv));
+         if ((*argv)[digits] == '-')
            {
-             p[wordlen] = '\0';
-             error ("Invalid signal %s given as argument to \"handle\".", p);
+             siglast =
+               (int) target_signal_from_command (atoi ((*argv) + digits + 1));
            }
-         if (signum == SIGTRAP || signum == SIGINT)
+         if (sigfirst > siglast)
            {
-             if (!query ("Signal %d is used by the debugger.\nAre you sure you want to change it? ", signum))
-               error ("Not confirmed.");
+             /* Bet he didn't figure we'd think of this case... */
+             signum = sigfirst;
+             sigfirst = siglast;
+             siglast = signum;
            }
        }
-      else if (signum == 0)
-       error ("First argument is not a signal number.");
-
-      /* Else, if already got a signal number, look for flag words
-        saying what to do for it.  */
-      else if (!strncmp (p, "stop", wordlen))
-       {
-         signal_stop[signum] = 1;
-         signal_print[signum] = 1;
-       }
-      else if (wordlen >= 2 && !strncmp (p, "print", wordlen))
-       signal_print[signum] = 1;
-      else if (wordlen >= 2 && !strncmp (p, "pass", wordlen))
-       signal_program[signum] = 1;
-      else if (!strncmp (p, "ignore", wordlen))
-       signal_program[signum] = 0;
-      else if (wordlen >= 3 && !strncmp (p, "nostop", wordlen))
-       signal_stop[signum] = 0;
-      else if (wordlen >= 4 && !strncmp (p, "noprint", wordlen))
+      else
        {
-         signal_print[signum] = 0;
-         signal_stop[signum] = 0;
+         oursig = target_signal_from_name (*argv);
+         if (oursig != TARGET_SIGNAL_UNKNOWN)
+           {
+             sigfirst = siglast = (int)oursig;
+           }
+         else
+           {
+             /* Not a number and not a recognized flag word => complain.  */
+             error ("Unrecognized or ambiguous flag word: \"%s\".", *argv);
+           }
        }
-      else if (wordlen >= 4 && !strncmp (p, "nopass", wordlen))
-       signal_program[signum] = 0;
-      else if (wordlen >= 3 && !strncmp (p, "noignore", wordlen))
-       signal_program[signum] = 1;
-      /* Not a number and not a recognized flag word => complain.  */
-      else
+
+      /* If any signal numbers or symbol names were found, set flags for
+        which signals to apply actions to. */
+
+      for (signum = sigfirst; signum >= 0 && signum <= siglast; signum++)
        {
-         p[wordlen] = 0;
-         error ("Unrecognized flag word: \"%s\".", p);
+         switch ((enum target_signal)signum)
+           {
+             case TARGET_SIGNAL_TRAP:
+             case TARGET_SIGNAL_INT:
+               if (!allsigs && !sigs[signum])
+                 {
+                   if (query ("%s is used by the debugger.\n\
+Are you sure you want to change it? ",
+                              target_signal_to_name
+                              ((enum target_signal)signum)))
+                     {
+                       sigs[signum] = 1;
+                     }
+                   else
+                     {
+                       printf_unfiltered ("Not confirmed, unchanged.\n");
+                       gdb_flush (gdb_stdout);
+                     }
+                 }
+               break;
+             case TARGET_SIGNAL_0:
+             case TARGET_SIGNAL_DEFAULT:
+             case TARGET_SIGNAL_UNKNOWN:
+               /* Make sure that "all" doesn't print these.  */
+               break;
+             default:
+               sigs[signum] = 1;
+               break;
+           }
        }
 
-      /* Find start of next word.  */
-      p += wordlen;
-      while (*p == ' ' || *p == '\t') p++;
+      argv++;
     }
 
+  target_notice_signals(inferior_pid);
+
   if (from_tty)
     {
       /* Show the results.  */
-      printf ("Number\tStop\tPrint\tPass to program\tDescription\n");
-      printf ("%d\t", signum);
-      printf ("%s\t", signal_stop[signum] ? "Yes" : "No");
-      printf ("%s\t", signal_print[signum] ? "Yes" : "No");
-      printf ("%s\t\t", signal_program[signum] ? "Yes" : "No");
-      printf ("%s\n", sys_siglist[signum]);
+      sig_print_header ();
+      for (signum = 0; signum < nsigs; signum++)
+       {
+         if (sigs[signum])
+           {
+             sig_print_info (signum);
+           }
+       }
     }
+
+  do_cleanups (old_chain);
 }
 
-/* Print current contents of the tables set by the handle command.  */
+/* Print current contents of the tables set by the handle command.
+   It is possible we should just be printing signals actually used
+   by the current target (but for things to work right when switching
+   targets, all signals should be in the signal tables).  */
 
 static void
-signals_info (signum_exp)
+signals_info (signum_exp, from_tty)
      char *signum_exp;
+     int from_tty;
 {
-  register int i;
-  printf ("Number\tStop\tPrint\tPass to program\tDescription\n");
+  enum target_signal oursig;
+  sig_print_header ();
 
   if (signum_exp)
     {
-      i = parse_and_eval_address (signum_exp);
-      printf ("%d\t", i);
-      printf ("%s\t", signal_stop[i] ? "Yes" : "No");
-      printf ("%s\t", signal_print[i] ? "Yes" : "No");
-      printf ("%s\t\t", signal_program[i] ? "Yes" : "No");
-      printf ("%s\n", sys_siglist[i]);
+      /* First see if this is a symbol name.  */
+      oursig = target_signal_from_name (signum_exp);
+      if (oursig == TARGET_SIGNAL_UNKNOWN)
+       {
+         /* No, try numeric.  */
+         oursig =
+           target_signal_from_command (parse_and_eval_address (signum_exp));
+       }
+      sig_print_info (oursig);
       return;
     }
 
-  printf ("\n");
-  for (i = 0; i < NSIG; i++)
+  printf_filtered ("\n");
+  /* These ugly casts brought to you by the native VAX compiler.  */
+  for (oursig = TARGET_SIGNAL_FIRST;
+       (int)oursig < (int)TARGET_SIGNAL_LAST;
+       oursig = (enum target_signal)((int)oursig + 1))
     {
       QUIT;
-      if (i > 0 && i % 16 == 0)
-       {
-         printf ("[Type Return to see more]");
-         fflush (stdout);
-         gdb_read_line (0, 0);
-       }
-      printf ("%d\t", i);
-      printf ("%s\t", signal_stop[i] ? "Yes" : "No");
-      printf ("%s\t", signal_print[i] ? "Yes" : "No");
-      printf ("%s\t\t", signal_program[i] ? "Yes" : "No");
-      printf ("%s\n", sys_siglist[i]);
+
+      if (oursig != TARGET_SIGNAL_UNKNOWN
+         && oursig != TARGET_SIGNAL_DEFAULT
+         && oursig != TARGET_SIGNAL_0)
+       sig_print_info (oursig);
     }
 
-  printf ("\nUse the \"handle\" command to change these tables.\n");
+  printf_filtered ("\nUse the \"handle\" command to change these tables.\n");
 }
 \f
 /* Save all of the information associated with the inferior<==>gdb
    connection.  INF_STATUS is a pointer to a "struct inferior_status"
    (defined in inferior.h).  */
 
-struct command_line *get_breakpoint_commands ();
-
 void
 save_inferior_status (inf_status, restore_stack_info)
      struct inferior_status *inf_status;
      int restore_stack_info;
 {
-  inf_status->pc_changed = pc_changed;
   inf_status->stop_signal = stop_signal;
   inf_status->stop_pc = stop_pc;
-  inf_status->stop_frame_address = stop_frame_address;
-  inf_status->stop_breakpoint = stop_breakpoint;
   inf_status->stop_step = stop_step;
   inf_status->stop_stack_dummy = stop_stack_dummy;
   inf_status->stopped_by_random_signal = stopped_by_random_signal;
@@ -1268,31 +2058,68 @@ save_inferior_status (inf_status, restore_stack_info)
   inf_status->step_range_end = step_range_end;
   inf_status->step_frame_address = step_frame_address;
   inf_status->step_over_calls = step_over_calls;
-  inf_status->step_resume_break_address = step_resume_break_address;
   inf_status->stop_after_trap = stop_after_trap;
-  inf_status->stop_after_attach = stop_after_attach;
-  inf_status->breakpoint_commands = get_breakpoint_commands ();
+  inf_status->stop_soon_quietly = stop_soon_quietly;
+  /* Save original bpstat chain here; replace it with copy of chain. 
+     If caller's caller is walking the chain, they'll be happier if we
+     hand them back the original chain when restore_i_s is called.  */
+  inf_status->stop_bpstat = stop_bpstat;
+  stop_bpstat = bpstat_copy (stop_bpstat);
+  inf_status->breakpoint_proceeded = breakpoint_proceeded;
   inf_status->restore_stack_info = restore_stack_info;
+  inf_status->proceed_to_finish = proceed_to_finish;
   
-  bcopy (stop_registers, inf_status->stop_registers, REGISTER_BYTES);
-  
+  memcpy (inf_status->stop_registers, stop_registers, REGISTER_BYTES);
+
+  read_register_bytes (0, inf_status->registers, REGISTER_BYTES);
+
   record_selected_frame (&(inf_status->selected_frame_address),
                         &(inf_status->selected_level));
   return;
 }
 
+struct restore_selected_frame_args {
+  CORE_ADDR frame_address;
+  int level;
+};
+
+static int restore_selected_frame PARAMS ((char *));
+
+/* Restore the selected frame.  args is really a struct
+   restore_selected_frame_args * (declared as char * for catch_errors)
+   telling us what frame to restore.  Returns 1 for success, or 0 for
+   failure.  An error message will have been printed on error.  */
+
+static int
+restore_selected_frame (args)
+     char *args;
+{
+  struct restore_selected_frame_args *fr =
+    (struct restore_selected_frame_args *) args;
+  struct frame_info *frame;
+  int level = fr->level;
+
+  frame = find_relative_frame (get_current_frame (), &level);
+
+  /* If inf_status->selected_frame_address is NULL, there was no
+     previously selected frame.  */
+  if (frame == NULL ||
+      FRAME_FP (frame) != fr->frame_address ||
+      level != 0)
+    {
+      warning ("Unable to restore previously selected frame.\n");
+      return 0;
+    }
+  select_frame (frame, fr->level);
+  return(1);
+}
+
 void
 restore_inferior_status (inf_status)
      struct inferior_status *inf_status;
 {
-  FRAME fid;
-  int level = inf_status->selected_level;
-
-  pc_changed = inf_status->pc_changed;
   stop_signal = inf_status->stop_signal;
   stop_pc = inf_status->stop_pc;
-  stop_frame_address = inf_status->stop_frame_address;
-  stop_breakpoint = inf_status->stop_breakpoint;
   stop_step = inf_status->stop_step;
   stop_stack_dummy = inf_status->stop_stack_dummy;
   stopped_by_random_signal = inf_status->stopped_by_random_signal;
@@ -1301,29 +2128,45 @@ restore_inferior_status (inf_status)
   step_range_end = inf_status->step_range_end;
   step_frame_address = inf_status->step_frame_address;
   step_over_calls = inf_status->step_over_calls;
-  step_resume_break_address = inf_status->step_resume_break_address;
   stop_after_trap = inf_status->stop_after_trap;
-  stop_after_attach = inf_status->stop_after_attach;
-  set_breakpoint_commands (inf_status->breakpoint_commands);
-
-  bcopy (inf_status->stop_registers, stop_registers, REGISTER_BYTES);
-
-  if (inf_status->restore_stack_info)
+  stop_soon_quietly = inf_status->stop_soon_quietly;
+  bpstat_clear (&stop_bpstat);
+  stop_bpstat = inf_status->stop_bpstat;
+  breakpoint_proceeded = inf_status->breakpoint_proceeded;
+  proceed_to_finish = inf_status->proceed_to_finish;
+
+  memcpy (stop_registers, inf_status->stop_registers, REGISTER_BYTES);
+
+  /* The inferior can be gone if the user types "print exit(0)"
+     (and perhaps other times).  */
+  if (target_has_execution)
+    write_register_bytes (0, inf_status->registers, REGISTER_BYTES);
+
+  /* The inferior can be gone if the user types "print exit(0)"
+     (and perhaps other times).  */
+
+  /* FIXME: If we are being called after stopping in a function which
+     is called from gdb, we should not be trying to restore the
+     selected frame; it just prints a spurious error message (The
+     message is useful, however, in detecting bugs in gdb (like if gdb
+     clobbers the stack)).  In fact, should we be restoring the
+     inferior status at all in that case?  .  */
+
+  if (target_has_stack && inf_status->restore_stack_info)
     {
-      fid = find_relative_frame (get_current_frame (),
-                                &level);
-      
-      if (FRAME_FP (fid) != inf_status->selected_frame_address ||
-         level != 0)
-       {
-         fprintf (stderr, "Unable to restore previously selected frame.\n");
-         select_frame (get_current_frame (), 0);
-         return;
-       }
-      
-      select_frame (fid, inf_status->selected_level);
+      struct restore_selected_frame_args fr;
+      fr.level = inf_status->selected_level;
+      fr.frame_address = inf_status->selected_frame_address;
+      /* The point of catch_errors is that if the stack is clobbered,
+        walking the stack might encounter a garbage pointer and error()
+        trying to dereference it.  */
+      if (catch_errors (restore_selected_frame, &fr,
+                       "Unable to restore previously selected frame:\n",
+                       RETURN_MASK_ERROR) == 0)
+       /* Error in restoring the selected frame.  Select the innermost
+          frame.  */
+       select_frame (get_current_frame (), 0);
     }
-  return;
 }
 
 \f
@@ -1331,22 +2174,42 @@ void
 _initialize_infrun ()
 {
   register int i;
+  register int numsigs;
 
   add_info ("signals", signals_info,
            "What debugger does when program gets various signals.\n\
-Specify a signal number as argument to print info on that signal only.");
+Specify a signal as argument to print info on that signal only.");
+  add_info_alias ("handle", "signals", 0);
 
   add_com ("handle", class_run, handle_command,
-          "Specify how to handle a signal.\n\
-Args are signal number followed by flags.\n\
-Flags allowed are \"stop\", \"print\", \"pass\",\n\
- \"nostop\", \"noprint\" or \"nopass\".\n\
-Print means print a message if this signal happens.\n\
+          concat ("Specify how to handle a signal.\n\
+Args are signals and actions to apply to those signals.\n\
+Symbolic signals (e.g. SIGSEGV) are recommended but numeric signals\n\
+from 1-15 are allowed for compatibility with old versions of GDB.\n\
+Numeric ranges may be specified with the form LOW-HIGH (e.g. 1-5).\n\
+The special arg \"all\" is recognized to mean all signals except those\n\
+used by the debugger, typically SIGTRAP and SIGINT.\n",
+"Recognized actions include \"stop\", \"nostop\", \"print\", \"noprint\",\n\
+\"pass\", \"nopass\", \"ignore\", or \"noignore\".\n\
 Stop means reenter debugger if this signal happens (implies print).\n\
+Print means print a message if this signal happens.\n\
 Pass means let program see this signal; otherwise program doesn't know.\n\
-Pass and Stop may be combined.");
-
-  for (i = 0; i < NSIG; i++)
+Ignore is a synonym for nopass and noignore is a synonym for pass.\n\
+Pass and Stop may be combined.", NULL));
+
+  stop_command = add_cmd ("stop", class_obscure, not_just_help_class_command,
+          "There is no `stop' command, but you can set a hook on `stop'.\n\
+This allows you to set a list of commands to be run each time execution\n\
+of the program stops.", &cmdlist);
+
+  numsigs = (int)TARGET_SIGNAL_LAST;
+  signal_stop = (unsigned char *)    
+    xmalloc (sizeof (signal_stop[0]) * numsigs);
+  signal_print = (unsigned char *)
+    xmalloc (sizeof (signal_print[0]) * numsigs);
+  signal_program = (unsigned char *)
+    xmalloc (sizeof (signal_program[0]) * numsigs);
+  for (i = 0; i < numsigs; i++)
     {
       signal_stop[i] = 1;
       signal_print[i] = 1;
@@ -1355,37 +2218,34 @@ Pass and Stop may be combined.");
 
   /* Signals caused by debugger's own actions
      should not be given to the program afterwards.  */
-  signal_program[SIGTRAP] = 0;
-  signal_program[SIGINT] = 0;
+  signal_program[TARGET_SIGNAL_TRAP] = 0;
+  signal_program[TARGET_SIGNAL_INT] = 0;
 
   /* Signals that are not errors should not normally enter the debugger.  */
-#ifdef SIGALRM
-  signal_stop[SIGALRM] = 0;
-  signal_print[SIGALRM] = 0;
-#endif /* SIGALRM */
-#ifdef SIGVTALRM
-  signal_stop[SIGVTALRM] = 0;
-  signal_print[SIGVTALRM] = 0;
-#endif /* SIGVTALRM */
-#ifdef SIGPROF
-  signal_stop[SIGPROF] = 0;
-  signal_print[SIGPROF] = 0;
-#endif /* SIGPROF */
-#ifdef SIGCHLD
-  signal_stop[SIGCHLD] = 0;
-  signal_print[SIGCHLD] = 0;
-#endif /* SIGCHLD */
-#ifdef SIGCLD
-  signal_stop[SIGCLD] = 0;
-  signal_print[SIGCLD] = 0;
-#endif /* SIGCLD */
-#ifdef SIGIO
-  signal_stop[SIGIO] = 0;
-  signal_print[SIGIO] = 0;
-#endif /* SIGIO */
-#ifdef SIGURG
-  signal_stop[SIGURG] = 0;
-  signal_print[SIGURG] = 0;
-#endif /* SIGURG */
+  signal_stop[TARGET_SIGNAL_ALRM] = 0;
+  signal_print[TARGET_SIGNAL_ALRM] = 0;
+  signal_stop[TARGET_SIGNAL_VTALRM] = 0;
+  signal_print[TARGET_SIGNAL_VTALRM] = 0;
+  signal_stop[TARGET_SIGNAL_PROF] = 0;
+  signal_print[TARGET_SIGNAL_PROF] = 0;
+  signal_stop[TARGET_SIGNAL_CHLD] = 0;
+  signal_print[TARGET_SIGNAL_CHLD] = 0;
+  signal_stop[TARGET_SIGNAL_IO] = 0;
+  signal_print[TARGET_SIGNAL_IO] = 0;
+  signal_stop[TARGET_SIGNAL_POLL] = 0;
+  signal_print[TARGET_SIGNAL_POLL] = 0;
+  signal_stop[TARGET_SIGNAL_URG] = 0;
+  signal_print[TARGET_SIGNAL_URG] = 0;
+
+#ifdef SOLIB_ADD
+  add_show_from_set
+    (add_set_cmd ("stop-on-solib-events", class_support, var_zinteger,
+                  (char *) &stop_on_solib_events,
+                 "Set stopping for shared library events.\n\
+If nonzero, gdb will give control to the user when the dynamic linker\n\
+notifies gdb of shared library events.  The most common event of interest\n\
+to the user would be loading/unloading of a new library.\n",
+                  &setlist),
+     &showlist);
+#endif
 }
-
This page took 0.076899 seconds and 4 git commands to generate.