packaging: bump to 2.24.90
[platform/upstream/binutils.git] / gdb / ser-unix.c
index 9b67230..6389712 100644 (file)
@@ -1,44 +1,58 @@
 /* Serial interface for local (hardwired) serial ports on Un*x like systems
-   Copyright 1992, 1993 Free Software Foundation, Inc.
 
-This file is part of GDB.
+   Copyright (C) 1992-2014 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 3 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., 675 Mass Ave, Cambridge, MA 02139, 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, see <http://www.gnu.org/licenses/>.  */
 
 #include "defs.h"
 #include "serial.h"
+#include "ser-base.h"
+#include "ser-unix.h"
+
 #include <fcntl.h>
 #include <sys/types.h>
+#include "terminal.h"
+#include <sys/socket.h>
+#include <sys/time.h>
 
-#if !defined (HAVE_TERMIOS) && !defined (HAVE_TERMIO) && !defined (HAVE_SGTTY)
-#define HAVE_SGTTY
-#endif
+#include "gdb_select.h"
+#include "gdbcmd.h"
+#include "filestuff.h"
 
 #ifdef HAVE_TERMIOS
-#include <termios.h>
-#include <unistd.h>
 
 struct hardwire_ttystate
+  {
+    struct termios termios;
+  };
+
+#ifdef CRTSCTS
+/* Boolean to explicitly enable or disable h/w flow control.  */
+static int serial_hwflow;
+static void
+show_serial_hwflow (struct ui_file *file, int from_tty,
+                   struct cmd_list_element *c, const char *value)
 {
-  struct termios termios;
-  pid_t process_group;
-};
+  fprintf_filtered (file, _("Hardware flow control is %s.\n"), value);
+}
+#endif
+
 #endif /* termios */
 
 #ifdef HAVE_TERMIO
-#include <termio.h>
 
 /* It is believed that all systems which have added job control to SVR3
    (e.g. sco) have also added termios.  Even if not, trying to figure out
@@ -46,57 +60,54 @@ struct hardwire_ttystate
    bewildering.  So we don't attempt it.  */
 
 struct hardwire_ttystate
-{
-  struct termio termio;
-};
+  {
+    struct termio termio;
+  };
 #endif /* termio */
 
 #ifdef HAVE_SGTTY
-/* Needed for the code which uses select().  We would include <sys/select.h>
-   too if it existed on all systems.  */
-#include <sys/time.h>
-
-#include <sgtty.h>
-
 struct hardwire_ttystate
-{
-  struct sgttyb sgttyb;
-  struct tchars tc;
-  struct ltchars ltc;
-  /* Line discipline flags.  */
-  int lmode;
-
-#ifdef SHORT_PGRP
-  /* This is only used for the ultra.  Does it have pid_t?  */
-  short process_group;
-#else
-  int process_group;
-#endif
-};
+  {
+    struct sgttyb sgttyb;
+    struct tchars tc;
+    struct ltchars ltc;
+    /* Line discipline flags.  */
+    int lmode;
+  };
 #endif /* sgtty */
 
-static int hardwire_open PARAMS ((serial_t scb, const char *name));
-static void hardwire_raw PARAMS ((serial_t scb));
-static int wait_for PARAMS ((serial_t scb, int timeout));
-static int hardwire_readchar PARAMS ((serial_t scb, int timeout));
-static int rate_to_code PARAMS ((int rate));
-static int hardwire_setbaudrate PARAMS ((serial_t scb, int rate));
-static int hardwire_write PARAMS ((serial_t scb, const char *str, int len));
-static void hardwire_restore PARAMS ((serial_t scb));
-static void hardwire_close PARAMS ((serial_t scb));
-static int get_tty_state PARAMS ((serial_t scb, struct hardwire_ttystate *state));
-static int set_tty_state PARAMS ((serial_t scb, struct hardwire_ttystate *state));
-static serial_ttystate hardwire_get_tty_state PARAMS ((serial_t scb));
-static int hardwire_set_tty_state PARAMS ((serial_t scb, serial_ttystate state));
-
-/* Open up a real live device for serial I/O */
+static int hardwire_open (struct serial *scb, const char *name);
+static void hardwire_raw (struct serial *scb);
+static int wait_for (struct serial *scb, int timeout);
+static int hardwire_readchar (struct serial *scb, int timeout);
+static int do_hardwire_readchar (struct serial *scb, int timeout);
+static int rate_to_code (int rate);
+static int hardwire_setbaudrate (struct serial *scb, int rate);
+static void hardwire_close (struct serial *scb);
+static int get_tty_state (struct serial *scb,
+                         struct hardwire_ttystate * state);
+static int set_tty_state (struct serial *scb,
+                         struct hardwire_ttystate * state);
+static serial_ttystate hardwire_get_tty_state (struct serial *scb);
+static int hardwire_set_tty_state (struct serial *scb, serial_ttystate state);
+static int hardwire_noflush_set_tty_state (struct serial *, serial_ttystate,
+                                          serial_ttystate);
+static void hardwire_print_tty_state (struct serial *, serial_ttystate,
+                                     struct ui_file *);
+static int hardwire_drain_output (struct serial *);
+static int hardwire_flush_output (struct serial *);
+static int hardwire_flush_input (struct serial *);
+static int hardwire_send_break (struct serial *);
+static int hardwire_setstopbits (struct serial *, int);
+
+void _initialize_ser_hardwire (void);
+
+/* Open up a real live device for serial I/O.  */
 
 static int
-hardwire_open(scb, name)
-     serial_t scb;
-     const char *name;
+hardwire_open (struct serial *scb, const char *name)
 {
-  scb->fd = open (name, O_RDWR);
+  scb->fd = gdb_open_cloexec (name, O_RDWR, 0);
   if (scb->fd < 0)
     return -1;
 
@@ -104,23 +115,12 @@ hardwire_open(scb, name)
 }
 
 static int
-get_tty_state(scb, state)
-     serial_t scb;
-     struct hardwire_ttystate *state;
+get_tty_state (struct serial *scb, struct hardwire_ttystate *state)
 {
 #ifdef HAVE_TERMIOS
-  pid_t new_process_group;
-
-  if (tcgetattr(scb->fd, &state->termios) < 0)
+  if (tcgetattr (scb->fd, &state->termios) < 0)
     return -1;
 
-  if (!job_control)
-    return 0;
-
-  new_process_group = tcgetpgrp (scb->fd);
-  if (new_process_group == (pid_t)-1)
-    return -1;
-  state->process_group = new_process_group;
   return 0;
 #endif
 
@@ -140,26 +140,18 @@ get_tty_state(scb, state)
   if (ioctl (scb->fd, TIOCLGET, &state->lmode) < 0)
     return -1;
 
-  if (!job_control)
-    return 0;
-
-  return ioctl (scb->fd, TIOCGPGRP, &state->process_group);
+  return 0;
 #endif
 }
 
 static int
-set_tty_state(scb, state)
-     serial_t scb;
-     struct hardwire_ttystate *state;
+set_tty_state (struct serial *scb, struct hardwire_ttystate *state)
 {
 #ifdef HAVE_TERMIOS
-  if (tcsetattr(scb->fd, TCSANOW, &state->termios) < 0)
+  if (tcsetattr (scb->fd, TCSANOW, &state->termios) < 0)
     return -1;
 
-  if (!job_control)
-    return 0;
-
-  return tcsetpgrp (scb->fd, state->process_group);
+  return 0;
 #endif
 
 #ifdef HAVE_TERMIO
@@ -171,68 +163,69 @@ set_tty_state(scb, state)
 #ifdef HAVE_SGTTY
   if (ioctl (scb->fd, TIOCSETN, &state->sgttyb) < 0)
     return -1;
+  if (ioctl (scb->fd, TIOCSETC, &state->tc) < 0)
+    return -1;
+  if (ioctl (scb->fd, TIOCSLTC, &state->ltc) < 0)
+    return -1;
+  if (ioctl (scb->fd, TIOCLSET, &state->lmode) < 0)
+    return -1;
 
-  if (!job_control)
-    return 0;
-
-  return ioctl (scb->fd, TIOCSPGRP, &state->process_group);
+  return 0;
 #endif
 }
 
 static serial_ttystate
-hardwire_get_tty_state(scb)
-     serial_t scb;
+hardwire_get_tty_state (struct serial *scb)
 {
   struct hardwire_ttystate *state;
 
-  state = (struct hardwire_ttystate *)xmalloc(sizeof *state);
+  state = (struct hardwire_ttystate *) xmalloc (sizeof *state);
 
-  if (get_tty_state(scb, state))
-    return NULL;
+  if (get_tty_state (scb, state))
+    {
+      xfree (state);
+      return NULL;
+    }
 
-  return (serial_ttystate)state;
+  return (serial_ttystate) state;
+}
+
+static serial_ttystate
+hardwire_copy_tty_state (struct serial *scb, serial_ttystate ttystate)
+{
+  struct hardwire_ttystate *state;
+
+  state = (struct hardwire_ttystate *) xmalloc (sizeof *state);
+  *state = *(struct hardwire_ttystate *) ttystate;
+
+  return (serial_ttystate) state;
 }
 
 static int
-hardwire_set_tty_state(scb, ttystate)
-     serial_t scb;
-     serial_ttystate ttystate;
+hardwire_set_tty_state (struct serial *scb, serial_ttystate ttystate)
 {
   struct hardwire_ttystate *state;
 
-  state = (struct hardwire_ttystate *)ttystate;
+  state = (struct hardwire_ttystate *) ttystate;
 
-  return set_tty_state(scb, state);
+  return set_tty_state (scb, state);
 }
 
 static int
-hardwire_noflush_set_tty_state (scb, new_ttystate, old_ttystate)
-     serial_t scb;
-     serial_ttystate new_ttystate;
-     serial_ttystate old_ttystate;
+hardwire_noflush_set_tty_state (struct serial *scb,
+                               serial_ttystate new_ttystate,
+                               serial_ttystate old_ttystate)
 {
   struct hardwire_ttystate new_state;
+#ifdef HAVE_SGTTY
   struct hardwire_ttystate *state = (struct hardwire_ttystate *) old_ttystate;
-
-  new_state = *(struct hardwire_ttystate *)new_ttystate;
-
-#ifdef HAVE_TERMIOS
-  /* I'm not sure whether this is necessary; the manpage makes no mention
-     of discarding input when switching to/from ICANON.  */
-  if (state->termios.c_lflag & ICANON)
-    new_state.termios.c_lflag |= ICANON;
-  else
-    new_state.termios.c_lflag &= ~ICANON;
 #endif
 
-#ifdef HAVE_TERMIO
-  /* I'm not sure whether this is necessary; the manpage makes no mention
-     of discarding input when switching to/from ICANON.  */
-  if (state->termio.c_lflag & ICANON)
-    new_state.termio.c_lflag |= ICANON;
-  else
-    new_state.termio.c_lflag &= ~ICANON;
-#endif
+  new_state = *(struct hardwire_ttystate *) new_ttystate;
+
+  /* Don't change in or out of raw mode; we don't want to flush input.
+     termio and termios have no such restriction; for them flushing input
+     is separate from setting the attributes.  */
 
 #ifdef HAVE_SGTTY
   if (state->sgttyb.sg_flags & RAW)
@@ -252,65 +245,96 @@ hardwire_noflush_set_tty_state (scb, new_ttystate, old_ttystate)
 }
 
 static void
-hardwire_print_tty_state (scb, ttystate)
-     serial_t scb;
-     serial_ttystate ttystate;
+hardwire_print_tty_state (struct serial *scb,
+                         serial_ttystate ttystate,
+                         struct ui_file *stream)
 {
   struct hardwire_ttystate *state = (struct hardwire_ttystate *) ttystate;
   int i;
 
 #ifdef HAVE_TERMIOS
-  printf_filtered ("Process group = %d\n", state->process_group);
-
-  printf_filtered ("c_iflag = 0x%x, c_oflag = 0x%x,\n",
-                  state->termios.c_iflag, state->termios.c_oflag);
-  printf_filtered ("c_cflag = 0x%x, c_lflag = 0x%x\n",
-                  state->termios.c_cflag, state->termios.c_lflag);
+  fprintf_filtered (stream, "c_iflag = 0x%x, c_oflag = 0x%x,\n",
+                   (int) state->termios.c_iflag,
+                   (int) state->termios.c_oflag);
+  fprintf_filtered (stream, "c_cflag = 0x%x, c_lflag = 0x%x\n",
+                   (int) state->termios.c_cflag,
+                   (int) state->termios.c_lflag);
 #if 0
   /* This not in POSIX, and is not really documented by those systems
      which have it (at least not Sun).  */
-  printf_filtered ("c_line = 0x%x.\n", state->termios.c_line);
+  fprintf_filtered (stream, "c_line = 0x%x.\n", state->termios.c_line);
 #endif
-  printf_filtered ("c_cc: ");
+  fprintf_filtered (stream, "c_cc: ");
   for (i = 0; i < NCCS; i += 1)
-    printf_filtered ("0x%x ", state->termios.c_cc[i]);
-  printf_filtered ("\n");
+    fprintf_filtered (stream, "0x%x ", state->termios.c_cc[i]);
+  fprintf_filtered (stream, "\n");
 #endif
 
 #ifdef HAVE_TERMIO
-  printf_filtered ("c_iflag = 0x%x, c_oflag = 0x%x,\n",
-                  state->termio.c_iflag, state->termio.c_oflag);
-  printf_filtered ("c_cflag = 0x%x, c_lflag = 0x%x, c_line = 0x%x.\n",
-                  state->termio.c_cflag, state->termio.c_lflag,
-                  state->termio.c_line);
-  printf_filtered ("c_cc: ");
+  fprintf_filtered (stream, "c_iflag = 0x%x, c_oflag = 0x%x,\n",
+                   state->termio.c_iflag, state->termio.c_oflag);
+  fprintf_filtered (stream, "c_cflag = 0x%x, c_lflag = 0x%x, c_line = 0x%x.\n",
+                   state->termio.c_cflag, state->termio.c_lflag,
+                   state->termio.c_line);
+  fprintf_filtered (stream, "c_cc: ");
   for (i = 0; i < NCC; i += 1)
-    printf_filtered ("0x%x ", state->termio.c_cc[i]);
-  printf_filtered ("\n");
+    fprintf_filtered (stream, "0x%x ", state->termio.c_cc[i]);
+  fprintf_filtered (stream, "\n");
 #endif
 
 #ifdef HAVE_SGTTY
-  printf_filtered ("Process group = %d\n", state->process_group);
+  fprintf_filtered (stream, "sgttyb.sg_flags = 0x%x.\n",
+                   state->sgttyb.sg_flags);
 
-  printf_filtered ("sgttyb.sg_flags = 0x%x.\n", state->sgttyb.sg_flags);
+  fprintf_filtered (stream, "tchars: ");
+  for (i = 0; i < (int) sizeof (struct tchars); i++)
+    fprintf_filtered (stream, "0x%x ", ((unsigned char *) &state->tc)[i]);
+  fprintf_filtered (stream, "\n");
 
-  printf_filtered ("tchars: ");
-  for (i = 0; i < (int)sizeof (struct tchars); i++)
-    printf_filtered ("0x%x ", ((unsigned char *)&state->tc)[i]);
-  printf_filtered ("\n");
+  fprintf_filtered (stream, "ltchars: ");
+  for (i = 0; i < (int) sizeof (struct ltchars); i++)
+    fprintf_filtered (stream, "0x%x ", ((unsigned char *) &state->ltc)[i]);
+  fprintf_filtered (stream, "\n");
 
-  printf_filtered ("ltchars: ");
-  for (i = 0; i < (int)sizeof (struct ltchars); i++)
-    printf_filtered ("0x%x ", ((unsigned char *)&state->ltc)[i]);
-  printf_filtered ("\n");
+  fprintf_filtered (stream, "lmode:  0x%x\n", state->lmode);
+#endif
+}
+
+/* Wait for the output to drain away, as opposed to flushing
+   (discarding) it.  */
+
+static int
+hardwire_drain_output (struct serial *scb)
+{
+#ifdef HAVE_TERMIOS
+  return tcdrain (scb->fd);
+#endif
 
-  printf_filtered ("lmode:  0x%x\n", state->lmode);
+#ifdef HAVE_TERMIO
+  return ioctl (scb->fd, TCSBRK, 1);
+#endif
+
+#ifdef HAVE_SGTTY
+  /* Get the current state and then restore it using TIOCSETP,
+     which should cause the output to drain and pending input
+     to be discarded.  */
+  {
+    struct hardwire_ttystate state;
+
+    if (get_tty_state (scb, &state))
+      {
+       return (-1);
+      }
+    else
+      {
+       return (ioctl (scb->fd, TIOCSETP, &state.sgttyb));
+      }
+  }
 #endif
 }
 
 static int
-hardwire_flush_output (scb)
-     serial_t scb;
+hardwire_flush_output (struct serial *scb)
 {
 #ifdef HAVE_TERMIOS
   return tcflush (scb->fd, TCOFLUSH);
@@ -323,13 +347,14 @@ hardwire_flush_output (scb)
 #ifdef HAVE_SGTTY
   /* This flushes both input and output, but we can't do better.  */
   return ioctl (scb->fd, TIOCFLUSH, 0);
-#endif  
+#endif
 }
 
 static int
-hardwire_flush_input (scb)
-     serial_t scb;
+hardwire_flush_input (struct serial *scb)
 {
+  ser_base_flush_input (scb);
+
 #ifdef HAVE_TERMIOS
   return tcflush (scb->fd, TCIFLUSH);
 #endif
@@ -341,15 +366,12 @@ hardwire_flush_input (scb)
 #ifdef HAVE_SGTTY
   /* This flushes both input and output, but we can't do better.  */
   return ioctl (scb->fd, TIOCFLUSH, 0);
-#endif  
+#endif
 }
 
 static int
-hardwire_send_break (scb)
-     serial_t scb;
+hardwire_send_break (struct serial *scb)
 {
-  int status;
-
 #ifdef HAVE_TERMIOS
   return tcsendbreak (scb->fd, 0);
 #endif
@@ -359,28 +381,49 @@ hardwire_send_break (scb)
 #endif
 
 #ifdef HAVE_SGTTY
-  status = ioctl (scb->fd, TIOCSBRK, 0);
-  usleep (250000);
-  status = ioctl (scb->fd, TIOCCBRK, 0);
-  return status;
-#endif  
+  {
+    int status;
+
+    status = ioctl (scb->fd, TIOCSBRK, 0);
+
+    /* Can't use usleep; it doesn't exist in BSD 4.2.  */
+    /* Note that if this gdb_select() is interrupted by a signal it will not
+       wait the full length of time.  I think that is OK.  */
+    gdb_usleep (250000);
+    status = ioctl (scb->fd, TIOCCBRK, 0);
+    return status;
+  }
+#endif
 }
 
 static void
-hardwire_raw(scb)
-     serial_t scb;
+hardwire_raw (struct serial *scb)
 {
   struct hardwire_ttystate state;
 
-  if (get_tty_state(scb, &state))
-    fprintf(stderr, "get_tty_state failed: %s\n", safe_strerror(errno));
+  if (get_tty_state (scb, &state))
+    fprintf_unfiltered (gdb_stderr, "get_tty_state failed: %s\n",
+                       safe_strerror (errno));
 
 #ifdef HAVE_TERMIOS
   state.termios.c_iflag = 0;
   state.termios.c_oflag = 0;
   state.termios.c_lflag = 0;
-  state.termios.c_cflag &= ~(CSIZE|PARENB);
-  state.termios.c_cflag |= CS8;
+  state.termios.c_cflag &= ~(CSIZE | PARENB);
+  state.termios.c_cflag |= CLOCAL | CS8;
+#ifdef CRTSCTS
+  /* h/w flow control.  */
+  if (serial_hwflow)
+    state.termios.c_cflag |= CRTSCTS;
+  else
+    state.termios.c_cflag &= ~CRTSCTS;
+#ifdef CRTS_IFLOW
+  if (serial_hwflow)
+    state.termios.c_cflag |= CRTS_IFLOW;
+  else
+    state.termios.c_cflag &= ~CRTS_IFLOW;
+#endif
+#endif
   state.termios.c_cc[VMIN] = 0;
   state.termios.c_cc[VTIME] = 0;
 #endif
@@ -389,8 +432,8 @@ hardwire_raw(scb)
   state.termio.c_iflag = 0;
   state.termio.c_oflag = 0;
   state.termio.c_lflag = 0;
-  state.termio.c_cflag &= ~(CSIZE|PARENB);
-  state.termio.c_cflag |= CS8;
+  state.termio.c_cflag &= ~(CSIZE | PARENB);
+  state.termio.c_cflag |= CLOCAL | CS8;
   state.termio.c_cc[VMIN] = 0;
   state.termio.c_cc[VTIME] = 0;
 #endif
@@ -403,40 +446,49 @@ hardwire_raw(scb)
   scb->current_timeout = 0;
 
   if (set_tty_state (scb, &state))
-    fprintf(stderr, "set_tty_state failed: %s\n", safe_strerror(errno));
+    fprintf_unfiltered (gdb_stderr, "set_tty_state failed: %s\n",
+                       safe_strerror (errno));
 }
 
 /* Wait for input on scb, with timeout seconds.  Returns 0 on success,
    otherwise SERIAL_TIMEOUT or SERIAL_ERROR.
 
    For termio{s}, we actually just setup VTIME if necessary, and let the
-   timeout occur in the read() in hardwire_read().
- */
+   timeout occur in the read() in hardwire_read().  */
+
+/* FIXME: cagney/1999-09-16: Don't replace this with the equivalent
+   ser_base*() until the old TERMIOS/SGTTY/... timer code has been
+   flushed. .  */
+
+/* NOTE: cagney/1999-09-30: Much of the code below is dead.  The only
+   possible values of the TIMEOUT parameter are ONE and ZERO.
+   Consequently all the code that tries to handle the possability of
+   an overflowed timer is unnecessary.  */
 
 static int
-wait_for(scb, timeout)
-     serial_t scb;
-     int timeout;
+wait_for (struct serial *scb, int timeout)
 {
-  int numfds;
-
 #ifdef HAVE_SGTTY
-  struct timeval tv;
-  fd_set readfds;
+  while (1)
+    {
+      struct timeval tv;
+      fd_set readfds;
+      int numfds;
 
-  FD_ZERO (&readfds);
+      /* NOTE: Some OS's can scramble the READFDS when the select()
+         call fails (ex the kernel with Red Hat 5.2).  Initialize all
+         arguments before each call.  */
 
-  tv.tv_sec = timeout;
-  tv.tv_usec = 0;
+      tv.tv_sec = timeout;
+      tv.tv_usec = 0;
 
-  FD_SET(scb->fd, &readfds);
+      FD_ZERO (&readfds);
+      FD_SET (scb->fd, &readfds);
 
-  while (1)
-    {
       if (timeout >= 0)
-       numfds = select(scb->fd+1, &readfds, 0, 0, &tv);
+       numfds = gdb_select (scb->fd + 1, &readfds, 0, 0, &tv);
       else
-       numfds = select(scb->fd+1, &readfds, 0, 0, 0);
+       numfds = gdb_select (scb->fd + 1, &readfds, 0, 0, 0);
 
       if (numfds <= 0)
        if (numfds == 0)
@@ -444,76 +496,176 @@ wait_for(scb, timeout)
        else if (errno == EINTR)
          continue;
        else
-         return SERIAL_ERROR;  /* Got an error from select or poll */
+         return SERIAL_ERROR;  /* Got an error from select or poll */
 
       return 0;
     }
-
-#endif /* HAVE_SGTTY */
+#endif /* HAVE_SGTTY */
 
 #if defined HAVE_TERMIO || defined HAVE_TERMIOS
   if (timeout == scb->current_timeout)
     return 0;
 
+  scb->current_timeout = timeout;
+
   {
     struct hardwire_ttystate state;
 
-    if (get_tty_state(scb, &state))
-      fprintf(stderr, "get_tty_state failed: %s\n", safe_strerror(errno));
+    if (get_tty_state (scb, &state))
+      fprintf_unfiltered (gdb_stderr, "get_tty_state failed: %s\n",
+                         safe_strerror (errno));
 
 #ifdef HAVE_TERMIOS
-    state.termios.c_cc[VTIME] = timeout * 10;
+    if (timeout < 0)
+      {
+       /* No timeout.  */
+       state.termios.c_cc[VTIME] = 0;
+       state.termios.c_cc[VMIN] = 1;
+      }
+    else
+      {
+       state.termios.c_cc[VMIN] = 0;
+       state.termios.c_cc[VTIME] = timeout * 10;
+       if (state.termios.c_cc[VTIME] != timeout * 10)
+         {
+
+           /* If c_cc is an 8-bit signed character, we can't go 
+              bigger than this.  If it is always unsigned, we could use
+              25.  */
+
+           scb->current_timeout = 12;
+           state.termios.c_cc[VTIME] = scb->current_timeout * 10;
+           scb->timeout_remaining = timeout - scb->current_timeout;
+         }
+      }
 #endif
 
 #ifdef HAVE_TERMIO
-    state.termio.c_cc[VTIME] = timeout * 10;
+    if (timeout < 0)
+      {
+       /* No timeout.  */
+       state.termio.c_cc[VTIME] = 0;
+       state.termio.c_cc[VMIN] = 1;
+      }
+    else
+      {
+       state.termio.c_cc[VMIN] = 0;
+       state.termio.c_cc[VTIME] = timeout * 10;
+       if (state.termio.c_cc[VTIME] != timeout * 10)
+         {
+           /* If c_cc is an 8-bit signed character, we can't go 
+              bigger than this.  If it is always unsigned, we could use
+              25.  */
+
+           scb->current_timeout = 12;
+           state.termio.c_cc[VTIME] = scb->current_timeout * 10;
+           scb->timeout_remaining = timeout - scb->current_timeout;
+         }
+      }
 #endif
 
-    scb->current_timeout = timeout;
-
     if (set_tty_state (scb, &state))
-      fprintf(stderr, "set_tty_state failed: %s\n", safe_strerror(errno));
+      fprintf_unfiltered (gdb_stderr, "set_tty_state failed: %s\n",
+                         safe_strerror (errno));
 
     return 0;
   }
-#endif /* HAVE_TERMIO || HAVE_TERMIOS */
+#endif /* HAVE_TERMIO || HAVE_TERMIOS */
 }
 
-/* Read a character with user-specified timeout.  TIMEOUT is number of seconds
-   to wait, or -1 to wait forever.  Use timeout of 0 to effect a poll.  Returns
-   char if successful.  Returns -2 if timeout expired, EOF if line dropped
-   dead, or -3 for any other error (see errno in that case). */
+/* Read a character with user-specified timeout.  TIMEOUT is number of
+   seconds to wait, or -1 to wait forever.  Use timeout of 0 to effect
+   a poll.  Returns char if successful.  Returns SERIAL_TIMEOUT if
+   timeout expired, EOF if line dropped dead, or SERIAL_ERROR for any
+   other error (see errno in that case).  */
+
+/* FIXME: cagney/1999-09-16: Don't replace this with the equivalent
+   ser_base*() until the old TERMIOS/SGTTY/... timer code has been
+   flushed.  */
+
+/* NOTE: cagney/1999-09-16: This function is not identical to
+   ser_base_readchar() as part of replacing it with ser_base*()
+   merging will be required - this code handles the case where read()
+   times out due to no data while ser_base_readchar() doesn't expect
+   that.  */
 
 static int
-hardwire_readchar(scb, timeout)
-     serial_t scb;
-     int timeout;
+do_hardwire_readchar (struct serial *scb, int timeout)
 {
-  int status;
+  int status, delta;
+  int detach = 0;
 
-  if (scb->bufcnt-- > 0)
-    return *scb->bufp++;
+  if (timeout > 0)
+    timeout++;
 
-  status = wait_for(scb, timeout);
+  /* We have to be able to keep the GUI alive here, so we break the
+     original timeout into steps of 1 second, running the "keep the
+     GUI alive" hook each time through the loop.
 
-  if (status < 0)
-    return status;
+     Also, timeout = 0 means to poll, so we just set the delta to 0,
+     so we will only go through the loop once.  */
 
-  scb->bufcnt = read(scb->fd, scb->buf, BUFSIZ);
+  delta = (timeout == 0 ? 0 : 1);
+  while (1)
+    {
 
-  if (scb->bufcnt <= 0)
-    if (scb->bufcnt == 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 */
+      /* N.B. The UI may destroy our world (for instance by calling
+         remote_stop,) in which case we want to get out of here as
+         quickly as possible.  It is not safe to touch scb, since
+         someone else might have freed it.  The
+         deprecated_ui_loop_hook signals that we should exit by
+         returning 1.  */
+
+      if (deprecated_ui_loop_hook)
+       detach = deprecated_ui_loop_hook (0);
+
+      if (detach)
+       return SERIAL_TIMEOUT;
+
+      scb->timeout_remaining = (timeout < 0 ? timeout : timeout - delta);
+      status = wait_for (scb, delta);
+
+      if (status < 0)
+       return status;
+
+      status = read (scb->fd, scb->buf, BUFSIZ);
+
+      if (status <= 0)
+       {
+         if (status == 0)
+           {
+             /* Zero characters means timeout (it could also be EOF, but
+                we don't (yet at least) distinguish).  */
+             if (scb->timeout_remaining > 0)
+               {
+                 timeout = scb->timeout_remaining;
+                 continue;
+               }
+             else if (scb->timeout_remaining < 0)
+               continue;
+             else
+               return SERIAL_TIMEOUT;
+           }
+         else if (errno == EINTR)
+           continue;
+         else
+           return SERIAL_ERROR;        /* Got an error from read.  */
+       }
+
+      scb->bufcnt = status;
+      scb->bufcnt--;
+      scb->bufp = scb->buf;
+      return *scb->bufp++;
+    }
+}
 
-  scb->bufcnt--;
-  scb->bufp = scb->buf;
-  return *scb->bufp++;
+static int
+hardwire_readchar (struct serial *scb, int timeout)
+{
+  return generic_readchar (scb, timeout, do_hardwire_readchar);
 }
 
+
 #ifndef B19200
 #define B19200 EXTA
 #endif
@@ -532,50 +684,153 @@ static struct
 }
 baudtab[] =
 {
-  {50, B50},
-  {75, B75},
-  {110, B110},
-  {134, B134},
-  {150, B150},
-  {200, B200},
-  {300, B300},
-  {600, B600},
-  {1200, B1200},
-  {1800, B1800},
-  {2400, B2400},
-  {4800, B4800},
-  {9600, B9600},
-  {19200, B19200},
-  {38400, B38400},
-  {-1, -1},
+  {
+    50, B50
+  }
+  ,
+  {
+    75, B75
+  }
+  ,
+  {
+    110, B110
+  }
+  ,
+  {
+    134, B134
+  }
+  ,
+  {
+    150, B150
+  }
+  ,
+  {
+    200, B200
+  }
+  ,
+  {
+    300, B300
+  }
+  ,
+  {
+    600, B600
+  }
+  ,
+  {
+    1200, B1200
+  }
+  ,
+  {
+    1800, B1800
+  }
+  ,
+  {
+    2400, B2400
+  }
+  ,
+  {
+    4800, B4800
+  }
+  ,
+  {
+    9600, B9600
+  }
+  ,
+  {
+    19200, B19200
+  }
+  ,
+  {
+    38400, B38400
+  }
+  ,
+#ifdef B57600
+  {
+    57600, B57600
+  }
+  ,
+#endif
+#ifdef B115200
+  {
+    115200, B115200
+  }
+  ,
+#endif
+#ifdef B230400
+  {
+    230400, B230400
+  }
+  ,
+#endif
+#ifdef B460800
+  {
+    460800, B460800
+  }
+  ,
+#endif
+  {
+    -1, -1
+  }
+  ,
 };
 
-static int 
-rate_to_code(rate)
-     int rate;
+static int
+rate_to_code (int rate)
 {
   int i;
 
   for (i = 0; baudtab[i].rate != -1; i++)
-    if (rate == baudtab[i].rate)  
-      return baudtab[i].code;
-
+    {
+      /* test for perfect macth.  */
+      if (rate == baudtab[i].rate)
+        return baudtab[i].code;
+      else
+        {
+         /* check if it is in between valid values.  */
+          if (rate < baudtab[i].rate)
+           {
+             if (i)
+               {
+                 warning (_("Invalid baud rate %d.  "
+                            "Closest values are %d and %d."),
+                          rate, baudtab[i - 1].rate, baudtab[i].rate);
+               }
+             else
+               {
+                 warning (_("Invalid baud rate %d.  Minimum value is %d."),
+                          rate, baudtab[0].rate);
+               }
+             return -1;
+           }
+        }
+    }
+  /* The requested speed was too large.  */
+  warning (_("Invalid baud rate %d.  Maximum value is %d."),
+            rate, baudtab[i - 1].rate);
   return -1;
 }
 
 static int
-hardwire_setbaudrate(scb, rate)
-     serial_t scb;
-     int rate;
+hardwire_setbaudrate (struct serial *scb, int rate)
 {
   struct hardwire_ttystate state;
+  int baud_code = rate_to_code (rate);
+  
+  if (baud_code < 0)
+    {
+      /* The baud rate was not valid.
+         A warning has already been issued.  */
+      errno = EINVAL;
+      return -1;
+    }
 
-  if (get_tty_state(scb, &state))
+  if (get_tty_state (scb, &state))
     return -1;
 
 #ifdef HAVE_TERMIOS
-  cfsetospeed (&state.termios, rate_to_code (rate));
-  cfsetispeed (&state.termios, rate_to_code (rate));
+  cfsetospeed (&state.termios, baud_code);
+  cfsetispeed (&state.termios, baud_code);
 #endif
 
 #ifdef HAVE_TERMIO
@@ -584,148 +839,140 @@ hardwire_setbaudrate(scb, rate)
 #endif
 
   state.termio.c_cflag &= ~(CBAUD | CIBAUD);
-  state.termio.c_cflag |= rate_to_code (rate);
+  state.termio.c_cflag |= baud_code;
 #endif
 
 #ifdef HAVE_SGTTY
-  state.sgttyb.sg_ispeed = rate_to_code (rate);
-  state.sgttyb.sg_ospeed = rate_to_code (rate);
+  state.sgttyb.sg_ispeed = baud_code;
+  state.sgttyb.sg_ospeed = baud_code;
 #endif
 
   return set_tty_state (scb, &state);
 }
 
 static int
-hardwire_set_process_group (scb, ttystate, group)
-     serial_t scb;
-     serial_ttystate ttystate;
-     int group;
+hardwire_setstopbits (struct serial *scb, int num)
 {
-#if defined (HAVE_SGTTY) || defined (HAVE_TERMIOS)
-  ((struct hardwire_ttystate *)ttystate)->process_group = group;
-#endif
-  return 0;
-}
+  struct hardwire_ttystate state;
+  int newbit;
 
-static int
-hardwire_write(scb, str, len)
-     serial_t scb;
-     const char *str;
-     int len;
-{
-  int cc;
+  if (get_tty_state (scb, &state))
+    return -1;
 
-  while (len > 0)
+  switch (num)
     {
-      cc = write(scb->fd, str, len);
-
-      if (cc < 0)
-       return 1;
-      len -= cc;
-      str += cc;
+    case SERIAL_1_STOPBITS:
+      newbit = 0;
+      break;
+    case SERIAL_1_AND_A_HALF_STOPBITS:
+    case SERIAL_2_STOPBITS:
+      newbit = 1;
+      break;
+    default:
+      return 1;
     }
-  return 0;
+
+#ifdef HAVE_TERMIOS
+  if (!newbit)
+    state.termios.c_cflag &= ~CSTOPB;
+  else
+    state.termios.c_cflag |= CSTOPB;   /* two bits */
+#endif
+
+#ifdef HAVE_TERMIO
+  if (!newbit)
+    state.termio.c_cflag &= ~CSTOPB;
+  else
+    state.termio.c_cflag |= CSTOPB;    /* two bits */
+#endif
+
+#ifdef HAVE_SGTTY
+  return 0;                    /* sgtty doesn't support this */
+#endif
+
+  return set_tty_state (scb, &state);
 }
 
 static void
-hardwire_close(scb)
-     serial_t scb;
+hardwire_close (struct serial *scb)
 {
   if (scb->fd < 0)
     return;
 
-  close(scb->fd);
+  close (scb->fd);
   scb->fd = -1;
 }
+\f
+\f
+
+/* The hardwire ops.  */
 
-static struct serial_ops hardwire_ops =
+static const struct serial_ops hardwire_ops =
 {
   "hardwire",
-  0,
   hardwire_open,
   hardwire_close,
+  NULL,
+  /* FIXME: Don't replace this with the equivalent ser_base*() until
+     the old TERMIOS/SGTTY/... timer code has been flushed.  cagney
+     1999-09-16.  */
   hardwire_readchar,
-  hardwire_write,
+  ser_base_write,
   hardwire_flush_output,
   hardwire_flush_input,
   hardwire_send_break,
   hardwire_raw,
   hardwire_get_tty_state,
+  hardwire_copy_tty_state,
   hardwire_set_tty_state,
   hardwire_print_tty_state,
   hardwire_noflush_set_tty_state,
   hardwire_setbaudrate,
-  hardwire_set_process_group
+  hardwire_setstopbits,
+  hardwire_drain_output,
+  ser_base_async,
+  ser_unix_read_prim,
+  ser_unix_write_prim
 };
 
-int job_control;
-#if defined (HAVE_TERMIOS)
-#include <unistd.h>
-#endif
+void
+_initialize_ser_hardwire (void)
+{
+  serial_add_interface (&hardwire_ops);
 
-/* This is here because this is where we figure out whether we (probably)
-   have job control.  Just using job_control only does part of it because
-   setpgid or setpgrp might not exist on a system without job control.
-   It might be considered misplaced (on the other hand, process groups and
-   job control are closely related to ttys).
+#ifdef HAVE_TERMIOS
+#ifdef CRTSCTS
+  add_setshow_boolean_cmd ("remoteflow", no_class,
+                          &serial_hwflow, _("\
+Set use of hardware flow control for remote serial I/O."), _("\
+Show use of hardware flow control for remote serial I/O."), _("\
+Enable or disable hardware flow control (RTS/CTS) on the serial port\n\
+when debugging using remote targets."),
+                          NULL,
+                          show_serial_hwflow,
+                          &setlist, &showlist);
+#endif
+#endif
+}
 
-   For a more clean implementation, in libiberty, put a setpgid which merely
-   calls setpgrp and a setpgrp which does nothing (any system with job control
-   will have one or the other).  */
 int
-gdb_setpgid ()
+ser_unix_read_prim (struct serial *scb, size_t count)
 {
-  int retval = 0;
-  if (job_control)
+  int status;
+
+  while (1)
     {
-#if defined (NEED_POSIX_SETPGID) || defined (HAVE_TERMIOS)
-      /* Do all systems with termios have setpgid?  I hope so.  */
-      /* setpgid (0, 0) is supposed to work and mean the same thing as
-        this, but on Ultrix 4.2A it fails with EPERM (and
-        setpgid (getpid (), getpid ()) succeeds).  */
-      retval = setpgid (getpid (), getpid ());
-#else
-#if defined (TIOCGPGRP)
-#if defined(USG) && !defined(SETPGRP_ARGS)
-      retval = setpgrp ();
-#else
-      retval = setpgrp (getpid (), getpid ());
-#endif /* USG */
-#endif /* TIOCGPGRP.  */
-#endif /* NEED_POSIX_SETPGID */
+      status = read (scb->fd, scb->buf, count);
+      if (status != -1 || errno != EINTR)
+       break;
     }
-  return retval;
+  return status;
 }
 
-void
-_initialize_ser_hardwire ()
+int
+ser_unix_write_prim (struct serial *scb, const void *buf, size_t len)
 {
-  serial_add_interface (&hardwire_ops);
-
-  /* OK, figure out whether we have job control.  */
-
-#if defined (HAVE_TERMIOS)
-  /* Do all systems with termios have the POSIX way of identifying job
-     control?  I hope so.  */
-#ifdef _POSIX_JOB_CONTROL
-  job_control = 1;
-#else
-  job_control = sysconf (_SC_JOB_CONTROL);
-#endif
-#endif /* termios */
-
-#ifdef HAVE_TERMIO
-  /* See comment at top of file about trying to support process groups
-     with termio.  */
-  job_control = 0;
-#endif /* termio */
-
-#ifdef HAVE_SGTTY
-#ifdef TIOCGPGRP
-  job_control = 1;
-#else
-  job_control = 0;
-#endif /* TIOCGPGRP */
-#endif /* sgtty */
-
+  /* ??? Historically, GDB has not retried calls to "write" that
+     result in EINTR.  */
+  return write (scb->fd, buf, len);
 }