X-Git-Url: https://git.libre-soc.org/?a=blobdiff_plain;f=gdb%2Fserial.c;h=0a8c7abf615e721e1ea4be0527671e943208f821;hb=62ec8e6dad018082e33e7dcb0633f28332f3520c;hp=7471a6b5ab53073f5b6ff63a1e6f17c3ee2e6ab4;hpb=4ab76ea3fb057381852a9d3b7ac9270fec8e7a7c;p=binutils-gdb.git diff --git a/gdb/serial.c b/gdb/serial.c index 7471a6b5ab5..0a8c7abf615 100644 --- a/gdb/serial.c +++ b/gdb/serial.c @@ -1,6 +1,6 @@ /* Generic serial interface routines - Copyright (C) 1992-2014 Free Software Foundation, Inc. + Copyright (C) 1992-2022 Free Software Foundation, Inc. This file is part of GDB. @@ -20,22 +20,16 @@ #include "defs.h" #include #include "serial.h" -#include #include "gdbcmd.h" #include "cli/cli-utils.h" -extern void _initialize_serial (void); - /* Is serial being debugged? */ static unsigned int global_serial_debug_p; -typedef const struct serial_ops *serial_ops_p; -DEF_VEC_P (serial_ops_p); - /* Serial I/O handlers. */ -VEC (serial_ops_p) *serial_ops_list = NULL; +static std::vector serial_ops_list; /* Pointer to list of scb's. */ @@ -44,7 +38,7 @@ static struct serial *scb_base; /* Non-NULL gives filename which contains a recording of the remote session, suitable for playback by gdbserver. */ -static char *serial_logfile = NULL; +static std::string serial_logfile; static struct ui_file *serial_logfp = NULL; static const struct serial_ops *serial_interface_lookup (const char *); @@ -71,59 +65,59 @@ serial_logchar (struct ui_file *stream, int ch_type, int ch, int timeout) { if (ch_type != serial_current_type) { - fprintf_unfiltered (stream, "\n%c ", ch_type); + gdb_printf (stream, "\n%c ", ch_type); serial_current_type = ch_type; } if (serial_logbase != logbase_ascii) - fputc_unfiltered (' ', stream); + gdb_putc (' ', stream); switch (ch) { case SERIAL_TIMEOUT: - fprintf_unfiltered (stream, "", timeout); + gdb_printf (stream, "", timeout); return; case SERIAL_ERROR: - fprintf_unfiltered (stream, "", safe_strerror (errno)); + gdb_printf (stream, "", safe_strerror (errno)); return; case SERIAL_EOF: - fputs_unfiltered ("", stream); + gdb_puts ("", stream); return; case SERIAL_BREAK: - fputs_unfiltered ("", stream); + gdb_puts ("", stream); return; default: if (serial_logbase == logbase_hex) - fprintf_unfiltered (stream, "%02x", ch & 0xff); + gdb_printf (stream, "%02x", ch & 0xff); else if (serial_logbase == logbase_octal) - fprintf_unfiltered (stream, "%03o", ch & 0xff); + gdb_printf (stream, "%03o", ch & 0xff); else switch (ch) { case '\\': - fputs_unfiltered ("\\\\", stream); + gdb_puts ("\\\\", stream); break; case '\b': - fputs_unfiltered ("\\b", stream); + gdb_puts ("\\b", stream); break; case '\f': - fputs_unfiltered ("\\f", stream); + gdb_puts ("\\f", stream); break; case '\n': - fputs_unfiltered ("\\n", stream); + gdb_puts ("\\n", stream); break; case '\r': - fputs_unfiltered ("\\r", stream); + gdb_puts ("\\r", stream); break; case '\t': - fputs_unfiltered ("\\t", stream); + gdb_puts ("\\t", stream); break; case '\v': - fputs_unfiltered ("\\v", stream); + gdb_puts ("\\v", stream); break; default: - fprintf_unfiltered (stream, - isprint (ch) ? "%c" : "\\x%02x", ch & 0xFF); + gdb_printf (stream, + isprint (ch) ? "%c" : "\\x%02x", ch & 0xFF); break; } } @@ -137,8 +131,8 @@ serial_log_command (struct target_ops *self, const char *cmd) serial_current_type = 'c'; - fputs_unfiltered ("\nc ", serial_logfp); - fputs_unfiltered (cmd, serial_logfp); + gdb_puts ("\nc ", serial_logfp); + gdb_puts (cmd, serial_logfp); /* Make sure that the log file is as up-to-date as possible, in case we are getting ready to dump core or something. */ @@ -149,10 +143,7 @@ serial_log_command (struct target_ops *self, const char *cmd) static const struct serial_ops * serial_interface_lookup (const char *name) { - const struct serial_ops *ops; - int i; - - for (i = 0; VEC_iterate (serial_ops_p, serial_ops_list, i, ops); ++i) + for (const serial_ops *ops : serial_ops_list) if (strcmp (name, ops->name) == 0) return ops; @@ -162,7 +153,7 @@ serial_interface_lookup (const char *name) void serial_add_interface (const struct serial_ops *optable) { - VEC_safe_push (serial_ops_p, serial_ops_list, optable); + serial_ops_list.push_back (optable); } /* Return the open serial device for FD, if found, or NULL if FD is @@ -180,45 +171,69 @@ serial_for_fd (int fd) return NULL; } +/* Create a new serial for OPS. */ + +static struct serial * +new_serial (const struct serial_ops *ops) +{ + struct serial *scb; + + scb = XCNEW (struct serial); + + scb->ops = ops; + + scb->bufp = scb->buf; + scb->error_fd = -1; + scb->refcnt = 1; + + return scb; +} + +static struct serial *serial_open_ops_1 (const struct serial_ops *ops, + const char *open_name); + /* Open up a device or a network socket, depending upon the syntax of NAME. */ struct serial * serial_open (const char *name) { - struct serial *scb; const struct serial_ops *ops; const char *open_name = name; - if (strcmp (name, "pc") == 0) - ops = serial_interface_lookup ("pc"); - else if (strncmp (name, "lpt", 3) == 0) - ops = serial_interface_lookup ("parallel"); - else if (strncmp (name, "|", 1) == 0) - { - ops = serial_interface_lookup ("pipe"); - /* Discard ``|'' and any space before the command itself. */ - ++open_name; - open_name = skip_spaces_const (open_name); - } + if (startswith (name, "|")) + ops = serial_interface_lookup ("pipe"); /* Check for a colon, suggesting an IP address/port pair. Do this *after* checking for all the interesting prefixes. We don't want to constrain the syntax of what can follow them. */ else if (strchr (name, ':')) ops = serial_interface_lookup ("tcp"); else - ops = serial_interface_lookup ("hardwire"); + { +#ifndef USE_WIN32API + /* Check to see if name is a socket. If it is, then treat it + as such. Otherwise assume that it's a character device. */ + struct stat sb; + if (stat (name, &sb) == 0 && (sb.st_mode & S_IFMT) == S_IFSOCK) + ops = serial_interface_lookup ("local"); + else +#endif + ops = serial_interface_lookup ("hardwire"); + } if (!ops) return NULL; - scb = XNEW (struct serial); + return serial_open_ops_1 (ops, open_name); +} - scb->ops = ops; +/* Open up a serial for OPS, passing OPEN_NAME to the open method. */ - scb->bufcnt = 0; - scb->bufp = scb->buf; - scb->error_fd = -1; - scb->refcnt = 1; +static struct serial * +serial_open_ops_1 (const struct serial_ops *ops, const char *open_name) +{ + struct serial *scb; + + scb = new_serial (ops); /* `...->open (...)' would get expanded by the open(2) syscall macro. */ if ((*scb->ops->open) (scb, open_name)) @@ -227,24 +242,31 @@ serial_open (const char *name) return NULL; } - scb->name = xstrdup (name); + scb->name = open_name != NULL ? xstrdup (open_name) : NULL; scb->next = scb_base; - scb->debug_p = 0; - scb->async_state = 0; - scb->async_handler = NULL; - scb->async_context = NULL; scb_base = scb; - if (serial_logfile != NULL) + if (!serial_logfile.empty ()) { - serial_logfp = gdb_fopen (serial_logfile, "w"); - if (serial_logfp == NULL) - perror_with_name (serial_logfile); + stdio_file_up file (new stdio_file ()); + + if (!file->open (serial_logfile.c_str (), "w")) + perror_with_name (serial_logfile.c_str ()); + + serial_logfp = file.release (); } return scb; } +/* See serial.h. */ + +struct serial * +serial_open_ops (const struct serial_ops *ops) +{ + return serial_open_ops_1 (ops, NULL); +} + /* Open a new serial stream using a file handle, using serial interface ops OPS. */ @@ -257,27 +279,16 @@ serial_fdopen_ops (const int fd, const struct serial_ops *ops) { ops = serial_interface_lookup ("terminal"); if (!ops) - ops = serial_interface_lookup ("hardwire"); + ops = serial_interface_lookup ("hardwire"); } if (!ops) return NULL; - scb = XCNEW (struct serial); - - scb->ops = ops; - - scb->bufcnt = 0; - scb->bufp = scb->buf; - scb->error_fd = -1; - scb->refcnt = 1; + scb = new_serial (ops); scb->name = NULL; scb->next = scb_base; - scb->debug_p = 0; - scb->async_state = 0; - scb->async_handler = NULL; - scb->async_context = NULL; scb_base = scb; if ((ops->fdopen) != NULL) @@ -301,11 +312,11 @@ do_serial_close (struct serial *scb, int really_close) if (serial_logfp) { - fputs_unfiltered ("\nEnd of log\n", serial_logfp); + gdb_puts ("\nEnd of log\n", serial_logfp); serial_current_type = 0; /* XXX - What if serial_logfp == gdb_stdout or gdb_stderr? */ - ui_file_delete (serial_logfp); + delete serial_logfp; serial_logfp = NULL; } @@ -316,8 +327,7 @@ do_serial_close (struct serial *scb, int really_close) if (really_close) scb->ops->close (scb); - if (scb->name) - xfree (scb->name); + xfree (scb->name); /* For serial_is_open. */ scb->bufp = NULL; @@ -386,14 +396,14 @@ serial_readchar (struct serial *scb, int timeout) serial_logchar (serial_logfp, 'r', ch, timeout); /* Make sure that the log file is as up-to-date as possible, - in case we are getting ready to dump core or something. */ + in case we are getting ready to dump core or something. */ gdb_flush (serial_logfp); } if (serial_debug_p (scb)) { - fprintf_unfiltered (gdb_stdlog, "["); + gdb_printf (gdb_stdlog, "["); serial_logchar (gdb_stdlog, 'r', ch, timeout); - fprintf_unfiltered (gdb_stdlog, "]"); + gdb_printf (gdb_stdlog, "]"); gdb_flush (gdb_stdlog); } @@ -405,26 +415,26 @@ serial_write (struct serial *scb, const void *buf, size_t count) { if (serial_logfp != NULL) { - const char *str = buf; + const char *str = (const char *) buf; size_t c; for (c = 0; c < count; c++) serial_logchar (serial_logfp, 'w', str[c] & 0xff, 0); /* Make sure that the log file is as up-to-date as possible, - in case we are getting ready to dump core or something. */ + in case we are getting ready to dump core or something. */ gdb_flush (serial_logfp); } if (serial_debug_p (scb)) { - const char *str = buf; + const char *str = (const char *) buf; size_t c; for (c = 0; c < count; c++) { - fprintf_unfiltered (gdb_stdlog, "["); - serial_logchar (gdb_stdlog, 'w', str[count] & 0xff, 0); - fprintf_unfiltered (gdb_stdlog, "]"); + gdb_printf (gdb_stdlog, "["); + serial_logchar (gdb_stdlog, 'w', str[c] & 0xff, 0); + gdb_printf (gdb_stdlog, "]"); } gdb_flush (gdb_stdlog); } @@ -433,16 +443,14 @@ serial_write (struct serial *scb, const void *buf, size_t count) } void -serial_printf (struct serial *desc, const char *format,...) +serial_printf (struct serial *desc, const char *format, ...) { va_list args; - char *buf; va_start (args, format); - buf = xstrvprintf (format, args); - serial_write (desc, buf, strlen (buf)); + std::string buf = string_vprintf (format, args); + serial_write (desc, buf.c_str (), buf.length ()); - xfree (buf); va_end (args); } @@ -505,14 +513,6 @@ serial_print_tty_state (struct serial *scb, scb->ops->print_tty_state (scb, ttystate, stream); } -int -serial_noflush_set_tty_state (struct serial *scb, - serial_ttystate new_ttystate, - serial_ttystate old_ttystate) -{ - return scb->ops->noflush_set_tty_state (scb, new_ttystate, old_ttystate); -} - int serial_setbaudrate (struct serial *scb, int rate) { @@ -525,6 +525,14 @@ serial_setstopbits (struct serial *scb, int num) return scb->ops->setstopbits (scb, num); } +/* See serial.h. */ + +int +serial_setparity (struct serial *scb, int parity) +{ + return scb->ops->setparity (scb, parity); +} + int serial_can_async_p (struct serial *scb) { @@ -610,37 +618,45 @@ serial_pipe (struct serial *scbs[2]) static struct cmd_list_element *serial_set_cmdlist; static struct cmd_list_element *serial_show_cmdlist; -static void -serial_set_cmd (char *args, int from_tty) -{ - printf_unfiltered ("\"set serial\" must be followed " - "by the name of a command.\n"); - help_list (serial_set_cmdlist, "set serial ", -1, gdb_stdout); -} +/* See serial.h. */ + +int baud_rate = -1; static void -serial_show_cmd (char *args, int from_tty) +serial_baud_show_cmd (struct ui_file *file, int from_tty, + struct cmd_list_element *c, const char *value) { - cmd_show_list (serial_show_cmdlist, from_tty, ""); + gdb_printf (file, _("Baud rate for remote serial I/O is %s.\n"), + value); } -/* Baud rate specified for talking to serial target systems. Default - is left as -1, so targets can choose their own defaults. */ -/* FIXME: This means that "show serial baud" and gr_files_info can - print -1 or (unsigned int)-1. This is a Bad User Interface. */ +/* See serial.h. */ -int baud_rate = -1; +int serial_parity = GDBPARITY_NONE; + +static const char parity_none[] = "none"; +static const char parity_odd[] = "odd"; +static const char parity_even[] = "even"; +static const char *const parity_enums[] = + {parity_none, parity_odd, parity_even, NULL}; +static const char *parity = parity_none; + +/* Set serial_parity value. */ static void -serial_baud_show_cmd (struct ui_file *file, int from_tty, - struct cmd_list_element *c, const char *value) +set_parity (const char *ignore_args, int from_tty, struct cmd_list_element *c) { - fprintf_filtered (file, _("Baud rate for remote serial I/O is %s.\n"), - value); + if (parity == parity_odd) + serial_parity = GDBPARITY_ODD; + else if (parity == parity_even) + serial_parity = GDBPARITY_EVEN; + else + serial_parity = GDBPARITY_NONE; } +void _initialize_serial (); void -_initialize_serial (void) +_initialize_serial () { #if 0 add_com ("connect", class_obscure, connect_command, _("\ @@ -648,17 +664,11 @@ Connect the terminal directly up to the command monitor.\n\ Use ~. or ~^D to break out.")); #endif /* 0 */ - add_prefix_cmd ("serial", class_maintenance, serial_set_cmd, _("\ -Set default serial/parallel port configuration."), - &serial_set_cmdlist, "set serial ", - 0/*allow-unknown*/, - &setlist); - - add_prefix_cmd ("serial", class_maintenance, serial_show_cmd, _("\ -Show default serial/parallel port configuration."), - &serial_show_cmdlist, "show serial ", - 0/*allow-unknown*/, - &showlist); + add_setshow_prefix_cmd ("serial", class_maintenance, + _("Set default serial/parallel port configuration."), + _("Show default serial/parallel port configuration."), + &serial_set_cmdlist, &serial_show_cmdlist, + &setlist, &showlist); /* If target is open when baud changes, it doesn't take effect until the next open (I think, not sure). */ @@ -671,33 +681,13 @@ using remote targets."), serial_baud_show_cmd, &serial_set_cmdlist, &serial_show_cmdlist); - /* The commands "set/show serial baud" used to have a different name. - Add aliases to those names to facilitate the transition, and mark - them as deprecated, in order to make users aware of the fact that - the command names have been changed. */ - { - const char *cmd_name; - struct cmd_list_element *cmd; - - /* FIXME: There is a limitation in the deprecation mechanism, - and the warning ends up not being displayed for prefixed - aliases. So use a real command instead of an alias. */ - add_setshow_zinteger_cmd ("remotebaud", class_alias, &baud_rate, _("\ -Set baud rate for remote serial I/O."), _("\ -Show baud rate for remote serial I/O."), _("\ -This value is used to set the speed of the serial port when debugging\n\ -using remote targets."), - NULL, - serial_baud_show_cmd, - &setlist, &showlist); - cmd_name = "remotebaud"; - cmd = lookup_cmd (&cmd_name, setlist, "", -1, 1); - deprecate_cmd (cmd, "set serial baud"); - cmd_name - = "remotebaud"; /* needed because lookup_cmd updates the pointer */ - cmd = lookup_cmd (&cmd_name, showlist, "", -1, 1); - deprecate_cmd (cmd, "show serial baud"); - } + add_setshow_enum_cmd ("parity", no_class, parity_enums, + &parity, _("\ +Set parity for remote serial I/O."), _("\ +Show parity for remote serial I/O."), NULL, + set_parity, + NULL, /* FIXME: i18n: */ + &serial_set_cmdlist, &serial_show_cmdlist); add_setshow_filename_cmd ("remotelogfile", no_class, &serial_logfile, _("\ Set filename for remote session recording."), _("\ @@ -710,8 +700,8 @@ by gdbserver."), add_setshow_enum_cmd ("remotelogbase", no_class, logbase_enums, &serial_logbase, _("\ -Set numerical base for remote session logging"), _("\ -Show numerical base for remote session logging"), NULL, +Set numerical base for remote session logging."), _("\ +Show numerical base for remote session logging."), NULL, NULL, NULL, /* FIXME: i18n: */ &setlist, &showlist);