Mon Dec 21 12:40:10 1992 Ian Lance Taylor (ian@cygnus.com)
[binutils-gdb.git] / gdb / remote-hms.c
index 48df7a7bfaa925cf0f2c998525ac138b88916b7a..216e5cf3cc72ff018282af8faee8a52411ef6695 100644 (file)
@@ -1,9 +1,7 @@
 /* Remote debugging interface for Hitachi HMS Monitor Version 1.0
-
    Copyright 1992 Free Software Foundation, Inc.
-
-   Contributed by Steve Chamberlain sac@cygnus.com
-
+   Contributed by Cygnus Support.  Written by Steve Chamberlain
+   (sac@cygnus.com).
 
 This file is part of GDB.
 
@@ -21,14 +19,11 @@ 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.  */
 
-
-
-#include <stdio.h>
-#include <string.h>
 #include "defs.h"
 #include "inferior.h"
 #include "wait.h"
 #include "value.h"
+#include <string.h>
 #include <ctype.h>
 #include <fcntl.h>
 #include <signal.h>
@@ -41,9 +36,6 @@ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
 /* External data declarations */
 extern int stop_soon_quietly;           /* for wait_for_inferior */
 
-/* External function declarations */
-extern struct value *call_function_by_hand();
-
 /* Forward data declarations */
 extern struct target_ops hms_ops;              /* Forward declaration */
 
@@ -57,15 +49,6 @@ extern struct target_ops hms_ops;
 
 static int quiet = 1;
 
-#ifdef DEBUG
-# define DENTER(NAME) if (!quiet)  (printf_filtered("Entering %s\n",NAME), fflush(stdout))
-# define DEXIT(NAME)  if (!quiet)  (printf_filtered("Exiting  %s\n",NAME), fflush(stdout))
-#else
-# define DENTER(NAME)
-# define DEXIT(NAME)
-#endif
-
-
 /***********************************************************************/
 /* Caching stuff stolen from remote-nindy.c  */
 
@@ -242,75 +225,36 @@ dcache_init ()
  ***********************************************************************/
 
 static int timeout = 2;
-static char *dev_name  = "/dev/ttya";
-
 
+static const char *dev_name;
 
 
 /* Descriptor for I/O to remote machine.  Initialize it to -1 so that
    hms_open knows that we don't have a file open when the program
    starts.  */
-int hms_desc = -1;
-#define OPEN(x) ((x) >= 0)
-
 
-void hms_open();
 
-#define ON     1
-#define OFF    0
-static void
-rawmode(desc, turnon)
-int    desc;
-int    turnon;
+int is_open = 0;
+int check_open()
 {
-  TERMINAL sg;
-
-  if (desc < 0)
-    return;
-
-  ioctl (desc, TIOCGETP, &sg);
-
-  if (turnon) {
-#ifdef HAVE_TERMIO
-       sg.c_lflag &= ~(ICANON);
-#else
-       sg.sg_flags |= RAW;
-#endif
-  } else {
-#ifdef HAVE_TERMIO
-       sg.c_lflag |= ICANON;
-#else
-       sg.sg_flags &= ~(RAW);
-#endif
+  if (!is_open)
+  {
+    error("remote device not open");
   }
-  ioctl (desc, TIOCSETP, &sg);
 }
 
+#define ON     1
+#define OFF    0
 
 /* Read a character from the remote system, doing all the fancy
    timeout stuff.  */
 static int
 readchar ()
 {
-  char buf;
+  int buf;
+  buf = serial_readchar(timeout);
 
-  buf = '\0';
-#ifdef HAVE_TERMIO
-  /* termio does the timeout for us.  */
-  read (hms_desc, &buf, 1);
-#else
-  alarm (timeout);
-  if (read (hms_desc, &buf, 1) < 0)
-  {
-    if (errno == EINTR)
-     error ("Timeout reading from remote system.");
-    else
-     perror_with_name ("remote");
-  }
-  alarm (0);
-#endif
-
-  if (buf == '\0')
+  if (buf < 0)
    error ("Timeout reading from remote system.");
 
   if (!quiet)
@@ -322,26 +266,14 @@ readchar ()
 static int
 readchar_nofail ()
 {
-  char buf;
-
-  buf = '\0';
-#ifdef HAVE_TERMIO
-  /* termio does the timeout for us.  */
-  read (hms_desc, &buf, 1);
-#else
-  alarm (timeout);
-  if (read (hms_desc, &buf, 1) < 0)
-  {
-    return 0;
-  }
-  alarm (0);
-#endif
-
-  if (buf == '\0') 
-  {
-    return 0;
-  }
+  int buf;
+  buf = serial_readchar(timeout);
+  if (buf < 0) buf = 0;
+  if (!quiet)
+   printf("%c",buf);
+  
   return buf & 0x7f;
+
 }
 
 /* Keep discarding input from the remote system, until STRING is found. 
@@ -387,7 +319,6 @@ expect (string)
 static void
 expect_prompt ()
 {
-
   expect ("HMS>");
 }
 
@@ -443,21 +374,7 @@ get_hex_word()
   return val;
 }
 /* Called when SIGALRM signal sent due to alarm() timeout.  */
-#ifndef HAVE_TERMIO
-
-#ifndef __STDC__
-# ifndef volatile
-#  define volatile /**/
-# endif
-#endif
-volatile int n_alarms;
 
-void
-hms_timer ()
-{
-  n_alarms++;
-}
-#endif
 
 
 /* Number of SIGTRAPs we need to simulate.  That is, the next
@@ -474,12 +391,7 @@ int        from_tty;
 
 }
 
-static check_open()
-{
-  if (!OPEN(hms_desc)) {
-      hms_open("",0);
-    }  
-}
+
 
 /*
  * Download a file specified in 'args', to the hms. 
@@ -493,8 +405,7 @@ int fromtty;
   asection *s;
   int  n;
   char buffer[1024];
-       
-  DENTER("hms_load()");
+
   check_open();
 
   dcache_flush();
@@ -517,16 +428,31 @@ int       fromtty;
   {
     if (s->flags & SEC_LOAD) 
     {
-      char *buffer = xmalloc(s->_raw_size);
-      bfd_get_section_contents(abfd, s, buffer, 0, s->_raw_size);
+      int i;
 
-      hms_write_inferior_memory(s->vma, buffer, s->_raw_size);
+      
+#define DELTA 1024
+      char *buffer = xmalloc(DELTA);
+      printf_filtered("%s\t: 0x%4x .. 0x%4x  ",s->name, s->vma, s->vma + s->_raw_size);
+      for (i = 0; i < s->_raw_size; i+= DELTA) 
+      {
+       int delta = DELTA;
+       if (delta > s->_raw_size - i)
+        delta = s->_raw_size - i ;
+
+       bfd_get_section_contents(abfd, s, buffer, i, delta);
+       hms_write_inferior_memory(s->vma + i, buffer, delta);
+       printf_filtered("*");
+       fflush(stdout);
+      }
+      printf_filtered(  "\n");      
       free(buffer);
     }
     s = s->next;
   }
-  
-  DEXIT("hms_load()");
+  sprintf(buffer, "r PC=%x", abfd->start_address);
+  hms_write_cr(buffer);
+  expect_prompt();
 }
 
 /* This is called not only when we first attach, but also when the
@@ -538,8 +464,7 @@ hms_create_inferior (execfile, args, env)
      char **env;
 {
   int entry_pt;
-
-  DENTER("hms_create_inferior()");
+  char buffer[100];    
 
   if (args && *args)
    error ("Can't pass arguments to remote hms process.");
@@ -549,58 +474,16 @@ hms_create_inferior (execfile, args, env)
 
   entry_pt = (int) bfd_get_start_address (exec_bfd);
   check_open();
-  
-  if (OPEN(hms_desc))
-  {
-    
-    hms_kill(NULL,NULL);        
-    hms_clear_breakpoints();
-    init_wait_for_inferior ();
-    /* Clear the input because what the hms sends back is different
-     * depending on whether it was running or not.
-     */
-
-    hms_write_cr("r");
-       
-    expect_prompt();
-
-
-    insert_breakpoints ();     /* Needed to get correct instruction in cache */
-    proceed(entry_pt, -1, 0);
-
-
-  } 
-  DEXIT("hms_create_inferior()");
-}
-
-/* Translate baud rates from integers to damn B_codes.  Unix should
-   have outgrown this crap years ago, but even POSIX wouldn't buck it.  */
-
-#ifndef B19200
-#define B19200 EXTA
-#endif
-#ifndef B38400
-#define B38400 EXTB
-#endif
 
-static struct {int rate, damn_b;} baudtab[] = {
-       {9600, B9600},
-       {19200, B19200},
-       {300, B300},
-       {1200, B1200},
-       {2400, B2400},
-       {4800, B4800},
-       {-1, -1},
-};
 
-static int damn_b (rate)
-     int rate;
-{
-  int i;
+  hms_kill(NULL,NULL);  
+  hms_clear_breakpoints();
+  init_wait_for_inferior ();
+  hms_write_cr("");
+  expect_prompt();
 
-  for (i = 0; baudtab[i].rate != -1; i++)
-    if (rate == baudtab[i].rate) return baudtab[i].damn_b;
-  return B19200;
+  insert_breakpoints ();       /* Needed to get correct instruction in cache */
+  proceed(entry_pt, -1, 0);
 }
 
 
@@ -651,66 +534,49 @@ static int baudrate = 9600;
 static int
 is_baudrate_right()
 {
-  
-
+  int ok;
   /* Put this port into NORMAL mode, send the 'normal' character */
+
   hms_write("\001", 1);                /* Control A */
   hms_write("\r", 1);          /* Cr */
-
-  while ( readchar_nofail()) /* Skip noise we put there */
-   ;
   
-  hms_write("r");
+  while (1) 
+  {
+    ok = serial_readchar(timeout);
+    if (ok < 0) break;
+  }
+
+  hms_write("r",1);
+
   if (readchar_nofail() == 'r')  
    return 1;
 
   /* Not the right baudrate, or the board's not on */
   return 0;
-  
-
 }
 static void
 set_rate()
 {
-  TERMINAL sg;    
-  ioctl (hms_desc, TIOCGETP, &sg);
-#ifdef HAVE_TERMIO
-  sg.c_cc[VMIN] = 0;           /* read with timeout.  */
-  sg.c_cc[VTIME] = timeout * 10;
-  sg.c_lflag &= ~(ICANON | ECHO);
-  sg.c_cflag = (sg.c_cflag & ~CBAUD) | damn_b (baudrate);
-#else
-  sg.sg_ispeed = damn_b (baudrate);
-  sg.sg_ospeed = damn_b (baudrate);
-  sg.sg_flags |= RAW | ANYP;
-  sg.sg_flags &= ~ECHO;
-#endif
-
-  ioctl (hms_desc, TIOCSETP, &sg);
+  if (!serial_setbaudrate(baudrate))
+   error("Can't set baudrate");
 }
 
 static void
 get_baudrate_right()
 {
-
-  int which_rate = 0;
-  
+#if 0
   while (!is_baudrate_right()) 
   {
-    if (baudtab[which_rate].rate == -1)
-    {
-      which_rate = 0;
-    }
-    else
-    {
-      which_rate++;
+    baudrate = serial_nextbaudrate(baudrate);
+    if (baudrate == 0) {
+      printf_filtered("Board not yet in sync\n");
+      break;
     }
-    
-    baudrate = baudtab[which_rate].rate;
     printf_filtered("Board not responding, trying %d baud\n",baudrate);
     QUIT;
-    set_rate();
+    serial_setbaudrate(baudrate);
   }
+#endif
 }
 
 static void
@@ -722,54 +588,31 @@ hms_open (name, from_tty)
   unsigned int prl;
   char *p;
   
-  DENTER("hms_open()");
   if(name == 0) 
   {
     name = "";
-    
   }
-  
-  hms_close (0);
-
-  hms_desc = open (dev_name, O_RDWR);
-  if (hms_desc < 0)
-   perror_with_name (dev_name);
-
-  set_rate();
+  if (is_open)  
+    hms_close (0);
+  if (name && strlen(name))
+    dev_name = strdup(name);
+  if (!serial_open(dev_name))
+    perror_with_name ((char *)dev_name);
+  serial_raw();
+  is_open = 1;
 
   dcache_init();
 
-
-  /* start_remote ();              /* Initialize gdb process mechanisms */
-
-
-#ifndef HAVE_TERMIO
-#ifndef NO_SIGINTERRUPT
-  /* Cause SIGALRM's to make reads fail with EINTR instead of resuming
-     the read.  */
-  if (siginterrupt (SIGALRM, 1) != 0)
-   perror ("hms_open: error in siginterrupt");
-#endif
-
-  /* Set up read timeout timer.  */
-  if ((void (*)) signal (SIGALRM, hms_timer) == (void (*)) -1)
-   perror ("hms_open: error in signal");
-#endif
-
   get_baudrate_right();
   
   /* Hello?  Are you there?  */
-  write (hms_desc, "\r", 1);
+  serial_write("\r",1);
   expect_prompt ();
 
   /* Clear any break points */
   hms_clear_breakpoints();
 
-
-  printf_filtered("Remote debugging on an H8/300 HMS via %s.\n",dev_name);
-
-  DEXIT("hms_open()");
+  printf_filtered("Connected to remote H8/300 HMS system.\n");
 }
 
 /* Close out all files and local state before this target loses control. */
@@ -779,60 +622,17 @@ hms_close (quitting)
      int quitting;
 {
 
-  DENTER("hms_close()");
 
   /* Clear any break points */
   hms_clear_breakpoints();
 
   /* Put this port back into REMOTE mode */ 
-  if (OPEN(hms_desc)) {
-
-     sleep(1);         /* Let any output make it all the way back */
-     write(hms_desc, "R\r", 2);
-  }
-
-  /* Due to a bug in Unix, fclose closes not only the stdio stream,
-     but also the file descriptor.  So we don't actually close
-     hms_desc.  */
-  if (OPEN(hms_desc))
-    close (hms_desc); 
-
-  /* Do not try to close hms_desc again, later in the program.  */
-
-  hms_desc = -1;
-
-  DEXIT("hms_close()");
-}
-
-/* Attach to the target that is already loaded and possibly running */
-static void
-hms_attach (args, from_tty)
-     char *args;
-     int from_tty;
-{
-
-  DENTER("hms_attach()");
-
-  /* push_target(&hms_ops);    /* This done in hms_open() */
-
-  mark_breakpoints_out ();
-
-  /* Send the hms a kill. It is ok if it is not already running */
-#if 0
-  fprintf(hms_stream, "K\r"); 
-  expect_prompt();             /* Slurp the echo */
-#endif
-  /* We will get a task spawn event immediately.  */
-  init_wait_for_inferior ();
-  clear_proceed_status ();
-  stop_soon_quietly = 1;
-  wait_for_inferior ();
-  stop_soon_quietly = 0;
-  normal_stop ();
-  DEXIT("hms_attach()");
+  sleep(1);                    /* Let any output make it all the way back */
+  serial_write("R\r", 2);
+  serial_close();
+  is_open = 0;
 }
 
-
 /* Terminate the open connection to the remote debugger.
    Use this when you want to detach and do something else
    with your gdb.  */
@@ -841,18 +641,14 @@ hms_detach (args,from_tty)
      char *args;
      int from_tty;
 {
-  DENTER("hms_detach()");
-  if (OPEN(hms_desc)) { /* Send it on its way (tell it to continue)  */
+  if (is_open)
+  { 
        hms_clear_breakpoints();
-#if 0
-       fprintf(hms_stream,"G\r");
-#endif
   }
  
   pop_target();                /* calls hms_close to do the real work */
   if (from_tty)
     printf_filtered ("Ending remote %s debugging\n", target_shortname);
-  DEXIT("hms_detach()");
 }
  
 /* Tell the remote machine to resume.  */
@@ -861,15 +657,13 @@ void
 hms_resume (step, sig)
      int step, sig;
 {
-  DENTER("hms_resume()");
   dcache_flush();
   
   if (step)    
   {
     hms_write_cr("s");
-
-    hms_write("\003",1);
-    expect_prompt();
+    expect("Step>");
+    
     /* Force the next hms_wait to return a trap.  Not doing anything
        about I/O from the target means that the user has to type
        "continue" to see any.  FIXME, this should be fixed.  */
@@ -878,9 +672,8 @@ hms_resume (step, sig)
   else
   {
     hms_write_cr("g");
-    expect("g\r");
+    expect("g");
   }
-  DEXIT("hms_resume()");
 }
 
 /* Wait until the remote machine stops, then return,
@@ -895,7 +688,7 @@ hms_wait (status)
      of the string cannot recur in the string, or we will not
      find some cases of the string in the input.  */
   
-  static char bpt[] = "At breakpoint:\r";
+  static char bpt[] = "At breakpoint:";
   /* It would be tempting to look for "\n[__exit + 0x8]\n"
      but that requires loading symbols with "yc i" and even if
      we did do that we don't know that the file has symbols.  */
@@ -914,8 +707,6 @@ hms_wait (status)
   int old_immediate_quit = immediate_quit;
   int swallowed_cr = 0;
   
-  DENTER("hms_wait()");
-
   WSETEXIT ((*status), 0);
 
   if (need_artificial_trap != 0)
@@ -925,48 +716,58 @@ hms_wait (status)
     return 0;
   }
 
-  timeout = 0;                 /* Don't time out -- user program is running. */
+  timeout = 99999;     /* Don't time out -- user program is running. */
   immediate_quit = 1;          /* Helps ability to QUIT */
-  while (1) {
-      QUIT;                    /* Let user quit and leave process running */
-      ch_handled = 0;
-      ch = readchar ();
-      if (ch == *bp) {
-         bp++;
-         if (*bp == '\0')
-          break;
-         ch_handled = 1;
-
-         *swallowed_p++ = ch;
-       } else
-        bp = bpt;
-      if (ch == *ep || *ep == '?') {
-         ep++;
-         if (*ep == '\0')
-          break;
-
-         if (!ch_handled)
-          *swallowed_p++ = ch;
-         ch_handled = 1;
-       } else
-        ep = exitmsg;
-      if (!ch_handled) {
-         char *p;
-         /* Print out any characters which have been swallowed.  */
-         for (p = swallowed; p < swallowed_p; ++p)
-          putc (*p, stdout);
-         swallowed_p = swallowed;
+  while (1) 
+  {
+    QUIT;                      /* Let user quit and leave process running */
+    ch_handled = 0;
+    ch = readchar ();
+    if (ch == *bp) 
+    {
+       bp++;
+       if (*bp == '\0')
+        break;
+       ch_handled = 1;
+
+       *swallowed_p++ = ch;
+      } 
+    else 
+    {
+      bp = bpt;
+    }
+    if (ch == *ep || *ep == '?') 
+    {
+       ep++;
+       if (*ep == '\0')
+        break;
+
+       if (!ch_handled)
+        *swallowed_p++ = ch;
+       ch_handled = 1;
+      } 
+    else 
+    {
+      ep = exitmsg;
+    }
+      
+    if (!ch_handled) {
+       char *p;
+       /* Print out any characters which have been swallowed.  */
+       for (p = swallowed; p < swallowed_p; ++p)
+        putc (*p, stdout);
+       swallowed_p = swallowed;
 
          
-         if ((ch != '\r' && ch != '\n') || swallowed_cr>10) 
-         {
-           putc (ch, stdout);
-           swallowed_cr = 10;
-         }
-         swallowed_cr ++;
-         
+       if ((ch != '\r' && ch != '\n') || swallowed_cr>10) 
+       {
+         putc (ch, stdout);
+         swallowed_cr = 10;
        }
-    }
+       swallowed_cr ++;
+         
+      }
+  }
   if (*bp== '\0') 
   {
     WSETSTOP ((*status), SIGTRAP);
@@ -979,7 +780,6 @@ hms_wait (status)
   
   timeout = old_timeout;
   immediate_quit = old_immediate_quit;
-  DEXIT("hms_wait()");
   return 0;
 }
 
@@ -991,11 +791,8 @@ static char *
 get_reg_name (regno)
      int regno;
 {
-static  char *rn[NUM_REGS]= REGISTER_NAMES;
-  
-
+  static  char *rn[NUM_REGS]= REGISTER_NAMES;
   return rn[regno];
-
 }
 
 /* Read the remote registers.  */
@@ -1050,23 +847,25 @@ hms_write(a,l)
 char *a;
 {
   int i;
-  write(hms_desc,a,l);
-if (!quiet)
-  for (i = 0; i < l ; i++)
-  {
-    printf("%c", a[i]);
-  }
+  serial_write(a, l);
+
+  if (!quiet)
+   for (i = 0; i < l ; i++)
+   {
+     printf("%c", a[i]);
+   }
 }
 
 hms_write_cr(s)
 char *s;
 {
-hms_write( s, strlen(s));
-hms_write("\r",1);  
+  hms_write( s, strlen(s));
+  hms_write("\r",1);  
 }
 
 static void
-hms_fetch_registers ()
+hms_fetch_register (dummy)
+int dummy;
 {
 #define REGREPLY_SIZE 79
   char linebuf[REGREPLY_SIZE+1];
@@ -1088,8 +887,6 @@ hms_fetch_registers ()
     linebuf[REGREPLY_SIZE] = 0;
     gottok = 0;    
     if (linebuf[0] == 'r' &&
-       linebuf[1] == '\r' &&
-       linebuf[2] == '\n' &&
        linebuf[3] == 'P' &&
        linebuf[4] == 'C' &&
        linebuf[5] == '=' && 
@@ -1115,43 +912,30 @@ hms_fetch_registers ()
   while (!gottok);
   for (i = 0; i < NUM_REGS; i++) 
   {
-    supply_register (i, reg+i);
-  }  
-}
-
-/* Fetch register REGNO, or all registers if REGNO is -1.
- */
-static void
-hms_fetch_register (regno)
-     int regno;
-{
-
-    hms_fetch_registers ();
+    char swapped[2];
+    swapped[1] = reg[i];
+    swapped[0] = (reg[i])>> 8;
 
+    supply_register (i, swapped);
+  }  
 }
 
-/* Store the remote registers from the contents of the block REGS.  */
-
-static int 
-hms_store_registers ()
-{
-  int i;
-  for (i = 0; i < NUM_REGS; i++)
-   hms_store_register(i);
-  return 0;
-  
-}
 
 /* Store register REGNO, or all if REGNO == -1.
    Return errno value.  */
-int
+static void
 hms_store_register (regno)
      int regno;
 {
 
   /* printf("hms_store_register() called.\n"); fflush(stdout); /* */
-  if (regno == -1)
-   hms_store_registers ();
+  if (regno == -1) 
+  {
+    for (regno = 0; regno < NUM_REGS; regno ++) 
+    {
+      hms_store_register(regno);
+    }
+  }
   else
   {
     char *name = get_reg_name (regno);
@@ -1160,11 +944,10 @@ hms_store_register (regno)
     hms_write_cr(buffer);
     expect_prompt();
   }
-  
-  DEXIT("hms_store_registers()");
-  return 0;
 }
 
+
+
 /* Get ready to modify the registers array.  On machines which store
    individual registers, this doesn't need to do anything.  On machines
    which store all the registers in one fell swoop, this makes sure
@@ -1284,50 +1067,53 @@ hms_xfer_inferior_memory(memaddr, myaddr, len, write, target)
   return len;
 }
 
-#if 0
-int
-hms_xfer_inferior_memory (memaddr, myaddr, len, write)
-     CORE_ADDR memaddr;
-     char *myaddr;
-     int len;
-     int write;
-{
-  memaddr &= 0xffff;
-  if (write)
-   return hms_write_inferior_memory (memaddr, myaddr, len);
-  else
-   return hms_read_inferior_memory (memaddr, myaddr, len);
-
-}
-#endif
-
 int
 hms_write_inferior_memory (memaddr, myaddr, len)
      CORE_ADDR memaddr;
-     char *myaddr;
+     unsigned char *myaddr;
      int len;
 {
+  bfd_vma addr;
+  int done;
+  int todo ;
+  done = 0;
+  while(done < len) 
+  {
+    char buffer[20];
+    int thisgo;
+    int idx;
+    thisgo = len - done;
+    if (thisgo > 20) thisgo = 20;
+
+    sprintf(buffer,"M.B %4x =", memaddr+done);
+    hms_write(buffer,10);
+    for (idx = 0; idx < thisgo; idx++) 
+    {
+      char buf[20];
+      sprintf(buf, "%2x ", myaddr[idx+done]); 
+      hms_write(buf, 3);
+    }
+    hms_write_cr("");
+    expect_prompt();
+    done += thisgo;
+  }
+
 
-  bfd *abfd = bfd_openw(dev_name, "srec");
-  asection *a;
-
-  bfd_set_format(abfd, bfd_object);
-  a = bfd_make_section(abfd, ".text");
-  a->vma = memaddr;
-  a->_raw_size = len;
-  a->flags = SEC_LOAD | SEC_HAS_CONTENTS;
-  hms_write_cr("tl");          /* tell hms here comes the recs */  
-  bfd_set_section_contents(abfd, a, myaddr, 0, len);
-  bfd_close(abfd);
-  
-  expect_prompt();
 }
 
 void
 hms_files_info ()
 {
-  printf_filtered("\tAttached to %s at %d baud and running program %s\n",
-         dev_name, baudrate, bfd_get_filename(exec_bfd));
+char *file = "nothing";
+if (exec_bfd)
+ file = bfd_get_filename(exec_bfd);
+
+if (exec_bfd)
+#ifdef __GO32__
+   printf_filtered("\tAttached to DOS asynctsr and running program %s\n",file);
+#else
+  printf_filtered("\tAttached to %s at %d baud and running program %s\n",file);
+#endif
   printf_filtered("\ton an H8/300 processor.\n");
 }
 
@@ -1366,7 +1152,7 @@ hms_read_inferior_memory(memaddr, myaddr, len)
   if (memaddr & 0xf) abort();
   if (len != 16) abort();
   
-  sprintf(buffer, "m %4x %4x", start, end);
+  sprintf(buffer, "m %4x %4x", start & 0xffff, end & 0xffff);
   hms_write_cr(buffer);
   /* drop the echo and newline*/
   for (i = 0; i < 13; i++)
@@ -1414,17 +1200,9 @@ hms_read_inferior_memory(memaddr, myaddr, len)
        
     }
   } 
-  
-  
-  
-  hms_write("\003",1);
+  hms_write_cr(" ");
   expect_prompt();
-  
-
-  
-  
   return len;
-
 }
 
 /* This routine is run as a hook, just before the main command loop is
@@ -1440,29 +1218,6 @@ hms_before_main_loop ()
   extern jmp_buf to_top_level;
 
   push_target (&hms_ops);
-#if 0
-
-  while (current_target != &hms_ops) {
-      /* remote tty not specified yet */
-      if ( instream == stdin ){
-         printf("\nEnter device and filename, or \"quit\" to quit:  ");
-         fflush( stdout );
-       }
-      fgets( ttyname, sizeof(ttyname)-1, stdin );
-
-      if ( !strcmp("quit", ttyname) ){
-         exit(1);
-       }
-
-      hms_open( ttyname, 1 );
-
-      /* Now that we have a tty open for talking to the remote machine,
-        download the executable file if one was specified.  */
-      if ( !setjmp(to_top_level) && exec_bfd ) {
-         target_load (bfd_get_filename (exec_bfd), 1);
-       }
-    }
-#endif
 }
 
 
@@ -1473,24 +1228,23 @@ hms_insert_breakpoint(addr, save)
 CORE_ADDR      addr;
 char           *save;  /* Throw away, let hms save instructions */
 {
-
-  DENTER("hms_insert_breakpoint()"); 
   check_open();
   
-  if (num_brkpts < MAX_BREAKS) {
-      char buffer[100];
-      num_brkpts++;
-      sprintf(buffer,"b %x", addr & 0xffff);
-      hms_write_cr(buffer);
-      expect_prompt ();
-      DEXIT("hms_insert_breakpoint() success"); 
-      return(0);               /* Success */
-    } else {
-       fprintf_filtered(stderr,
-                        "Too many break points, break point not installed\n");
-       DEXIT("hms_insert_breakpoint() failure"); 
-       return(1);              /* Failure */
-      }
+  if (num_brkpts < MAX_BREAKS) 
+  {
+    char buffer[100];
+    num_brkpts++;
+    sprintf(buffer,"b %x", addr & 0xffff);
+    hms_write_cr(buffer);
+    expect_prompt ();
+    return(0);
+  }
+  else 
+  {
+    fprintf_filtered(stderr,
+                    "Too many break points, break point not installed\n");
+    return(1);
+  }
 
 
 }
@@ -1499,17 +1253,16 @@ hms_remove_breakpoint(addr, save)
 CORE_ADDR      addr;
 char           *save;  /* Throw away, let hms save instructions */
 {
-  DENTER("hms_remove_breakpoint()");
-  if (num_brkpts > 0) {
-      char buffer[100];
+  if (num_brkpts > 0) 
+  {
+    char buffer[100];
       
-      num_brkpts--;
-      sprintf(buffer,"b - %x", addr & 0xffff);
-      hms_write_cr(buffer);
-      expect_prompt();
+    num_brkpts--;
+    sprintf(buffer,"b - %x", addr & 0xffff);
+    hms_write_cr(buffer);
+    expect_prompt();
       
-    }
-  DEXIT("hms_remove_breakpoint()");
+  }
   return(0);
 }
 
@@ -1518,53 +1271,23 @@ static int
 hms_clear_breakpoints() 
 { 
 
-  DENTER("hms_clear_breakpoint()");
-  if (OPEN(hms_desc)) {
-      hms_write_cr("b -");
-      expect_prompt ();
-    }
+  if (is_open) {
+    hms_write_cr("b -");
+    expect_prompt ();
+  }
   num_brkpts = 0;
-
-  DEXIT("hms_clear_breakpoint()");
 }
 static void
 hms_mourn() 
 { 
-  DENTER("hms_mourn()");
   hms_clear_breakpoints();
-/*  pop_target ();                /* Pop back to no-child state */
   generic_mourn_inferior ();
-  DEXIT("hms_mourn()");
 }
 
-/* Display everthing we read in from the hms until we match/see the
- * specified string
- */
-static int
-display_until(str)
-char   *str;
-{
-       int     i=0,j,c;
-
-       while (c=readchar()) {
-               if (c==str[i]) {
-                       i++;
-                       if (i == strlen(str)) return;
-               } else {
-                       if (i) {
-                           for (j=0 ; j<i ; j++) /* Put everthing we matched */
-                               putchar(str[j]);
-                           i=0;
-                       }
-                       putchar(c);
-               }       
-       }
-
-}
 
 
 /* Put a command string, in args, out to the hms.  The hms is assumed to
-   be in raw mode, all writing/reading done through hms_desc.
+   be in raw mode, all writing/reading done through desc.
    Ouput from the hms is placed on the users terminal until the
    prompt from the hms is seen.
    FIXME: Can't handle commands that take input.  */
@@ -1580,11 +1303,9 @@ hms_com (args, fromtty)
        
   /* Clear all input so only command relative output is displayed */
 
-
   hms_write_cr(args);
   hms_write("\030",1);
   expect_prompt();
-
 }
 
 /* Define the target subroutine names */
@@ -1595,19 +1316,20 @@ struct target_ops hms_ops = {
 by a serial line.",
 
        hms_open, hms_close, 
-       hms_attach, hms_detach, hms_resume, hms_wait,
+       0, hms_detach, hms_resume, hms_wait,    /* attach */
        hms_fetch_register, hms_store_register,
-       hms_prepare_to_store, 0, 0,     /* conv_to, conv_from */
+       hms_prepare_to_store,
        hms_xfer_inferior_memory, 
        hms_files_info,
        hms_insert_breakpoint, hms_remove_breakpoint, /* Breakpoints */
        0, 0, 0, 0, 0,          /* Terminal handling */
        hms_kill,               /* FIXME, kill */
        hms_load, 
-       call_function_by_hand,
        0,                      /* lookup_symbol */
        hms_create_inferior,    /* create_inferior */ 
        hms_mourn,              /* mourn_inferior FIXME */
+       0,                      /* can_run */
+       0,                      /* notice_signals */
        process_stratum, 0, /* next */
        1, 1, 1, 1, 1,  /* all mem, mem, stack, regs, exec */
        0,0,            /* Section pointers */
@@ -1617,38 +1339,39 @@ by a serial line.",
 
 hms_quiet()
 {
-  quiet = ! quiet;  
+ quiet = ! quiet;  
+   if (quiet)
+     printf_filtered("Snoop disabled\n");
+   else
+     printf_filtered("Snoop enabled\n");
+
 }
 
 hms_device(s)
 char *s;
 {
-  if (s) {
-      dev_name = get_word(&s);
-    }  
+  if (s) 
+  {
+    dev_name = get_word(&s);
+  }  
 }
 
-static hms_speed(s)
+
+static 
+hms_speed(s)
 char *s;
 {
+  check_open();
+  
   if (s) 
   {
     char buffer[100];
     int newrate = atoi(s);
     int which = 0;
-    while (baudtab[which].rate != newrate) 
-    {
-      if (baudtab[which].rate == -1) 
-      {
+    if (!serial_setbaudrate(newrate))
        error("Can't use %d baud\n", newrate);
-      }
-      which++;
-    }
-    
-
     
     printf_filtered("Checking target is in sync\n");
-
     
     get_baudrate_right();
     baudrate = newrate;
@@ -1656,7 +1379,7 @@ char *s;
                    baudrate);    
     
     sprintf(buffer, "tm %d. N 8 1", baudrate);
-    hms_write(buffer);
+    hms_write_cr(buffer);
   }    
 }
 
@@ -1670,12 +1393,15 @@ _initialize_remote_hms ()
        "Send a command to the HMS monitor.");
   add_com ("snoop", class_obscure, hms_quiet,
           "Show what commands are going to the monitor");
+
   add_com ("device", class_obscure, hms_device,
           "Set the terminal line for HMS communications");
 
   add_com ("speed", class_obscure, hms_speed,
           "Set the terminal line speed for HMS communications");
   
+#if 0
+  dev_name = serial_default_name();
+#endif
+  dev_name = NULL;
 }
-
-