2002-04-09 Daniel Jacobowitz <drow@mvista.com>
[deliverable/binutils-gdb.git] / gdb / remote-sds.c
index 80564a936ea855bb8c00fdc65182b49d88a75e79..507ac5be208a0a8ee8e90403c227bda3439d2bb7 100644 (file)
@@ -1,21 +1,24 @@
 /* Remote target communications for serial-line targets using SDS' protocol.
-   Copyright 1997 Free Software Foundation, Inc.
 
-This file is part of GDB.
+   Copyright 1997, 1998, 1999, 2000, 2001, 2002 Free Software
+   Foundation, Inc.
 
-This program is free software; you can redistribute it and/or modify
-it under the terms of the GNU General Public License as published by
-the Free Software Foundation; either version 2 of the License, or
-(at your option) any later version.
+   This file is part of GDB.
 
-This program is distributed in the hope that it will be useful,
-but WITHOUT ANY WARRANTY; without even the implied warranty of
-MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-GNU General Public License for more details.
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 2 of the License, or
+   (at your option) any later version.
 
-You should have received a copy of the GNU General Public License
-along with this program; if not, write to the Free Software
-Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
+
+   You should have received a copy of the GNU General Public License
+   along with this program; if not, write to the Free Software
+   Foundation, Inc., 59 Temple Place - Suite 330,
+   Boston, MA 02111-1307, USA.  */
 
 /* This interface was written by studying the behavior of the SDS
    monitor on an ADS 821/860 board, and by consulting the
@@ -30,13 +33,12 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 #include "bfd.h"
 #include "symfile.h"
 #include "target.h"
-#include "wait.h"
 #include "gdbcmd.h"
 #include "objfiles.h"
 #include "gdb-stabs.h"
 #include "gdbthread.h"
-
-#include "dcache.h"
+#include "gdbcore.h"
+#include "regcache.h"
 
 #ifdef USG
 #include <sys/types.h>
@@ -45,155 +47,132 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 #include <signal.h>
 #include "serial.h"
 
+extern void _initialize_remote_sds (void);
+
 /* Declarations of local functions. */
 
-static int sds_write_bytes PARAMS ((CORE_ADDR, char *, int));
+static int sds_write_bytes (CORE_ADDR, char *, int);
+
+static int sds_read_bytes (CORE_ADDR, char *, int);
 
-static int sds_read_bytes PARAMS ((CORE_ADDR, char *, int));
+static void sds_files_info (struct target_ops *ignore);
 
-static void sds_files_info PARAMS ((struct target_ops *ignore));
+static int sds_xfer_memory (CORE_ADDR, char *, int, int, 
+                           struct mem_attrib *, struct target_ops *);
 
-static int sds_xfer_memory PARAMS ((CORE_ADDR, char *,
-                                   int, int, struct target_ops *));
+static void sds_prepare_to_store (void);
 
-static void sds_prepare_to_store PARAMS ((void));
+static void sds_fetch_registers (int);
 
-static void sds_fetch_registers PARAMS ((int));
+static void sds_resume (ptid_t, int, enum target_signal);
 
-static void sds_resume PARAMS ((int, int, enum target_signal));
+static int sds_start_remote (PTR);
 
-static int sds_start_remote PARAMS ((char *));
+static void sds_open (char *, int);
 
-static void sds_open PARAMS ((char *, int));
+static void sds_close (int);
 
-static void sds_close PARAMS ((int));
+static void sds_store_registers (int);
 
-static void sds_store_registers PARAMS ((int));
+static void sds_mourn (void);
 
-static void sds_mourn PARAMS ((void));
+static void sds_create_inferior (char *, char *, char **);
 
-static void sds_restart PARAMS ((void));
+static void sds_load (char *, int);
 
-static void sds_create_inferior PARAMS ((char *, char *, char **));
+static int getmessage (unsigned char *, int);
 
-static int getmessage PARAMS ((unsigned char *, int));
+static int putmessage (unsigned char *, int);
 
-static int putmessage PARAMS ((unsigned char *, int));
+static int sds_send (unsigned char *, int);
 
-static int sds_send PARAMS ((unsigned char *, int));
+static int readchar (int);
 
-static int readchar PARAMS ((int));
+static ptid_t sds_wait (ptid_t, struct target_waitstatus *);
 
-static int sds_wait PARAMS ((int, struct target_waitstatus *));
+static void sds_kill (void);
 
-static void sds_kill PARAMS ((void));
+static int tohex (int);
 
-static int tohex PARAMS ((int));
+static int fromhex (int);
 
-static int fromhex PARAMS ((int));
+static void sds_detach (char *, int);
 
-static void sds_detach PARAMS ((char *, int));
+static void sds_interrupt (int);
 
-static void sds_interrupt PARAMS ((int));
+static void sds_interrupt_twice (int);
 
-static void sds_interrupt_twice PARAMS ((int));
+static void interrupt_query (void);
 
-static void interrupt_query PARAMS ((void));
+static int read_frame (char *);
 
-static int read_frame PARAMS ((char *));
+static int sds_insert_breakpoint (CORE_ADDR, char *);
 
-static int sds_insert_breakpoint PARAMS ((CORE_ADDR, char *));
+static int sds_remove_breakpoint (CORE_ADDR, char *);
 
-static int sds_remove_breakpoint PARAMS ((CORE_ADDR, char *));
+static void init_sds_ops (void);
 
+static void sds_command (char *args, int from_tty);
 
-static struct target_ops sds_ops;      /* Forward decl */
+/* Define the target operations vector. */
+
+static struct target_ops sds_ops;
 
 /* This was 5 seconds, which is a long time to sit and wait.
    Unless this is going though some terminal server or multiplexer or
    other form of hairy serial connection, I would think 2 seconds would
    be plenty.  */
 
-/* Changed to allow option to set timeout value.
-   was static int sds_timeout = 2; */
 static int sds_timeout = 2;
 
-/* This variable chooses whether to send a ^C or a break when the user
-   requests program interruption.  Although ^C is usually what remote
-   systems expect, and that is the default here, sometimes a break is
-   preferable instead.  */
-
-static int sds_break;
-
 /* Descriptor for I/O to remote machine.  Initialize it to NULL so
    that sds_open knows that we don't have a file open when the program
    starts.  */
 
-static serial_t sds_desc = NULL;
+static struct serial *sds_desc = NULL;
 
-/* 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).  */
-#define        PBUFSIZ 400
+/* This limit comes from the monitor.  */
+
+#define        PBUFSIZ 250
 
 /* Maximum number of bytes to read/write at once.  The value here
    is chosen to fill up a packet (the headers account for the 32).  */
 #define MAXBUFBYTES ((PBUFSIZ-32)/2)
 
-/* Round up PBUFSIZ to hold all the registers, at least.  */
-/* The blank line after the #if seems to be required to work around a
-   bug in HP's PA compiler.  */
-#if REGISTER_BYTES > MAXBUFBYTES
-
-#undef PBUFSIZ
-#define        PBUFSIZ (REGISTER_BYTES * 2 + 32)
-#endif
-
 static int next_msg_id;
 
 static int just_started;
 
 static int message_pending;
-
 \f
-/*  Restart the remote side; this is an extended protocol operation.  */
 
-static void
-sds_restart ()
-{
-}
-\f
 /* Clean up connection to a remote debugger.  */
 
 /* ARGSUSED */
 static void
-sds_close (quitting)
-     int quitting;
+sds_close (int quitting)
 {
   if (sds_desc)
-    SERIAL_CLOSE (sds_desc);
+    serial_close (sds_desc);
   sds_desc = NULL;
 }
 
 /* Stub for catch_errors.  */
 
 static int
-sds_start_remote (dummy)
-     char *dummy;
+sds_start_remote (PTR dummy)
 {
-  char c;
+  int c;
   unsigned char buf[200];
 
-  immediate_quit = 1;          /* Allow user to interrupt it */
+  immediate_quit++;            /* Allow user to interrupt it */
 
   /* Ack any packet which the remote side has already sent.  */
-  SERIAL_WRITE (sds_desc, "{#*\r\n", 5);
-  SERIAL_WRITE (sds_desc, "{#}\r\n", 5);
+  serial_write (sds_desc, "{#*\r\n", 5);
+  serial_write (sds_desc, "{#}\r\n", 5);
 
   while ((c = readchar (1)) >= 0)
-    printf_unfiltered ("%c");
+    printf_unfiltered ("%c", c);
   printf_unfiltered ("\n");
 
   next_msg_id = 251;
@@ -204,7 +183,7 @@ sds_start_remote (dummy)
   buf[0] = 0;
   sds_send (buf, 1);
 
-  immediate_quit = 0;
+  immediate_quit--;
 
   start_remote ();             /* Initialize gdb process mechanisms */
   return 1;
@@ -213,12 +192,8 @@ sds_start_remote (dummy)
 /* Open a connection to a remote debugger.
    NAME is the filename used for communication.  */
 
-static DCACHE *sds_dcache;
-
 static void
-sds_open (name, from_tty)
-     char *name;
-     int from_tty;
+sds_open (char *name, int from_tty)
 {
   if (name == 0)
     error ("To open a remote debug connection, you need to specify what serial\n\
@@ -228,27 +203,25 @@ device is attached to the remote system (e.g. /dev/ttya).");
 
   unpush_target (&sds_ops);
 
-  sds_dcache = dcache_init (sds_read_bytes, sds_write_bytes);
-
-  sds_desc = SERIAL_OPEN (name);
+  sds_desc = serial_open (name);
   if (!sds_desc)
     perror_with_name (name);
 
   if (baud_rate != -1)
     {
-      if (SERIAL_SETBAUDRATE (sds_desc, baud_rate))
+      if (serial_setbaudrate (sds_desc, baud_rate))
        {
-         SERIAL_CLOSE (sds_desc);
+         serial_close (sds_desc);
          perror_with_name (name);
        }
     }
 
 
-  SERIAL_RAW (sds_desc);
+  serial_raw (sds_desc);
 
   /* If there is something sitting in the buffer we might take it as a
      response to a command, which would be bad.  */
-  SERIAL_FLUSH_INPUT (sds_desc);
+  serial_flush_input (sds_desc);
 
   if (from_tty)
     {
@@ -258,21 +231,12 @@ device is attached to the remote system (e.g. /dev/ttya).");
     }
   push_target (&sds_ops);      /* Switch to using remote target now */
 
-  /* Without this, some commands which require an active target (such
-     as kill) won't work.  This variable serves (at least) double duty
-     as both the pid of the target process (if it has such), and as a
-     flag indicating that a target is active.  These functions should
-     be split out into seperate variables, especially since GDB will
-     someday have a notion of debugging several processes.  */
-
-  inferior_pid = 42000;
-
   just_started = 1;
 
   /* Start the remote connection; if error (0), discard this target.
      In particular, if the user quits, be sure to discard it (we'd be
      in an inconsistent state otherwise).  */
-  if (!catch_errors (sds_start_remote, (char *)0, 
+  if (!catch_errors (sds_start_remote, NULL,
                     "Couldn't establish connection to remote target\n",
                     RETURN_MASK_ALL))
     pop_target ();
@@ -284,9 +248,7 @@ device is attached to the remote system (e.g. /dev/ttya).");
    die when it hits one.  */
 
 static void
-sds_detach (args, from_tty)
-     char *args;
-     int from_tty;
+sds_detach (char *args, int from_tty)
 {
   char buf[PBUFSIZ];
 
@@ -307,59 +269,55 @@ sds_detach (args, from_tty)
 /* Convert hex digit A to a number.  */
 
 static int
-fromhex (a)
-     int a;
+fromhex (int a)
 {
   if (a >= '0' && a <= '9')
     return a - '0';
   else if (a >= 'a' && a <= 'f')
     return a - 'a' + 10;
-  else 
+  else
     error ("Reply contains invalid hex digit %d", a);
 }
 
 /* Convert number NIB to a hex digit.  */
 
 static int
-tohex (nib)
-     int nib;
+tohex (int nib)
 {
   if (nib < 10)
-    return '0'+nib;
+    return '0' + nib;
   else
-    return 'a'+nib-10;
+    return 'a' + nib - 10;
 }
 
 static int
-tob64 (inbuf, outbuf, len)
-     unsigned char *inbuf;
-     char *outbuf;
-     int len;
+tob64 (unsigned char *inbuf, char *outbuf, int len)
 {
   int i, sum;
+  char *p;
 
   if (len % 3 != 0)
     error ("bad length");
 
+  p = outbuf;
   for (i = 0; i < len; i += 3)
     {
       /* Collect the next three bytes into a number.  */
       sum = ((long) *inbuf++) << 16;
-      sum |= ((long) *inbuf++) <<  8;
+      sum |= ((long) *inbuf++) << 8;
       sum |= ((long) *inbuf++);
 
       /* Spit out 4 6-bit encodings.  */
-      *outbuf++ = ((sum >> 18) & 0x3f) + '0';
-      *outbuf++ = ((sum >> 12) & 0x3f) + '0';
-      *outbuf++ = ((sum >>  6) & 0x3f) + '0';
-      *outbuf++ = (sum & 0x3f) + '0';
+      *p++ = ((sum >> 18) & 0x3f) + '0';
+      *p++ = ((sum >> 12) & 0x3f) + '0';
+      *p++ = ((sum >> 6) & 0x3f) + '0';
+      *p++ = (sum & 0x3f) + '0';
     }
+  return (p - outbuf);
 }
 
 static int
-fromb64 (inbuf, outbuf, len)
-     char *inbuf, *outbuf;
-     int len;
+fromb64 (char *inbuf, char *outbuf, int len)
 {
   int i, sum;
 
@@ -369,74 +327,68 @@ fromb64 (inbuf, outbuf, len)
   for (i = 0; i < len; i += 4)
     {
       /* Collect 4 6-bit digits.  */
-      sum  = (*inbuf++ - '0') << 18;
+      sum = (*inbuf++ - '0') << 18;
       sum |= (*inbuf++ - '0') << 12;
-      sum |= (*inbuf++ - '0') <<  6;
+      sum |= (*inbuf++ - '0') << 6;
       sum |= (*inbuf++ - '0');
 
       /* Now take the resulting 24-bit number and get three bytes out
          of it.  */
       *outbuf++ = (sum >> 16) & 0xff;
-      *outbuf++ = (sum >>  8) & 0xff;
+      *outbuf++ = (sum >> 8) & 0xff;
       *outbuf++ = sum & 0xff;
     }
 
   return (len / 4) * 3;
 }
-
 \f
+
 /* Tell the remote machine to resume.  */
 
 static enum target_signal last_sent_signal = TARGET_SIGNAL_0;
 int last_sent_step;
 
 static void
-sds_resume (pid, step, siggnal)
-     int pid, step;
-     enum target_signal siggnal;
+sds_resume (ptid_t ptid, int step, enum target_signal siggnal)
 {
   unsigned char buf[PBUFSIZ];
 
-  dcache_flush (sds_dcache);
-
   last_sent_signal = siggnal;
   last_sent_step = step;
 
   buf[0] = (step ? 21 : 20);
-  buf[1] = 0;  /* (should be signal?) */
+  buf[1] = 0;                  /* (should be signal?) */
 
   sds_send (buf, 2);
 }
 \f
-/* Send ^C to target to halt it.  Target will respond, and send us a
-   packet.  */
+/* Send a message to target to halt it.  Target will respond, and send
+   us a message pending notice.  */
 
 static void
-sds_interrupt (signo)
-     int signo;
+sds_interrupt (int signo)
 {
+  unsigned char buf[PBUFSIZ];
+
   /* If this doesn't work, try more severe steps.  */
   signal (signo, sds_interrupt_twice);
-  
+
   if (remote_debug)
-    printf_unfiltered ("sds_interrupt called\n");
+    fprintf_unfiltered (gdb_stdlog, "sds_interrupt called\n");
 
-  /* Send a break or a ^C, depending on user preference.  */
-  if (sds_break)
-    SERIAL_SEND_BREAK (sds_desc);
-  else
-    SERIAL_WRITE (sds_desc, "\003", 1);
+  buf[0] = 25;
+  sds_send (buf, 1);
 }
 
-static void (*ofunc)();
+static void (*ofunc) ();
 
 /* The user typed ^C twice.  */
+
 static void
-sds_interrupt_twice (signo)
-     int signo;
+sds_interrupt_twice (int signo)
 {
   signal (signo, ofunc);
-  
+
   interrupt_query ();
 
   signal (signo, sds_interrupt);
@@ -445,7 +397,7 @@ sds_interrupt_twice (signo)
 /* Ask the user what to do when an interrupt is received.  */
 
 static void
-interrupt_query ()
+interrupt_query (void)
 {
   target_terminal_ours ();
 
@@ -453,7 +405,7 @@ interrupt_query ()
 Give up (and stop debugging it)? "))
     {
       target_mourn_inferior ();
-      return_to_top_level (RETURN_QUIT);
+      throw_exception (RETURN_QUIT);
     }
 
   target_terminal_inferior ();
@@ -466,10 +418,8 @@ int kill_kludge;
    STATUS just as `wait' would.  Returns "pid" (though it's not clear
    what, if anything, that means in the case of this target).  */
 
-static int
-sds_wait (pid, status)
-     int pid;
-     struct target_waitstatus *status;
+static ptid_t
+sds_wait (ptid_t ptid, struct target_waitstatus *status)
 {
   unsigned char buf[PBUFSIZ];
   int retlen;
@@ -485,7 +435,7 @@ sds_wait (pid, status)
     {
       just_started = 0;
       status->kind = TARGET_WAITKIND_STOPPED;
-      return inferior_pid;
+      return inferior_ptid;
     }
 
   while (1)
@@ -496,145 +446,108 @@ sds_wait (pid, status)
        {
          buf[0] = 26;
          retlen = sds_send (buf, 1);
+         if (remote_debug)
+           {
+             fprintf_unfiltered (gdb_stdlog, "Signals: %02x%02x %02x %02x\n",
+                                 buf[0], buf[1],
+                                 buf[2], buf[3]);
+           }
          message_pending = 0;
          status->kind = TARGET_WAITKIND_STOPPED;
+         status->value.sig = TARGET_SIGNAL_TRAP;
          goto got_status;
        }
     }
- got_status:
-  return inferior_pid;
+got_status:
+  return inferior_ptid;
 }
 
-/* Number of bytes of registers this stub implements.  */
-static int register_bytes_found;
+static unsigned char sprs[16];
 
 /* Read the remote registers into the block REGS.  */
 /* Currently we just read all the registers, so we don't use regno.  */
 
 /* ARGSUSED */
 static void
-sds_fetch_registers (regno)
-     int regno;
+sds_fetch_registers (int regno)
 {
   unsigned char buf[PBUFSIZ];
-  int i, len;
-  char *p;
+  int i, retlen;
   char regs[REGISTER_BYTES];
 
   /* Unimplemented registers read as all bits zero.  */
   memset (regs, 0, REGISTER_BYTES);
 
   buf[0] = 18;
-  buf[1] = 2;
+  buf[1] = 1;
   buf[2] = 0;
-  len = sds_send (buf, 3);
+  retlen = sds_send (buf, 3);
 
-  /* Reply describes registers byte by byte.  Suck them all up, then
-     supply them to the register cacheing/storage mechanism.  */
-
-  for (i = 0; i < len; i++)
-    regs[i] = buf[i];
+  for (i = 0; i < 4 * 6; ++i)
+    regs[i + 4 * 32 + 8 * 32] = buf[i];
+  for (i = 0; i < 4 * 4; ++i)
+    sprs[i] = buf[i + 4 * 7];
 
   buf[0] = 18;
-  buf[1] = 1;
+  buf[1] = 2;
   buf[2] = 0;
-  len = sds_send (buf, 3);
+  retlen = sds_send (buf, 3);
 
-  for (i = 0; i < 4 * 6; i++)
-    {
-      regs[i + 4 * 32 + 8 * 32] = buf[i];
-    }
+  for (i = 0; i < retlen; i++)
+    regs[i] = buf[i];
 
   /* (should warn about reply too short) */
 
- supply_them:
   for (i = 0; i < NUM_REGS; i++)
-    supply_register (i, &regs[REGISTER_BYTE(i)]);
+    supply_register (i, &regs[REGISTER_BYTE (i)]);
 }
 
-/* Prepare to store registers.  Since we may send them all (using a
-   'G' request), we have to read out the ones we don't want to change
-   first.  */
+/* Prepare to store registers.  Since we may send them all, we have to
+   read out the ones we don't want to change first.  */
 
-static void 
-sds_prepare_to_store ()
+static void
+sds_prepare_to_store (void)
 {
   /* Make sure the entire registers array is valid.  */
-  read_register_bytes (0, (char *)NULL, REGISTER_BYTES);
+  read_register_bytes (0, (char *) NULL, REGISTER_BYTES);
 }
 
 /* Store register REGNO, or all registers if REGNO == -1, from the contents
    of REGISTERS.  FIXME: ignores errors.  */
 
 static void
-sds_store_registers (regno)
-     int regno;
+sds_store_registers (int regno)
 {
-  unsigned char buf[PBUFSIZ];
+  unsigned char *p, buf[PBUFSIZ];
   int i;
-  char *p;
-
-  buf[0] = 19;
-  buf[1] = 2;
-  buf[2] = 0;
-  buf[3] = 0;
-
-  p = buf + 4;
 
+  /* Store all the special-purpose registers.  */
+  p = buf;
+  *p++ = 19;
+  *p++ = 1;
+  *p++ = 0;
+  *p++ = 0;
+  for (i = 0; i < 4 * 6; i++)
+    *p++ = registers[i + 4 * 32 + 8 * 32];
+  for (i = 0; i < 4 * 1; i++)
+    *p++ = 0;
+  for (i = 0; i < 4 * 4; i++)
+    *p++ = sprs[i];
+
+  sds_send (buf, p - buf);
+
+  /* Store all the general-purpose registers.  */
+  p = buf;
+  *p++ = 19;
+  *p++ = 2;
+  *p++ = 0;
+  *p++ = 0;
   for (i = 0; i < 4 * 32; i++)
-    p[i] = registers[i];
+    *p++ = registers[i];
 
-  sds_send (buf, 4 + 4 * 32);
+  sds_send (buf, p - buf);
 
-  buf[0] = 19;
-  buf[1] = 1;
-  buf[2] = 0;
-  buf[3] = 0;
-
-  p = buf + 4;
-
-  for (i = 0; i < 4 * 10; i++)
-    p[i] = registers[i];
-
-  sds_send (buf, 4 + 4 * 10);
-}
-
-/* 
-   Use of the data cache *used* to be disabled because it loses for looking at
-   and changing hardware I/O ports and the like.  Accepting `volatile'
-   would perhaps be one way to fix it.  Another idea would be to use the
-   executable file for the text segment (for all SEC_CODE sections?
-   For all SEC_READONLY sections?).  This has problems if you want to
-   actually see what the memory contains (e.g. self-modifying code,
-   clobbered memory, user downloaded the wrong thing).  
-
-   Because it speeds so much up, it's now enabled, if you're playing
-   with registers you turn it of (set remotecache 0)
-*/
-
-/* Read a word from remote address ADDR and return it.
-   This goes through the data cache.  */
-
-#if 0  /* unused? */
-static int
-sds_fetch_word (addr)
-     CORE_ADDR addr;
-{
-  return dcache_fetch (sds_dcache, addr);
-}
-
-/* Write a word WORD into remote address ADDR.
-   This goes through the data cache.  */
-
-static void
-sds_store_word (addr, word)
-     CORE_ADDR addr;
-     int word;
-{
-  dcache_poke (sds_dcache, addr, word);
 }
-#endif /* 0 (unused?) */
-
 \f
 /* Write memory data directly to the remote machine.  This does not
    inform the data cache; the data cache uses this.  MEMADDR is the
@@ -644,10 +557,7 @@ sds_store_word (addr, word)
    Returns number of bytes transferred, or 0 for error.  */
 
 static int
-sds_write_bytes (memaddr, myaddr, len)
-     CORE_ADDR memaddr;
-     char *myaddr;
-     int len;
+sds_write_bytes (CORE_ADDR memaddr, char *myaddr, int len)
 {
   int max_buf_size;            /* Max size of packet output buffer */
   int origlen;
@@ -668,8 +578,8 @@ sds_write_bytes (memaddr, myaddr, len)
       buf[1] = 0;
       buf[2] = (int) (memaddr >> 24) & 0xff;
       buf[3] = (int) (memaddr >> 16) & 0xff;
-      buf[4] = (int) (memaddr >>  8) & 0xff;
-      buf[5] = (int) (memaddr      ) & 0xff;
+      buf[4] = (int) (memaddr >> 8) & 0xff;
+      buf[5] = (int) (memaddr) & 0xff;
       buf[6] = 1;
       buf[7] = 0;
 
@@ -695,10 +605,7 @@ sds_write_bytes (memaddr, myaddr, len)
    Returns number of bytes transferred, or 0 for error.  */
 
 static int
-sds_read_bytes (memaddr, myaddr, len)
-     CORE_ADDR memaddr;
-     char *myaddr;
-     int len;
+sds_read_bytes (CORE_ADDR memaddr, char *myaddr, int len)
 {
   int max_buf_size;            /* Max size of packet output buffer */
   int origlen, retlen;
@@ -719,10 +626,10 @@ sds_read_bytes (memaddr, myaddr, len)
       buf[1] = 0;
       buf[2] = (int) (memaddr >> 24) & 0xff;
       buf[3] = (int) (memaddr >> 16) & 0xff;
-      buf[4] = (int) (memaddr >>  8) & 0xff;
-      buf[5] = (int) (memaddr      ) & 0xff;
+      buf[4] = (int) (memaddr >> 8) & 0xff;
+      buf[5] = (int) (memaddr) & 0xff;
       buf[6] = (int) (todo >> 8) & 0xff;
-      buf[7] = (int) (todo     ) & 0xff;
+      buf[7] = (int) (todo) & 0xff;
       buf[8] = 1;
 
       retlen = sds_send (buf, 9);
@@ -748,26 +655,28 @@ sds_read_bytes (memaddr, myaddr, len)
 /* Read or write LEN bytes from inferior memory at MEMADDR,
    transferring to or from debugger address MYADDR.  Write to inferior
    if SHOULD_WRITE is nonzero.  Returns length of data written or
-   read; 0 for error.  */
+   read; 0 for error.  TARGET is unused.  */
 
 /* ARGSUSED */
 static int
-sds_xfer_memory(memaddr, myaddr, len, should_write, target)
-     CORE_ADDR memaddr;
-     char *myaddr;
-     int len;
-     int should_write;
-     struct target_ops *target;                        /* ignored */
+sds_xfer_memory (CORE_ADDR memaddr, char *myaddr, int len, int should_write,
+                struct mem_attrib *attrib, struct target_ops *target)
 {
-  return dcache_xfer_memory (sds_dcache, memaddr, myaddr, len, should_write);
-}
+  int res;
 
+  if (should_write)
+    res = sds_write_bytes (memaddr, myaddr, len);
+  else
+    res = sds_read_bytes (memaddr, myaddr, len);
+  
+  return res;
+}
 \f
+
 static void
-sds_files_info (ignore)
-     struct target_ops *ignore;
+sds_files_info (struct target_ops *ignore)
 {
-  puts_filtered ("Debugging a target over a serial line.\n");
+  puts_filtered ("Debugging over a serial connection, using SDS protocol.\n");
 }
 \f
 /* Stuff for dealing with the packets which are part of this protocol.
@@ -776,15 +685,14 @@ sds_files_info (ignore)
 /* Read a single character from the remote end, masking it down to 7 bits. */
 
 static int
-readchar (timeout)
-     int timeout;
+readchar (int timeout)
 {
   int ch;
 
-  ch = SERIAL_READCHAR (sds_desc, timeout);
+  ch = serial_readchar (sds_desc, timeout);
 
   if (remote_debug > 1 && ch >= 0)
-    printf_unfiltered("%c(%x)", ch, ch);
+    fprintf_unfiltered (gdb_stdlog, "%c(%x)", ch, ch);
 
   switch (ch)
     {
@@ -799,10 +707,11 @@ readchar (timeout)
     }
 }
 
+/* An SDS-style checksum is a sum of the bytes modulo 253.  (Presumably
+   because 253, 254, and 255 are special flags in the protocol.)  */
+
 static int
-compute_checksum (csum, buf, len)
-     int csum, len;
-     char *buf;
+compute_checksum (int csum, char *buf, int len)
 {
   int i;
 
@@ -817,9 +726,7 @@ compute_checksum (csum, buf, len)
    into BUF also.  */
 
 static int
-sds_send (buf, len)
-     unsigned char *buf;
-     int len;
+sds_send (unsigned char *buf, int len)
 {
   putmessage (buf, len);
 
@@ -829,60 +736,52 @@ sds_send (buf, len)
 /* Send a message to the remote machine.  */
 
 static int
-putmessage (buf, len)
-     unsigned char *buf;
-     int len;
+putmessage (unsigned char *buf, int len)
 {
-  int i;
+  int i, enclen;
   unsigned char csum = 0;
   char buf2[PBUFSIZ], buf3[PBUFSIZ];
   unsigned char header[3];
-  int ch;
-  int tcount = 0;
   char *p;
 
   /* Copy the packet into buffer BUF2, encapsulating it
      and giving it a checksum.  */
 
-  if (len > (int) sizeof (buf2) - 5)           /* Prosanity check */
-    abort();
+  if (len > 170)               /* Prosanity check */
+    internal_error (__FILE__, __LINE__, "failed internal consistency check");
 
   if (remote_debug)
     {
-      fprintf_unfiltered (gdb_stderr, "Message to send: \"");
+      fprintf_unfiltered (gdb_stdlog, "Message to send: \"");
       for (i = 0; i < len; ++i)
-       fprintf_unfiltered (gdb_stderr, "%02x", buf[i]);
-      fprintf_unfiltered (gdb_stderr, "\"\n");
+       fprintf_unfiltered (gdb_stdlog, "%02x", buf[i]);
+      fprintf_unfiltered (gdb_stdlog, "\"\n");
     }
 
   p = buf2;
   *p++ = '$';
 
-  header[1] = next_msg_id;
-
   if (len % 3 != 0)
     {
       buf[len] = '\0';
-      buf[len+1] = '\0';
+      buf[len + 1] = '\0';
     }
 
-  len = ((len + 2) / 3) * 3;
+  header[1] = next_msg_id;
 
   header[2] = len;
 
   csum = compute_checksum (csum, buf, len);
-  csum = compute_checksum (csum, header+1, 2);
+  csum = compute_checksum (csum, header + 1, 2);
 
   header[0] = csum;
 
   tob64 (header, p, 3);
   p += 4;
-  tob64 (buf, buf3, len);
+  enclen = tob64 (buf, buf3, ((len + 2) / 3) * 3);
 
-  for (i = 0; i < (len / 3) * 4; i++)
-    {
-      *p++ = buf3[i];
-    }
+  for (i = 0; i < enclen; ++i)
+    *p++ = buf3[i];
   *p++ = '\r';
   *p++ = '\n';
 
@@ -892,31 +791,27 @@ putmessage (buf, len)
 
   while (1)
     {
-      int started_error_output = 0;
-
       if (remote_debug)
        {
          *p = '\0';
-         printf_unfiltered ("Sending encoded: \"%s\"", buf2);
-         printf_unfiltered ("  (Checksum %d, id %d, length %d)\n",
-                            header[0], header[1], header[2]);
-         gdb_flush (gdb_stdout);
+         fprintf_unfiltered (gdb_stdlog, "Sending encoded: \"%s\"", buf2);
+         fprintf_unfiltered (gdb_stdlog,
+                             "  (Checksum %d, id %d, length %d)\n",
+                             header[0], header[1], header[2]);
+         gdb_flush (gdb_stdlog);
        }
-      if (SERIAL_WRITE (sds_desc, buf2, p - buf2))
+      if (serial_write (sds_desc, buf2, p - buf2))
        perror_with_name ("putmessage: write failed");
 
       return 1;
-
     }
-
 }
 
 /* Come here after finding the start of the frame.  Collect the rest
    into BUF.  Returns 0 on any error, 1 on success.  */
 
 static int
-read_frame (buf)
-     char *buf;
+read_frame (char *buf)
 {
   char *bp;
   int c;
@@ -931,11 +826,12 @@ read_frame (buf)
        {
        case SERIAL_TIMEOUT:
          if (remote_debug)
-           puts_filtered ("Timeout in mid-message, retrying\n");
+           fputs_filtered ("Timeout in mid-message, retrying\n", gdb_stdlog);
          return 0;
        case '$':
          if (remote_debug)
-           puts_filtered ("Saw new packet start in middle of old one\n");
+           fputs_filtered ("Saw new packet start in middle of old one\n",
+                           gdb_stdlog);
          return 0;             /* Start a new packet, count retries */
        case '\r':
          break;
@@ -944,7 +840,8 @@ read_frame (buf)
          {
            *bp = '\000';
            if (remote_debug)
-             printf_filtered ("Received encoded: \"%s\"\n", buf);
+             fprintf_unfiltered (gdb_stdlog, "Received encoded: \"%s\"\n",
+                                 buf);
            return 1;
          }
 
@@ -971,9 +868,7 @@ read_frame (buf)
    while the target is executing user code.  */
 
 static int
-getmessage (buf, forever)
-     unsigned char *buf;
-     int forever;
+getmessage (unsigned char *buf, int forever)
 {
   int c, c2, c3;
   int tries;
@@ -986,11 +881,7 @@ getmessage (buf, forever)
 
   if (forever)
     {
-#ifdef MAINTENANCE_CMDS
       timeout = watchdog > 0 ? watchdog : -1;
-#else
-      timeout = -1;
-#endif
     }
 
   else
@@ -1001,12 +892,12 @@ getmessage (buf, forever)
   for (tries = 1; tries <= MAX_TRIES; tries++)
     {
       /* This can loop forever if the remote side sends us characters
-        continuously, but if it pauses, we'll get a zero from readchar
-        because of timeout.  Then we'll count that as a retry.  */
+         continuously, but if it pauses, we'll get a zero from readchar
+         because of timeout.  Then we'll count that as a retry.  */
 
       /* Note that we will only wait forever prior to the start of a packet.
-        After that, we expect characters to arrive at a brisk pace.  They
-        should show up within sds_timeout intervals.  */
+         After that, we expect characters to arrive at a brisk pace.  They
+         should show up within sds_timeout intervals.  */
 
       do
        {
@@ -1014,22 +905,20 @@ getmessage (buf, forever)
 
          if (c == SERIAL_TIMEOUT)
            {
-#ifdef MAINTENANCE_CMDS
              if (forever)      /* Watchdog went off.  Kill the target. */
                {
                  target_mourn_inferior ();
                  error ("Watchdog has expired.  Target detached.\n");
                }
-#endif
              if (remote_debug)
-               puts_filtered ("Timed out.\n");
+               fputs_filtered ("Timed out.\n", gdb_stdlog);
              goto retry;
            }
        }
       while (c != '$' && c != '{');
 
       /* We might have seen a "trigraph", a sequence of three characters
-        that indicate various sorts of communication state.  */
+         that indicate various sorts of communication state.  */
 
       if (c == '{')
        {
@@ -1037,12 +926,12 @@ getmessage (buf, forever)
          c2 = readchar (timeout);
          c3 = readchar (timeout);
          if (remote_debug)
-           fprintf_unfiltered (gdb_stderr, "Trigraph %c%c%c received\n",
+           fprintf_unfiltered (gdb_stdlog, "Trigraph %c%c%c received\n",
                                c, c2, c3);
          if (c3 == '+')
            {
              message_pending = 1;
-             return;
+             return 0;         /*???? */
            }
          continue;
        }
@@ -1063,7 +952,7 @@ getmessage (buf, forever)
 
          if (csum != header[0])
            fprintf_unfiltered (gdb_stderr,
-                               "Checksum mismatch: computed %d, received %d\n",
+                           "Checksum mismatch: computed %d, received %d\n",
                                csum, header[0]);
 
          if (header[2] == 0xff)
@@ -1071,15 +960,15 @@ getmessage (buf, forever)
 
          if (remote_debug)
            {
-             fprintf_unfiltered (gdb_stderr,
-                                 "... (Got checksum %d, id %d, length %d)\n",
+             fprintf_unfiltered (gdb_stdlog,
+                               "... (Got checksum %d, id %d, length %d)\n",
                                  header[0], header[1], header[2]);
-             fprintf_unfiltered (gdb_stderr, "Message received: \"");
+             fprintf_unfiltered (gdb_stdlog, "Message received: \"");
              for (i = 0; i < len; ++i)
                {
-                 fprintf_unfiltered (gdb_stderr, "%02x", (unsigned char) buf[i]);
+                 fprintf_unfiltered (gdb_stdlog, "%02x", (unsigned char) buf[i]);
                }
-             fprintf_unfiltered (gdb_stderr, "\"\n");
+             fprintf_unfiltered (gdb_stdlog, "\"\n");
            }
 
          /* no ack required? */
@@ -1094,152 +983,129 @@ getmessage (buf, forever)
   /* We have tried hard enough, and just can't receive the packet.  Give up. */
 
   printf_unfiltered ("Ignoring packet error, continuing...\n");
+  return 0;
 }
 \f
 static void
-sds_kill ()
+sds_kill (void)
 {
-  /* For some mysterious reason, wait_for_inferior calls kill instead of
-     mourn after it gets TARGET_WAITKIND_SIGNALLED.  Work around it.  */
-  if (kill_kludge)
-    {
-      kill_kludge = 0;
-      target_mourn_inferior ();
-      return;
-    }
-
-#if 0 /* fix to use 1-arg fn */
-  /* Use catch_errors so the user can quit from gdb even when we aren't on
-     speaking terms with the remote system.  */
-  catch_errors (putmessage, "k", "", RETURN_MASK_ERROR);
-#endif
-
-  /* Don't wait for it to die.  I'm not really sure it matters whether
-     we do or not.  For the existing stubs, kill is a noop.  */
-  target_mourn_inferior ();
+  /* Don't try to do anything to the target.  */
 }
 
 static void
-sds_mourn ()
+sds_mourn (void)
 {
   unpush_target (&sds_ops);
   generic_mourn_inferior ();
 }
 
 static void
-sds_create_inferior (exec_file, args, env)
-     char *exec_file;
-     char *args;
-     char **env;
+sds_create_inferior (char *exec_file, char *args, char **env)
 {
-  /* Rip out the breakpoints; we'll reinsert them after restarting
-     the remote server.  */
-  remove_breakpoints ();
-
-  /* Now restart the remote server.  */
-  sds_restart ();
-
-  /* Now put the breakpoints back in.  This way we're safe if the
-     restart function works via a unix fork on the remote side.  */
-  insert_breakpoints ();
+  inferior_ptid = pid_to_ptid (42000);
 
   /* Clean up from the last time we were running.  */
   clear_proceed_status ();
 
   /* Let the remote process run.  */
-  proceed (-1, TARGET_SIGNAL_0, 0);
+  proceed (bfd_get_start_address (exec_bfd), TARGET_SIGNAL_0, 0);
 }
 
+static void
+sds_load (char *filename, int from_tty)
+{
+  generic_load (filename, from_tty);
+
+  inferior_ptid = null_ptid;
+}
 \f
 /* The SDS monitor has commands for breakpoint insertion, although it
    it doesn't actually manage the breakpoints, it just returns the
    replaced instruction back to the debugger.  */
 
 static int
-sds_insert_breakpoint (addr, contents_cache)
-     CORE_ADDR addr;
-     char *contents_cache;
+sds_insert_breakpoint (CORE_ADDR addr, char *contents_cache)
 {
-  int retlen;
-  unsigned char buf[PBUFSIZ];
+  int i, retlen;
+  unsigned char *p, buf[PBUFSIZ];
+
+  p = buf;
+  *p++ = 16;
+  *p++ = 0;
+  *p++ = (int) (addr >> 24) & 0xff;
+  *p++ = (int) (addr >> 16) & 0xff;
+  *p++ = (int) (addr >> 8) & 0xff;
+  *p++ = (int) (addr) & 0xff;
 
-  buf[0] = 16;
-  buf[1] = 0;
+  retlen = sds_send (buf, p - buf);
 
-  retlen = sds_send (buf, 7);
+  for (i = 0; i < 4; ++i)
+    contents_cache[i] = buf[i + 2];
+
+  return 0;
 }
 
 static int
-sds_remove_breakpoint (addr, contents_cache)
-     CORE_ADDR addr;
-     char *contents_cache;
+sds_remove_breakpoint (CORE_ADDR addr, char *contents_cache)
 {
-  int retlen;
-  unsigned char buf[PBUFSIZ];
-
-  buf[0] = 17;
-  buf[1] = 0;
-
-  retlen = sds_send (buf, 7);
+  int i, retlen;
+  unsigned char *p, buf[PBUFSIZ];
+
+  p = buf;
+  *p++ = 17;
+  *p++ = 0;
+  *p++ = (int) (addr >> 24) & 0xff;
+  *p++ = (int) (addr >> 16) & 0xff;
+  *p++ = (int) (addr >> 8) & 0xff;
+  *p++ = (int) (addr) & 0xff;
+  for (i = 0; i < 4; ++i)
+    *p++ = contents_cache[i];
+
+  retlen = sds_send (buf, p - buf);
+
+  return 0;
 }
 \f
-/* Define the target operations vector. */
-
-static struct target_ops sds_ops =
+static void
+init_sds_ops (void)
 {
-  "sds",                       /* to_shortname */
-  "Remote serial target with SDS protocol",    /* to_longname */
-  "Use a remote computer via a serial line, using the SDS protocol.\n\
-Specify the serial device it is connected to (e.g. /dev/ttya).",  /* to_doc */
-  sds_open,                    /* to_open */
-  sds_close,                   /* to_close */
-  NULL,                                /* to_attach */
-  sds_detach,                  /* to_detach */
-  sds_resume,                  /* to_resume */
-  sds_wait,                    /* to_wait */
-  sds_fetch_registers,         /* to_fetch_registers */
-  sds_store_registers,         /* to_store_registers */
-  sds_prepare_to_store,                /* to_prepare_to_store */
-  sds_xfer_memory,             /* to_xfer_memory */
-  sds_files_info,              /* to_files_info */
-  sds_insert_breakpoint,       /* to_insert_breakpoint */
-  sds_remove_breakpoint,       /* to_remove_breakpoint */
-  NULL,                                /* to_terminal_init */
-  NULL,                                /* to_terminal_inferior */
-  NULL,                                /* to_terminal_ours_for_output */
-  NULL,                                /* to_terminal_ours */
-  NULL,                                /* to_terminal_info */
-  sds_kill,                    /* to_kill */
-  generic_load,                        /* to_load */
-  NULL,                                /* to_lookup_symbol */
-  sds_create_inferior,         /* to_create_inferior */
-  sds_mourn,                   /* to_mourn_inferior */
-  0,                           /* to_can_run */
-  0,                           /* to_notice_signals */
-  0,                           /* to_thread_alive */
-  0,                           /* to_stop */
-  process_stratum,             /* to_stratum */
-  NULL,                                /* to_next */
-  1,                           /* to_has_all_memory */
-  1,                           /* to_has_memory */
-  1,                           /* to_has_stack */
-  1,                           /* to_has_registers */
-  1,                           /* to_has_execution */
-  NULL,                                /* sections */
-  NULL,                                /* sections_end */
-  OPS_MAGIC                    /* to_magic */
-};
+  sds_ops.to_shortname = "sds";
+  sds_ops.to_longname = "Remote serial target with SDS protocol";
+  sds_ops.to_doc = "Use a remote computer via a serial line; using the SDS protocol.\n\
+Specify the serial device it is connected to (e.g. /dev/ttya).";
+  sds_ops.to_open = sds_open;
+  sds_ops.to_close = sds_close;
+  sds_ops.to_detach = sds_detach;
+  sds_ops.to_resume = sds_resume;
+  sds_ops.to_wait = sds_wait;
+  sds_ops.to_fetch_registers = sds_fetch_registers;
+  sds_ops.to_store_registers = sds_store_registers;
+  sds_ops.to_prepare_to_store = sds_prepare_to_store;
+  sds_ops.to_xfer_memory = sds_xfer_memory;
+  sds_ops.to_files_info = sds_files_info;
+  sds_ops.to_insert_breakpoint = sds_insert_breakpoint;
+  sds_ops.to_remove_breakpoint = sds_remove_breakpoint;
+  sds_ops.to_kill = sds_kill;
+  sds_ops.to_load = sds_load;
+  sds_ops.to_create_inferior = sds_create_inferior;
+  sds_ops.to_mourn_inferior = sds_mourn;
+  sds_ops.to_stratum = process_stratum;
+  sds_ops.to_has_all_memory = 1;
+  sds_ops.to_has_memory = 1;
+  sds_ops.to_has_stack = 1;
+  sds_ops.to_has_registers = 1;
+  sds_ops.to_has_execution = 1;
+  sds_ops.to_magic = OPS_MAGIC;
+}
 
 /* Put a command string, in args, out to the monitor and display the
    reply message.  */
 
 static void
-sds_command (args, from_tty)
-     char *args;
-     int from_tty;
+sds_command (char *args, int from_tty)
 {
   char *p;
-  int i, len, resp_len;
+  int i, len, retlen;
   unsigned char buf[1000];
 
   /* Convert hexadecimal chars into a byte buffer.  */
@@ -1253,26 +1119,27 @@ sds_command (args, from_tty)
       p += 2;
     }
 
-  len = sds_send (buf, len);
+  retlen = sds_send (buf, len);
 
   printf_filtered ("Reply is ");
-  for (i = 0; i < len; ++i)
+  for (i = 0; i < retlen; ++i)
     {
       printf_filtered ("%02x", buf[i]);
-    }  
+    }
   printf_filtered ("\n");
 }
 
 void
-_initialize_remote_sds ()
+_initialize_remote_sds (void)
 {
+  init_sds_ops ();
   add_target (&sds_ops);
 
   add_show_from_set (add_set_cmd ("sdstimeout", no_class,
-                                 var_integer, (char *)&sds_timeout,
-                                 "Set timeout value for sds read.\n", &setlist),
+                                 var_integer, (char *) &sds_timeout,
+                            "Set timeout value for sds read.\n", &setlist),
                     &showlist);
 
   add_com ("sds", class_obscure, sds_command,
-          "Send a command to the SDS monitor."); 
+          "Send a command to the SDS monitor.");
 }
This page took 0.042759 seconds and 4 git commands to generate.