* remote-mips.c (mips_insert_breakpoint, mips_remove_breakpoint):

	New functions.
	(mips_store_word): Change calling convention to return errors, and
	to provide old contents if the caller wants it.
	(mips_xfer_memory): Deal with errors from mips_store_word.
	* config/mips/tm-idt.h, config/mips/tm-idtl.h: Remove BREAKPOINT
	define now that remote-mips.c doesn't use BREAKPOINT.

4 files changed, 91 insertions, 27 deletions

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 10bbb4d..5c1225f 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,17 @@
 Sat Feb  5 08:03:41 1994  Jim Kingdon  (kingdon@lioth.cygnus.com)
 
+	* remote-mips.c (mips_insert_breakpoint, mips_remove_breakpoint):
+	New functions.
+	(mips_store_word): Change calling convention to return errors, and
+	to provide old contents if the caller wants it.
+	(mips_xfer_memory): Deal with errors from mips_store_word.
+	* config/mips/tm-idt.h, config/mips/tm-idtl.h: Remove BREAKPOINT
+	define now that remote-mips.c doesn't use BREAKPOINT.
+
+	* remote-mips.c (mips_create_inferior): Call warning if arguments
+	specified, and then execute "set args" command.  Call error, not
+	mips_error, if executable file not specified.
+
 	* remote-e7000.c: Replace "snoop" command (e7000_noecho) with
 	remote_debug.
 
diff --git a/gdb/config/mips/tm-idt.h b/gdb/config/mips/tm-idt.h
index 1a5a415..bb4697d 100644
--- a/gdb/config/mips/tm-idt.h
+++ b/gdb/config/mips/tm-idt.h
@@ -17,9 +17,3 @@ along with this program; if not, write to the Free Software
 Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
 
 #include "mips/tm-bigmips.h"
-
-/* The IDT board uses an unusual breakpoint value, and sometimes gets
-   confused when it sees the usual MIPS breakpoint instruction.  */
-
-#undef BREAKPOINT
-#define BREAKPOINT {0, 0, 0x0a, 0x0d}
diff --git a/gdb/config/mips/tm-idtl.h b/gdb/config/mips/tm-idtl.h
index 632495d..3410bca 100644
--- a/gdb/config/mips/tm-idtl.h
+++ b/gdb/config/mips/tm-idtl.h
@@ -17,9 +17,3 @@ along with this program; if not, write to the Free Software
 Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
 
 #include "mips/tm-mips.h"
-
-/* The IDT board uses an unusual breakpoint value, and sometimes gets
-   confused when it sees the usual MIPS breakpoint instruction.  */
-
-#undef BREAKPOINT
-#define BREAKPOINT {0x0d, 0x0a, 0, 0}
diff --git a/gdb/remote-mips.c b/gdb/remote-mips.c
index 599d1ec..ae793f5 100644
--- a/gdb/remote-mips.c
+++ b/gdb/remote-mips.c
@@ -93,8 +93,8 @@ mips_store_registers PARAMS ((int regno));
 static int
 mips_fetch_word PARAMS ((CORE_ADDR addr));
 
-static void
-mips_store_word PARAMS ((CORE_ADDR addr, int value));
+static int
+mips_store_word PARAMS ((CORE_ADDR addr, int value, char *old_contents));
 
 static int
 mips_xfer_memory PARAMS ((CORE_ADDR memaddr, char *myaddr, int len,
@@ -1248,25 +1248,34 @@ mips_fetch_word (addr)
   return val;
 }
 
-/* Store a word to the target board.  */
+/* Store a word to the target board.  Returns errno code or zero for
+   success.  If OLD_CONTENTS is non-NULL, put the old contents of that
+   memory location there.  */
 
-static void
-mips_store_word (addr, val)
+static int
+mips_store_word (addr, val, old_contents)
      CORE_ADDR addr;
      int val;
+     char *old_contents;
 {
   int err;
+  unsigned int oldcontents;
 
-  mips_request ('D', (unsigned int) addr, (unsigned int) val, &err,
-		mips_receive_wait);
+  oldcontents = mips_request ('D', (unsigned int) addr, (unsigned int) val,
+			      &err,
+			      mips_receive_wait);
   if (err)
     {
       /* Data space failed; try instruction space.  */
-      mips_request ('I', (unsigned int) addr, (unsigned int) val, &err,
-		    mips_receive_wait);
+      oldcontents = mips_request ('I', (unsigned int) addr,
+				  (unsigned int) val, &err,
+				  mips_receive_wait);
       if (err)
-	mips_error ("Can't write address 0x%x: %s", addr, safe_strerror (errno));
+	return errno;
     }
+  if (old_contents != NULL)
+    store_unsigned_integer (old_contents, 4, oldcontents);
+  return 0;
 }
 
 /* Read or write LEN bytes from inferior memory at MEMADDR,
@@ -1292,6 +1301,8 @@ mips_xfer_memory (memaddr, myaddr, len, write, ignore)
   /* Allocate buffer of that many longwords.  */
   register char *buffer = alloca (count * 4);
 
+  int status;
+
   if (write)
     {
       /* Fill start and end extra bytes of buffer with existing data.  */
@@ -1317,7 +1328,14 @@ mips_xfer_memory (memaddr, myaddr, len, write, ignore)
 
       for (i = 0; i < count; i++, addr += 4)
 	{
-	  mips_store_word (addr, extract_unsigned_integer (&buffer[i*4], 4));
+	  status = mips_store_word (addr,
+				    extract_unsigned_integer (&buffer[i*4], 4),
+				    NULL);
+	  if (status)
+	    {
+	      errno = status;
+	      return 0;
+	    }
 	  /* FIXME: Do we want a QUIT here?  */
 	}
     }
@@ -1378,10 +1396,15 @@ mips_create_inferior (execfile, args, env)
   CORE_ADDR entry_pt;
 
   if (args && *args)
-    mips_error ("Can't pass arguments to remote MIPS board.");
+    {
+      warning ("\
+Can't pass arguments to remote MIPS board; arguments ignored.");
+      /* And don't try to use them on the next "run" command.  */
+      execute_command ("set args", 0);
+    }
 
   if (execfile == 0 || exec_bfd == 0)
-    mips_error ("No exec file specified");
+    error ("No executable file specified");
 
   entry_pt = (CORE_ADDR) bfd_get_start_address (exec_bfd);
 
@@ -1401,6 +1424,47 @@ mips_mourn_inferior ()
   generic_mourn_inferior ();
 }
 
+/* We can write a breakpoint and read the shadow contents in one
+   operation.  */
+
+/* The IDT board uses an unusual breakpoint value, and sometimes gets
+   confused when it sees the usual MIPS breakpoint instruction.  */
+
+#if TARGET_BYTE_ORDER == BIG_ENDIAN
+static unsigned char break_insn[] = {0, 0, 0x0a, 0x0d};
+#else
+static unsigned char break_insn[] = {0x0d, 0x0a, 0, 0};
+#endif
+
+/* Insert a breakpoint on targets that don't have any better breakpoint
+   support.  We read the contents of the target location and stash it,
+   then overwrite it with a breakpoint instruction.  ADDR is the target
+   location in the target machine.  CONTENTS_CACHE is a pointer to 
+   memory allocated for saving the target contents.  It is guaranteed
+   by the caller to be long enough to save sizeof BREAKPOINT bytes (this
+   is accomplished via BREAKPOINT_MAX).  */
+
+static int
+mips_insert_breakpoint (addr, contents_cache)
+     CORE_ADDR addr;
+     char *contents_cache;
+{
+  int status;
+
+  return
+    mips_store_word (addr,
+		     extract_unsigned_integer (break_insn, sizeof break_insn),
+		     contents_cache);
+}
+
+static int
+mips_remove_breakpoint (addr, contents_cache)
+     CORE_ADDR addr;
+     char *contents_cache;
+{
+  return target_write_memory (addr, contents_cache, sizeof break_insn);
+}
+
 /* The target vector.  */
 
 struct target_ops mips_ops =
@@ -1422,8 +1486,8 @@ HOST:PORT to access a board over a network",  /* to_doc */
   mips_prepare_to_store,	/* to_prepare_to_store */
   mips_xfer_memory,		/* to_xfer_memory */
   mips_files_info,		/* to_files_info */
-  NULL,				/* to_insert_breakpoint */
-  NULL,				/* to_remove_breakpoint */
+  mips_insert_breakpoint,	/* to_insert_breakpoint */
+  mips_remove_breakpoint,	/* to_remove_breakpoint */
   NULL,				/* to_terminal_init */
   NULL,				/* to_terminal_inferior */
   NULL,				/* to_terminal_ours_for_output */