Replacing a bogus file with a semi-bogus one (sharing through devo).
[deliverable/binutils-gdb.git] / gdb / monitor.c
index 270f10772e96d91c3529f6d4527a3a726b490dee..bd07a2af4d20a9277c654a4cb387124903bb5878 100644 (file)
@@ -1,6 +1,8 @@
 /* Remote debugging interface for boot monitors, for GDB.
-   Copyright 1990, 1991, 1992, 1993, 1995 Free Software Foundation, Inc.
-   Contributed by Cygnus Support. Written by Rob Savoye for Cygnus.
+   Copyright 1990, 1991, 1992, 1993, 1995, 1996, 1997
+   Free Software Foundation, Inc.
+   Contributed by Cygnus Support.  Written by Rob Savoye for Cygnus.
+   Resurrected from the ashes by Stu Grossman.
 
 This file is part of GDB.
 
@@ -16,7 +18,7 @@ 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., 675 Mass Ave, Cambridge, MA 02139, USA.  */
+Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 
 /* This file was derived from various remote-* modules. It is a collection
    of generic support functions so GDB can talk directly to a ROM based
@@ -28,54 +30,103 @@ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
    (or possibly TELNET) stream to a terminal multiplexor,
    which in turn talks to the target board.  */
 
+/* FIXME 32x64: This code assumes that registers and addresses are at
+   most 32 bits long.  If they can be larger, you will need to declare
+   values as LONGEST and use %llx or some such to print values when
+   building commands to send to the monitor.  Since we don't know of
+   any actual 64-bit targets with ROM monitors that use this code,
+   it's not an issue right now.  -sts 4/18/96  */
+
 #include "defs.h"
 #include "gdbcore.h"
 #include "target.h"
 #include "wait.h"
+#ifdef ANSI_PROTOTYPES
+#include <stdarg.h>
+#else
 #include <varargs.h>
+#endif
 #include <signal.h>
-#include <string.h>
+#include <ctype.h>
+#include "gdb_string.h"
 #include <sys/types.h>
 #include "command.h"
 #include "serial.h"
 #include "monitor.h"
 #include "gdbcmd.h"
 #include "inferior.h"
+#include "gnu-regex.h"
+#include "dcache.h"
+#include "srec.h"
+
+static char *dev_name;
+static struct target_ops *targ_ops;
 
-static void make_xmodem_packet ();
-static void print_xmodem_packet ();
+static void monitor_vsprintf PARAMS ((char *sndbuf, char *pattern, va_list args));
 
-static void monitor_load_ascii_srec PARAMS ((char *file, int fromtty));
+static int readchar PARAMS ((int timeout));
 
-static int monitor_make_srec PARAMS ((char *buffer, int type,
-                                     CORE_ADDR memaddr,
-                                     unsigned char *myaddr, int len));
+static void monitor_command PARAMS ((char *args, int fromtty));
 
 static void monitor_fetch_register PARAMS ((int regno));
 static void monitor_store_register PARAMS ((int regno));
 
-static int from_hex ();
+static void monitor_detach PARAMS ((char *args, int from_tty));
+static void monitor_resume PARAMS ((int pid, int step, enum target_signal sig));
+static void monitor_interrupt PARAMS ((int signo));
+static void monitor_interrupt_twice PARAMS ((int signo));
+static void monitor_interrupt_query PARAMS ((void));
+static void monitor_wait_cleanup PARAMS ((int old_timeout));
+
+static int monitor_wait PARAMS ((int pid, struct target_waitstatus *status));
+static void monitor_fetch_registers PARAMS ((int regno));
+static void monitor_store_registers PARAMS ((int regno));
+static void monitor_prepare_to_store PARAMS ((void));
+static int monitor_xfer_memory PARAMS ((CORE_ADDR memaddr, char *myaddr, int len, int write, struct target_ops *target));
+static void monitor_files_info PARAMS ((struct target_ops *ops));
+static int monitor_insert_breakpoint PARAMS ((CORE_ADDR addr, char *shadow));
+static int monitor_remove_breakpoint PARAMS ((CORE_ADDR addr, char *shadow));
+static void monitor_kill PARAMS ((void));
+static void monitor_load PARAMS ((char *file, int from_tty));
+static void monitor_mourn_inferior PARAMS ((void));
+static void monitor_stop PARAMS ((void));
+
+static int monitor_read_memory PARAMS ((CORE_ADDR addr, char *myaddr,int len));
+static int monitor_write_memory PARAMS ((CORE_ADDR addr, char *myaddr,int len));
+static int monitor_write_memory_bytes PARAMS ((CORE_ADDR addr,
+                                              char *myaddr,int len));
+static int monitor_write_memory_block PARAMS((
+                                             CORE_ADDR memaddr ,
+                                             char * myaddr ,
+                                             int len)) ;
+static int monitor_expect_regexp PARAMS ((struct re_pattern_buffer *pat,
+                                         char *buf, int buflen));
+static void monitor_dump_regs PARAMS((void)) ;
+#if 0
+static int from_hex PARAMS ((int a));
 static unsigned long get_hex_word PARAMS ((void));
+#endif
+static void parse_register_dump PARAMS ((char *, int));
 
-struct monitor_ops *current_monitor;
-
-static char *loadtype_str = "srec";
-static char *loadproto_str = "none";
+static struct monitor_ops *current_monitor;
 
-static int hashmark;                           /* flag set by "set hash" */
+static int hashmark;           /* flag set by "set hash" */
 
 static int timeout = 30;
 
-static int expect PARAMS ((char *string, char *buf, int buflen));
-static int expect_prompt PARAMS ((char *buf, int buflen));
+static int in_monitor_wait = 0;        /* Non-zero means we are in monitor_wait() */
 
-/* Having this larger than 400 causes us to be incompatible with m68k-stub.c
-   and i386-stub.c.  Normally, no one would notice because it only matters
-   for writing large chunks of memory (e.g. in downloads).  Also, this needs
-   to be more than 400 if required to hold the registers (see below, where
-   we round it up based on REGISTER_BYTES).  */
+static void (*ofunc)();                /* Old SIGINT signal handler */
 
-#define PBUFSIZ 400
+/* Extra remote debugging for developing a new rom monitor variation */
+#if ! defined(EXTRA_RDEBUG)
+#define EXTRA_RDEBUG 1
+#endif
+#if EXTRA_RDEBUG
+#define RDEBUG(stuff) { if (remote_debug) printf stuff ; }
+#else
+#define RDEBUG(stuff) {}
+#endif
 
 /* Descriptor for I/O to remote machine.  Initialize it to NULL so
    that monitor_open knows that we don't have a file open when the
@@ -83,89 +134,226 @@ static int expect_prompt PARAMS ((char *buf, int buflen));
 
 static serial_t monitor_desc = NULL;
 
-static void monitor_load_srec ();
+/* Pointer to regexp pattern matching data */
 
-/* These definitions are for xmodem protocol. */
+static struct re_pattern_buffer register_pattern;
+static char register_fastmap[256];
 
-#define SOH    0x01
-#define ACK    0x06
-#define NAK    0x15
-#define EOT    0x04
-#define CANCEL 0x18
-#define GETACK         getacknak(ACK)
-#define GETNAK         getacknak(NAK)
-#define XMODEM_DATASIZE        128             /* the data   size is ALWAYS 128 */
-#define XMODEM_PACKETSIZE      131     /* the packet size is ALWAYS 132 (zero based) */
-#define XMODEM         1
+static struct re_pattern_buffer getmem_resp_delim_pattern;
+static char getmem_resp_delim_fastmap[256];
 
-static unsigned char output_buf[0x200];
-static int obp;
+static int dump_reg_flag;      /* Non-zero means do a dump_registers cmd when
+                                  monitor_wait wakes up.  */
 
-static void
-debug_save_output (buf, len)
-     unsigned char *buf;
-     int len;
+static DCACHE *remote_dcache;
+static int first_time=0;       /* is this the first time we're executing after 
+                                       gaving created the child proccess? */
+
+/* Convert hex digit A to a number.  */
+
+static int
+fromhex (a)
+     int a;
 {
-#if 0
-  for (; len > 0; len--)
-    output_buf[obp++ & 0x1ff] = *buf++;
-#else
-  fputs_unfiltered (buf, gdb_stdout);
-#endif
+  if (a >= '0' && a <= '9')
+    return a - '0';
+  else if (a >= 'a' && a <= 'f')
+    return a - 'a' + 10;
+  else
+    if (a >= 'A' && a <= 'F')
+      return a - 'A' + 10 ;
+  else error ("Invalid hex digit %d", a);
 }
 
-static unsigned char input_buf[0x200];
-static int ibp;
+/* monitor_vsprintf - similar to vsprintf but handles 64-bit addresses
+
+   This function exists to get around the problem that many host platforms
+   don't have a printf that can print 64-bit addresses.  The %A format
+   specification is recognized as a special case, and causes the argument
+   to be printed as a 64-bit hexadecimal address.
 
+   Only format specifiers of the form "[0-9]*[a-z]" are recognized.
+   If it is a '%s' format, the argument is a string; otherwise the
+   argument is assumed to be a long integer.
+*/
+  
 static void
-debug_save_input_char (c)
-     int c;
+monitor_vsprintf (sndbuf, pattern, args)
+     char *sndbuf;
+     char *pattern;
+     va_list args;
 {
-#if 0
-  input_buf[ibp++ & 0x1ff] = c;
+  char format[10];
+  char fmt;
+  char *p;
+  int i;
+  long arg_int;
+  CORE_ADDR arg_addr;
+  char *arg_string;
+
+  for (p = pattern; *p; p++)
+    {
+      if (*p == '%')
+       {
+         /* Copy the format specifier to a separate buffer.  */
+         format[0] = *p++;
+         for (i = 1; *p >= '0' && *p <= '9' && i < (int) sizeof (format) - 2;
+              i++, p++)
+           format[i] = *p;
+         format[i] = fmt = *p;
+         format[i+1] = '\0';
+
+         /* Fetch the next argument and print it.  */
+         switch (fmt)
+           {
+           case 'A':
+             arg_addr = va_arg (args, CORE_ADDR);
+             strcpy (sndbuf, paddr_nz (arg_addr));
+             break;
+           case 's':
+             arg_string = va_arg (args, char *);
+             sprintf (sndbuf, format, arg_string);
+             break;
+           default:
+             arg_int = va_arg (args, long);
+             sprintf (sndbuf, format, arg_int);
+             break;
+           }
+         sndbuf += strlen (sndbuf);
+       }
+      else
+       *sndbuf++ = *p;
+    }
+  *sndbuf = '\0';
+}
+
+
+/* monitor_printf_noecho -- Send data to monitor, but don't expect an echo.
+   Works just like printf.  */
+
+void
+#ifdef ANSI_PROTOTYPES
+monitor_printf_noecho (char *pattern, ...)
+#else
+monitor_printf_noecho (va_alist)
+     va_dcl
+#endif
+{
+  va_list args;
+  char sndbuf[2000];
+  int len;
+
+#if ANSI_PROTOTYPES
+  va_start (args, pattern);
 #else
-  fputc_unfiltered (c, gdb_stdout);
+  char *pattern;
+  va_start (args);
+  pattern = va_arg (args, char *);
+#endif
+
+  monitor_vsprintf (sndbuf, pattern, args);
+
+#if 0
+  if (remote_debug > 0)
+    puts_debug ("sent -->", sndbuf, "<--");
 #endif
+  RDEBUG(("sent[%s]\n",sndbuf)) ;
+  
+  len = strlen (sndbuf);
+
+  if (len + 1 > sizeof sndbuf)
+    abort ();
+
+  monitor_write (sndbuf, len);
 }
 
-/* printf_monitor -- send data to monitor.  Works just like printf. */
+/* monitor_printf -- Send data to monitor and check the echo.  Works just like
+   printf.  */
 
-static void
-printf_monitor (va_alist)
+void
+#ifdef ANSI_PROTOTYPES
+monitor_printf (char *pattern, ...)
+#else
+monitor_printf (va_alist)
      va_dcl
+#endif
 {
   va_list args;
-  char *pattern;
-  char buf[PBUFSIZ];
+  char sndbuf[2000];
+  int len;
 
+#ifdef ANSI_PROTOTYPES
+  va_start (args, pattern);
+#else
+  char *pattern;
   va_start (args);
-
   pattern = va_arg (args, char *);
+#endif
 
-  vsprintf (buf, pattern, args);
+  monitor_vsprintf (sndbuf, pattern, args);
 
+#if 0
   if (remote_debug > 0)
-    debug_save_output (buf, strlen (buf));
+    puts_debug ("sent -->", sndbuf, "<--");
+#endif
+  RDEBUG(("sent[%s]\n",sndbuf)) 
+  len = strlen (sndbuf);
+  if (len + 1 > sizeof sndbuf)
+    abort ();
+
+  monitor_write (sndbuf, len);
+
+  /* We used to expect that the next immediate output was the characters we
+     just output, but sometimes some extra junk appeared before the characters
+     we expected, like an extra prompt, or a portmaster sending telnet negotiations.
+     So, just start searching for what we sent, and skip anything unknown.  */
+  RDEBUG(("ExpectEcho\n"))
+  monitor_expect (sndbuf, (char *)0, 0);
+}
 
-  if (strlen (buf) > PBUFSIZ)
-    error ("printf_monitor(): string too long");
-  if (SERIAL_WRITE(monitor_desc, buf, strlen (buf)))
+
+/* Write characters to the remote system.  */
+
+void
+monitor_write (buf, buflen)
+     char *buf;
+     int buflen;
+{
+  if (SERIAL_WRITE(monitor_desc, buf, buflen))
     fprintf_unfiltered (stderr, "SERIAL_WRITE failed: %s\n", safe_strerror (errno));
 }
 
-/* Send raw data to monitor.  */
 
-static void
-write_monitor (data, len)
-     char *data;
-     int len;
+/* Read a binary character from the remote system, doing all the fancy
+   timeout stuff, but without interpreting the character in any way,
+   and without printing remote debug information.  */
+
+int
+monitor_readchar ()
 {
-  if (SERIAL_WRITE (monitor_desc, data, len))
-    fprintf_unfiltered (stderr, "SERIAL_WRITE failed: %s\n", safe_strerror(errno));
-  *(data + len + 1) = '\0';
+  int c;
+  int looping;
+
+  do
+    {
+      looping = 0;
+      c = SERIAL_READCHAR (monitor_desc, timeout);
+
+      if (c >= 0)
+       c &= 0xff;                      /* don't lose bit 7 */
+    }
+  while (looping);
+
+  if (c >= 0)
+    return c;
+
+  if (c == SERIAL_TIMEOUT)
+      error ("Timeout reading from remote system.");
+
+  perror_with_name ("remote-monitor");
 }
 
+
 /* Read a character from the remote system, doing all the fancy
    timeout stuff.  */
 
@@ -174,31 +362,80 @@ readchar (timeout)
      int timeout;
 {
   int c;
+  static enum { last_random, last_nl, last_cr, last_crnl } state = last_random;
+  int looping;
 
-  c = SERIAL_READCHAR (monitor_desc, timeout);
+  do
+    {
+      looping = 0;
+      c = SERIAL_READCHAR (monitor_desc, timeout);
 
-  if (remote_debug > 0)
-    debug_save_input_char (c & 0x7f);
+      if (c >= 0)
+       {
+         c &= 0x7f;
+#if 0
+         /* This seems to interfere with proper function of the
+            input stream */
+         if (remote_debug > 0)
+           {
+             char buf[2];
+             buf[0] = c;
+             buf[1] = '\0';
+             puts_debug ("read -->", buf, "<--");
+           }
+         
+#endif   
+       }
+
+      /* Canonicialize \n\r combinations into one \r */
+      if ((current_monitor->flags & MO_HANDLE_NL) != 0)
+       {
+         if ((c == '\r' && state == last_nl)
+             || (c == '\n' && state == last_cr))
+           {
+             state = last_crnl;
+             looping = 1;
+           }
+         else if (c == '\r')
+           state = last_cr;
+         else if (c != '\n')
+           state = last_random;
+         else
+           {
+             state = last_nl;
+             c = '\r';
+           }
+       }
+    }
+  while (looping);
 
   if (c >= 0)
-    return c & 0x7f;
+    return c;
 
   if (c == SERIAL_TIMEOUT)
-    {
-      if (timeout == 0)
-       return c;               /* Polls shouldn't generate timeout errors */
+#if 0 /* MAINTENANCE_CMDS */
+    /* I fail to see how detaching here can be useful
+    if (in_monitor_wait)       /* Watchdog went off */
+      {
+       target_mourn_inferior ();
+       error ("GDB serial timeout has expired.  Target detached.\n");
+      }
+    else
+#endif
       error ("Timeout reading from remote system.");
-    }
+
   perror_with_name ("remote-monitor");
 }
 
 /* Scan input from the remote system, until STRING is found.  If BUF is non-
-   zero, then collect input until either STRING has been collected or BUFLEN
-   chars have been collected.  If input overflows BUF because STRING can't be
-   found, return -1, else return number of chars in BUF (including STRING).  */
+   zero, then collect input until we have collected either STRING or BUFLEN-1
+   chars.  In either case we terminate BUF with a 0.  If input overflows BUF
+   because STRING can't be found, return -1, else return number of chars in BUF
+   (minus the terminating NUL).  Note that in the non-overflow case, STRING
+   will be at the end of BUF.  */
 
-static int
-expect (string, buf, buflen)
+int
+monitor_expect (string, buf, buflen)
      char *string;
      char *buf;
      int buflen;
@@ -206,34 +443,53 @@ expect (string, buf, buflen)
   char *p = string;
   int obuflen = buflen;
   int c;
+  extern struct target_ops *targ_ops;
+  RDEBUG(("MON Expecting '%s'\n",string)) ;
 
   immediate_quit = 1;
   while (1)
     {
       if (buf)
        {
-         if (buflen <= 0)
+         if (buflen < 2)
            {
+             *buf = '\000';
              immediate_quit = 0;
              return -1;
            }
 
          c = readchar (timeout);
+         if (c == '\000')
+           continue;
          *buf++ = c;
          buflen--;
        }
       else
        c = readchar (timeout);
 
-      if (c == *p++)
+      /* Don't expect any ^C sent to be echoed */
+       
+      if (*p == '\003' || c == *p)
        {
+         p++;
          if (*p == '\0')
            {
              immediate_quit = 0;
 
-             return obuflen - buflen;
+             if (buf)
+               {
+                 *buf++ = '\000';
+                 return obuflen - buflen;
+               }
+             else
+               return 0;
            }
        }
+      else if ((c == '\021' || c == '\023') &&
+              (strcmp(targ_ops->to_shortname, "m32r") == 0))
+       { /* m32r monitor emits random DC1/DC3 chars */
+         continue;
+       }
       else
        {
          p = string;
@@ -243,98 +499,75 @@ expect (string, buf, buflen)
     }
 }
 
-/* Keep discarding input until we see the MONITOR prompt.
-
-   The convention for dealing with the prompt is that you
-   o give your command
-   o *then* wait for the prompt.
-
-   Thus the last thing that a procedure does with the serial line
-   will be an expect_prompt().  Exception:  monitor_resume does not
-   wait for the prompt, because the terminal is being handed over
-   to the inferior.  However, the next thing which happens after that
-   is a monitor_wait which does wait for the prompt.
-   Note that this includes abnormal exit, e.g. error().  This is
-   necessary to prevent getting into states from which we can't
-   recover.  */
+/* Search for a regexp.  */
 
 static int
-expect_prompt (buf, buflen)
+monitor_expect_regexp (pat, buf, buflen)
+     struct re_pattern_buffer *pat;
      char *buf;
      int buflen;
 {
-  return expect (PROMPT, buf, buflen);
-}
-
-/* Ignore junk characters. Returns a 1 if junk, 0 otherwise.  */
-
-static int
-junk (ch)
-     char ch;
-{
-  switch (ch)
+  char *mybuf;
+  char *p;
+  RDEBUG(("MON Expecting regexp\n")) ;
+  if (buf)
+    mybuf = buf;
+  else
     {
-    case '\0':
-    case ' ':
-    case '-':
-    case '\t':
-    case '\r':
-    case '\n':
-      return 1;
-    default:
-      return 0;
+      mybuf = alloca (1024);
+      buflen = 1024;
     }
-}
-
-/* Get a hex digit from the remote system & return its value.  If
-   ignore is nonzero, ignore spaces, newline & tabs.  */
-
-static int
-get_hex_digit (ignore)
-     int ignore;
-{
-  static int ch;
 
+  p = mybuf;
   while (1)
     {
-      ch = readchar (timeout);
-      if (junk (ch))
-       continue;
-
-      if (ch >= '0' && ch <= '9')
-       return ch - '0';
-      else if (ch >= 'A' && ch <= 'F')
-       return ch - 'A' + 10;
-      else if (ch >= 'a' && ch <= 'f')
-       return ch - 'a' + 10;
-      else if (ch == ' ' && ignore)
-       ;
-      else
-       {
-         expect_prompt (NULL, 0);
-         error ("Invalid hex digit from remote system. (0x%x)", ch);
+      int retval;
+
+      if (p - mybuf >= buflen)
+       {                       /* Buffer about to overflow */
+
+/* On overflow, we copy the upper half of the buffer to the lower half.  Not
+   great, but it usually works... */
+
+         memcpy (mybuf, mybuf + buflen / 2, buflen / 2);
+         p = mybuf + buflen / 2;
        }
+
+      *p++ = readchar (timeout);
+
+      retval = re_search (pat, mybuf, p - mybuf, 0, p - mybuf, NULL);
+      if (retval >= 0)
+       return 1;
     }
 }
 
-/* Get a byte from monitor and put it in *BYT. Accept any number of
-   leading spaces.  */
+/* Keep discarding input until we see the MONITOR prompt.
 
-static void
-get_hex_byte (byt)
-     char *byt;
-{
-  int val;
+   The convention for dealing with the prompt is that you
+   o give your command
+   o *then* wait for the prompt.
 
-  val = get_hex_digit (1) << 4;
-  val |= get_hex_digit (0);
-  *byt = val;
+   Thus the last thing that a procedure does with the serial line will
+   be an monitor_expect_prompt().  Exception: monitor_resume does not
+   wait for the prompt, because the terminal is being handed over to
+   the inferior.  However, the next thing which happens after that is
+   a monitor_wait which does wait for the prompt.  Note that this
+   includes abnormal exit, e.g. error().  This is necessary to prevent
+   getting into states from which we can't recover.  */
+
+int
+monitor_expect_prompt (buf, buflen)
+     char *buf;
+     int buflen;
+{
+  RDEBUG(("MON Expecting prompt\n"))
+  return monitor_expect (current_monitor->prompt, buf, buflen);
 }
 
 /* Get N 32-bit words from remote, each preceded by a space, and put
    them in registers starting at REGNO.  */
 
+#if 0
 static unsigned long
 get_hex_word ()
 {
@@ -358,61 +591,46 @@ get_hex_word ()
 
   return val;
 }
+#endif
 
-/* This is called not only when we first attach, but also when the
-   user types "run" after having attached.  */
-
-void
-monitor_create_inferior (execfile, args, env)
-     char *execfile;
-     char *args;
-     char **env;
+static void
+compile_pattern (pattern, compiled_pattern, fastmap)
+     char *pattern;
+     struct re_pattern_buffer *compiled_pattern;
+     char *fastmap;
 {
-  int entry_pt;
-
-  if (args && *args)
-    error ("Can't pass arguments to remote MONITOR process");
-
-  if (execfile == 0 || exec_bfd == 0)
-    error ("No exec file specified");
+  int tmp;
+  char *val;
 
-  entry_pt = (int) bfd_get_start_address (exec_bfd);
+  compiled_pattern->fastmap = fastmap;
 
-  /* The "process" (board) is already stopped awaiting our commands, and
-     the program is already downloaded.  We just set its PC and go.  */
+  tmp = re_set_syntax (RE_SYNTAX_EMACS);
+  val = re_compile_pattern (pattern,
+                           strlen (pattern),
+                           compiled_pattern);
+  re_set_syntax (tmp);
 
-  clear_proceed_status ();
-
-  /* Tell wait_for_inferior that we've started a new process.  */
-  init_wait_for_inferior ();
-
-  /* Set up the "saved terminal modes" of the inferior
-     based on what modes we are starting it with.  */
-  target_terminal_init ();
-
-  /* Install inferior's terminal modes.  */
-  target_terminal_inferior ();
-
-  /* insert_step_breakpoint ();  FIXME, do we need this?  */
+  if (val)
+    error ("compile_pattern: Can't compile pattern string `%s': %s!", pattern, val);
 
-  /* Let 'er rip... */
-  proceed ((CORE_ADDR)entry_pt, TARGET_SIGNAL_DEFAULT, 0);
+  if (fastmap)
+    re_compile_fastmap (compiled_pattern);
 }
 
 /* Open a connection to a remote debugger. NAME is the filename used
    for communication.  */
 
-static char *dev_name;
-
 void
 monitor_open (args, mon_ops, from_tty)
      char *args;
      struct monitor_ops *mon_ops;
      int from_tty;
 {
-  struct target_ops *targ_ops;
   char *name;
-  int i;
+  char **p;
+
+  if (mon_ops->magic != MONITOR_OPS_MAGIC)
+    error ("Magic number of monitor_ops struct wrong.");
 
   targ_ops = mon_ops->target;
   name = targ_ops->to_shortname;
@@ -423,6 +641,16 @@ monitor_open (args, mon_ops, from_tty)
 
   target_preopen (from_tty);
 
+  /* Setup pattern for register dump */
+
+  if (mon_ops->register_pattern)
+    compile_pattern (mon_ops->register_pattern, &register_pattern,
+                    register_fastmap);
+
+  if (mon_ops->getmem.resp_delim)
+    compile_pattern (mon_ops->getmem.resp_delim, &getmem_resp_delim_pattern,
+                    getmem_resp_delim_fastmap);
+
   unpush_target (targ_ops);
 
   if (dev_name)
@@ -453,34 +681,41 @@ monitor_open (args, mon_ops, from_tty)
 
   current_monitor = mon_ops;
 
-  /* see if the target is alive. For a ROM monitor, we can just try to
-     force the prompt to print a few times.  */
-
-  /* wake up the monitor and see if it's alive */
-  printf_monitor (mon_ops->init);
-  expect_prompt (NULL, 0);     /* See if we get a prompt */
+  /* See if we can wake up the monitor.  First, try sending a stop sequence,
+     then send the init strings.  Last, remove all breakpoints.  */
 
-  /* try again to be sure */
-  printf_monitor (mon_ops->init);
-  expect_prompt (NULL, 0);     /* See if we get a prompt */
+  if (current_monitor->stop)
+    {
+      monitor_stop ();
+      if ((current_monitor->flags & MO_NO_ECHO_ON_OPEN) == 0)
+        {
+         RDEBUG(("EXP Open echo\n")) ;
+         monitor_expect_prompt (NULL, 0); 
+      }
+    }
 
-  /* Setup the suffixes for the `set remoteloadtype' command */
+  /* wake up the monitor and see if it's alive */
+  for (p = mon_ops->init; *p != NULL; p++)
+    {
+      /* Some of the characters we send may not be echoed,
+        but we hope to get a prompt at the end of it all. */
+        
+      if ((current_monitor->flags & MO_NO_ECHO_ON_OPEN) == 0)
+        monitor_printf(*p); 
+      else
+        monitor_printf_noecho (*p);
+      monitor_expect_prompt (NULL, 0);
+    }
 
-  add_show_from_set (add_set_enum_cmd ("remoteloadtype", no_class,
-                                      mon_ops->loadtypes,
-                                      (char *)&loadtype_str,
-                                      "Set the remote load type.",
-                                      &setlist),
-                    &showlist);
+  SERIAL_FLUSH_INPUT (monitor_desc);
 
-  /* Setup the suffixes for the `set remoteloadprotocol' command */
+  /* Remove all breakpoints */
 
-  add_show_from_set (add_set_enum_cmd ("remoteloadprotocol", no_class,
-                                      mon_ops->loadprotos,
-                                      (char *)&loadproto_str,
-                                      "Set the remote load protocol.",
-                                      &setlist),
-                    &showlist);
+  if (mon_ops->clr_all_break)
+    {
+      monitor_printf (mon_ops->clr_all_break);
+      monitor_expect_prompt (NULL, 0);
+    }
 
   if (from_tty)
     printf_unfiltered ("Remote target %s connected to %s\n", name, dev_name);
@@ -489,8 +724,14 @@ monitor_open (args, mon_ops, from_tty)
 
   inferior_pid = 42000;                /* Make run command think we are busy... */
 
-  printf_monitor ("\r");
+  /* Give monitor_wait something to read */
+
+  monitor_printf (current_monitor->line_term);
 
+  if (current_monitor->flags & MO_HAS_BLOCKWRITES)
+    remote_dcache = dcache_init (monitor_read_memory, monitor_write_memory_block);
+  else
+    remote_dcache = dcache_init (monitor_read_memory, monitor_write_memory);
   start_remote ();
 }
 
@@ -509,7 +750,7 @@ monitor_close (quitting)
 /* Terminate the open connection to the remote debugger.  Use this
    when you want to detach and do something else with your gdb.  */
 
-void
+static void
 monitor_detach (args, from_tty)
      char *args;
      int from_tty;
@@ -519,56 +760,271 @@ monitor_detach (args, from_tty)
     printf_unfiltered ("Ending remote %s debugging\n", target_shortname);
 }
 
+/* Convert VALSTR into the target byte-ordered value of REGNO and store it.  */
+
+char *
+monitor_supply_register (regno, valstr)
+     int regno;
+     char *valstr;
+{
+  unsigned int val;
+  unsigned char regbuf[MAX_REGISTER_RAW_SIZE];
+  char *p;
+
+  val = strtoul (valstr, &p, 16);
+  RDEBUG(("Supplying Register %d %s\n",regno,valstr)) ;
+
+  if (val == 0 && valstr == p)
+    error ("monitor_supply_register (%d):  bad value from monitor: %s.",
+          regno, valstr);
+
+  /* supply register stores in target byte order, so swap here */
+
+  store_unsigned_integer (regbuf, REGISTER_RAW_SIZE (regno), val);
+
+  supply_register (regno, regbuf);
+
+  return p;
+}
+
 /* Tell the remote machine to resume.  */
 
-void
+int flush_monitor_dcache(void) {  dcache_flush (remote_dcache); }
+
+static void
 monitor_resume (pid, step, sig)
      int pid, step;
      enum target_signal sig;
 {
+  /* Some monitors require a different command when starting a program */
+  RDEBUG(("MON resume\n")) ;
+  if (current_monitor->flags & MO_RUN_FIRST_TIME && first_time == 1)
+    {
+      first_time = 0;
+      monitor_printf ("run\r");
+      if (current_monitor->flags & MO_NEED_REGDUMP_AFTER_CONT)
+           dump_reg_flag = 1;
+      return;
+    }
+  dcache_flush (remote_dcache);
   if (step)
-    printf_monitor (STEP_CMD);
+    monitor_printf (current_monitor->step);
   else
-    printf_monitor (CONT_CMD);
+    {
+      if (current_monitor->continue_hook)
+       (*current_monitor->continue_hook)() ;
+      else monitor_printf (current_monitor->cont);
+      if (current_monitor->flags & MO_NEED_REGDUMP_AFTER_CONT)
+       dump_reg_flag = 1;
+    }
 }
 
-/* Wait until the remote machine stops, then return, storing status in
-   status just as `wait' would.  */
+/* Parse the output of a register dump command.  A monitor specific
+   regexp is used to extract individual register descriptions of the
+   form REG=VAL.  Each description is split up into a name and a value
+   string which are passed down to monitor specific code.  */
 
-int
-monitor_wait (pid, status)
-     int pid;
-     struct target_waitstatus *status;
+static void
+parse_register_dump (buf, len)
+     char *buf;
+     int len;
 {
-  int old_timeout = timeout;
+  RDEBUG(("MON Parsing  register dump\n")) 
+  while (1)
+    {
+      int regnamelen, vallen;
+      char *regname, *val;
+      /* Element 0 points to start of register name, and element 1
+        points to the start of the register value.  */
+      struct re_registers register_strings;
+
+      if (re_search (&register_pattern, buf, len, 0, len,
+                    &register_strings) == -1)
+       break;
 
-  status->kind = TARGET_WAITKIND_EXITED;
-  status->value.integer = 0;
+      regnamelen = register_strings.end[1] - register_strings.start[1];
+      regname = buf + register_strings.start[1];
+      vallen = register_strings.end[2] - register_strings.start[2];
+      val = buf + register_strings.start[2];
 
-  timeout = -1;                /* Don't time out -- user program is running. */
+      current_monitor->supply_register (regname, regnamelen, val, vallen);
 
-  expect_prompt (NULL, 0);     /* Wait for prompt, outputting extraneous text */
+      buf += register_strings.end[0];
+      len -= register_strings.end[0];
+    }
+}
 
-  status->kind = TARGET_WAITKIND_STOPPED;
-  status->value.sig = TARGET_SIGNAL_TRAP;
+/* Send ^C to target to halt it.  Target will respond, and send us a
+   packet.  */
 
-  timeout = old_timeout;
+static void
+monitor_interrupt (signo)
+     int signo;
+{
+  /* If this doesn't work, try more severe steps.  */
+  signal (signo, monitor_interrupt_twice);
+  
+  if (remote_debug)
+    printf_unfiltered ("monitor_interrupt called\n");
 
-  return inferior_pid;
+  target_stop ();
 }
 
-/* Return the name of register number regno in the form input and output by
-   monitor.  Currently, register_names just happens to contain exactly what
-   monitor wants.  Lets take advantage of that just as long as possible! */
+/* The user typed ^C twice.  */
 
-static char *
-get_reg_name (regno)
-     int regno;
+static void
+monitor_interrupt_twice (signo)
+     int signo;
 {
-  if (regno < 0 || regno > NUM_REGS)
-    return NULL;
-  else
-    return REGNAMES (regno);
+  signal (signo, ofunc);
+  
+  monitor_interrupt_query ();
+
+  signal (signo, monitor_interrupt);
+}
+
+/* Ask the user what to do when an interrupt is received.  */
+
+static void
+monitor_interrupt_query ()
+{
+  target_terminal_ours ();
+
+  if (query ("Interrupted while waiting for the program.\n\
+Give up (and stop debugging it)? "))
+    {
+      target_mourn_inferior ();
+      return_to_top_level (RETURN_QUIT);
+    }
+
+  target_terminal_inferior ();
+}
+
+static void
+monitor_wait_cleanup (old_timeout)
+     int old_timeout;
+{
+  timeout = old_timeout;
+  signal (SIGINT, ofunc);
+  in_monitor_wait = 0;
+}
+
+
+
+void monitor_wait_filter(char * buf,
+                        int bufmax,
+                        int * ext_resp_len, 
+                        struct target_waitstatus * status
+                        )
+{
+  int resp_len ;
+  do
+    {
+      resp_len = monitor_expect_prompt (buf, bufmax);
+      * ext_resp_len =resp_len ;
+
+      if (resp_len <= 0)
+       fprintf_unfiltered (gdb_stderr, "monitor_wait:  excessive response from monitor: %s.", buf);
+    }
+  while (resp_len < 0);
+
+  /* Print any output characters that were preceded by ^O.  */
+  /* FIXME - This would be great as a user settabgle flag */
+  if (remote_debug ||
+      current_monitor->flags & MO_PRINT_PROGRAM_OUTPUT)
+    {
+      int i;
+
+      for (i = 0; i < resp_len - 1; i++)
+       if (buf[i] == 0x0f)
+         putchar_unfiltered (buf[++i]);
+    }
+}
+
+
+
+/* Wait until the remote machine stops, then return, storing status in
+   status just as `wait' would.  */
+
+static int
+monitor_wait (pid, status)
+     int pid;
+     struct target_waitstatus *status;
+{
+  int old_timeout = timeout;
+  char buf[1024];
+  int resp_len;
+  struct cleanup *old_chain;
+
+  status->kind = TARGET_WAITKIND_EXITED;
+  status->value.integer = 0;
+
+  old_chain = make_cleanup (monitor_wait_cleanup, old_timeout);
+  RDEBUG(("MON wait\n"))
+
+#if 0 /* MAINTENANCE_CMDS */
+    /* This is somthing other than a maintenance command */
+  in_monitor_wait = 1;
+  timeout = watchdog > 0 ? watchdog : -1;
+#else
+  timeout = -1;                /* Don't time out -- user program is running. */
+#endif
+
+  ofunc = (void (*)()) signal (SIGINT, monitor_interrupt);
+
+  if (current_monitor->wait_filter)
+    (*current_monitor->wait_filter)(buf,sizeof (buf),&resp_len,status) ;
+  else          monitor_wait_filter(buf,sizeof (buf),&resp_len,status) ;
+      
+#if 0 /* Transferred to monitor wait filter */
+  do
+    {
+      resp_len = monitor_expect_prompt (buf, sizeof (buf));
+
+      if (resp_len <= 0)
+       fprintf_unfiltered (gdb_stderr, "monitor_wait:  excessive response from monitor: %s.", buf);
+    }
+  while (resp_len < 0);
+
+  /* Print any output characters that were preceded by ^O.  */
+  /* FIXME - This would be great as a user settabgle flag */
+  if (remote_debug ||
+      current_monitor->flags & MO_PRINT_PROGRAM_OUTPUT)
+    {
+      int i;
+
+      for (i = 0; i < resp_len - 1; i++)
+       if (buf[i] == 0x0f)
+         putchar_unfiltered (buf[++i]);
+    }
+#endif  
+
+  signal (SIGINT, ofunc);
+
+  timeout = old_timeout;
+#if 0
+  if (dump_reg_flag && current_monitor->dump_registers)
+    {
+      dump_reg_flag = 0;
+      monitor_printf (current_monitor->dump_registers);
+      resp_len = monitor_expect_prompt (buf, sizeof (buf));
+    }
+
+  if (current_monitor->register_pattern)
+    parse_register_dump (buf, resp_len);
+#else
+  RDEBUG(("Wait fetching registers after stop\n")) ;
+  monitor_dump_regs() ;
+#endif  
+
+  status->kind = TARGET_WAITKIND_STOPPED;
+  status->value.sig = TARGET_SIGNAL_TRAP;
+
+  discard_cleanups (old_chain);
+
+  in_monitor_wait = 0;
+
+  return inferior_pid;
 }
 
 /* Fetch register REGNO, or all registers if REGNO is -1. Returns
@@ -578,85 +1034,150 @@ static void
 monitor_fetch_register (regno)
      int regno;
 {
-  unsigned LONGEST val;
-  unsigned char regbuf[MAX_REGISTER_RAW_SIZE];
-  char buf[200];
-  char *p, *p1;
   char *name;
-  int resp_len;
+  static char zerobuf[MAX_REGISTER_RAW_SIZE] = {0};
+  char regbuf[MAX_REGISTER_RAW_SIZE * 2 + 1];
+  int i;
 
-  name = get_reg_name (regno);
+  name = current_monitor->regnames[regno];
+  RDEBUG(("MON fetchreg %d '%s'\n",regno,name)) 
 
-  if (!name)
-    return;
+  if (!name || (*name == '\0'))
+    { RDEBUG(("No register known for %d\n",regno))
+      supply_register (regno, zerobuf);
+      return;
+    }
 
- /* send the register examine command */
 /* send the register examine command */
 
-  printf_monitor (current_monitor->getreg.cmd, name);
+  monitor_printf (current_monitor->getreg.cmd, name);
 
-/* If TERM is present, we wait for that to show up.  Also, (if TERM is
-   present), we will send TERM_CMD if that is present.  In any case, we collect
-   all of the output into buf, and then wait for the normal prompt.  */
+  /* If RESP_DELIM is specified, we search for that as a leading
+     delimiter for the register value.  Otherwise, we just start
+     searching from the start of the buf.  */
 
-  if (current_monitor->getreg.term)
+  if (current_monitor->getreg.resp_delim)
     {
-      resp_len = expect (current_monitor->getreg.term, buf, sizeof buf); /* get response */
-
-      if (resp_len <= 0)
-       error ("monitor_fetch_register (%d):  excessive response from monitor: %.*s.", resp_len, buf);
-
-      if (current_monitor->getreg.term_cmd)
-       {
-         SERIAL_WRITE (monitor_desc, current_monitor->getreg.term_cmd,
-                       strlen (current_monitor->getreg.term_cmd));
-         expect_prompt (NULL, 0);
+      RDEBUG(("EXP getreg.resp_delim\n")) 
+      monitor_expect (current_monitor->getreg.resp_delim, NULL, 0);
+      /* Handle case of first 32 registers listed in pairs.  */
+      if (current_monitor->flags & MO_32_REGS_PAIRED
+         && regno & 1 == 1 && regno < 32)
+       { RDEBUG(("EXP getreg.resp_delim\n")) ;
+         monitor_expect (current_monitor->getreg.resp_delim, NULL, 0);
        }
     }
-  else
-    resp_len = expect_prompt (buf, sizeof buf); /* get response */
 
+  /* Skip leading spaces and "0x" if MO_HEX_PREFIX flag is set */
+  if (current_monitor->flags & MO_HEX_PREFIX) 
+    {
+      int c;
+      c = readchar (timeout);
+      while (c == ' ')
+       c = readchar (timeout);
+      if ((c == '0') && ((c = readchar (timeout)) == 'x'))
+       ;
+      else
+         error ("Bad value returned from monitor while fetching register %x.",
+                regno);
+    }
 
-  /* If RESP_DELIM is specified, we search for that as a leading delimiter for
-     the register value.  Otherwise, we just start searching from the start of
-     the buf.  */
+  /* Read upto the maximum number of hex digits for this register, skipping
+     spaces, but stop reading if something else is seen.  Some monitors
+     like to drop leading zeros.  */
 
-  if (current_monitor->getreg.resp_delim)
+  for (i = 0; i < REGISTER_RAW_SIZE (regno) * 2; i++)
     {
-      p = strstr (buf, current_monitor->getreg.resp_delim);
-      if (!p)
-       error ("monitor_fetch_register (%d):  bad response from monitor: %.*s.", resp_len, buf);
-      p += strlen (current_monitor->getreg.resp_delim);
+      int c;
+      c = readchar (timeout);
+      while (c == ' ')
+       c = readchar (timeout);
+
+      if (!isxdigit (c))
+       break;
+
+      regbuf[i] = c;
     }
-  else
-    p = buf;
 
-  val = strtoul (p, &p1, 16);
+  regbuf[i] = '\000';          /* terminate the number */
+  RDEBUG(("REGVAL '%s'\n",regbuf)) ;
 
-  if (val == 0 && p == p1)
-    error ("monitor_fetch_register (%d):  bad value from monitor: %.*s.",
-          resp_len, buf);
+  /* If TERM is present, we wait for that to show up.  Also, (if TERM
+     is present), we will send TERM_CMD if that is present.  In any
+     case, we collect all of the output into buf, and then wait for
+     the normal prompt.  */
 
-  /* supply register stores in target byte order, so swap here */
+  if (current_monitor->getreg.term)
+    {
+      RDEBUG(("EXP getreg.term\n"))
+      monitor_expect (current_monitor->getreg.term, NULL, 0); /* get response */
+    }
 
-  store_unsigned_integer (regbuf, REGISTER_RAW_SIZE (regno), val);
+  if (current_monitor->getreg.term_cmd)
+    { RDEBUG(("EMIT getreg.term.cmd\n"))
+       monitor_printf (current_monitor->getreg.term_cmd);
+    }
+  if (! current_monitor->getreg.term ||    /* Already expected or */
+      current_monitor->getreg.term_cmd)    /* ack expected */
+    monitor_expect_prompt (NULL, 0); /* get response */
 
-  supply_register (regno, regbuf);
+  monitor_supply_register (regno, regbuf);
+}
+
+/* Sometimes, it takes several commands to dump the registers */
+/* This is a primitive for use by variations of monitor interfaces in
+   case they need to compose the operation.
+   */
+int monitor_dump_reg_block(char * block_cmd)
+{
+  char buf[1024];
+  int resp_len;
+  monitor_printf (block_cmd);
+  resp_len = monitor_expect_prompt (buf, sizeof (buf));
+  parse_register_dump (buf, resp_len);
+  return 1 ;
 }
 
+
 /* Read the remote registers into the block regs.  */
+/* Call the specific function if it has been provided */
 
-void
+static void
+monitor_dump_regs ()
+{
+  char buf[1024];
+  int resp_len;
+  if (current_monitor->dumpregs)
+     (*(current_monitor->dumpregs))() ; /* call supplied function */
+  else
+    if (current_monitor->dump_registers) /* default version */
+    { monitor_printf (current_monitor->dump_registers);
+      resp_len = monitor_expect_prompt (buf, sizeof (buf));
+      parse_register_dump (buf, resp_len);
+    }
+  else
+    abort(); /* Need some way to read registers */
+}
+
+static void
 monitor_fetch_registers (regno)
      int regno;
 {
-  if (regno >= 0)
+  RDEBUG(("MON fetchregs\n")) ;
+  if (current_monitor->getreg.cmd) 
     {
-      monitor_fetch_register (regno);
-      return;
-    }
+      if (regno >= 0)
+       {
+         monitor_fetch_register (regno);
+         return;
+       }
 
-  for (regno = 0; regno < NUM_REGS; regno++)
-    monitor_fetch_register (regno);
+      for (regno = 0; regno < NUM_REGS; regno++)
+       monitor_fetch_register (regno);
+    }
+  else {
+    monitor_dump_regs ();
+  }
 }
 
 /* Store register REGNO, or all if REGNO == 0.  Return errno value.  */
@@ -666,29 +1187,45 @@ monitor_store_register (regno)
      int regno;
 {
   char *name;
-  unsigned LONGEST val;
+  unsigned int val;
 
-  name = get_reg_name (regno);
-  if (!name)
-    return;
+  name = current_monitor->regnames[regno];
+  if (!name || (*name == '\0'))
+    { RDEBUG(("MON Cannot store unknown register\n"))
+      return;
+    }
 
   val = read_register (regno);
+  RDEBUG(("MON storeg %d %08x\n",regno,(unsigned int)val))
 
- /* send the register deposit command */
 /* send the register deposit command */
 
-  printf_monitor (current_monitor->setreg.cmd, name, val);
-
-/* It's possible that there are actually some monitors out there that will
-   prompt you when you set a register.  In that case, you may need to add some
-   code here to deal with TERM and TERM_CMD (see monitor_fetch_register to get
-   an idea of what's needed...) */
-
-  expect_prompt (NULL, 0);
-}
+  if (current_monitor->flags & MO_REGISTER_VALUE_FIRST)
+    monitor_printf (current_monitor->setreg.cmd, val, name);
+  else if (current_monitor->flags & MO_SETREG_INTERACTIVE)
+    monitor_printf (current_monitor->setreg.cmd, name);
+  else
+    monitor_printf (current_monitor->setreg.cmd, name, val);
+
+  if (current_monitor->setreg.term)
+    { RDEBUG(("EXP setreg.term\n"))
+      monitor_expect (current_monitor->setreg.term, NULL, 0);
+      if (current_monitor->flags & MO_SETREG_INTERACTIVE)
+       monitor_printf ("%x\r", val);
+      monitor_expect_prompt (NULL, 0);
+    }
+  else
+    monitor_expect_prompt (NULL, 0);
+  if (current_monitor->setreg.term_cmd) /* Mode exit required */
+    { RDEBUG(("EXP setreg_termcmd\n")) ;
+      monitor_printf("%s",current_monitor->setreg.term_cmd) ;
+      monitor_expect_prompt(NULL,0) ;
+    }
+} /* monitor_store_register */
 
 /* Store the remote registers.  */
 
-void
+static void
 monitor_store_registers (regno)
      int regno;
 {
@@ -708,41 +1245,428 @@ monitor_store_registers (regno)
    that registers contains all the registers from the program being
    debugged.  */
 
-void
+static void
 monitor_prepare_to_store ()
 {
   /* Do nothing, since we can store individual regs */
 }
 
-void
-monitor_files_info ()
+static void
+monitor_files_info (ops)
+     struct target_ops *ops;
 {
   printf_unfiltered ("\tAttached to %s at %d baud.\n", dev_name, baud_rate);
 }
 
-static int
-monitor_write_memory (memaddr, myaddr, len)
-     CORE_ADDR memaddr;
-     unsigned char *myaddr;
-     int len;
-{
- /* send the memory deposit command */
+static int
+monitor_write_memory (memaddr, myaddr, len)
+     CORE_ADDR memaddr;
+     char *myaddr;
+     int len;
+{
+  unsigned int val, hostval ;
+  char *cmd;
+  int i;
+
+  RDEBUG(("MON write %d %08x\n",len,(unsigned long)memaddr))
+
+  if (current_monitor->flags & MO_ADDR_BITS_REMOVE)
+    memaddr = ADDR_BITS_REMOVE (memaddr);
+
+  /* Use memory fill command for leading 0 bytes.  */
+
+  if (current_monitor->fill)
+    {
+      for (i = 0; i < len; i++)
+       if (myaddr[i] != 0)
+         break;
+
+      if (i > 4)               /* More than 4 zeros is worth doing */
+       {
+         RDEBUG(("MON FILL %d\n",i))
+           if (current_monitor->flags & MO_FILL_USES_ADDR)
+             monitor_printf (current_monitor->fill, memaddr, (memaddr + i)-1, 0);
+           else
+             monitor_printf (current_monitor->fill, memaddr, i, 0);
+
+         monitor_expect_prompt (NULL, 0);
+
+         return i;
+       }
+    }
+
+#if 0
+  /* Can't actually use long longs if VAL is an int (nice idea, though).  */
+  if ((memaddr & 0x7) == 0 && len >= 8 && current_monitor->setmem.cmdll)
+    {
+      len = 8;
+      cmd = current_monitor->setmem.cmdll;
+    }
+  else
+#endif
+  if ((memaddr & 0x3) == 0 && len >= 4 && current_monitor->setmem.cmdl)
+    {
+      len = 4;
+      cmd = current_monitor->setmem.cmdl;
+    }
+  else if ((memaddr & 0x1) == 0 && len >= 2 && current_monitor->setmem.cmdw)
+    {
+      len = 2;
+      cmd = current_monitor->setmem.cmdw;
+    }
+  else
+    {
+      len = 1;
+      cmd = current_monitor->setmem.cmdb;
+    }
+
+  val = extract_unsigned_integer (myaddr, len);
+  
+  if (len == 4)
+    { hostval = * (unsigned int *) myaddr ;
+      RDEBUG(("Hostval(%08x) val(%08x)\n",hostval,val)) ;
+    }
+
+
+  if (current_monitor->flags & MO_NO_ECHO_ON_SETMEM)
+    monitor_printf_noecho (cmd, memaddr, val);
+  else if (current_monitor->flags & MO_SETMEM_INTERACTIVE)
+    {
+
+      monitor_printf_noecho (cmd, memaddr);
+
+      if (current_monitor->setmem.term)
+       { RDEBUG(("EXP setmem.term")) ;
+         monitor_expect (current_monitor->setmem.term, NULL, 0);
+         monitor_printf ("%x\r", val);
+       }
+      if (current_monitor->setmem.term_cmd)
+       { /* Emit this to get out of the memory editing state */
+         monitor_printf("%s",current_monitor->setmem.term_cmd) ;
+         /* Drop through to expecting a prompt */
+       }
+    }
+  else
+    monitor_printf (cmd, memaddr, val);
+
+  monitor_expect_prompt (NULL, 0);
+
+  return len;
+}
+
+
+static monitor_write_even_block(memaddr,myaddr,len)
+     CORE_ADDR memaddr ;
+     char * myaddr ;
+     int len ;
+{
+  unsigned int val ;
+  int written = 0 ;;
+  /* Enter the sub mode */
+  monitor_printf(current_monitor->setmem.cmdl,memaddr) ;
+  monitor_expect_prompt(NULL,0) ;
+  while (len)
+    {
+      val = extract_unsigned_integer(myaddr,4) ; /* REALLY */
+      monitor_printf("%x\r",val) ;
+      myaddr += 4 ;
+      memaddr += 4 ;
+      written += 4 ;
+      RDEBUG((" @ %08x\n",memaddr))
+      /* If we wanted to, here we could validate the address */
+      monitor_expect_prompt(NULL,0) ;
+    }
+  /* Now exit the sub mode */
+  monitor_printf (current_monitor->getreg.term_cmd);
+  monitor_expect_prompt(NULL,0) ;
+  return written ;
+}
+
+
+static int monitor_write_memory_bytes(memaddr,myaddr,len)
+     CORE_ADDR memaddr ;
+     char * myaddr ;
+     int len ;
+{
+  unsigned char val ;
+  int written = 0 ;
+  if (len == 0) return 0 ;
+  /* Enter the sub mode */
+  monitor_printf(current_monitor->setmem.cmdb,memaddr) ;
+  monitor_expect_prompt(NULL,0) ;
+  while (len)
+    {
+      val = *myaddr ;
+      monitor_printf("%x\r",val) ;
+      myaddr++ ;
+      memaddr++ ;
+      written++ ;
+      /* If we wanted to, here we could validate the address */
+      monitor_expect_prompt(NULL,0) ;
+      len-- ;
+    }
+  /* Now exit the sub mode */
+  monitor_printf (current_monitor->getreg.term_cmd);
+  monitor_expect_prompt(NULL,0) ;
+  return written ;
+}
+
+
+static longlongendswap(unsigned char * a)
+{
+  int i,j ;
+  unsigned char x ;
+  i = 0 ; j = 7 ;
+  while (i < 4)
+    { x = *(a+i) ;
+      *(a+i) = *(a+j) ;
+      *(a+j) = x ;
+      i++ , j-- ;
+    }
+}
+/* Format 32 chars of long long value, advance the pointer */
+static char * hexlate = "0123456789abcdef" ;
+static char * longlong_hexchars(unsigned long long value,
+                                char * outbuff )
+{
+  if (value == 0) { *outbuff++ = '0' ; return outbuff ; }
+  else
+    { static unsigned char disbuf[8] ; /* disassembly buffer */
+      unsigned char * scan , * limit ; /* loop controls */
+      unsigned char c , nib ;
+      int leadzero = 1 ;
+      scan = disbuf ; limit = scan + 8 ;
+      { unsigned long long * dp ;
+       dp = (unsigned long long *) scan ;
+       *dp = value ;
+      }
+      longlongendswap(disbuf) ; /* FIXME: ONly on big endian hosts */
+      while (scan < limit)
+       { c = *scan++ ; /* a byte of our long long value */
+         if (leadzero)
+           if (c == 0) continue ;
+           else leadzero = 0 ; /* henceforth we print even zeroes */
+         nib = c >> 4 ;        /* high nibble bits */
+         *outbuff++ = hexlate[nib] ;
+         nib = c & 0x0f ;      /* low nibble bits */
+         *outbuff++ = hexlate[nib] ;
+       }
+      return outbuff ;
+    }
+} /* longlong_hexchars */
+
+
+
+/* I am only going to call this when writing virtual byte streams.
+   Which possably entails endian conversions
+   */
+static int monitor_write_memory_longlongs(memaddr,myaddr,len)
+     CORE_ADDR memaddr ;
+     char * myaddr ;
+     int len ;
+{
+  static char hexstage[20] ; /* At least 16 digits required, plus null */
+  char * endstring ;
+  long long * llptr ;
+  long long value ;
+  int written = 0 ;
+  llptr = (unsigned long long *) myaddr ;
+  if (len == 0 ) return 0 ;
+  monitor_printf(current_monitor->setmem.cmdll,memaddr) ;
+  monitor_expect_prompt(NULL,0) ;
+  while (len >= 8 )
+    {
+      value = *llptr ;
+      endstring = longlong_hexchars(*llptr,hexstage) ;
+      *endstring = '\0' ; /* NUll terminate for printf */
+      monitor_printf("%s\r",hexstage) ;
+      llptr++ ;
+      memaddr += 8 ;
+      written += 8 ;
+      /* If we wanted to, here we could validate the address */
+      monitor_expect_prompt(NULL,0) ;
+      len -= 8 ;
+    }
+  /* Now exit the sub mode */
+  monitor_printf (current_monitor->getreg.term_cmd);
+  monitor_expect_prompt(NULL,0) ;
+  return written ;
+} /* */
+
+
+
+/* ----- MONITOR_WRITE_MEMORY_BLOCK ---------------------------- */
+/* This is for the large blocks of memory which may occur in downloading.
+   And for monitors which use interactive entry,
+   And for monitors which do not have other downloading methods.
+   Without this, we will end up calling monitor_write_memory many times
+   and do the entry and exit of the sub mode many times
+   This currently assumes...
+     MO_SETMEM_INTERACTIVE
+     ! MO_NO_ECHO_ON_SETMEM
+     To use this, the you have to patch the monitor_cmds block with
+     this function. Otherwise, its not tuned up for use by all
+     monitor variations.
+   */
+
+static int monitor_write_memory_block(memaddr,myaddr,len)
+     CORE_ADDR memaddr ;
+     char * myaddr ;
+     int len ;
+{
+  int written ;
+  written = 0 ;
+  /* FIXME: This would be a good place to put the zero test */
+#if 1 
+  if ((len > 8) && (((len & 0x07)) == 0) && current_monitor->setmem.cmdll)
+       {
+        return monitor_write_memory_longlongs(memaddr,myaddr,len) ;
+       }
+#endif  
+#if 0 
+  if (len > 4)
+    {
+      int sublen ;
+      written = monitor_write_even_block(memaddr,myaddr,len) ;
+      /* Adjust calling parameters by written amount */
+      memaddr += written ;
+      myaddr += written ;
+      len -= written ;
+    }
+#endif
+  written = monitor_write_memory_bytes(memaddr,myaddr,len) ;
+  return written ;
+}
+
+/* This is an alternate form of monitor_read_memory which is used for monitors
+   which can only read a single byte/word/etc. at a time.  */
+
+static int
+monitor_read_memory_single (memaddr, myaddr, len)
+     CORE_ADDR memaddr;
+     char *myaddr;
+     int len;
+{
+  unsigned int val;
+  char membuf[sizeof(int) * 2 + 1];
+  char *p;
+  char *cmd;
+  int i;
+
+  RDEBUG(("MON read single\n")) ;
+#if 0
+  /* Can't actually use long longs (nice idea, though).  In fact, the
+     call to strtoul below will fail if it tries to convert a value
+     that's too big to fit in a long.  */
+  if ((memaddr & 0x7) == 0 && len >= 8 && current_monitor->getmem.cmdll)
+    {
+      len = 8;
+      cmd = current_monitor->getmem.cmdll;
+    }
+  else
+#endif
+  if ((memaddr & 0x3) == 0 && len >= 4 && current_monitor->getmem.cmdl)
+    {
+      len = 4;
+      cmd = current_monitor->getmem.cmdl;
+    }
+  else if ((memaddr & 0x1) == 0 && len >= 2 && current_monitor->getmem.cmdw)
+    {
+      len = 2;
+      cmd = current_monitor->getmem.cmdw;
+    }
+  else
+    {
+      len = 1;
+      cmd = current_monitor->getmem.cmdb;
+    }
+
+  /* Send the examine command.  */
+
+  monitor_printf (cmd, memaddr);
+
+  /* If RESP_DELIM is specified, we search for that as a leading
+     delimiter for the memory value.  Otherwise, we just start
+     searching from the start of the buf.  */
+
+  if (current_monitor->getmem.resp_delim)
+    { RDEBUG(("EXP getmem.resp_delim\n")) ;
+      monitor_expect_regexp (&getmem_resp_delim_pattern, NULL, 0);
+    }
+
+  /* Now, read the appropriate number of hex digits for this loc,
+     skipping spaces.  */
+
+  /* Skip leading spaces and "0x" if MO_HEX_PREFIX flag is set. */
+  if (current_monitor->flags & MO_HEX_PREFIX) 
+    {
+      int c;
+
+      c = readchar (timeout);
+      while (c == ' ')
+       c = readchar (timeout);
+      if ((c == '0') && ((c = readchar (timeout)) == 'x'))
+       ;
+      else
+       error ("monitor_read_memory_single (0x%x):  bad response from monitor: %.*s%c.",
+              memaddr, i, membuf, c);
+    }
+  for (i = 0; i < len * 2; i++)
+    {
+      int c;
+
+      while (1)
+       {
+         c = readchar (timeout);
+         if (isxdigit (c))
+           break;
+         if (c == ' ')
+           continue;
+
+         error ("monitor_read_memory_single (0x%x):  bad response from monitor: %.*s%c.",
+                memaddr, i, membuf, c);
+       }
+
+      membuf[i] = c;
+    }
+
+  membuf[i] = '\000';          /* terminate the number */
+
+/* If TERM is present, we wait for that to show up.  Also, (if TERM is
+   present), we will send TERM_CMD if that is present.  In any case, we collect
+   all of the output into buf, and then wait for the normal prompt.  */
+
+  if (current_monitor->getmem.term)
+    {
+      monitor_expect (current_monitor->getmem.term, NULL, 0); /* get response */
+
+      if (current_monitor->getmem.term_cmd)
+       {
+         monitor_printf (current_monitor->getmem.term_cmd);
+         monitor_expect_prompt (NULL, 0);
+       }
+    }
+  else
+    monitor_expect_prompt (NULL, 0); /* get response */
+
+  p = membuf;
+  val = strtoul (membuf, &p, 16);
 
-  printf_monitor (current_monitor->setmem.cmd, memaddr, *myaddr);
+  if (val == 0 && membuf == p)
+    error ("monitor_read_memory_single (0x%x):  bad value from monitor: %s.",
+          memaddr, membuf);
 
-/* It's possible that there are actually some monitors out there that will
-   prompt you when you deposit to memory.  In that case, you may need to add
-   some code here to deal with TERM and TERM_CMD (see monitor_read_memory to
-   get an idea of what's needed...) */
+  /* supply register stores in target byte order, so swap here */
 
-  expect_prompt (NULL, 0);
+  store_unsigned_integer (myaddr, len, val);
 
-  return 1;
+  return len;
 }
 
-/* Copy LEN bytes of data from debugger memory at MYADDR to inferior's memory
-   at MEMADDR.  Returns length moved.  Currently, we only do one byte at a
-   time.  */
+/* Copy LEN bytes of data from debugger memory at MYADDR to inferior's
+   memory at MEMADDR.  Returns length moved.  Currently, we do no more
+   than 16 bytes at a time.  */
 
 static int
 monitor_read_memory (memaddr, myaddr, len)
@@ -750,66 +1674,169 @@ monitor_read_memory (memaddr, myaddr, len)
      char *myaddr;
      int len;
 {
-  unsigned LONGEST val;
-  unsigned char regbuf[MAX_REGISTER_RAW_SIZE];
-  char buf[200];
+  unsigned int val;
+  char buf[512];
   char *p, *p1;
-  char *name;
   int resp_len;
+  int i;
+  CORE_ADDR dumpaddr;
 
- /* send the memory examine command */
+  if (remote_debug) printf("MON read block ta(%08x) ha(%08x) %d\n",
+         (unsigned long) memaddr , (unsigned long)myaddr, len);
 
-  printf_monitor (current_monitor->getmem.cmd, memaddr);
+  if (current_monitor->flags & MO_ADDR_BITS_REMOVE)
+    memaddr = ADDR_BITS_REMOVE (memaddr);
 
-/* If TERM is present, we wait for that to show up.  Also, (if TERM is
-   present), we will send TERM_CMD if that is present.  In any case, we collect
-   all of the output into buf, and then wait for the normal prompt.  */
+  if (current_monitor->flags & MO_GETMEM_READ_SINGLE)
+    return monitor_read_memory_single (memaddr, myaddr, len);
+
+  len = min (len, 16);
+
+  /* Some dumpers align the first data with the preceeding 16
+     byte boundary. Some print blanks and start at the
+     requested boundary. EXACT_DUMPADDR
+     */
+
+  dumpaddr = (current_monitor->flags & MO_EXACT_DUMPADDR)
+    ? memaddr : memaddr & ~ 0x0f ;
+
+  /* See if xfer would cross a 16 byte boundary.  If so, clip it.  */
+  if (((memaddr ^ (memaddr + len - 1)) & ~0xf) != 0)
+    len = ((memaddr + len) & ~0xf) - memaddr;
+
+  /* send the memory examine command */
+
+  if (current_monitor->flags & MO_GETMEM_NEEDS_RANGE)
+    monitor_printf (current_monitor->getmem.cmdb, memaddr, memaddr + len - 1);
+  else if (current_monitor->flags & MO_GETMEM_16_BOUNDARY)
+    monitor_printf (current_monitor->getmem.cmdb, dumpaddr);
+  else
+    monitor_printf (current_monitor->getmem.cmdb, memaddr, len);
+
+  /* If TERM is present, we wait for that to show up.  Also, (if TERM
+     is present), we will send TERM_CMD if that is present.  In any
+     case, we collect all of the output into buf, and then wait for
+     the normal prompt.  */
 
   if (current_monitor->getmem.term)
     {
-      resp_len = expect (current_monitor->getmem.term, buf, sizeof buf); /* get response */
+      resp_len = monitor_expect (current_monitor->getmem.term, buf, sizeof buf); /* get response */
 
       if (resp_len <= 0)
-       error ("monitor_read_memory (%d):  excessive response from monitor: %.*s.", resp_len, buf);
+       error ("monitor_read_memory (0x%x):  excessive response from monitor: %.*s.",
+              memaddr, resp_len, buf);
 
       if (current_monitor->getmem.term_cmd)
        {
          SERIAL_WRITE (monitor_desc, current_monitor->getmem.term_cmd,
                        strlen (current_monitor->getmem.term_cmd));
-         expect_prompt (NULL, 0);
+         monitor_expect_prompt (NULL, 0);
        }
     }
   else
-    resp_len = expect_prompt (buf, sizeof buf); /* get response */
+    resp_len = monitor_expect_prompt (buf, sizeof buf); /* get response */
 
-  /* If RESP_DELIM is specified, we search for that as a leading delimiter for
-     the register value.  Otherwise, we just start searching from the start of
-     the buf.  */
+  p = buf;
+
+  /* If RESP_DELIM is specified, we search for that as a leading
+     delimiter for the values.  Otherwise, we just start searching
+     from the start of the buf.  */
 
   if (current_monitor->getmem.resp_delim)
     {
-      p = strstr (buf, current_monitor->getmem.resp_delim);
+      int retval, tmp;
+      struct re_registers resp_strings;
+      RDEBUG(("MON getmem.resp_delim %s\n",current_monitor->getmem.resp_delim)) ;
+
+      tmp = strlen (p);
+      retval = re_search (&getmem_resp_delim_pattern, p, tmp, 0, tmp,
+                         &resp_strings);
+
+      if (retval < 0)
+       error ("monitor_read_memory (0x%x):  bad response from monitor: %.*s.",
+              memaddr, resp_len, buf);
+
+      p += resp_strings.end[0];
+#if 0
+      p = strstr (p, current_monitor->getmem.resp_delim);
       if (!p)
-       error ("monitor_read_memory (%d):  bad response from monitor: %.*s.", resp_len, buf);
+       error ("monitor_read_memory (0x%x):  bad response from monitor: %.*s.",
+              memaddr, resp_len, buf);
       p += strlen (current_monitor->getmem.resp_delim);
+#endif
     }
-  else
-    p = buf;
+  if (remote_debug) printf("MON scanning  %d ,%08x '%s'\n",len,p,p) ;
+  if (current_monitor->flags & MO_GETMEM_16_BOUNDARY)
+    {
+      char c ;
+      int fetched = 0 ;
+      i = len;
+      c = *p ;
+
+                           
+      while (!(c  == '\000' || c == '\n' || c == '\r') && i > 0)
+       { if (isxdigit (c))
+           { if ((dumpaddr >= memaddr) && (i > 0))
+               { val = fromhex (c) * 16 + fromhex (*(p+1));
+                 *myaddr++ = val;
+                 if (remote_debug) printf("[%02x]",val) ;
+                 --i;
+                 fetched++ ;
+               }
+             ++dumpaddr;
+             ++p;
+           }
+         ++p; /* skip a blank or other non hex char */
+         c = *p ;
+       }
+      if (fetched == 0) error("Failed to read via monitor") ;
+      if (remote_debug) printf("\n") ;
+      return fetched ; /* Return the number of bytes actually read */
+    }
+  RDEBUG(("MON scanning bytes\n")) ;
+
+  for (i = len; i > 0; i--)
+    {
+      /* Skip non-hex chars, but bomb on end of string and newlines */
 
-  val = strtoul (p, &p1, 16);
+      while (1)
+       {
+         if (isxdigit (*p))
+           break;
+
+         if (*p == '\000' || *p == '\n' || *p == '\r')
+           error ("monitor_read_memory (0x%x):  badly terminated response from monitor: %.*s", memaddr, resp_len, buf);
+         p++;
+       }
+
+      val = strtoul (p, &p1, 16);
 
-  if (val == 0 && p == p1)
-    error ("monitor_read_memory (%d):  bad value from monitor: %.*s.",
-          resp_len, buf);
+      if (val == 0 && p == p1)
+       error ("monitor_read_memory (0x%x):  bad value from monitor: %.*s.", memaddr,
+              resp_len, buf);
 
-  *myaddr = val;
+      *myaddr++ = val;
+
+      if (i == 1)
+       break;
+
+      p = p1;
+    }
 
-  return 1;                    /* Got 1 byte */
+  return len;
 }
 
-/* FIXME-someday!  merge these two.  */
+/* This version supports very large reads by looping on multiline
+   dump bytes outputs. Beware of buffering limits.
+   */
+static int monotor_read_memory_block(memaddr,myaddr,len)
+     CORE_ADDR memaddr ;
+     char * myaddr ;
+     int len ;
+{
+}
 
-int
+static int
 monitor_xfer_memory (memaddr, myaddr, len, write, target)
      CORE_ADDR memaddr;
      char *myaddr;
@@ -817,29 +1844,41 @@ monitor_xfer_memory (memaddr, myaddr, len, write, target)
      int write;
      struct target_ops *target;                /* ignored */
 {
-  if (write)
-    return monitor_write_memory (memaddr, myaddr, len);
-  else
-    return monitor_read_memory (memaddr, myaddr, len);
+  return dcache_xfer_memory (remote_dcache, memaddr, myaddr, len, write);
 }
 
-void
-monitor_kill (args, from_tty)
-     char *args;
-     int from_tty;
+static void
+monitor_kill ()
 {
   return;              /* ignore attempts to kill target system */
 }
 
+/* All we actually do is set the PC to the start address of exec_bfd, and start
+   the program at that point.  */
+
+static void
+monitor_create_inferior (exec_file, args, env)
+     char *exec_file;
+     char *args;
+     char **env;
+{
+  if (args && (*args != '\000'))
+    error ("Args are not supported by the monitor.");
+
+  first_time = 1;
+  clear_proceed_status ();
+  proceed (bfd_get_start_address (exec_bfd), TARGET_SIGNAL_0, 0);
+}
+
 /* Clean up when a program exits.
    The program actually lives on in the remote processor's RAM, and may be
    run again without a download.  Don't leave it full of breakpoint
    instructions.  */
 
-void
+static void
 monitor_mourn_inferior ()
 {
-  remove_breakpoints ();
+  unpush_target (targ_ops);
   generic_mourn_inferior ();   /* Do all the proper things now */
 }
 
@@ -849,22 +1888,33 @@ static CORE_ADDR breakaddr[NUM_MONITOR_BREAKPOINTS] = {0};
 
 /* Tell the monitor to add a breakpoint.  */
 
-int
+static int
 monitor_insert_breakpoint (addr, shadow)
      CORE_ADDR addr;
      char *shadow;
 {
   int i;
-  static unsigned char break_insn[] = BREAKPOINT;
+  unsigned char *bp;
+  int bplen;
+
+  RDEBUG(("MON inst bkpt %08x\n",addr)) 
+  if (current_monitor->set_break == NULL)
+    error ("No set_break defined for this monitor");
+
+  if (current_monitor->flags & MO_ADDR_BITS_REMOVE)
+    addr = ADDR_BITS_REMOVE (addr);
+
+  /* Determine appropriate breakpoint size for this address.  */
+  bp = memory_breakpoint_from_pc (&addr, &bplen);
 
   for (i = 0; i < NUM_MONITOR_BREAKPOINTS; i++)
     {
       if (breakaddr[i] == 0)
        {
          breakaddr[i] = addr;
-         monitor_read_memory (addr, shadow, sizeof (break_insn));
-         printf_monitor (SET_BREAK_CMD, addr);
-         expect_prompt (NULL, 0);
+         monitor_read_memory (addr, shadow, bplen);
+         monitor_printf (current_monitor->set_break, addr);
+         monitor_expect_prompt (NULL, 0);
          return 0;
        }
     }
@@ -874,24 +1924,33 @@ monitor_insert_breakpoint (addr, shadow)
 
 /* Tell the monitor to remove a breakpoint.  */
 
-int
+static int
 monitor_remove_breakpoint (addr, shadow)
      CORE_ADDR addr;
      char *shadow;
 {
   int i;
 
+  RDEBUG(("MON rmbkpt %08x\n",addr))
+  if (current_monitor->clr_break == NULL)
+    error ("No clr_break defined for this monitor");
+
+  if (current_monitor->flags & MO_ADDR_BITS_REMOVE)
+    addr = ADDR_BITS_REMOVE (addr);
+
   for (i = 0; i < NUM_MONITOR_BREAKPOINTS; i++)
     {
       if (breakaddr[i] == addr)
        {
          breakaddr[i] = 0;
          /* some monitors remove breakpoints based on the address */
-         if (CLR_BREAK_ADDR)   
-           printf_monitor(CLR_BREAK_CMD, addr);
+         if (current_monitor->flags & MO_CLR_BREAK_USES_ADDR)
+           monitor_printf (current_monitor->clr_break, addr);
+         else if (current_monitor->flags & MO_CLR_BREAK_1_BASED)
+           monitor_printf (current_monitor->clr_break, i + 1);
          else
-           printf_monitor(CLR_BREAK_CMD, i);
-         expect_prompt (NULL, 0);
+           monitor_printf (current_monitor->clr_break, i);
+         monitor_expect_prompt (NULL, 0);
          return 0;
        }
     }
@@ -899,527 +1958,127 @@ monitor_remove_breakpoint (addr, shadow)
   return 1;
 }
 
-/* monitor_load -- load a file. This file determines which of the
- *     supported formats to use. The current types are:
- *     FIXME: not all types supported yet.
- *     default - reads any file using bfd and writes it to memory. This
- *             is really slow.
- *     srec    - reads binary file using bfd and writes it as an
- *             ascii srecord.
- *     xmodem-bin - reads a binary file using bfd, and  downloads it
- *              using xmodem protocol.
- *     xmodem-srec - reads a binary file using bfd, and after converting
- *              it downloads it as an srecord using xmodem protocol.
- *     ascii-srec - reads a ascii srecord file and downloads it
- *             without a change.
- *     ascii-xmodem - reads a ascii file and downloads using xmodem
- *             protocol.
- */
+/* monitor_wait_srec_ack -- wait for the target to send an acknowledgement for
+   an S-record.  Return non-zero if the ACK is received properly.  */
 
-void
-monitor_load (file, fromtty)
-    char *file;
-    int  fromtty;
+static int
+monitor_wait_srec_ack ()
 {
-  /* default, load a binary */
-  if (STREQ (loadtype_str, "default"))
-    {
-      error ("default load type not supported.");
-    }
-
-  /* load an srecord by converting */
-  if ((STREQ (loadtype_str, "srec")) && STREQ (loadproto_str, "xmodem"))
-    {
-      monitor_load_srec (file, XMODEM);
-      return;
-    }
-
-  /* load an srecord by converting */
-  if (STREQ (loadtype_str, "srec"))
-    {
-      monitor_load_srec (file, 0);                     /* if from a binary */
-      return;
-    }
-
-  /* load an srecord by converting */
-  if (STREQ (loadtype_str, "none"))
-    {
-      error ("Unimplemented");
-      return;
-    }
+  int i, ch;
 
-  /* load an srecord file */
-  if (STREQ (loadproto_str, "none"))
+  if (current_monitor->flags & MO_SREC_ACK_PLUS)
     {
-      monitor_load_ascii_srec (file, fromtty);         /* if from a binary */
-      return;
+      return (readchar (timeout) == '+');
     }
-
-  if (STREQ (loadproto_str, "xmodem"))
+  else if (current_monitor->flags & MO_SREC_ACK_ROTATE)
     {
-      monitor_load_srec (file, XMODEM);
-      return;
+      /* Eat two backspaces, a "rotating" char (|/-\), and a space.  */
+      if ((ch = readchar (1)) < 0)
+       return 0;
+      if ((ch = readchar (1)) < 0)
+       return 0;
+      if ((ch = readchar (1)) < 0)
+       return 0;
+      if ((ch = readchar (1)) < 0)
+       return 0;
     }
+  return 1;
 }
 
-/* Download an ASCII srecord file.  */
-
-#define DOWNLOAD_LINE_SIZE 100
+/* monitor_load -- download a file. */
 
 static void
-monitor_load_ascii_srec (file, fromtty)
+monitor_load (file, from_tty)
     char *file;
-    int fromtty;
-{
-  FILE *download;
-  char buf[DOWNLOAD_LINE_SIZE];
-  int i, bytes_read;
-
-  download = fopen (file, "r");
-  if (download == NULL)
-    {
-      error ("%s does not exist", file);
-      return;
-    }
-
-  printf_monitor (LOAD_CMD);
-  sleep (1);
-  while (!feof (download))
-    {
-      bytes_read = fread (buf, sizeof (char), DOWNLOAD_LINE_SIZE, download);
-      if (hashmark)
-       {
-         putchar_unfiltered ('.');
-         gdb_flush (gdb_stdout);
-       }
-      if (SERIAL_WRITE (monitor_desc, buf, bytes_read))
-       {
-         fprintf_unfiltered (stderr, "SERIAL_WRITE failed: (while downloading) %s\n",
-                  safe_strerror (errno));
-         break;
-       }
-      i = 0;
-      while (i++ <=200) {} ;   /* Ugly HACK, probably needs flow control */
-      if (bytes_read < DOWNLOAD_LINE_SIZE)
-       {
-         if (!feof (download))
-           error ("Only read %d bytes\n", bytes_read);
-         break;
-       }
-    }
-  
-  if (hashmark)
-    putchar_unfiltered ('\n');
-
-  if (!feof (download))
-    error ("Never got EOF while downloading");
-  expect_prompt (NULL, 0);
-  fclose (download);
-}
-
-/* Put a command string, in args, out to MONITOR.  Output from MONITOR
-   is placed on the users terminal until the prompt is seen. FIXME: We
-   read the characters ourseleves here cause of a nasty echo.  */
-
-void
-monitor_command (args, fromtty)
-     char *args;
-     int fromtty;
-{
-  char *p;
-
-  p = PROMPT;
-
-  if (monitor_desc == NULL)
-    error ("monitor target not open.");
-
-  /* Send the command.  Note that if no args were supplied, then we're
-     just sending the monitor a newline, which is sometimes useful.  */
-
-  printf_monitor ("%s\n", (args ? args : ""));
-
-  expect_prompt (NULL, 0);
-}
-
-/*  Download a binary file by converting it to srecords. This
-    will also use xmodem to download the resulting file.
-  
-       A download goes like this when using xmodem:
-       Receiver:               Sender
-       NAK ---------->
-               <-------- (packet)      [SOH|1|1|data|SUM]
-       ACK ---------->
-               <-------- (packet)      [SOH|2|2|data|SUM]
-       ACK ---------->
-               <-------- EOT
-       ACK ---------->
-  
-       ACK = 0x06
-       NAK = 0x15
-       EOT = 0x04
-   */
-
-static void
-monitor_load_srec (args, protocol)
-     char *args;
-     int protocol;
+    int  from_tty;
 {
-  bfd *abfd;
-  asection *s;
-  char *buffer, srec[1024];
-  char packet[XMODEM_PACKETSIZE];
-  int i;
-  int retries;
-  int type = 0;                        /* default to a type 0, header record */
-  int srec_frame = 57;         /* FIXME: this must be 57 There is 12 bytes
-                                of header, and 2 bytes of checksum at the end.
-                                The problem is an xmodem packet holds exactly
-                                128 bytes. */
-
-  abfd = bfd_openr (args, 0);
-  if (!abfd)
-    {
-      printf_filtered ("Unable to open file %s\n", args);
-      return;
-    }
-
-  if (bfd_check_format (abfd, bfd_object) == 0)
-    {
-      printf_filtered ("File is not an object file\n");
-      return;
-    }
-  
-  printf_monitor (LOAD_CMD);                   /* tell the monitor to load */
-  sleep (3);
-  /* get the NAK from the target */
-  if (protocol == XMODEM)
-    {
-      if (!GETNAK)
-       {
-         printf_monitor ("%c", EOT);
-         error ("Never got the NAK to start loading");
-       }
-    }
-  
-  s = abfd->sections;
-  while (s != (asection *) NULL)
-    {
-      if (s->flags & SEC_LOAD)
-       {
-         buffer = xmalloc (srec_frame);
-
-         printf_filtered ("%s\t: 0x%4x .. 0x%4x  ",
-                          s->name, s->vma, s->vma + s->_raw_size);
-         gdb_flush (gdb_stdout);
-         for (i = 0; i < s->_raw_size; i += srec_frame)
-           {
-             if (srec_frame > s->_raw_size - i)
-               srec_frame = s->_raw_size - i;
-       
-             bfd_get_section_contents (abfd, s, buffer, i, srec_frame);
-             monitor_make_srec (srec, type, s->vma + i, buffer, srec_frame);
-             /* send a packet using xmodem */
-             if (protocol == XMODEM)
-               {
-                 make_xmodem_packet (packet, srec, XMODEM_DATASIZE);
-                 write_monitor (packet, XMODEM_PACKETSIZE+1);
-                 retries = 0;
-                 while (retries++ <= 3)
-                   {
-                     /* Resend packet */
-                     if (GETNAK)
-                       {
-                         sleep (1);
-                         /* send it again */
-                         write_monitor (packet, XMODEM_PACKETSIZE+1);
-                         if (GETACK)   /* ACKnowledged, get next data chunk */
-                           break;
-                       }
-                     else
-                       {                       /* assume we got an ACK */
-                         if (hashmark)
-                           {
-                             putchar_unfiltered ('#');
-                             gdb_flush (gdb_stdout);
-                           }
-                         break;
-                       }
-                   }
-                 if (retries >= 4)
-                   {                   /* too many tries, must be hosed */
-                     printf_monitor ("%c", EOT);
-                     error ("Never got a ACK after sending an xmodem packet");
-                   }
-               }
-             else
-               {                               /* no protocols at all */
-                 printf_monitor ("%s\n", srec);
-               }
-             if (hashmark)
-               {
-                 putchar_unfiltered ('#');
-                 gdb_flush (gdb_stdout);
-               }
-             type = 3;                         /* switch to a 4 byte address record */
-             gdb_flush (gdb_stdout);
-           }
-         free (buffer);
-       }
-      s = s->next;
-    }
-  putchar_unfiltered ('\n');
-  
-  /* Write a type 7 terminator record. no data for a type 7, and there
-     is no data, so len is 0.  */
+  dcache_flush (remote_dcache);
+  RDEBUG(("MON load\n"))
 
-  if (protocol == XMODEM)
-    {
-      /* send a packet using xmodem */
-      monitor_make_srec (srec, 7, abfd->start_address, "", 0);
-      make_xmodem_packet (packet, srec, XMODEM_DATASIZE);  
-      write_monitor (packet, XMODEM_PACKETSIZE+1);
-    }
+  if (current_monitor->load_routine)
+    current_monitor->load_routine (monitor_desc, file, hashmark);
   else
-    {
-      monitor_make_srec (srec, 7, abfd->start_address, "", 0);
-      printf_monitor ("%s\n", srec);
-    }
-  if (protocol == XMODEM)
-    {
-      printf_monitor ("%c", EOT);
-      if (!GETACK)
-       error ("Never got ACK after sending EOT");
-    }
+    {                          /* The default is ascii S-records */
+      int n;
+      unsigned long load_offset;
+      char buf[128];
+
+      /* enable user to specify address for downloading as 2nd arg to load */
+      n = sscanf (file, "%s 0x%lx", buf, &load_offset);
+      if (n > 1)
+       file = buf;
+      else
+       load_offset = 0;
 
-  if (hashmark) 
-    putchar_unfiltered ('\n');
-  
-  expect_prompt (NULL, 0);
-}
+      monitor_printf (current_monitor->load);
+      if (current_monitor->loadresp)
+       monitor_expect (current_monitor->loadresp, NULL, 0);
 
-/* Get an ACK or a NAK from the target.  returns 1 (true) or 0 (false)
-   This is for xmodem. ANy string starting with "***" is an error
-   message from the target.  Here's a few from the WinBond w89k
-   "Cougar" PA board:
-   *** Too many errors found.
-   *** Bad command
-   *** Command syntax error
-   */
+      load_srec (monitor_desc, file, (bfd_vma) load_offset,
+                32, SREC_ALL, hashmark,
+                current_monitor->flags & MO_SREC_ACK ?
+                  monitor_wait_srec_ack : NULL);
 
-int
-getacknak (byte)
-     int byte;
-{
-  char character;
-  int i;
-  
-  i = 0;
-  while (i++ < 60)
-    {
-      character = (char) readchar (0);
-      if ((character == 0xfffffffe) || (character == 0x7f))
-       {               /* empty uart */
-         sleep (1);
-         continue;
-       }
-      if (character == CANCEL)
-       {                       /* target aborted load */
-         expect_prompt (NULL, 0);
-         error ("Got a CANCEL from the target.");
-       }
-    if (character == '*')
-      {                        /* look for missed error message */
-       expect_prompt (NULL, 0);
-       error ("Got an error message from the target");
-      }
-      if (character == byte)                   /* got what we wanted */
-       return 1;
-      if (character == ((byte == ACK) ? NAK : ACK))
-       return 0;
-      sleep (1); 
+      monitor_expect_prompt (NULL, 0);
     }
 
-  return 0;
-}
-
-/*
- * monitor_make_srec -- make an srecord. This writes each line, one at a
- *     time, each with it's own header and trailer line.
- *     An srecord looks like this:
- *
- * byte count-+     address
- * start ---+ |        |       data        +- checksum
- *         | |        |                   |
- *       S01000006F6B692D746573742E73726563E4
- *       S315000448600000000000000000FC00005900000000E9
- *       S31A0004000023C1400037DE00F023604000377B009020825000348D
- *       S30B0004485A0000000000004E
- *       S70500040000F6
- *
- *     S<type><length><address><data><checksum>
- *
- *      Where
- *      - length
- *        is the number of bytes following upto the checksum. Note that
- *        this is not the number of chars following, since it takes two
- *        chars to represent a byte.
- *      - type
- *        is one of:
- *        0) header record
- *        1) two byte address data record
- *        2) three byte address data record
- *        3) four byte address data record
- *        7) four byte address termination record
- *        8) three byte address termination record
- *        9) two byte address termination record
- *       
- *      - address
- *        is the start address of the data following, or in the case of
- *        a termination record, the start address of the image
- *      - data
- *        is the data.
- *      - checksum
- *       is the sum of all the raw byte data in the record, from the length
- *        upwards, modulo 256 and subtracted from 255.
- */
-
-static int
-monitor_make_srec (buffer, type, memaddr, myaddr, len)
-     char *buffer;
-     int type;
-     CORE_ADDR memaddr;
-     unsigned char *myaddr;
-     int len;
-{
-  int checksum;
-  int i;
-  char *buf;
+/* Finally, make the PC point at the start address */
 
-  buf = buffer;
+  if (exec_bfd)
+    write_pc (bfd_get_start_address (exec_bfd));
 
-  checksum = 0;
-  
-  /* Create the header for the srec. 4 is the number of bytes in the address,
-     and 1 is the number of bytes in the count.  */
+  inferior_pid = 0;            /* No process now */
 
-  if (type == 0)                               /* FIXME: type 0 is optional */
-    type = 3;                                  /* so use data as it works */
-  sprintf (buf, "S%d%02X%08X", type, len + 4 + 1, memaddr);
-  buf += 12;
-  
-  checksum += (len + 4 + 1                     /* calculate the checksum */
-              + (memaddr & 0xff)
-              + ((memaddr >>  8) & 0xff)
-              + ((memaddr >> 16) & 0xff)
-              + ((memaddr >> 24) & 0xff));
-  
-  /* build the srecord */
-  for (i = 0; i < len; i++)
-    {
-      sprintf (buf, "%02X", myaddr[i]);
-      checksum += myaddr[i];
-      buf += 2;
-    }
+/* This is necessary because many things were based on the PC at the time that
+   we attached to the monitor, which is no longer valid now that we have loaded
+   new code (and just changed the PC).  Another way to do this might be to call
+   normal_stop, except that the stack may not be valid, and things would get
+   horribly confused... */
 
-  sprintf(buf, "%02X", ~checksum & 0xff);      /* add the checksum */
-  
-  return 0;
+  clear_symtab_users ();
 }
 
-/* Take 128 bytes of data and make a packet out of it.
- *
- *     Each packet looks like this:
- *     +-----+-------+-------+------+-----+
- *     | SOH | Seq1. | Seq2. | data | SUM |
- *     +-----+-------+-------+------+-----+
- *     SOH  = 0x01
- *     Seq1 = The sequence number.
- *     Seq2 = The complement of the sequence number.
- *     Data = A 128 bytes of data.
- *     SUM  = Add the contents of the 128 bytes and use the low-order
- *            8 bits of the result.
- */
-
 static void
-make_xmodem_packet (packet, data, len)
-     unsigned char *packet;
-     unsigned char *data;
-     int len;
+monitor_stop ()
 {
-  static int sequence = 1;
-  int i, sum;
-  unsigned char *buf;
-  
-  buf = data;
-  /* build the packet header */
-  packet[0] = SOH;
-  packet[1] = sequence;
-  packet[2] = 255 - sequence;
-  sequence++;
-#if 0
-  packet[2] = ~sequence++;     /* the complement is the sequence checksum */
-#endif
-  
-  sum = 0;                             /* calculate the data checksum */
-  for (i = 3; i <= len + 2; i++) {
-    packet[i] = *buf;
-    sum += *buf;
-    buf++;
-  }
-
-  /* add padding for the rest of the packet */
-  for (i = len+1 ; i <= XMODEM_DATASIZE ; i++)
-    packet[i] = '0';
-
-  packet[XMODEM_PACKETSIZE] = sum & 0xff;      /* add the checksum */
+  RDEBUG(("MON stop\n")) ;
+  if ((current_monitor->flags & MO_SEND_BREAK_ON_STOP) != 0)
+    SERIAL_SEND_BREAK (monitor_desc);
+  if (current_monitor->stop)
+    monitor_printf_noecho (current_monitor->stop);
 }
 
-/* Print the packet as a debug check.  */
+/* Put a command string, in args, out to MONITOR.  Output from MONITOR
+   is placed on the users terminal until the prompt is seen. FIXME: We
+   read the characters ourseleves here cause of a nasty echo.  */
 
 static void
-print_xmodem_packet (packet)
-     char *packet;
+monitor_command (args, from_tty)
+     char *args;
+     int from_tty;
 {
-  int i;
-  static int lastseq;
-  int sum;
+  char *p;
+  int resp_len;
+  char buf[1000];
 
-  /* take apart the packet header the packet header */
-  if (packet[0] == SOH)
-    printf_unfiltered ("SOH");
-  else
-    error ("xmodem: SOH is wrong");
-  
-  /* check the sequence */
-  if (packet[1] != 0)
-    {
-      lastseq = packet[1];
-      if (packet[2] != ~lastseq)
-       error ("xmodem: Sequence checksum is wrong");
-      else
-       printf_filtered (" %d %d", lastseq, ~lastseq);
-    }
-  
-  /* check the data checksum */
-  sum = 0;
-  for (i = 3; i <= XMODEM_DATASIZE; i++)
-    sum += packet[i];
+  if (monitor_desc == NULL)
+    error ("monitor target not open.");
 
-  /* ignore the data */
-#if 0
-  printf_unfiltered (" [128 bytes of data] %d\n", sum & 0xff);
-#endif
-  printf_filtered (" [%s] %d\n", packet, sum & 0xff);
+  p = current_monitor->prompt;
+
+  /* Send the command.  Note that if no args were supplied, then we're
+     just sending the monitor a newline, which is sometimes useful.  */
 
-  if ((packet[XMODEM_PACKETSIZE] & 0xff) != (sum & 0xff))
-    printf_unfiltered ("xmodem: data checksum wrong, got a %d",
-                      packet[XMODEM_PACKETSIZE] & 0xff);
+  monitor_printf ("%s\r", (args ? args : ""));
 
-  putchar_unfiltered ('\n');
+  resp_len = monitor_expect_prompt (buf, sizeof buf);
+
+  fputs_unfiltered (buf, gdb_stdout); /* Output the response */
 }
 
 /* Convert hex digit A to a number.  */
 
+#if 0
 static int
 from_hex (a)
      int a;
@@ -1433,6 +2092,66 @@ from_hex (a)
 
   error ("Reply contains invalid hex digit 0x%x", a);
 }
+#endif
+
+char *
+monitor_get_dev_name ()
+{
+  return dev_name;
+}
+
+static struct target_ops monitor_ops =
+{
+  NULL,                                /* to_shortname */
+  NULL,                                /* to_longname */
+  NULL,                                /* to_doc */
+  NULL,                                /* to_open */
+  monitor_close,               /* to_close */
+  NULL,                                /* to_attach */
+  monitor_detach,              /* to_detach */
+  monitor_resume,              /* to_resume */
+  monitor_wait,                        /* to_wait */
+  monitor_fetch_registers,     /* to_fetch_registers */
+  monitor_store_registers,     /* to_store_registers */
+  monitor_prepare_to_store,    /* to_prepare_to_store */
+  monitor_xfer_memory,         /* to_xfer_memory */
+  monitor_files_info,          /* to_files_info */
+  monitor_insert_breakpoint,   /* to_insert_breakpoint */
+  monitor_remove_breakpoint,   /* to_remove_breakpoint */
+  0,                           /* to_terminal_init */
+  0,                           /* to_terminal_inferior */
+  0,                           /* to_terminal_ours_for_output */
+  0,                           /* to_terminal_ours */
+  0,                           /* to_terminal_info */
+  monitor_kill,                        /* to_kill */
+  monitor_load,                        /* to_load */
+  0,                           /* to_lookup_symbol */
+  monitor_create_inferior,     /* to_create_inferior */
+  monitor_mourn_inferior,      /* to_mourn_inferior */
+  0,                           /* to_can_run */
+  0,                           /* to_notice_signals */
+  0,                           /* to_thread_alive */
+  monitor_stop,                        /* to_stop */
+  process_stratum,             /* to_stratum */
+  0,                           /* to_next */
+  1,                           /* to_has_all_memory */
+  1,                           /* to_has_memory */
+  1,                           /* to_has_stack */
+  1,                           /* to_has_registers */
+  1,                           /* to_has_execution */
+  0,                           /* sections */
+  0,                           /* sections_end */
+  OPS_MAGIC                    /* to_magic */
+};
+
+/* Init the target_ops structure pointed at by OPS */
+
+void
+init_monitor_ops (ops)
+     struct target_ops *ops;
+{
+  memcpy (ops, &monitor_ops, sizeof monitor_ops);
+}
 
 /* Define additional commands that are usually only used by monitors.  */
 
This page took 0.052628 seconds and 4 git commands to generate.