From 9e294fb82fe60926d4665f492053c1fa77d65ed2 Mon Sep 17 00:00:00 2001 From: Andrew Cagney Date: Sat, 1 Jul 2000 08:02:53 +0000 Subject: [PATCH] Revert previous ser-unix change. Locks up serial device. --- gdb/ChangeLog | 6 ++++++ gdb/ser-unix.c | 65 +++++++++++++++++++++++++++++++++++----------------------- 2 files changed, 45 insertions(+), 26 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 437ac13..e751c3e 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,9 @@ +Sat Jul 1 17:47:08 2000 Andrew Cagney + + * ser-unix.c (do_unix_readchar): Revert Tue Mar 28 18:19:50 2000 + Andrew Cagney . Locks up when no data is + pending. + Sat Jul 1 15:40:14 2000 Andrew Cagney * Makefile.in (SER_HARDWIRE): Restore code to set it by configure. diff --git a/gdb/ser-unix.c b/gdb/ser-unix.c index 47b6647..919c9fd 100644 --- a/gdb/ser-unix.c +++ b/gdb/ser-unix.c @@ -910,9 +910,9 @@ do_unix_readchar (serial_t scb, int timeout) each time through the loop. Also, timeout = 0 means to poll, so we just set the delta to 0, so we - will only go through the loop once. timeout < 0 means to wait forever. */ + will only go through the loop once. */ - delta = (timeout <= 0 ? 0 : 1); + delta = (timeout == 0 ? 0 : 1); while (1) { @@ -928,38 +928,51 @@ do_unix_readchar (serial_t scb, int timeout) return SERIAL_TIMEOUT; } - status = ser_unix_wait_for (scb, timeout < 0 ? timeout : delta); + status = ser_unix_wait_for (scb, delta); timeout -= delta; - /* If we got an error back from wait_for, then we can return */ + /* If we got a character or an error back from wait_for, then we can + break from the loop before the timeout is completed. */ - if (status == SERIAL_ERROR) - return status; + if (status != SERIAL_TIMEOUT) + { + break; + } - status = read (scb->fd, scb->buf, BUFSIZ); + /* If we have exhausted the original timeout, then generate + a SERIAL_TIMEOUT, and pass it out of the loop. */ - if (status <= 0) - { - if (status == 0) - { - if (timeout != 0) - continue; - else - return SERIAL_TIMEOUT; /* 0 chars means timeout [may need to - distinguish between EOF & timeouts - someday] */ - } - else if (errno == EINTR) - continue; - else - return SERIAL_ERROR; /* Got an error from read */ + else if (timeout == 0) + { + status = SERIAL_TIMEOUT; + break; } + } - scb->bufcnt = status; - scb->bufcnt--; - scb->bufp = scb->buf; - return *scb->bufp++; + if (status < 0) + return status; + + while (1) + { + status = read (scb->fd, scb->buf, BUFSIZ); + if (status != -1 || errno != EINTR) + break; + } + + if (status <= 0) + { + if (status == 0) + return SERIAL_TIMEOUT; /* 0 chars means timeout [may need to + distinguish between EOF & timeouts + someday] */ + else + return SERIAL_ERROR; /* Got an error from read */ } + + scb->bufcnt = status; + scb->bufcnt--; + scb->bufp = scb->buf; + return *scb->bufp++; } /* Perform operations common to both old and new readchar. */ -- 2.7.4