* monitor.c (monitor_printf): Changed format specification in
authorJ.T. Conklin <jtc@acorntoolworks.com>
Wed, 17 May 1995 23:13:26 +0000 (23:13 +0000)
committerJ.T. Conklin <jtc@acorntoolworks.com>
Wed, 17 May 1995 23:13:26 +0000 (23:13 +0000)
  error message to work with pre-ansi compilers.
  (monitor_load_srec): reduced length of s-records from 128 to 32
  bytes so download is more reliable with the rom68k monitor.

* rom68k-rom.c: Added trailing space to prompt string.

* config/i386/xm-i386sco.h (HAVE_STRSTR): Removed.
* config/i386/xm-go32.h, mswin/xm.h (SYS_SIGLIST_MISSING):
  Removed.
* defs.h, config/{xm-lynx.h, xm-nbsd.h},
  config/i386/{xm-i386bsd.h, xm-linux.h},
  config/m68k/xm-hp300bsd.h, config/mips/xm-irix4.h,
  config/ns32k/xm-ns32km3.h, doc/gdbint.texinfo
  (PSIGNAL_IN_SIGNAL_H): Removed.

13 files changed:
gdb/ChangeLog
gdb/config/i386/xm-go32.h
gdb/config/i386/xm-i386bsd.h
gdb/config/i386/xm-i386sco.h
gdb/config/i386/xm-linux.h
gdb/config/m68k/xm-hp300bsd.h
gdb/config/mips/xm-irix4.h
gdb/config/ns32k/xm-ns32km3.h
gdb/config/xm-lynx.h
gdb/config/xm-nbsd.h
gdb/defs.h
gdb/monitor.c
gdb/rom68k-rom.c

index 51eae308e98391aefc710020b0152015b339e76c..a39ed7a75a309ed21916a2eaae8d14991d2e9474 100644 (file)
@@ -1,5 +1,25 @@
+Wed May 17 15:55:53 1995  J.T. Conklin  <jtc@rtl.cygnus.com>
+
+       * monitor.c (monitor_printf): Changed format specification in
+         error message to work with pre-ansi compilers.
+         (monitor_load_srec): reduced length of s-records from 128 to 32
+         bytes so download is more reliable with the rom68k monitor.
+
+       * rom68k-rom.c: Added trailing space to prompt string.
+
+       * config/i386/xm-i386sco.h (HAVE_STRSTR): Removed.
+       * config/i386/xm-go32.h, mswin/xm.h (SYS_SIGLIST_MISSING):
+         Removed.
+       * defs.h, config/{xm-lynx.h, xm-nbsd.h},
+         config/i386/{xm-i386bsd.h, xm-linux.h},
+         config/m68k/xm-hp300bsd.h, config/mips/xm-irix4.h,
+         config/ns32k/xm-ns32km3.h, doc/gdbint.texinfo
+         (PSIGNAL_IN_SIGNAL_H): Removed.
+
 Tue May 16 13:16:06 1995  J.T. Conklin  <jtc@rtl.cygnus.com>
 
+       * Makefile.in (Makefile): Added config.status to dependency list.
+
        * configure.in: Added INIT-CMDS argument to AC_OUTPUT which sets
          gdb_host_cpu, gdb_target_cpu and nativefile.
        * configure: regenerated.
index eee636ff975f3c1626632e0a863c7b4532df72b6..fe9d5f860e05e4b5d4ac92b37915dbdc34c90026 100644 (file)
@@ -17,7 +17,6 @@ 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.  */
 
-#define SYS_SIGLIST_MISSING 1
 #define HOST_BYTE_ORDER LITTLE_ENDIAN
 #include "fopen-bin.h"
 
index 2f3d70e02f1ebb7ad95810631c1773f7eba67134..fce0263db48df08aabc81a5e48d1516049a2e98f 100644 (file)
@@ -21,7 +21,3 @@ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
 
 #include <machine/limits.h>            /* for INT_MIN, to avoid "INT_MIN
                                           redefined" warnings from defs.h */
-
-/* psignal() is in <signal.h>.  */
-
-#define PSIGNAL_IN_SIGNAL_H
index 129cda08cb0d36f26235ebc7f95c754d989b1d10..ddcdc9b8083a067126c818fa4cbd70053428fa39 100644 (file)
@@ -37,7 +37,6 @@ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
    So we use dots instead.  This item must be coordinated with G++. */
 #undef CPLUS_MARKER
 #define CPLUS_MARKER '.'
-#define HAVE_STRSTR
 
 /* Use setpgid instead of setpgrp on SCO */
 #define NEED_POSIX_SETPGID
index 31dcc39b383a0d81f530a8650192485d2f156b5f..cf1b4a4d2774cea176534afdbcf43c36b7b89564 100644 (file)
@@ -24,7 +24,6 @@ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
 /* This is the amount to subtract from u.u_ar0
    to get the offset in the core file of the register values.  */
 #define KERNEL_U_ADDR 0x0
-#define PSIGNAL_IN_SIGNAL_H
 
 #define NEED_POSIX_SETPGID
 
index 80e04d34865cfe7118a63a31ee8b8a75d9fb9f33..e59b4c6fb730245a35af4a8b0d0a5f17cc931a57 100644 (file)
@@ -31,12 +31,6 @@ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
 /* Kernel is a bit tenacious about sharing text segments, disallowing bpts.  */
 #define        ONE_PROCESS_WRITETEXT
 
-/* psignal's definition in 4.4BSD conflicts with the one in defs.h. 
-   But there *is* no psignal definition in 4.3BSD.  So we avoid the defs.h
-   version here, and supply our own (matching) one.  */
-#define PSIGNAL_IN_SIGNAL_H
-void    psignal PARAMS ((unsigned int, const char *));
-
 extern char *strdup PARAMS ((const char *));
 \f
 /* Interface definitions for kernel debugger KDB.  */
index 7c6ad8d1dbed86c1b2c11586cd6f52cd79a4d0ba..069a175c77d7cd48b028d54ab020815c1b2094e1 100644 (file)
@@ -22,11 +22,6 @@ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
 
 #include "mips/xm-irix3.h"
 
-/* Irix defines psignal() in signal.h, in a way which conflicts with the
-   definition in defs.h.  */
-
-#define PSIGNAL_IN_SIGNAL_H
-
 #define BROKEN_SIGINFO_H       /* <sys/siginfo.h> si_pid & si_uid are bogus */
 
 /* Irix 4.0.1 and later have termios.  Not sure about earlier versions.  */
index ef3bc7c568382c884c0f96e8b536a20eb4029385..4147fa8ed3754dd4702b715be75b3b1714c2f326 100644 (file)
@@ -26,6 +26,3 @@ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
 
 /* Do implement the attach and detach commands.  */
 #define ATTACH_DETACH  1
-
-/* signal.h has declaration for psignal() */
-#define PSIGNAL_IN_SIGNAL_H
index 5161884b78293ae60d78ba2a5553ece86f9fb9fc..5b9a3891247402877bfcee4251660f0b59084244 100644 (file)
@@ -21,8 +21,4 @@ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
 
 #include <limits.h>
 
-/* psignal() is in <signal.h>.  */
-
-#define PSIGNAL_IN_SIGNAL_H
-
 #define MEM_FNS_DECLARED
index a942308e44d2904f929253f8f744c1d0ed9b444a..2399743b7addb27993f52c742ebdedbf612c7583 100644 (file)
@@ -30,6 +30,3 @@ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
 
 /* NetBSD has termios facilities. */
 #define HAVE_TERMIOS
-
-/* psignal() is in <signal.h>.  */
-#define PSIGNAL_IN_SIGNAL_H
index 9bce3c48e94dcdcf0d939ef385fe26811f191a00..25c6fbc71f5821f1ce6c883c528af92034468fab 100644 (file)
@@ -266,7 +266,7 @@ extern char *re_comp PARAMS ((const char *));
 
 extern void symbol_file_command PARAMS ((char *, int));
 
-/* From main.c */
+/* From top.c */
 
 extern char *skip_quoted PARAMS ((char *));
 
@@ -278,6 +278,8 @@ extern void print_prompt PARAMS ((void));
 
 extern int input_from_terminal_p PARAMS ((void));
 
+extern int info_verbose;
+
 /* From printcmd.c */
 
 extern void set_next_address PARAMS ((CORE_ADDR));
@@ -587,10 +589,6 @@ extern char *getenv PARAMS ((const char *));
 
 /* From other system libraries */
 
-#ifndef PSIGNAL_IN_SIGNAL_H
-extern void psignal PARAMS ((unsigned, const char *));
-#endif
-
 #ifdef __STDC__
 #include <stddef.h>
 #include <stdlib.h>
@@ -837,8 +835,6 @@ extern int (*target_wait_hook) PARAMS ((int pid,
 extern void (*call_command_hook) PARAMS ((struct cmd_list_element *c,
                                          char *cmd, int from_tty));
 
-extern NORETURN void (*error_hook) PARAMS (());
-
 /* Inhibit window interface if non-zero. */
 
 extern int use_windows;
index 73236a9327d4934db7f1be1a192d66ec05539fc8..5693d4c69fee0c431b0b725f69cee0d8dba510f5 100644 (file)
@@ -43,6 +43,8 @@ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
 #include "inferior.h"
 #include "regex.h"
 
+static int readchar PARAMS ((int timeout));
+
 static void monitor_command PARAMS ((char *args, int fromtty));
 static void monitor_load_srec PARAMS ((char *args));
 
@@ -56,6 +58,11 @@ static void monitor_store_register PARAMS ((int regno));
 static void monitor_close PARAMS ((int quitting));
 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));
@@ -78,6 +85,8 @@ static int hashmark;          /* flag set by "set hash" */
 
 static int timeout = 30;
 
+static void (*ofunc)();                /* Old SIGINT signal handler */
+
 /* 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
    program starts.  */
@@ -98,7 +107,38 @@ static char fastmap[256];
 static int dump_reg_flag;      /* Non-zero means do a dump_registers cmd when
                                   monitor_wait wakes up.  */
 
-/* monitor_printf -- send data to monitor.  Works just like printf. */
+/* monitor_printf_noecho -- Send data to monitor, but don't expect an echo.
+   Works just like printf.  */
+
+void
+monitor_printf_noecho (va_alist)
+     va_dcl
+{
+  va_list args;
+  char *pattern;
+  char sndbuf[2000];
+  int len;
+
+  va_start (args);
+
+  pattern = va_arg (args, char *);
+
+  vsprintf (sndbuf, pattern, args);
+
+  if (remote_debug > 0)
+    fputs_unfiltered (sndbuf, gdb_stderr);
+
+  len = strlen (sndbuf);
+
+  if (len + 1 > sizeof sndbuf)
+    abort ();
+
+  if (SERIAL_WRITE(monitor_desc, sndbuf, len))
+    fprintf_unfiltered (stderr, "SERIAL_WRITE failed: %s\n", safe_strerror (errno));
+}
+
+/* monitor_printf -- Send data to monitor and check the echo.  Works just like
+   printf.  */
 
 void
 monitor_printf (va_alist)
@@ -106,25 +146,43 @@ monitor_printf (va_alist)
 {
   va_list args;
   char *pattern;
-  char buf[2000];
+  char sndbuf[2000];
   int len;
+  int i, c;
 
   va_start (args);
 
   pattern = va_arg (args, char *);
 
-  vsprintf (buf, pattern, args);
+  vsprintf (sndbuf, pattern, args);
 
   if (remote_debug > 0)
-    fputs_unfiltered (buf, gdb_stderr);
+    fputs_unfiltered (sndbuf, gdb_stderr);
 
-  len = strlen (buf);
+  len = strlen (sndbuf);
 
-  if (len + 1 > sizeof buf)
+  if (len + 1 > sizeof sndbuf)
     abort ();
 
-  if (SERIAL_WRITE(monitor_desc, buf, len))
+  if (SERIAL_WRITE(monitor_desc, sndbuf, len))
     fprintf_unfiltered (stderr, "SERIAL_WRITE failed: %s\n", safe_strerror (errno));
+
+  for (i = 0; i < len; i++)
+    {
+    trycr:
+      c = readchar (timeout);
+
+      if (c != sndbuf[i])
+       {
+#if 0
+         if (sndbuf[i] == '\r'
+             && c == '\n')
+           goto trycr;
+#endif
+         error ("monitor_printf:  Bad echo.  Sent: \"%s\", Got: \"%.*s%c\".",
+                sndbuf, i, sndbuf, c);
+       }
+    }
 }
 
 /* Read a character from the remote system, doing all the fancy
@@ -339,7 +397,11 @@ monitor_open (args, mon_ops, from_tty)
   /* See if we can wake up the monitor.  First, try sending a stop sequence,
      then send the init strings.  Last, remove all breakpoints.  */
 
-  monitor_stop ();
+  if (current_monitor->stop)
+    {
+      monitor_stop ();
+      monitor_expect_prompt (NULL, 0);
+    }
 
   /* wake up the monitor and see if it's alive */
   for (p = mon_ops->init; *p != NULL; p++)
@@ -348,6 +410,8 @@ monitor_open (args, mon_ops, from_tty)
       monitor_expect_prompt (NULL, 0);
     }
 
+  SERIAL_FLUSH_INPUT (monitor_desc);
+
   /* Remove all breakpoints */
 
   if (mon_ops->clr_all_break)
@@ -363,7 +427,9 @@ monitor_open (args, mon_ops, from_tty)
 
   inferior_pid = 42000;                /* Make run command think we are busy... */
 
-  monitor_printf ("\r");       /* Give monitor_wait something to read */
+  /* Give monitor_wait something to read */
+
+  monitor_printf (current_monitor->line_term);
 
   start_remote ();
 }
@@ -467,6 +533,60 @@ parse_register_dump (buf, len)
     }
 }
 
+/* Send ^C to target to halt it.  Target will respond, and send us a
+   packet.  */
+
+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");
+
+  target_stop ();
+}
+
+/* The user typed ^C twice.  */
+
+static void
+monitor_interrupt_twice (signo)
+     int signo;
+{
+  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);
+}
+
 /* Wait until the remote machine stops, then return, storing status in
    status just as `wait' would.  */
 
@@ -478,12 +598,17 @@ monitor_wait (pid, 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);
+
   timeout = -1;                /* Don't time out -- user program is running. */
 
+  ofunc = (void (*)()) signal (SIGINT, monitor_interrupt);
+
   do
     {
       resp_len = monitor_expect_prompt (buf, sizeof (buf));
@@ -493,6 +618,8 @@ monitor_wait (pid, status)
     }
   while (resp_len < 0);
 
+  signal (SIGINT, ofunc);
+
   timeout = old_timeout;
 
   if (dump_reg_flag && current_monitor->dump_registers)
@@ -509,6 +636,8 @@ monitor_wait (pid, status)
   status->kind = TARGET_WAITKIND_STOPPED;
   status->value.sig = TARGET_SIGNAL_TRAP;
 
+  discard_cleanups (old_chain);
+
   return inferior_pid;
 }
 
@@ -519,11 +648,10 @@ static void
 monitor_fetch_register (regno)
      int regno;
 {
-  char buf[200];
-  char *p;
   char *name;
-  int resp_len;
   static char zerobuf[MAX_REGISTER_RAW_SIZE] = {0};
+  char regbuf[MAX_REGISTER_RAW_SIZE * 2 + 1];
+  int i;
 
   name = REGNAMES (regno);
 
@@ -537,45 +665,55 @@ monitor_fetch_register (regno)
 
   monitor_printf (current_monitor->getreg.cmd, name);
 
+/* 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.resp_delim)
+    monitor_expect (current_monitor->getreg.resp_delim, NULL, 0);
+
+/* Now, read the appropriate number of hex digits for this register, skipping
+   spaces.  */
+
+  for (i = 0; i < REGISTER_RAW_SIZE (regno) * 2; i++)
+    {
+      int c;
+
+      while (1)
+       {
+         c = readchar (timeout);
+         if (isxdigit (c))
+           break;
+         if (c == ' ')
+           continue;
+
+         error ("monitor_fetch_register (%d):  bad response from monitor: %.*s%c.",
+                regno, i, regbuf, c);
+       }
+
+      regbuf[i] = c;
+    }
+
+  regbuf[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->getreg.term)
     {
-      resp_len = monitor_expect (current_monitor->getreg.term, buf, sizeof buf); /* get response */
-
-      if (resp_len <= 0)
-       error ("monitor_fetch_register (%d):  excessive response from monitor: %.*s.",
-              regno, resp_len, buf);
+      monitor_expect (current_monitor->getreg.term, NULL, 0); /* get response */
 
       if (current_monitor->getreg.term_cmd)
        {
-         SERIAL_WRITE (monitor_desc, current_monitor->getreg.term_cmd,
-                       strlen (current_monitor->getreg.term_cmd));
+         monitor_printf (current_monitor->getreg.term_cmd);
          monitor_expect_prompt (NULL, 0);
        }
     }
   else
-    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.  */
-
-  if (current_monitor->getreg.resp_delim)
-    {
-      p = strstr (buf, current_monitor->getreg.resp_delim);
-      if (!p)
-       error ("monitor_fetch_register (%d):  bad response from monitor: %.*s.",
-              regno, resp_len, buf);
-      p += strlen (current_monitor->getreg.resp_delim);
-    }
-  else
-    p = buf;
+    monitor_expect_prompt (NULL, 0); /* get response */
 
-  monitor_supply_register (regno, p);
+  monitor_supply_register (regno, regbuf);
 }
 
 /* Read the remote registers into the block regs.  */
@@ -717,6 +855,108 @@ monitor_write_memory (memaddr, myaddr, len)
   return len;
 }
 
+/* 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;
+     unsigned char *myaddr;
+     int len;
+{
+  unsigned LONGEST val;
+  char membuf[sizeof(LONGEST) * 2 + 1];
+  char *p;
+  char *cmd;
+  int i;
+
+  if ((memaddr & 0x7) == 0 && len >= 8 && current_monitor->getmem.cmdll)
+    {
+      len = 8;
+      cmd = current_monitor->getmem.cmdll;
+    }
+  else 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 register value.  Otherwise, we just start searching from the start of
+   the buf.  */
+
+  if (current_monitor->getmem.resp_delim)
+    monitor_expect (current_monitor->getmem.resp_delim, NULL, 0);
+
+/* Now, read the appropriate number of hex digits for this loc, skipping
+   spaces.  */
+
+  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);
+
+  if (val == 0 && membuf == p)
+    error ("monitor_read_memory_single (0x%x):  bad value from monitor: %s.",
+          memaddr, membuf);
+
+  /* supply register stores in target byte order, so swap here */
+
+  store_unsigned_integer (myaddr, len, val);
+
+  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.  */
@@ -735,6 +975,9 @@ monitor_read_memory (memaddr, myaddr, len)
   int resp_len;
   int i;
 
+  if (current_monitor->flags & MO_GETMEM_READ_SINGLE)
+    return monitor_read_memory_single (memaddr, myaddr, len);
+
   len = min (len, 16);
 
 /* See if xfer would cross a 16 byte boundary.  If so, clip it.  */
@@ -772,9 +1015,6 @@ monitor_read_memory (memaddr, myaddr, len)
 
   p = buf;
 
-  while (*p != '\r')           /* Skip command echo and line delim */
-    p++;
-
   /* 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.
    */
@@ -818,8 +1058,6 @@ monitor_read_memory (memaddr, myaddr, len)
   return len;
 }
 
-/* FIXME-someday!  merge these two.  */
-
 static int
 monitor_xfer_memory (memaddr, myaddr, len, write, target)
      CORE_ADDR memaddr;
@@ -954,11 +1192,8 @@ monitor_load (file, from_tty)
 static void
 monitor_stop ()
 {
-  if (!current_monitor->stop)
-    return;
-
-  monitor_printf(current_monitor->stop);
-  monitor_expect_prompt (NULL, 0);
+  if (current_monitor->stop)
+    monitor_printf_noecho (current_monitor->stop);
 }
 
 /* Put a command string, in args, out to MONITOR.  Output from MONITOR
@@ -999,7 +1234,7 @@ monitor_load_srec (args)
   asection *s;
   char *buffer, srec[1024];
   int i;
-  int srec_frame = 128;
+  int srec_frame = 32;
   int reclen;
 
   buffer = alloca (srec_frame * 2 + 256);
@@ -1038,7 +1273,7 @@ monitor_load_srec (args)
 
            reclen = monitor_make_srec (srec, 3, s->vma + i, buffer, numbytes);
 
-           monitor_printf ("%.*s\r", reclen, srec);
+           monitor_printf_noecho ("%.*s\r", reclen, srec);
 
            if (hashmark)
              {
@@ -1058,11 +1293,13 @@ monitor_load_srec (args)
 
   reclen = monitor_make_srec (srec, 7, abfd->start_address, NULL, 0);
 
-  monitor_printf ("%.*s\r", reclen, srec);
+  monitor_printf_noecho ("%.*s\r", reclen, srec);
 
-  monitor_printf ("\r\r");     /* Some monitors need these to wake up */
+  monitor_printf_noecho ("\r\r"); /* Some monitors need these to wake up */
 
   monitor_expect_prompt (NULL, 0);
+
+  SERIAL_FLUSH_INPUT (monitor_desc);
 }
 
 /*
index d628c6eacd35e1978457eb06cc69daa4587d2728..1de71008da720133685b798f3f9af8fc15866b3f 100644 (file)
@@ -141,7 +141,7 @@ static struct monitor_ops rom68k_cmds =
   NULL,                                /* load_routine (defaults to SRECs) */
   "dc\r",                      /* download command */
   "Waiting for S-records from host... ", /* Load response */
-  "ROM68K :->",                        /* monitor command prompt */
+  "ROM68K :-> ",               /* monitor command prompt */
   "\r",                                /* end-of-line terminator */
   ".\r",                       /* optional command terminator */
   &rom68k_ops,                 /* target operations */