From: John Gilmore <gnu@cygnus>
Date: Wed, 5 Aug 1992 08:46:44 +0000 (+0000)
Subject: * remote.c (getpkt):  Only force retransmission ten times;
X-Git-Url: http://drtracing.org/?a=commitdiff_plain;h=38094c60330686e8f69dfcc8c2c5d2316f622cd1;p=deliverable%2Fbinutils-gdb.git

* remote.c (getpkt):  Only force retransmission ten times;
after that, assume bug in target code, and handle pkt anyway.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index f608dea6f6..c4f2f0afb2 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+Wed Aug  5 01:42:40 1992  John Gilmore  (gnu at cygnus.com)
+
+	* remote.c (getpkt):  Only force retransmission ten times;
+	after that, assume bug in target code, and handle pkt anyway.
+
 Mon Aug  3 17:06:20 1992  Stu Grossman  (grossman at cygnus.com)
 
 	* configure.in:  Add support for apollo 68k under BSD environment.
diff --git a/gdb/remote.c b/gdb/remote.c
index f8372db58b..cb1ad341f1 100644
--- a/gdb/remote.c
+++ b/gdb/remote.c
@@ -420,7 +420,7 @@ remote_wait (status)
 
   WSETEXIT ((*status), 0);
 
-  ofunc = (void (*)) signal (SIGINT, remote_interrupt);
+  ofunc = (void (*)()) signal (SIGINT, remote_interrupt);
   getpkt ((char *) buf);
   signal (SIGINT, ofunc);
 
@@ -793,6 +793,8 @@ getpkt (buf)
   unsigned char csum;
   int c;
   unsigned char c1, c2;
+  int retries = 0;
+#define MAX_RETRIES	10
 
 #if 0
   /* Sorry, this will cause all hell to break loose, i.e. we'll end
@@ -839,9 +841,18 @@ getpkt (buf)
 	break;
       printf_filtered ("Bad checksum, sentsum=0x%x, csum=0x%x, buf=%s\n",
 	      (c1 << 4) + c2, csum & 0xff, buf);
+
       /* Try the whole thing again.  */
 whole:
-      write (remote_desc, "-", 1);
+      if (++retries < MAX_RETRIES)
+	{
+	  write (remote_desc, "-", 1);
+	}
+      else
+	{
+	  printf ("Ignoring packet error, continuing...\n");
+	  break;
+	}
     }
 
 #if 0