Moved the position of the #### lines so that the makefile fragments
[deliverable/binutils-gdb.git] / gdb / remote.c
index 27a73222baaf9d057c7c11d5b3b301096eaa3bfb..821524392ade8d955ae13464c644b305ec59a80b 100644 (file)
@@ -1,22 +1,21 @@
 /* Memory-access and commands for inferior process, for GDB.
-   Copyright (C)  1988 Free Software Foundation, Inc.
-
-GDB is distributed in the hope that it will be useful, but WITHOUT ANY
-WARRANTY.  No author or distributor accepts responsibility to anyone
-for the consequences of using it or for whether it serves any
-particular purpose or works at all, unless he says so in writing.
-Refer to the GDB General Public License for full details.
-
-Everyone is granted permission to copy, modify and redistribute GDB,
-but only under the conditions described in the GDB General Public
-License.  A copy of this license is supposed to have been given to you
-along with GDB so you can know your rights and responsibilities.  It
-should be in a file named COPYING.  Among other things, the copyright
-notice and this notice must be preserved on all copies.
-
-In other words, go ahead and share GDB, but don't try to stop
-anyone else from sharing it farther.  Help stamp out software hoarding!
-*/
+   Copyright (C) 1988-1991 Free Software Foundation, Inc.
+
+This file is part of GDB.
+
+GDB 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 1, or (at your option)
+any later version.
+
+GDB 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 GDB; see the file COPYING.  If not, write to
+the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.  */
 
 /* Remote communication protocol.
    All values are encoded in ascii hex digits.
@@ -55,6 +54,11 @@ anyone else from sharing it farther.  Help stamp out software hoarding!
                                        If AA..AA is omitted,
                                        resume at same address.
 
+       last signal     ?               Reply the current reason for stopping.
+                                        This is the same reply as is generated
+                                       for step or cont : SAA where AA is the
+                                       signal number.
+
        There is no immediate reply to step or cont.
        The reply comes when the machine stops.
        It is           SAA             AA is the "signal number"
@@ -63,49 +67,85 @@ anyone else from sharing it farther.  Help stamp out software hoarding!
 */
 
 #include <stdio.h>
-#include <signal.h>
-
+#include <string.h>
+#include <fcntl.h>
 #include "defs.h"
-#include "initialize.h"
 #include "param.h"
 #include "frame.h"
 #include "inferior.h"
-
+#include "target.h"
 #include "wait.h"
-#include <sys/ioctl.h>
-#include <a.out.h>
-#include <sys/file.h>
+#include "terminal.h"
 
-#ifdef HAVE_TERMIO
-#include <termio.h>
-#undef TIOCGETP
-#define TIOCGETP TCGETA
-#undef TIOCSETN
-#define TIOCSETN TCSETA
-#undef TIOCSETP
-#define TIOCSETP TCSETAF
-#define TERMINAL struct termio
-#else
-#include <sgtty.h>
-#define TERMINAL struct sgttyb
+#ifdef USG
+#include <sys/types.h>
 #endif
 
-int kiodebug;
+#include <signal.h>
+
+extern void add_syms_addr_command ();
+extern struct value *call_function_by_hand();
+extern void start_remote ();
+
+extern struct target_ops remote_ops;   /* Forward decl */
 
+static int kiodebug;
+static int timeout = 5;
+
+#if 0
 int icache;
+#endif
 
-/* Descriptor for I/O to remote machine.  */
-int remote_desc;
+/* Descriptor for I/O to remote machine.  Initialize it to -1 so that
+   remote_open knows that we don't have a file open when the program
+   starts.  */
+int remote_desc = -1;
 
-#define        PBUFSIZ 300
+#define        PBUFSIZ 400
+
+/* 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)
 
 static void remote_send ();
 static void putpkt ();
 static void getpkt ();
+#if 0
 static void dcache_flush ();
+#endif
 
-START_FILE
 \f
+/* Called when SIGALRM signal sent due to alarm() timeout.  */
+#ifndef HAVE_TERMIO
+void
+remote_timer ()
+{
+  if (kiodebug)
+    printf ("remote_timer called\n");
+
+  alarm (timeout);
+}
+#endif
+
+/* Initialize remote connection */
+
+void
+remote_start()
+{
+}
+
+/* Clean up connection to a remote debugger.  */
+
+/* ARGSUSED */
+void
+remote_close (quitting)
+     int quitting;
+{
+  if (remote_desc >= 0)
+    close (remote_desc);
+  remote_desc = -1;
+}
+
 /* Open a connection to a remote debugger.
    NAME is the filename used for communication.  */
 
@@ -116,8 +156,18 @@ remote_open (name, from_tty)
 {
   TERMINAL sg;
 
-  remote_debugging = 0;
+  if (name == 0)
+    error (
+"To open a remote debug connection, you need to specify what serial\n\
+device is attached to the remote system (e.g. /dev/ttya).");
+
+  target_preopen (from_tty);
+
+  remote_close (0);
+
+#if 0
   dcache_init ();
+#endif
 
   remote_desc = open (name, O_RDWR);
   if (remote_desc < 0)
@@ -125,7 +175,9 @@ remote_open (name, from_tty)
 
   ioctl (remote_desc, TIOCGETP, &sg);
 #ifdef HAVE_TERMIO
-  sg.c_lflag &= ~ICANON;
+  sg.c_cc[VMIN] = 0;           /* read with timeout.  */
+  sg.c_cc[VTIME] = timeout * 10;
+  sg.c_lflag &= ~(ICANON | ECHO);
 #else
   sg.sg_flags = RAW;
 #endif
@@ -133,7 +185,46 @@ remote_open (name, from_tty)
 
   if (from_tty)
     printf ("Remote debugging using %s\n", name);
-  remote_debugging = 1;
+  push_target (&remote_ops);   /* Switch to using remote target now */
+
+#ifndef HAVE_TERMIO
+#ifndef NO_SIGINTERRUPT
+  /* Cause SIGALRM's to make reads fail.  */
+  if (siginterrupt (SIGALRM, 1) != 0)
+    perror ("remote_open: error in siginterrupt");
+#endif
+
+  /* Set up read timeout timer.  */
+  if ((void (*)()) signal (SIGALRM, remote_timer) == (void (*)()) -1)
+    perror ("remote_open: error in signal");
+#endif
+
+  /* Ack any packet which the remote side has already sent.  */
+  write (remote_desc, "+", 1);
+  putpkt ("?");                        /* initiate a query from remote machine */
+
+  start_remote ();             /* Initialize gdb process mechanisms */
+}
+
+/* remote_detach()
+   takes a program previously attached to and detaches it.
+   We better not have left any breakpoints
+   in the program or it'll die when it hits one.
+   Close the open connection to the remote debugger.
+   Use this when you want to detach and do something else
+   with your gdb.  */
+
+static void
+remote_detach (args, from_tty)
+     char *args;
+     int from_tty;
+{
+  if (args)
+    error ("Argument given to \"detach\" when remotely debugging.");
+  
+  pop_target ();
+  if (from_tty)
+    printf ("Ending remote debugging.\n");
 }
 
 /* Convert hex digit A to a number.  */
@@ -148,6 +239,7 @@ fromhex (a)
     return a - 'a' + 10;
   else
     error ("Reply contains invalid hex digit");
+  return -1;
 }
 
 /* Convert number NIB to a hex digit.  */
@@ -164,13 +256,18 @@ tohex (nib)
 \f
 /* Tell the remote machine to resume.  */
 
-int
-remote_resume (step, signal)
-     int step, signal;
+void
+remote_resume (step, siggnal)
+     int step, siggnal;
 {
   char buf[PBUFSIZ];
 
+  if (siggnal)
+    error ("Can't send signals to a remote system.");
+
+#if 0
   dcache_flush ();
+#endif
 
   strcpy (buf, step ? "s": "c");
 
@@ -178,13 +275,15 @@ remote_resume (step, signal)
 }
 
 /* Wait until the remote machine stops, then return,
-   storing status in STATUS just as `wait' would.  */
+   storing status in STATUS just as `wait' would.
+   Returns "pid" (though it's not clear what, if anything, that
+   means in the case of this target).  */
 
 int
 remote_wait (status)
      WAITTYPE *status;
 {
-  char buf[PBUFSIZ];
+  unsigned char buf[PBUFSIZ];
 
   WSETEXIT ((*status), 0);
   getpkt (buf);
@@ -193,23 +292,28 @@ remote_wait (status)
   if (buf[0] != 'S')
     error ("Invalid remote reply: %s", buf);
   WSETSTOP ((*status), (((fromhex (buf[1])) << 4) + (fromhex (buf[2]))));
+  return 0;
 }
 
 /* Read the remote registers into the block REGS.  */
 
+/* Currently we just read all the registers, so we don't use regno.  */
+/* ARGSUSED */
 void
-remote_fetch_registers (regs)
-     char *regs;
+remote_fetch_registers (regno)
+     int regno;
 {
   char buf[PBUFSIZ];
   int i;
   char *p;
+  char regs[REGISTER_BYTES];
 
   sprintf (buf, "g");
   remote_send (buf);
 
-  /* Reply describes registers byte by byte,
-     each byte encoded as two hex characters.  */
+  /* Reply describes registers byte by byte, each byte encoded as two
+     hex characters.  Suck them all up, then supply them to the
+     register cacheing/storage mechanism.  */
 
   p = buf;
   for (i = 0; i < REGISTER_BYTES; i++)
@@ -219,13 +323,26 @@ remote_fetch_registers (regs)
       regs[i] = fromhex (p[0]) * 16 + fromhex (p[1]);
       p += 2;
     }
+  for (i = 0; i < NUM_REGS; i++)
+    supply_register (i, &regs[REGISTER_BYTE(i)]);
 }
 
-/* Store the remote registers from the contents of the block REGS.  */
+/* Prepare to store registers.  Since we send them all, we have to
+   read out the ones we don't want to change first.  */
 
-void
-remote_store_registers (regs)
-     char *regs;
+void 
+remote_prepare_to_store ()
+{
+  remote_fetch_registers (-1);
+}
+
+/* Store the remote registers from the contents of the block REGISTERS. 
+   FIXME, eventually just store one register if that's all that is needed.  */
+
+/* ARGSUSED */
+int
+remote_store_registers (regno)
+     int regno;
 {
   char buf[PBUFSIZ];
   int i;
@@ -239,13 +356,16 @@ remote_store_registers (regs)
   p = buf + 1;
   for (i = 0; i < REGISTER_BYTES; i++)
     {
-      *p++ = (regs[i] > 4) & 0xf;
-      *p++ = regs[i] & 0xf;
+      *p++ = tohex ((registers[i] >> 4) & 0xf);
+      *p++ = tohex (registers[i] & 0xf);
     }
+  *p = '\0';
 
   remote_send (buf);
+  return 0;
 }
 
+#if 0
 /* Read a word from remote address ADDR and return it.
    This goes through the data cache.  */
 
@@ -277,6 +397,7 @@ remote_store_word (addr, word)
 {
   dcache_poke (addr, word);
 }
+#endif /* 0 */
 \f
 /* Write memory data directly to the remote machine.
    This does not inform the data cache; the data cache uses this.
@@ -305,9 +426,10 @@ remote_write_bytes (memaddr, myaddr, len)
   p = buf + strlen (buf);
   for (i = 0; i < len; i++)
     {
-      *p++ = (myaddr[i] > 4) & 0xf;
-      *p++ = myaddr[i] & 0xf;
+      *p++ = tohex ((myaddr[i] >> 4) & 0xf);
+      *p++ = tohex (myaddr[i] & 0xf);
     }
+  *p = '\0';
 
   remote_send (buf);
 }
@@ -338,7 +460,7 @@ remote_read_bytes (memaddr, myaddr, len)
      each byte encoded as two hex characters.  */
 
   p = buf;
-  for (i = 0; i < REGISTER_BYTES; i++)
+  for (i = 0; i < len; i++)
     {
       if (p[0] == 0 || p[1] == 0)
        error ("Remote reply is too short: %s", buf);
@@ -347,6 +469,43 @@ remote_read_bytes (memaddr, myaddr, len)
     }
 }
 \f
+/* 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.  */
+
+int
+remote_xfer_inferior_memory(memaddr, myaddr, len, should_write)
+     CORE_ADDR memaddr;
+     char *myaddr;
+     int len;
+     int should_write;
+{
+  int origlen = len;
+  int xfersize;
+  while (len > 0)
+    {
+      if (len > MAXBUFBYTES)
+       xfersize = MAXBUFBYTES;
+      else
+       xfersize = len;
+
+      if (should_write)
+        remote_write_bytes(memaddr, myaddr, xfersize);
+      else
+       remote_read_bytes (memaddr, myaddr, xfersize);
+      memaddr += xfersize;
+      myaddr  += xfersize;
+      len     -= xfersize;
+    }
+  return origlen; /* no error possible */
+}
+
+void
+remote_files_info ()
+{
+  printf ("remote files info missing here.  FIXME.\n");
+}
+\f
 /*
 
 A debug packet whose contents are <data>
@@ -368,6 +527,24 @@ Receiver responds with:
 
 */
 
+static int
+readchar ()
+{
+  char buf;
+
+  buf = '\0';
+#ifdef HAVE_TERMIO
+  /* termio does the timeout for us.  */
+  read (remote_desc, &buf, 1);
+#else
+  alarm (timeout);
+  read (remote_desc, &buf, 1);
+  alarm (0);
+#endif
+
+  return buf & 0x7f;
+}
+
 /* Send the command in BUF to the remote machine,
    and read the reply into BUF.
    Report an error if we get an error reply.  */
@@ -376,7 +553,7 @@ static void
 remote_send (buf)
      char *buf;
 {
-  int i;
+
   putpkt (buf);
   getpkt (buf);
 
@@ -392,15 +569,12 @@ putpkt (buf)
      char *buf;
 {
   int i;
-  char csum = 0;
+  unsigned char csum = 0;
   char buf2[500];
-  char buf3[1];
   int cnt = strlen (buf);
+  char ch;
   char *p;
 
-  if (kiodebug)
-    fprintf (stderr, "Sending packet: %s\n", buf);
-
   /* Copy the packet into buffer BUF2, encapsulating it
      and giving it a checksum.  */
 
@@ -419,17 +593,18 @@ putpkt (buf)
   /* Send it over and over until we get a positive ack.  */
 
   do {
+    if (kiodebug)
+      {
+       *p = '\0';
+       printf ("Sending packet: %s (%s)\n", buf2, buf);
+      }
     write (remote_desc, buf2, p - buf2);
-    read (remote_desc, buf3, 1);
-  } while (buf3[0] != '+');
-}
 
-static int
-readchar ()
-{
-  char buf[1];
-  while (read (remote_desc, buf, 1) != 1) ;
-  return buf[0] & 0x7f;
+    /* read until either a timeout occurs (\0) or '+' is read */
+    do {
+      ch = readchar ();
+    } while ((ch != '+') && (ch != '\0'));
+  } while (ch != '+');
 }
 
 /* Read a packet from the remote machine, with error checking,
@@ -440,12 +615,30 @@ getpkt (buf)
      char *buf;
 {
   char *bp;
-  char csum = 0;
-  int c, c1, c2;
-  extern kiodebug;
+  unsigned char csum;
+  int c;
+  unsigned char c1, c2;
+
+#if 0
+  /* Sorry, this will cause all hell to break loose, i.e. we'll end
+     up in the command loop with an inferior, but (at least if this
+     happens in remote_wait or some such place) without a current_frame,
+     having set up prev_* in wait_for_inferior, etc.
+
+     If it is necessary to have such an "emergency exit", seems like
+     the only plausible thing to do is to say the inferior died, and
+     make the user reattach if they want to.  Perhaps with a prompt
+     asking for confirmation.  */
+
+  /* allow immediate quit while reading from device, it could be hung */
+  immediate_quit++;
+#endif /* 0 */
 
   while (1)
     {
+      /* Force csum to be zero here because of possible error retry.  */
+      csum = 0;
+      
       while ((c = readchar()) != '$');
 
       bp = buf;
@@ -461,19 +654,33 @@ getpkt (buf)
 
       c1 = fromhex (readchar ());
       c2 = fromhex (readchar ());
-      if (csum == (c1 << 4) + c2)
+      if ((csum & 0xff) == (c1 << 4) + c2)
        break;
       printf ("Bad checksum, sentsum=0x%x, csum=0x%x, buf=%s\n",
-             (c1 << 4) + c2, csum, buf);
+             (c1 << 4) + c2, csum & 0xff, buf);
       write (remote_desc, "-", 1);
     }
 
+#if 0
+  immediate_quit--;
+#endif
+
   write (remote_desc, "+", 1);
 
   if (kiodebug)
     fprintf (stderr,"Packet received :%s\n", buf);
 }
 \f
+/* The data cache leads to incorrect results because it doesn't know about
+   volatile variables, thus making it impossible to debug functions which
+   use hardware registers.  Therefore it is #if 0'd out.  Effect on
+   performance is some, for backtraces of functions with a few
+   arguments each.  For functions with many arguments, the stack
+   frames don't fit in the cache blocks, which makes the cache less
+   helpful.  Disabling the cache is a big performance win for fetching
+   large structures, because the cache code fetched data in 16-byte
+   chunks.  */
+#if 0
 /* The data cache records all the data read from the remote machine
    since the last time it stopped.
 
@@ -615,9 +822,33 @@ dcache_init ()
   for (i=0;i<DCACHE_SIZE;i++,db++)
     insque (db, &dcache_free);
 }
+#endif /* 0 */
+
+/* Define the target subroutine names */
+
+struct target_ops remote_ops = {
+       "remote", "Remote serial target in gdb-specific protocol",
+       "Use a remote computer via a serial line, using a gdb-specific protocol.\n\
+Specify the serial device it is connected to (e.g. /dev/ttya).",
+       remote_open, remote_close,
+       0, remote_detach, remote_resume, remote_wait,  /* attach */
+       remote_fetch_registers, remote_store_registers,
+       remote_prepare_to_store, 0, 0, /* conv_from, conv_to */
+       remote_xfer_inferior_memory, remote_files_info,
+       0, 0, /* insert_breakpoint, remove_breakpoint, */
+       0, 0, 0, 0, 0,  /* Terminal crud */
+       0, /* kill */
+       0, add_syms_addr_command,  /* load */
+       call_function_by_hand,
+       0, /* lookup_symbol */
+       0, 0, /* create_inferior FIXME, mourn_inferior FIXME */
+       process_stratum, 0, /* next */
+       1, 1, 1, 1, 1,  /* all mem, mem, stack, regs, exec */
+       OPS_MAGIC,              /* Always the last thing */
+};
 
-static initialize ()
+void
+_initialize_remote ()
 {
+  add_target (&remote_ops);
 }
-
-END_FILE
This page took 0.032096 seconds and 4 git commands to generate.