]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commitdiff
New gdbserver option --debug-format=timestamp.
authorDoug Evans <dje@google.com>
Wed, 22 Jan 2014 22:17:39 +0000 (14:17 -0800)
committerDoug Evans <dje@google.com>
Wed, 22 Jan 2014 22:17:39 +0000 (14:17 -0800)
* NEWS: Mention it.

gdbserver/
* configure.ac (AC_CHECK_FUNCS): Add test for gettimeofday.
* configure: Regenerate.
* config.in: Regenerate.
* Makefile.in (SFILES): Add debug.c.
(OBS): Add debug.o.
* debug.c: New file.
* debug.h: New file.
* linux-aarch64-low.c (*): Update all debugging printfs to use
debug_printf instead of fprintf.
* linux-arm-low.c (*): Ditto.
* linux-cris-low.c (*): Ditto.
* linux-crisv32-low.c (*): Ditto.
* linux-m32r-low.c (*): Ditto.
* linux-sparc-low.c (*): Ditto.
* linux-x86.c (*): Ditto.
* linux-low.c (*): Ditto.
(linux_wait_1): Add calls to debug_enter, debug_exit.
(linux_wait): Remove redundant debugging printf.
(stop_all_lwps): Add calls to debug_enter, debug_exit.
(linux_resume, unstop_all_lwps): Ditto.
* mem-break.c (*): Update all debugging printfs to use
debug_printf instead of fprintf.
* remote-utils.c (*): Ditto.
* thread-db.c (*): Ditto.
* server.c #include <ctype.h>, "gdb_vecs.h".
(debug_threads): Moved to debug.c.
(*): Update all debugging printfs to use debug_printf instead of
fprintf.
(start_inferior): Replace call to fflush with call to debug_flush.
(monitor_show_help): Mention set debug-format.
(parse_debug_format_options): New function.
(handle_monitor_command): Handle "monitor set debug-format".
(gdbserver_usage): Mention --debug-format.
(main): Parse --debug-format.
* server.h (debug_threads): Declaration moved to debug.h.
#include "debug.h".
* tracepoint.c (trace_debug_1) [!IN_PROCESS_AGENT]: Add version of
trace_debug_1 that uses debug_printf.
(tracepoint_look_up_symbols): Update all debugging printfs to use
debug_printf instead of fprintf.

doc/
* gdb.texinfo (Server): Mention --debug-format=all|none|timestamp.
(gdbserver man): Ditto.

testsuite/
* gdb.server/server-mon.exp: Add tests for "set debug-format".

27 files changed:
gdb/ChangeLog
gdb/NEWS
gdb/doc/ChangeLog
gdb/doc/gdb.texinfo
gdb/gdbserver/ChangeLog
gdb/gdbserver/Makefile.in
gdb/gdbserver/config.in
gdb/gdbserver/configure
gdb/gdbserver/configure.ac
gdb/gdbserver/debug.c [new file with mode: 0644]
gdb/gdbserver/debug.h [new file with mode: 0644]
gdb/gdbserver/linux-aarch64-low.c
gdb/gdbserver/linux-arm-low.c
gdb/gdbserver/linux-cris-low.c
gdb/gdbserver/linux-crisv32-low.c
gdb/gdbserver/linux-low.c
gdb/gdbserver/linux-m32r-low.c
gdb/gdbserver/linux-sparc-low.c
gdb/gdbserver/linux-x86-low.c
gdb/gdbserver/mem-break.c
gdb/gdbserver/remote-utils.c
gdb/gdbserver/server.c
gdb/gdbserver/server.h
gdb/gdbserver/thread-db.c
gdb/gdbserver/tracepoint.c
gdb/testsuite/ChangeLog
gdb/testsuite/gdb.server/server-mon.exp

index ef14841ea3fd9529635f83465c3fd89a29c4a418..0198542920468ea683a45fb0839ce7a204bcbe64 100644 (file)
@@ -1,3 +1,8 @@
+2014-01-22  Doug Evans  <dje@google.com>
+
+       New gdbserver option --debug-format=timestamp.
+       * NEWS: Mention it.
+
 2014-01-22  Andreas Arnez  <arnez@vnet.linux.ibm.com>
 
        * syscalls/s390x-linux.xml: New file.
index 5a44a67352ef73a330c0e5f1fe2a5d16219b9ba9..6b1aacb731a97b95e903d5af4c793d79c51e54b2 100644 (file)
--- a/gdb/NEWS
+++ b/gdb/NEWS
@@ -3,6 +3,14 @@
 
 *** Changes since GDB 7.7
 
+* New features in the GDB remote stub, GDBserver
+
+  ** New option --debug-format=option1[,option2,...] allows one to add
+     additional text to each output.  At present only timestamps
+     are supported: --debug-format=timestamps.
+     Timestamps can also be turned on with the
+     "monitor set debug-format timestamps" command from GDB.
+
 * The 'record instruction-history' command now starts counting instructions
   at one.  This also affects the instruction ranges reported by the
   'record function-call-history' command when given the /i modifier.
index 74a2763b15f27061088a96a16b4c08b7f7d95e46..0ccad7e9a34501a809387cbdf05fadbb7016bc1b 100644 (file)
@@ -1,3 +1,8 @@
+2014-01-22  Doug Evans  <dje@google.com>
+
+       * gdb.texinfo (Server): Mention --debug-format=all|none|timestamp.
+       (gdbserver man): Ditto.
+
 2014-01-16  Markus Metzger  <markus.t.metzger@intel.com>
 
        * gdb.texinfo: Document limited reverse/replay support
index e77ca2aadc3261010a50ac99e39786a0fb42036f..a990d2ae00cc65eb0a4276b0cddab3aa7257fc5e 100644 (file)
@@ -18507,6 +18507,7 @@ connections and even in the @kbd{target extended-remote} mode.  The
 multiple instances of @code{gdbserver} running on the same host, since each
 instance closes its port after the first connection.
 
+@anchor{Other Command-Line Arguments for gdbserver}
 @subsubsection Other Command-Line Arguments for @code{gdbserver}
 
 @cindex @option{--debug}, @code{gdbserver} option
@@ -18517,6 +18518,23 @@ The @option{--remote-debug} option tells @code{gdbserver} to display
 remote protocol debug output.  These options are intended for
 @code{gdbserver} development and for bug reports to the developers.
 
+@cindex @option{--debug-format}, @code{gdbserver} option
+The @option{--debug-format=option1[,option2,...]} option tells
+@code{gdbserver} to include additional information in each output.
+Possible options are:
+
+@table @code
+@item none
+Turn off all extra information in debugging output.
+@item all
+Turn on all extra information in debugging output.
+@item timestamps
+Include a timestamp in each line of debugging output.
+@end table
+
+Options are processed in order.  Thus, for example, if @option{none}
+appears last then no additional information is added to debugging output.
+
 @cindex @option{--wrapper}, @code{gdbserver} option
 The @option{--wrapper} option specifies a wrapper to launch programs
 for debugging.  The option should be followed by the name of the
@@ -18587,6 +18605,22 @@ Disable or enable general debugging messages.
 Disable or enable specific debugging messages associated with the remote
 protocol (@pxref{Remote Protocol}).
 
+@item monitor set debug-format option1@r{[},option2,...@r{]}
+Specify additional text to add to debugging messages.
+Possible options are:
+
+@table @code
+@item none
+Turn off all extra information in debugging output.
+@item all
+Turn on all extra information in debugging output.
+@item timestamps
+Include a timestamp in each line of debugging output.
+@end table
+
+Options are processed in order.  Thus, for example, if @option{none}
+appears last then no additional information is added to debugging output.
+
 @item monitor set libthread-db-search-path [PATH]
 @cindex gdbserver, search path for @code{libthread_db}
 When this command is issued, @var{path} is a colon-separated list of
@@ -44342,6 +44376,11 @@ Instruct @code{gdbserver} to display remote protocol debug output.
 This option is intended for @code{gdbserver} development and for bug reports to
 the developers.
 
+@item --debug-format=option1@r{[},option2,...@r{]}
+Instruct @code{gdbserver} to include extra information in each line
+of debugging output.
+@xref{Other Command-Line Arguments for gdbserver}.
+
 @item --wrapper
 Specify a wrapper to launch programs
 for debugging.  The option should be followed by the name of the
index 1deea578a6bd7b4dca5f44393417d01db99ad96f..e6b3255b8fde9a39e0a26fab117f51ed16cc66f3 100644 (file)
@@ -1,3 +1,46 @@
+2014-01-22  Doug Evans  <dje@google.com>
+
+       * configure.ac (AC_CHECK_FUNCS): Add test for gettimeofday.
+       * configure: Regenerate.
+       * config.in: Regenerate.
+       * Makefile.in (SFILES): Add debug.c.
+       (OBS): Add debug.o.
+       * debug.c: New file.
+       * debug.h: New file.
+       * linux-aarch64-low.c (*): Update all debugging printfs to use
+       debug_printf instead of fprintf.
+       * linux-arm-low.c (*): Ditto.
+       * linux-cris-low.c (*): Ditto.
+       * linux-crisv32-low.c (*): Ditto.
+       * linux-m32r-low.c (*): Ditto.
+       * linux-sparc-low.c (*): Ditto.
+       * linux-x86.c (*): Ditto.
+       * linux-low.c (*): Ditto.
+       (linux_wait_1): Add calls to debug_enter, debug_exit.
+       (linux_wait): Remove redundant debugging printf.
+       (stop_all_lwps): Add calls to debug_enter, debug_exit.
+       (linux_resume, unstop_all_lwps): Ditto.
+       * mem-break.c (*): Update all debugging printfs to use
+       debug_printf instead of fprintf.
+       * remote-utils.c (*): Ditto.
+       * thread-db.c (*): Ditto.
+       * server.c #include <ctype.h>, "gdb_vecs.h".
+       (debug_threads): Moved to debug.c.
+       (*): Update all debugging printfs to use debug_printf instead of
+       fprintf.
+       (start_inferior): Replace call to fflush with call to debug_flush.
+       (monitor_show_help): Mention set debug-format.
+       (parse_debug_format_options): New function.
+       (handle_monitor_command): Handle "monitor set debug-format".
+       (gdbserver_usage): Mention --debug-format.
+       (main): Parse --debug-format.
+       * server.h (debug_threads): Declaration moved to debug.h.
+       #include "debug.h".
+       * tracepoint.c (trace_debug_1) [!IN_PROCESS_AGENT]: Add version of
+       trace_debug_1 that uses debug_printf.
+       (tracepoint_look_up_symbols): Update all debugging printfs to use
+       debug_printf instead of fprintf.
+
 2014-01-20  Baruch Siach  <baruch@tkos.co.il>
 
        * linux-xtensa-low.c: Include asm/ptrace.h instead of
index e72ee6bca963fc91eb2c137a6cb3e9a240d11b83..46324e64dc9b5b9097d5caebb708b14efefaa2c4 100644 (file)
@@ -141,7 +141,7 @@ SFILES=     $(srcdir)/gdbreplay.c $(srcdir)/inferiors.c $(srcdir)/dll.c \
        $(srcdir)/mem-break.c $(srcdir)/proc-service.c \
        $(srcdir)/proc-service.list $(srcdir)/regcache.c \
        $(srcdir)/remote-utils.c $(srcdir)/server.c $(srcdir)/target.c \
-       $(srcdir)/thread-db.c $(srcdir)/utils.c \
+       $(srcdir)/thread-db.c $(srcdir)/utils.c $(srcdir)/debug.c \
        $(srcdir)/linux-arm-low.c $(srcdir)/linux-bfin-low.c \
        $(srcdir)/linux-cris-low.c $(srcdir)/linux-crisv32-low.c \
        ${srcdir}/i386-low.c $(srcdir)/i387-fp.c \
@@ -173,7 +173,7 @@ SOURCES = $(SFILES)
 TAGFILES = $(SOURCES) ${HFILES} ${ALLPARAM} ${POSSLIBS}
 
 OBS = agent.o ax.o inferiors.o regcache.o remote-utils.o server.o signals.o \
-      target.o waitstatus.o utils.o version.o vec.o gdb_vecs.o \
+      target.o waitstatus.o utils.o debug.o version.o vec.o gdb_vecs.o \
       mem-break.o hostio.o event-loop.o tracepoint.o xml-utils.o \
       common-utils.o ptid.o buffer.o format.o filestuff.o dll.o notif.o \
       tdesc.o $(XML_BUILTIN) $(DEPFILES) $(LIBOBJS)
index e33cdead3fbb6feaa9e6ab330dbdf703e28d9995..1b9751f617a10b89a5641e35eee04c9d19f104c8 100644 (file)
@@ -66,6 +66,9 @@
 /* Define to 1 if you have the `getrlimit' function. */
 #undef HAVE_GETRLIMIT
 
+/* Define to 1 if you have the `gettimeofday' function. */
+#undef HAVE_GETTIMEOFDAY
+
 /* Define to 1 if you have the <inttypes.h> header file. */
 #undef HAVE_INTTYPES_H
 
index fac7fb3b7d9a47fdb819cce0d30289518c46e713..6d8a6a74b6c991f3a76a45c53a580e84f04962c1 100755 (executable)
@@ -4728,7 +4728,7 @@ fi
 
 done
 
-for ac_func in pread pwrite pread64 readlink
+for ac_func in gettimeofday pread pwrite pread64 readlink
 do :
   as_ac_var=`$as_echo "ac_cv_func_$ac_func" | $as_tr_sh`
 ac_fn_c_check_func "$LINENO" "$ac_func" "$as_ac_var"
index dc8131c8d6266ba62a866091ca45253fe5b92509..0dfa4a9cc3a8056abbd00cbd1ff7ec193e9b989f 100644 (file)
@@ -82,7 +82,7 @@ AC_CHECK_HEADERS(sgtty.h termio.h termios.h sys/reg.h string.h dnl
                 errno.h fcntl.h signal.h sys/file.h malloc.h dnl
                 sys/ioctl.h netinet/in.h sys/socket.h netdb.h dnl
                 netinet/tcp.h arpa/inet.h)
-AC_CHECK_FUNCS(pread pwrite pread64 readlink)
+AC_CHECK_FUNCS(gettimeofday pread pwrite pread64 readlink)
 AC_REPLACE_FUNCS(vasprintf vsnprintf)
 
 GDB_AC_COMMON
diff --git a/gdb/gdbserver/debug.c b/gdb/gdbserver/debug.c
new file mode 100644 (file)
index 0000000..6f4d531
--- /dev/null
@@ -0,0 +1,98 @@
+/* Debugging routines for the remote server for GDB.
+   Copyright (C) 2014 Free Software Foundation, Inc.
+
+   This file is part of GDB.
+
+   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.
+
+   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 "server.h"
+#include <sys/time.h>
+
+/* Enable miscellaneous debugging output.  The name is historical - it
+   was originally used to debug LinuxThreads support.  */
+int debug_threads;
+
+/* Include timestamps in debugging output.  */
+int debug_timestamp;
+
+/* Print a debugging message.
+   If the text begins a new line it is preceded by a timestamp, if the
+   system has gettimeofday.
+   We don't get fancy with newline checking, we just check whether the
+   previous call ended with "\n".  */
+
+void
+debug_printf (const char *msg, ...)
+{
+  va_list args;
+#if defined (HAVE_GETTIMEOFDAY) && !defined (IN_PROCESS_AGENT)
+  /* N.B. Not thread safe, and can't be used, as is, with IPA.  */
+  static int new_line = 1;
+
+  if (debug_timestamp && new_line)
+    {
+      struct timeval tm;
+
+      gettimeofday (&tm, NULL);
+
+      /* If gettimeofday doesn't exist, and as a portability solution it has
+        been replaced with, e.g., time, then it doesn't make sense to print
+        the microseconds field.  Is there a way to check for that?  */
+      fprintf (stderr, "%ld:%06ld ", (long) tm.tv_sec, (long) tm.tv_usec);
+    }
+#endif
+
+  va_start (args, msg);
+  vfprintf (stderr, msg, args);
+  va_end (args);
+
+#if defined (HAVE_GETTIMEOFDAY) && !defined (IN_PROCESS_AGENT)
+  if (*msg)
+    new_line = msg[strlen (msg) - 1] == '\n';
+#endif
+}
+
+/* Flush debugging output.
+   This is called, for example, when starting an inferior to ensure all debug
+   output thus far appears before any inferior output.  */
+
+void
+debug_flush (void)
+{
+  fflush (stderr);
+}
+
+/* Notify the user that the code is entering FUNCTION_NAME.
+   FUNCTION_NAME is the name of the calling function, or NULL if unknown.
+
+   This is intended to be called via the debug_enter macro.  */
+
+void
+do_debug_enter (const char *function_name)
+{
+  if (function_name != NULL)
+    debug_printf (">>>> entering %s\n", function_name);
+}
+
+/* Notify the user that the code is exiting FUNCTION_NAME.
+   FUNCTION_NAME is the name of the calling function, or NULL if unknown.
+
+   This is intended to be called via the debug_exit macro.  */
+
+void
+do_debug_exit (const char *function_name)
+{
+  if (function_name != NULL)
+    debug_printf ("<<<< exiting %s\n", function_name);
+}
diff --git a/gdb/gdbserver/debug.h b/gdb/gdbserver/debug.h
new file mode 100644 (file)
index 0000000..0f056ca
--- /dev/null
@@ -0,0 +1,55 @@
+/* Debugging routines for the remote server for GDB.
+   Copyright (C) 2014 Free Software Foundation, Inc.
+
+   This file is part of GDB.
+
+   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.
+
+   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/>.  */
+
+#ifndef DEBUG_H
+#define DEBUG_H
+
+/* We declare debug format variables here, and debug_threads but no other
+   debug content variables (e.g., not remote_debug) because while this file
+   is not currently used by IPA it may be some day, and IPA may have its own
+   set of debug content variables.  It's ok to declare debug_threads here
+   because it is misnamed - a better name is debug_basic or some such,
+   which can work for any program, gdbserver or IPA.  If/when this file is
+   used with IPA it is recommended to fix debug_thread's name.  */
+extern int debug_threads;
+extern int debug_timestamp;
+
+void debug_printf (const char *msg, ...) ATTRIBUTE_PRINTF (1, 2);
+void debug_flush (void);
+void do_debug_enter (const char *function_name);
+void do_debug_exit (const char *function_name);
+
+/* These macros are for use in major functions that produce a lot of
+   debugging output.  They help identify in the mass of debugging output
+   when these functions enter and exit.  debug_enter is intended to be
+   called at the start of a function, before any other debugging output.
+   debug_exit is intended to be called at the end of the same function,
+   after all debugging output.  */
+#ifdef FUNCTION_NAME
+#define debug_enter() \
+  do { do_debug_enter (FUNCTION_NAME); } while (0)
+#define debug_exit() \
+  do { do_debug_exit (FUNCTION_NAME); } while (0)
+#else
+#define debug_enter() \
+  do { } while (0)
+#define debug_exit() \
+  do { } while (0)
+#endif
+
+#endif /* DEBUG_H */
index 1b0da6cb8c8542d9a325399fe5200f0455256742..e7d3e4f9ad523bbfd5b98cbb5de29cddb805865e 100644 (file)
@@ -323,7 +323,7 @@ aarch64_get_pc (struct regcache *regcache)
 
   collect_register_by_name (regcache, "pc", &pc);
   if (debug_threads)
-    fprintf (stderr, "stop pc is %08lx\n", pc);
+    debug_printf ("stop pc is %08lx\n", pc);
   return pc;
 }
 
index da5085cdaf6ea287a1543040db4be958e924fdaa..fb6ff6815ff1271d4ae6bbf6372a7141770bffe8 100644 (file)
@@ -256,7 +256,7 @@ arm_get_pc (struct regcache *regcache)
   unsigned long pc;
   collect_register_by_name (regcache, "pc", &pc);
   if (debug_threads)
-    fprintf (stderr, "stop pc is %08lx\n", pc);
+    debug_printf ("stop pc is %08lx\n", pc);
   return pc;
 }
 
index d229d435509eb1675cce6d690b3d7d29a84f547e..2abd987768c76272f3be7bcd875a19b816d6f090 100644 (file)
@@ -67,7 +67,7 @@ cris_get_pc (struct regcache *regcache, void)
   unsigned long pc;
   collect_register_by_name (regcache, "pc", &pc);
   if (debug_threads)
-    fprintf (stderr, "stop pc is %08lx\n", pc);
+    debug_printf ("stop pc is %08lx\n", pc);
   return pc;
 }
 
index e4ebb62b711c75769da2217a9c95976010e9142c..a16a656b3d13b647c196a1d21ccbb2cdec78d01a 100644 (file)
@@ -63,7 +63,7 @@ cris_get_pc (struct regcache *regcache)
   unsigned long pc;
   collect_register_by_name (regcache, "pc", &pc);
   if (debug_threads)
-    fprintf (stderr, "stop pc is %08lx\n", pc);
+    debug_printf ("stop pc is %08lx\n", pc);
   return pc;
 }
 
index 01d0e84ac4534c9976e394d26639194690e408a6..bac6134d05b260d7809694d0b9ad5d98b3318f6f 100644 (file)
@@ -475,7 +475,7 @@ get_pc (struct lwp_info *lwp)
   pc = (*the_low_target.get_pc) (regcache);
 
   if (debug_threads)
-    fprintf (stderr, "pc is 0x%lx\n", (long) pc);
+    debug_printf ("pc is 0x%lx\n", (long) pc);
 
   current_inferior = saved_inferior;
   return pc;
@@ -520,7 +520,7 @@ get_stop_pc (struct lwp_info *lwp)
     stop_pc -= the_low_target.decr_pc_after_break;
 
   if (debug_threads)
-    fprintf (stderr, "stop pc is 0x%lx\n", (long) stop_pc);
+    debug_printf ("stop pc is 0x%lx\n", (long) stop_pc);
 
   return stop_pc;
 }
@@ -692,8 +692,7 @@ linux_attach_lwp_1 (unsigned long lwpid, int initial)
   if (linux_proc_pid_is_stopped (lwpid))
     {
       if (debug_threads)
-       fprintf (stderr,
-                "Attached to a stopped process\n");
+       debug_printf ("Attached to a stopped process\n");
 
       /* The process is definitely stopped.  It is in a job control
         stop, unless the kernel predates the TASK_STOPPED /
@@ -824,8 +823,8 @@ linux_attach (unsigned long pid)
                      new_threads_found++;
 
                      if (debug_threads)
-                       fprintf (stderr, "\
-Found and attached to new lwp %ld\n", lwp);
+                       debug_printf ("Found and attached to new lwp %ld\n",
+                                     lwp);
                    }
                }
 
@@ -898,18 +897,16 @@ linux_kill_one_lwp (struct lwp_info *lwp)
   errno = 0;
   kill (pid, SIGKILL);
   if (debug_threads)
-    fprintf (stderr,
-            "LKL:  kill (SIGKILL) %s, 0, 0 (%s)\n",
-            target_pid_to_str (ptid_of (lwp)),
-            errno ? strerror (errno) : "OK");
+    debug_printf ("LKL:  kill (SIGKILL) %s, 0, 0 (%s)\n",
+                 target_pid_to_str (ptid_of (lwp)),
+                 errno ? strerror (errno) : "OK");
 
   errno = 0;
   ptrace (PTRACE_KILL, pid, (PTRACE_TYPE_ARG3) 0, (PTRACE_TYPE_ARG4) 0);
   if (debug_threads)
-    fprintf (stderr,
-            "LKL:  PTRACE_KILL %s, 0, 0 (%s)\n",
-            target_pid_to_str (ptid_of (lwp)),
-            errno ? strerror (errno) : "OK");
+    debug_printf ("LKL:  PTRACE_KILL %s, 0, 0 (%s)\n",
+                 target_pid_to_str (ptid_of (lwp)),
+                 errno ? strerror (errno) : "OK");
 }
 
 /* Callback for `find_inferior'.  Kills an lwp of a given process,
@@ -934,8 +931,8 @@ kill_one_lwp_callback (struct inferior_list_entry *entry, void *args)
   if (lwpid_of (lwp) == pid)
     {
       if (debug_threads)
-       fprintf (stderr, "lkop: is last of process %s\n",
-                target_pid_to_str (entry->id));
+       debug_printf ("lkop: is last of process %s\n",
+                     target_pid_to_str (entry->id));
       return 0;
     }
 
@@ -975,14 +972,14 @@ linux_kill (int pid)
   if (lwp == NULL)
     {
       if (debug_threads)
-       fprintf (stderr, "lk_1: cannot find lwp %ld, for pid: %d\n",
-                lwpid_of (lwp), pid);
+       debug_printf ("lk_1: cannot find lwp %ld, for pid: %d\n",
+                     lwpid_of (lwp), pid);
     }
   else
     {
       if (debug_threads)
-       fprintf (stderr, "lk_1: killing lwp %ld, for pid: %d\n",
-                lwpid_of (lwp), pid);
+       debug_printf ("lk_1: killing lwp %ld, for pid: %d\n",
+                     lwpid_of (lwp), pid);
 
       do
        {
@@ -1031,9 +1028,8 @@ get_detach_signal (struct thread_info *thread)
   if (!WIFSTOPPED (status))
     {
       if (debug_threads)
-       fprintf (stderr,
-                "GPS: lwp %s hasn't stopped: no pending signal\n",
-                target_pid_to_str (ptid_of (lp)));
+       debug_printf ("GPS: lwp %s hasn't stopped: no pending signal\n",
+                     target_pid_to_str (ptid_of (lp)));
       return 0;
     }
 
@@ -1041,10 +1037,9 @@ get_detach_signal (struct thread_info *thread)
   if (WSTOPSIG (status) == SIGTRAP && status >> 16 != 0)
     {
       if (debug_threads)
-       fprintf (stderr,
-                "GPS: lwp %s had stopped with extended "
-                "status: no pending signal\n",
-                target_pid_to_str (ptid_of (lp)));
+       debug_printf ("GPS: lwp %s had stopped with extended "
+                     "status: no pending signal\n",
+                     target_pid_to_str (ptid_of (lp)));
       return 0;
     }
 
@@ -1053,10 +1048,9 @@ get_detach_signal (struct thread_info *thread)
   if (program_signals_p && !program_signals[signo])
     {
       if (debug_threads)
-       fprintf (stderr,
-                "GPS: lwp %s had signal %s, but it is in nopass state\n",
-                target_pid_to_str (ptid_of (lp)),
-                gdb_signal_to_string (signo));
+       debug_printf ("GPS: lwp %s had signal %s, but it is in nopass state\n",
+                     target_pid_to_str (ptid_of (lp)),
+                     gdb_signal_to_string (signo));
       return 0;
     }
   else if (!program_signals_p
@@ -1066,20 +1060,19 @@ get_detach_signal (struct thread_info *thread)
           && (signo == GDB_SIGNAL_TRAP || signo == GDB_SIGNAL_INT))
     {
       if (debug_threads)
-       fprintf (stderr,
-                "GPS: lwp %s had signal %s, "
-                "but we don't know if we should pass it.  Default to not.\n",
-                target_pid_to_str (ptid_of (lp)),
-                gdb_signal_to_string (signo));
+       debug_printf ("GPS: lwp %s had signal %s, "
+                     "but we don't know if we should pass it. "
+                     "Default to not.\n",
+                     target_pid_to_str (ptid_of (lp)),
+                     gdb_signal_to_string (signo));
       return 0;
     }
   else
     {
       if (debug_threads)
-       fprintf (stderr,
-                "GPS: lwp %s has pending signal %s: delivering it.\n",
-                target_pid_to_str (ptid_of (lp)),
-                gdb_signal_to_string (signo));
+       debug_printf ("GPS: lwp %s has pending signal %s: delivering it.\n",
+                     target_pid_to_str (ptid_of (lp)),
+                     gdb_signal_to_string (signo));
 
       return WSTOPSIG (status);
     }
@@ -1100,9 +1093,8 @@ linux_detach_one_lwp (struct inferior_list_entry *entry, void *args)
   if (lwp->stop_expected)
     {
       if (debug_threads)
-       fprintf (stderr,
-                "Sending SIGCONT to %s\n",
-                target_pid_to_str (ptid_of (lwp)));
+       debug_printf ("Sending SIGCONT to %s\n",
+                     target_pid_to_str (ptid_of (lwp)));
 
       kill_lwp (lwpid_of (lwp), SIGCONT);
       lwp->stop_expected = 0;
@@ -1276,7 +1268,7 @@ linux_wait_for_lwp (ptid_t ptid, int *wstatp, int options)
   struct lwp_info *child = NULL;
 
   if (debug_threads)
-    fprintf (stderr, "linux_wait_for_lwp: %s\n", target_pid_to_str (ptid));
+    debug_printf ("linux_wait_for_lwp: %s\n", target_pid_to_str (ptid));
 
   if (ptid_equal (ptid, minus_one_ptid))
     to_wait_for = -1;                  /* any child */
@@ -1297,7 +1289,7 @@ retry:
       && (!WIFSTOPPED (*wstatp)
          || (WSTOPSIG (*wstatp) != 32
              && WSTOPSIG (*wstatp) != 33)))
-    fprintf (stderr, "Got an event from %d (%x)\n", ret, *wstatp);
+    debug_printf ("Got an event from %d (%x)\n", ret, *wstatp);
 
   child = find_lwp_pid (pid_to_ptid (ret));
 
@@ -1404,7 +1396,7 @@ retry:
       current_inferior = get_lwp_thread (child);
       regcache = get_thread_regcache (current_inferior, 1);
       pc = (*the_low_target.get_pc) (regcache);
-      fprintf (stderr, "linux_wait_for_lwp: pc is 0x%lx\n", (long) pc);
+      debug_printf ("linux_wait_for_lwp: pc is 0x%lx\n", (long) pc);
       current_inferior = saved_inferior;
     }
 
@@ -1450,7 +1442,7 @@ handle_tracepoints (struct lwp_info *lwp)
   if (tpoint_related_event)
     {
       if (debug_threads)
-       fprintf (stderr, "got a tracepoint event\n");
+       debug_printf ("got a tracepoint event\n");
       return 1;
     }
 
@@ -1501,9 +1493,9 @@ maybe_move_out_of_jump_pad (struct lwp_info *lwp, int *wstat)
       int r;
 
       if (debug_threads)
-       fprintf (stderr, "\
-Checking whether LWP %ld needs to move out of the jump pad.\n",
-                lwpid_of (lwp));
+       debug_printf ("Checking whether LWP %ld needs to move out of the "
+                     "jump pad.\n",
+                     lwpid_of (lwp));
 
       r = linux_fast_tracepoint_collecting (lwp, &status);
 
@@ -1527,9 +1519,9 @@ Checking whether LWP %ld needs to move out of the jump pad.\n",
                }
 
              if (debug_threads)
-               fprintf (stderr, "\
-Checking whether LWP %ld needs to move out of the jump pad...it does\n",
-                lwpid_of (lwp));
+               debug_printf ("Checking whether LWP %ld needs to move out of "
+                             "the jump pad...it does\n",
+                             lwpid_of (lwp));
              current_inferior = saved_inferior;
 
              return 1;
@@ -1582,9 +1574,8 @@ Checking whether LWP %ld needs to move out of the jump pad...it does\n",
          if (lwp->exit_jump_pad_bkpt != NULL)
            {
              if (debug_threads)
-               fprintf (stderr,
-                        "Cancelling fast exit-jump-pad: removing bkpt. "
-                        "stopping all threads momentarily.\n");
+               debug_printf ("Cancelling fast exit-jump-pad: removing bkpt. "
+                             "stopping all threads momentarily.\n");
 
              stop_all_lwps (1, lwp);
              cancel_breakpoints ();
@@ -1600,9 +1591,9 @@ Checking whether LWP %ld needs to move out of the jump pad...it does\n",
     }
 
   if (debug_threads)
-    fprintf (stderr, "\
-Checking whether LWP %ld needs to move out of the jump pad...no\n",
-            lwpid_of (lwp));
+    debug_printf ("Checking whether LWP %ld needs to move out of the "
+                 "jump pad...no\n",
+                 lwpid_of (lwp));
 
   current_inferior = saved_inferior;
   return 0;
@@ -1617,8 +1608,8 @@ enqueue_one_deferred_signal (struct lwp_info *lwp, int *wstat)
   struct pending_signals *p_sig;
 
   if (debug_threads)
-    fprintf (stderr, "\
-Deferring signal %d for LWP %ld.\n", WSTOPSIG (*wstat), lwpid_of (lwp));
+    debug_printf ("Deferring signal %d for LWP %ld.\n",
+                 WSTOPSIG (*wstat), lwpid_of (lwp));
 
   if (debug_threads)
     {
@@ -1627,11 +1618,10 @@ Deferring signal %d for LWP %ld.\n", WSTOPSIG (*wstat), lwpid_of (lwp));
       for (sig = lwp->pending_signals_to_report;
           sig != NULL;
           sig = sig->prev)
-       fprintf (stderr,
-                "   Already queued %d\n",
-                sig->signal);
+       debug_printf ("   Already queued %d\n",
+                     sig->signal);
 
-      fprintf (stderr, "   (no more currently queued signals)\n");
+      debug_printf ("   (no more currently queued signals)\n");
     }
 
   /* Don't enqueue non-RT signals if they are already in the deferred
@@ -1648,11 +1638,10 @@ Deferring signal %d for LWP %ld.\n", WSTOPSIG (*wstat), lwpid_of (lwp));
          if (sig->signal == WSTOPSIG (*wstat))
            {
              if (debug_threads)
-               fprintf (stderr,
-                        "Not requeuing already queued non-RT signal %d"
-                        " for LWP %ld\n",
-                        sig->signal,
-                        lwpid_of (lwp));
+               debug_printf ("Not requeuing already queued non-RT signal %d"
+                             " for LWP %ld\n",
+                             sig->signal,
+                             lwpid_of (lwp));
              return;
            }
        }
@@ -1690,8 +1679,8 @@ dequeue_one_deferred_signal (struct lwp_info *lwp, int *wstat)
       *p_sig = NULL;
 
       if (debug_threads)
-       fprintf (stderr, "Reporting deferred signal %d for LWP %ld.\n",
-                WSTOPSIG (*wstat), lwpid_of (lwp));
+       debug_printf ("Reporting deferred signal %d for LWP %ld.\n",
+                     WSTOPSIG (*wstat), lwpid_of (lwp));
 
       if (debug_threads)
        {
@@ -1700,11 +1689,10 @@ dequeue_one_deferred_signal (struct lwp_info *lwp, int *wstat)
          for (sig = lwp->pending_signals_to_report;
               sig != NULL;
               sig = sig->prev)
-           fprintf (stderr,
-                    "   Still queued %d\n",
-                    sig->signal);
+           debug_printf ("   Still queued %d\n",
+                         sig->signal);
 
-         fprintf (stderr, "   (no more queued signals)\n");
+         debug_printf ("   (no more queued signals)\n");
        }
 
       return 1;
@@ -1734,9 +1722,8 @@ cancel_breakpoint (struct lwp_info *lwp)
   if ((*the_low_target.breakpoint_at) (lwp->stop_pc))
     {
       if (debug_threads)
-       fprintf (stderr,
-                "CB: Push back breakpoint for %s\n",
-                target_pid_to_str (ptid_of (lwp)));
+       debug_printf ("CB: Push back breakpoint for %s\n",
+                     target_pid_to_str (ptid_of (lwp)));
 
       /* Back up the PC if necessary.  */
       if (the_low_target.decr_pc_after_break)
@@ -1752,10 +1739,9 @@ cancel_breakpoint (struct lwp_info *lwp)
   else
     {
       if (debug_threads)
-       fprintf (stderr,
-                "CB: No breakpoint found at %s for [%s]\n",
-                paddress (lwp->stop_pc),
-                target_pid_to_str (ptid_of (lwp)));
+       debug_printf ("CB: No breakpoint found at %s for [%s]\n",
+                     paddress (lwp->stop_pc),
+                     target_pid_to_str (ptid_of (lwp)));
     }
 
   current_inferior = saved_inferior;
@@ -1788,7 +1774,7 @@ linux_wait_for_event (ptid_t ptid, int *wstat, int options)
       event_child = (struct lwp_info *)
        find_inferior (&all_lwps, status_pending_p_callback, &ptid);
       if (debug_threads && event_child)
-       fprintf (stderr, "Got a pending child %ld\n", lwpid_of (event_child));
+       debug_printf ("Got a pending child %ld\n", lwpid_of (event_child));
     }
   else
     {
@@ -1816,8 +1802,8 @@ linux_wait_for_event (ptid_t ptid, int *wstat, int options)
   if (event_child != NULL)
     {
       if (debug_threads)
-       fprintf (stderr, "Got an event from pending child %ld (%04x)\n",
-                lwpid_of (event_child), event_child->status_pending);
+       debug_printf ("Got an event from pending child %ld (%04x)\n",
+                     lwpid_of (event_child), event_child->status_pending);
       *wstat = event_child->status_pending;
       event_child->status_pending_p = 0;
       event_child->status_pending = 0;
@@ -1847,7 +1833,7 @@ linux_wait_for_event (ptid_t ptid, int *wstat, int options)
       if ((options & WNOHANG) && event_child == NULL)
        {
          if (debug_threads)
-           fprintf (stderr, "WNOHANG set, no event found\n");
+           debug_printf ("WNOHANG set, no event found\n");
          return 0;
        }
 
@@ -1873,14 +1859,14 @@ linux_wait_for_event (ptid_t ptid, int *wstat, int options)
       if (! WIFSTOPPED (*wstat))
        {
          if (debug_threads)
-           fprintf (stderr, "LWP %ld exiting\n", lwpid_of (event_child));
+           debug_printf ("LWP %ld exiting\n", lwpid_of (event_child));
 
          /* If the last thread is exiting, just return.  */
          if (last_thread_of_process_p (current_inferior))
            {
              if (debug_threads)
-               fprintf (stderr, "LWP %ld is last lwp of process\n",
-                        lwpid_of (event_child));
+               debug_printf ("LWP %ld is last lwp of process\n",
+                             lwpid_of (event_child));
              return lwpid_of (event_child);
            }
 
@@ -1888,14 +1874,14 @@ linux_wait_for_event (ptid_t ptid, int *wstat, int options)
            {
              current_inferior = (struct thread_info *) all_threads.head;
              if (debug_threads)
-               fprintf (stderr, "Current inferior is now %ld\n",
-                        lwpid_of (get_thread_lwp (current_inferior)));
+               debug_printf ("Current inferior is now %ld\n",
+                             lwpid_of (get_thread_lwp (current_inferior)));
            }
          else
            {
              current_inferior = NULL;
              if (debug_threads)
-               fprintf (stderr, "Current inferior is now <NULL>\n");
+               debug_printf ("Current inferior is now <NULL>\n");
            }
 
          /* If we were waiting for this particular child to do something...
@@ -1938,7 +1924,7 @@ linux_wait_for_event (ptid_t ptid, int *wstat, int options)
          int should_stop;
 
          if (debug_threads)
-           fprintf (stderr, "Expected stop.\n");
+           debug_printf ("Expected stop.\n");
          event_child->stop_expected = 0;
 
          should_stop = (current_inferior->last_resume_kind == resume_stop
@@ -2082,9 +2068,8 @@ select_event_lwp (struct lwp_info **orig_lp)
   if (event_lp != NULL)
     {
       if (debug_threads)
-       fprintf (stderr,
-                "SEL: Select single-step %s\n",
-                target_pid_to_str (ptid_of (event_lp)));
+       debug_printf ("SEL: Select single-step %s\n",
+                     target_pid_to_str (ptid_of (event_lp)));
     }
   else
     {
@@ -2099,9 +2084,8 @@ select_event_lwp (struct lwp_info **orig_lp)
        ((num_events * (double) rand ()) / (RAND_MAX + 1.0));
 
       if (debug_threads && num_events > 1)
-       fprintf (stderr,
-                "SEL: Found %d SIGTRAP events, selecting #%d\n",
-                num_events, random_selector);
+       debug_printf ("SEL: Found %d SIGTRAP events, selecting #%d\n",
+                     num_events, random_selector);
 
       event_lp = (struct lwp_info *) find_inferior (&all_lwps,
                                                    select_event_lwp_callback,
@@ -2191,8 +2175,8 @@ linux_stabilize_threads (void)
   if (lwp_stuck != NULL)
     {
       if (debug_threads)
-       fprintf (stderr, "can't stabilize, LWP %ld is stuck in jump pad\n",
-                lwpid_of (lwp_stuck));
+       debug_printf ("can't stabilize, LWP %ld is stuck in jump pad\n",
+                     lwpid_of (lwp_stuck));
       return;
     }
 
@@ -2243,8 +2227,8 @@ linux_stabilize_threads (void)
        = (struct lwp_info *) find_inferior (&all_lwps,
                                         stuck_in_jump_pad_callback, NULL);
       if (lwp_stuck != NULL)
-       fprintf (stderr, "couldn't stabilize, LWP %ld got stuck in jump pad\n",
-                lwpid_of (lwp_stuck));
+       debug_printf ("couldn't stabilize, LWP %ld got stuck in jump pad\n",
+                     lwpid_of (lwp_stuck));
     }
 }
 
@@ -2265,6 +2249,12 @@ linux_wait_1 (ptid_t ptid,
   int trace_event;
   int in_step_range;
 
+  if (debug_threads)
+    {
+      debug_enter ();
+      debug_printf ("linux_wait_1: [%s]\n", target_pid_to_str (ptid));
+    }
+
   /* Translate generic target options into linux options.  */
   options = __WALL;
   if (target_options & TARGET_WNOHANG)
@@ -2309,13 +2299,20 @@ retry:
   else
     {
       if (debug_threads)
-       fprintf (stderr, "step_over_bkpt set [%s], doing a blocking wait\n",
-                target_pid_to_str (step_over_bkpt));
+       debug_printf ("step_over_bkpt set [%s], doing a blocking wait\n",
+                     target_pid_to_str (step_over_bkpt));
       pid = linux_wait_for_event (step_over_bkpt, &w, options & ~WNOHANG);
     }
 
   if (pid == 0) /* only if TARGET_WNOHANG */
-    return null_ptid;
+    {
+      if (debug_threads)
+       {
+         debug_printf ("linux_wait_1 ret = null_ptid\n");
+         debug_exit ();
+       }
+      return null_ptid;
+    }
 
   event_child = get_thread_lwp (current_inferior);
 
@@ -2341,9 +2338,13 @@ retry:
              ourstatus->value.integer = WEXITSTATUS (w);
 
              if (debug_threads)
-               fprintf (stderr,
-                        "\nChild exited with retcode = %x \n",
-                        WEXITSTATUS (w));
+               {
+                 debug_printf ("linux_wait_1 ret = %s, exited with "
+                               "retcode %d\n",
+                               target_pid_to_str (ptid_of (event_child)),
+                               WEXITSTATUS (w));
+                 debug_exit ();
+               }
            }
          else
            {
@@ -2351,10 +2352,13 @@ retry:
              ourstatus->value.sig = gdb_signal_from_host (WTERMSIG (w));
 
              if (debug_threads)
-               fprintf (stderr,
-                        "\nChild terminated with signal = %x \n",
-                        WTERMSIG (w));
-
+               {
+                 debug_printf ("linux_wait_1 ret = %s, terminated with "
+                               "signal %d\n",
+                               target_pid_to_str (ptid_of (event_child)),
+                               WTERMSIG (w));
+                 debug_exit ();
+               }
            }
 
          return ptid_of (event_child);
@@ -2409,7 +2413,7 @@ retry:
             already handled it.  So next time we resume (from this
             PC), we should step over it.  */
          if (debug_threads)
-           fprintf (stderr, "Hit a gdbserver breakpoint.\n");
+           debug_printf ("Hit a gdbserver breakpoint.\n");
 
          if (breakpoint_here (event_child->stop_pc))
            event_child->need_step_over = 1;
@@ -2435,10 +2439,9 @@ retry:
       && agent_loaded_p ())
     {
       if (debug_threads)
-       fprintf (stderr,
-                "Got signal %d for LWP %ld.  Check if we need "
-                "to defer or adjust it.\n",
-                WSTOPSIG (w), lwpid_of (event_child));
+       debug_printf ("Got signal %d for LWP %ld.  Check if we need "
+                     "to defer or adjust it.\n",
+                     WSTOPSIG (w), lwpid_of (event_child));
 
       /* Allow debugging the jump pad itself.  */
       if (current_inferior->last_resume_kind != resume_step
@@ -2447,9 +2450,8 @@ retry:
          enqueue_one_deferred_signal (event_child, &w);
 
          if (debug_threads)
-           fprintf (stderr,
-                    "Signal %d for LWP %ld deferred (in jump pad)\n",
-                    WSTOPSIG (w), lwpid_of (event_child));
+           debug_printf ("Signal %d for LWP %ld deferred (in jump pad)\n",
+                         WSTOPSIG (w), lwpid_of (event_child));
 
          linux_resume_one_lwp (event_child, 0, 0, NULL);
          goto retry;
@@ -2459,11 +2461,10 @@ retry:
   if (event_child->collecting_fast_tracepoint)
     {
       if (debug_threads)
-       fprintf (stderr, "\
-LWP %ld was trying to move out of the jump pad (%d).  \
-Check if we're already there.\n",
-                lwpid_of (event_child),
-                event_child->collecting_fast_tracepoint);
+       debug_printf ("LWP %ld was trying to move out of the jump pad (%d). "
+                     "Check if we're already there.\n",
+                     lwpid_of (event_child),
+                     event_child->collecting_fast_tracepoint);
 
       trace_event = 1;
 
@@ -2476,9 +2477,8 @@ Check if we're already there.\n",
          if (event_child->exit_jump_pad_bkpt != NULL)
            {
              if (debug_threads)
-               fprintf (stderr,
-                        "No longer need exit-jump-pad bkpt; removing it."
-                        "stopping all threads momentarily.\n");
+               debug_printf ("No longer need exit-jump-pad bkpt; removing it."
+                             "stopping all threads momentarily.\n");
 
              /* Other running threads could hit this breakpoint.
                 We don't handle moribund locations like GDB does,
@@ -2502,25 +2502,33 @@ Check if we're already there.\n",
       if (event_child->collecting_fast_tracepoint == 0)
        {
          if (debug_threads)
-           fprintf (stderr,
-                    "fast tracepoint finished "
-                    "collecting successfully.\n");
+           debug_printf ("fast tracepoint finished "
+                         "collecting successfully.\n");
 
          /* We may have a deferred signal to report.  */
          if (dequeue_one_deferred_signal (event_child, &w))
            {
              if (debug_threads)
-               fprintf (stderr, "dequeued one signal.\n");
+               debug_printf ("dequeued one signal.\n");
            }
          else
            {
              if (debug_threads)
-               fprintf (stderr, "no deferred signals.\n");
+               debug_printf ("no deferred signals.\n");
 
              if (stabilizing_threads)
                {
                  ourstatus->kind = TARGET_WAITKIND_STOPPED;
                  ourstatus->value.sig = GDB_SIGNAL_0;
+
+                 if (debug_threads)
+                   {
+                     debug_printf ("linux_wait_1 ret = %s, stopped "
+                                   "while stabilizing threads\n",
+                                   target_pid_to_str (ptid_of (event_child)));
+                     debug_exit ();
+                   }
+
                  return ptid_of (event_child);
                }
            }
@@ -2554,8 +2562,8 @@ Check if we're already there.\n",
       siginfo_t info, *info_p;
 
       if (debug_threads)
-       fprintf (stderr, "Ignored signal %d for LWP %ld.\n",
-                WSTOPSIG (w), lwpid_of (event_child));
+       debug_printf ("Ignored signal %d for LWP %ld.\n",
+                     WSTOPSIG (w), lwpid_of (event_child));
 
       if (ptrace (PTRACE_GETSIGINFO, lwpid_of (event_child),
                  (PTRACE_TYPE_ARG3) 0, &info) == 0)
@@ -2600,16 +2608,16 @@ Check if we're already there.\n",
       if (debug_threads)
        {
          if (bp_explains_trap)
-           fprintf (stderr, "Hit a gdbserver breakpoint.\n");
+           debug_printf ("Hit a gdbserver breakpoint.\n");
          if (step_over_finished)
-           fprintf (stderr, "Step-over finished.\n");
+           debug_printf ("Step-over finished.\n");
          if (trace_event)
-           fprintf (stderr, "Tracepoint event.\n");
+           debug_printf ("Tracepoint event.\n");
          if (lwp_in_step_range (event_child))
-           fprintf (stderr, "Range stepping pc 0x%s [0x%s, 0x%s).\n",
-                    paddress (event_child->stop_pc),
-                    paddress (event_child->step_range_start),
-                    paddress (event_child->step_range_end));
+           debug_printf ("Range stepping pc 0x%s [0x%s, 0x%s).\n",
+                         paddress (event_child->stop_pc),
+                         paddress (event_child->step_range_start),
+                         paddress (event_child->step_range_end));
        }
 
       /* We're not reporting this breakpoint to GDB, so apply the
@@ -2629,7 +2637,7 @@ Check if we're already there.\n",
         going to keep waiting, so use proceed, which handles stepping
         over the next breakpoint.  */
       if (debug_threads)
-       fprintf (stderr, "proceeding all threads.\n");
+       debug_printf ("proceeding all threads.\n");
 
       if (step_over_finished)
        unsuspend_all_lwps (event_child);
@@ -2643,16 +2651,16 @@ Check if we're already there.\n",
       if (current_inferior->last_resume_kind == resume_step)
        {
          if (event_child->step_range_start == event_child->step_range_end)
-           fprintf (stderr, "GDB wanted to single-step, reporting event.\n");
+           debug_printf ("GDB wanted to single-step, reporting event.\n");
          else if (!lwp_in_step_range (event_child))
-           fprintf (stderr, "Out of step range, reporting event.\n");
+           debug_printf ("Out of step range, reporting event.\n");
        }
       if (event_child->stopped_by_watchpoint)
-       fprintf (stderr, "Stopped by watchpoint.\n");
+       debug_printf ("Stopped by watchpoint.\n");
       if (gdb_breakpoint_here (event_child->stop_pc))
-       fprintf (stderr, "Stopped by GDB breakpoint.\n");
+       debug_printf ("Stopped by GDB breakpoint.\n");
       if (debug_threads)
-       fprintf (stderr, "Hit a non-gdbserver trap event.\n");
+       debug_printf ("Hit a non-gdbserver trap event.\n");
     }
 
   /* Alright, we're going to report a stop.  */
@@ -2730,10 +2738,12 @@ Check if we're already there.\n",
   gdb_assert (ptid_equal (step_over_bkpt, null_ptid));
 
   if (debug_threads)
-    fprintf (stderr, "linux_wait ret = %s, %d, %d\n",
-            target_pid_to_str (ptid_of (event_child)),
-            ourstatus->kind,
-            ourstatus->value.sig);
+    {
+      debug_printf ("linux_wait_1 ret = %s, %d, %d\n",
+                   target_pid_to_str (ptid_of (event_child)),
+                   ourstatus->kind, ourstatus->value.sig);
+      debug_exit ();
+    }
 
   return ptid_of (event_child);
 }
@@ -2772,9 +2782,6 @@ linux_wait (ptid_t ptid,
 {
   ptid_t event_ptid;
 
-  if (debug_threads)
-    fprintf (stderr, "linux_wait: [%s]\n", target_pid_to_str (ptid));
-
   /* Flush the async file first.  */
   if (target_is_async_p ())
     async_file_flush ();
@@ -2837,13 +2844,13 @@ send_sigstop (struct lwp_info *lwp)
   if (lwp->stop_expected)
     {
       if (debug_threads)
-       fprintf (stderr, "Have pending sigstop for lwp %d\n", pid);
+       debug_printf ("Have pending sigstop for lwp %d\n", pid);
 
       return;
     }
 
   if (debug_threads)
-    fprintf (stderr, "Sending sigstop to lwp %d\n", pid);
+    debug_printf ("Sending sigstop to lwp %d\n", pid);
 
   lwp->stop_expected = 1;
   kill_lwp (pid, SIGSTOP);
@@ -2912,8 +2919,8 @@ wait_for_sigstop (struct inferior_list_entry *entry)
   if (lwp->stopped)
     {
       if (debug_threads)
-       fprintf (stderr, "wait_for_sigstop: LWP %ld already stopped\n",
-                lwpid_of (lwp));
+       debug_printf ("wait_for_sigstop: LWP %ld already stopped\n",
+                     lwpid_of (lwp));
       return;
     }
 
@@ -2926,7 +2933,7 @@ wait_for_sigstop (struct inferior_list_entry *entry)
   ptid = lwp->head.id;
 
   if (debug_threads)
-    fprintf (stderr, "wait_for_sigstop: pulling one event\n");
+    debug_printf ("wait_for_sigstop: pulling one event\n");
 
   pid = linux_wait_for_event (ptid, &wstat, __WALL);
 
@@ -2936,14 +2943,14 @@ wait_for_sigstop (struct inferior_list_entry *entry)
   if (WIFSTOPPED (wstat))
     {
       if (debug_threads)
-       fprintf (stderr, "LWP %ld stopped with signal %d\n",
-                lwpid_of (lwp), WSTOPSIG (wstat));
+       debug_printf ("LWP %ld stopped with signal %d\n",
+                     lwpid_of (lwp), WSTOPSIG (wstat));
 
       if (WSTOPSIG (wstat) != SIGSTOP)
        {
          if (debug_threads)
-           fprintf (stderr, "LWP %ld stopped with non-sigstop status %06x\n",
-                    lwpid_of (lwp), wstat);
+           debug_printf ("LWP %ld stopped with non-sigstop status %06x\n",
+                         lwpid_of (lwp), wstat);
 
          lwp->status_pending_p = 1;
          lwp->status_pending = wstat;
@@ -2952,7 +2959,7 @@ wait_for_sigstop (struct inferior_list_entry *entry)
   else
     {
       if (debug_threads)
-       fprintf (stderr, "Process %d exited while stopping LWPs\n", pid);
+       debug_printf ("Process %d exited while stopping LWPs\n", pid);
 
       lwp = find_lwp_pid (pid_to_ptid (pid));
       if (lwp)
@@ -2973,7 +2980,7 @@ wait_for_sigstop (struct inferior_list_entry *entry)
   else
     {
       if (debug_threads)
-       fprintf (stderr, "Previously current thread died.\n");
+       debug_printf ("Previously current thread died.\n");
 
       if (non_stop)
        {
@@ -3032,9 +3039,8 @@ move_out_of_jump_pad_callback (struct inferior_list_entry *entry)
       && maybe_move_out_of_jump_pad (lwp, wstat))
     {
       if (debug_threads)
-       fprintf (stderr,
-                "LWP %ld needs stabilizing (in jump pad)\n",
-                lwpid_of (lwp));
+       debug_printf ("LWP %ld needs stabilizing (in jump pad)\n",
+                     lwpid_of (lwp));
 
       if (wstat)
        {
@@ -3042,10 +3048,9 @@ move_out_of_jump_pad_callback (struct inferior_list_entry *entry)
          enqueue_one_deferred_signal (lwp, wstat);
 
          if (debug_threads)
-           fprintf (stderr,
-                    "Signal %d for LWP %ld deferred "
-                    "(in jump pad)\n",
-                    WSTOPSIG (*wstat), lwpid_of (lwp));
+           debug_printf ("Signal %d for LWP %ld deferred "
+                         "(in jump pad)\n",
+                         WSTOPSIG (*wstat), lwpid_of (lwp));
        }
 
       linux_resume_one_lwp (lwp, 0, 0, NULL);
@@ -3076,6 +3081,16 @@ stop_all_lwps (int suspend, struct lwp_info *except)
   /* Should not be called recursively.  */
   gdb_assert (stopping_threads == NOT_STOPPING_THREADS);
 
+  if (debug_threads)
+    {
+      debug_enter ();
+      debug_printf ("stop_all_lwps (%s, except=%s)\n",
+                   suspend ? "stop-and-suspend" : "stop",
+                   except != NULL
+                   ? target_pid_to_str (ptid_of (except))
+                   : "none");
+    }
+
   stopping_threads = (suspend
                      ? STOPPING_AND_SUSPENDING_THREADS
                      : STOPPING_THREADS);
@@ -3086,6 +3101,13 @@ stop_all_lwps (int suspend, struct lwp_info *except)
     find_inferior (&all_lwps, send_sigstop_callback, except);
   for_each_inferior (&all_lwps, wait_for_sigstop);
   stopping_threads = NOT_STOPPING_THREADS;
+
+  if (debug_threads)
+    {
+      debug_printf ("stop_all_lwps done, setting stopping_threads "
+                   "back to !stopping\n");
+      debug_exit ();
+    }
 }
 
 /* Resume execution of the inferior process.
@@ -3138,10 +3160,10 @@ linux_resume_one_lwp (struct lwp_info *lwp,
   if (lwp->status_pending_p)
     {
       if (debug_threads)
-       fprintf (stderr, "Not resuming lwp %ld (%s, signal %d, stop %s);"
-                " has pending status\n",
-                lwpid_of (lwp), step ? "step" : "continue", signal,
-                lwp->stop_expected ? "expected" : "not expected");
+       debug_printf ("Not resuming lwp %ld (%s, signal %d, stop %s);"
+                     " has pending status\n",
+                     lwpid_of (lwp), step ? "step" : "continue", signal,
+                     lwp->stop_expected ? "expected" : "not expected");
       return;
     }
 
@@ -3149,9 +3171,9 @@ linux_resume_one_lwp (struct lwp_info *lwp,
   current_inferior = get_lwp_thread (lwp);
 
   if (debug_threads)
-    fprintf (stderr, "Resuming lwp %ld (%s, signal %d, stop %s)\n",
-            lwpid_of (lwp), step ? "step" : "continue", signal,
-            lwp->stop_expected ? "expected" : "not expected");
+    debug_printf ("Resuming lwp %ld (%s, signal %d, stop %s)\n",
+                 lwpid_of (lwp), step ? "step" : "continue", signal,
+                 lwp->stop_expected ? "expected" : "not expected");
 
   /* This bit needs some thinking about.  If we get a signal that
      we must report while a single-step reinsert is still pending,
@@ -3166,8 +3188,8 @@ linux_resume_one_lwp (struct lwp_info *lwp,
   if (lwp->bp_reinsert != 0)
     {
       if (debug_threads)
-       fprintf (stderr, "  pending reinsert at 0x%s\n",
-                paddress (lwp->bp_reinsert));
+       debug_printf ("  pending reinsert at 0x%s\n",
+                     paddress (lwp->bp_reinsert));
 
       if (can_hardware_single_step ())
        {
@@ -3190,9 +3212,9 @@ linux_resume_one_lwp (struct lwp_info *lwp,
   if (fast_tp_collecting == 1)
     {
       if (debug_threads)
-       fprintf (stderr, "\
-lwp %ld wants to get out of fast tracepoint jump pad (exit-jump-pad-bkpt)\n",
-                lwpid_of (lwp));
+       debug_printf ("lwp %ld wants to get out of fast tracepoint jump pad"
+                     " (exit-jump-pad-bkpt)\n",
+                     lwpid_of (lwp));
 
       /* Postpone any pending signal.  It was enqueued above.  */
       signal = 0;
@@ -3200,9 +3222,9 @@ lwp %ld wants to get out of fast tracepoint jump pad (exit-jump-pad-bkpt)\n",
   else if (fast_tp_collecting == 2)
     {
       if (debug_threads)
-       fprintf (stderr, "\
-lwp %ld wants to get out of fast tracepoint jump pad single-stepping\n",
-                lwpid_of (lwp));
+       debug_printf ("lwp %ld wants to get out of fast tracepoint jump pad"
+                     " single-stepping\n",
+                     lwpid_of (lwp));
 
       if (can_hardware_single_step ())
        step = 1;
@@ -3226,9 +3248,8 @@ lwp %ld wants to get out of fast tracepoint jump pad single-stepping\n",
       && can_hardware_single_step ())
     {
       if (debug_threads)
-       fprintf (stderr,
-                "lwp %ld has a while-stepping action -> forcing step.\n",
-                lwpid_of (lwp));
+       debug_printf ("lwp %ld has a while-stepping action -> forcing step.\n",
+                     lwpid_of (lwp));
       step = 1;
     }
 
@@ -3236,7 +3257,7 @@ lwp %ld wants to get out of fast tracepoint jump pad single-stepping\n",
     {
       struct regcache *regcache = get_thread_regcache (current_inferior, 1);
       CORE_ADDR pc = (*the_low_target.get_pc) (regcache);
-      fprintf (stderr, "  resuming from pc 0x%lx\n", (long) pc);
+      debug_printf ("  resuming from pc 0x%lx\n", (long) pc);
     }
 
   /* If we have pending signals, consume one unless we are trying to
@@ -3334,11 +3355,12 @@ linux_set_resume_request (struct inferior_list_entry *entry, void *arg)
              && thread->last_resume_kind == resume_stop)
            {
              if (debug_threads)
-               fprintf (stderr, "already %s LWP %ld at GDB's request\n",
-                        thread->last_status.kind == TARGET_WAITKIND_STOPPED
-                        ? "stopped"
-                        : "stopping",
-                        lwpid_of (lwp));
+               debug_printf ("already %s LWP %ld at GDB's request\n",
+                             (thread->last_status.kind
+                              == TARGET_WAITKIND_STOPPED)
+                             ? "stopped"
+                             : "stopping",
+                             lwpid_of (lwp));
 
              continue;
            }
@@ -3359,10 +3381,9 @@ linux_set_resume_request (struct inferior_list_entry *entry, void *arg)
              lwp->status_pending_p = 1;
 
              if (debug_threads)
-               fprintf (stderr,
-                        "Dequeueing deferred signal %d for LWP %ld, "
-                        "leaving status pending.\n",
-                        WSTOPSIG (lwp->status_pending), lwpid_of (lwp));
+               debug_printf ("Dequeueing deferred signal %d for LWP %ld, "
+                             "leaving status pending.\n",
+                             WSTOPSIG (lwp->status_pending), lwpid_of (lwp));
            }
 
          return 0;
@@ -3413,9 +3434,8 @@ need_step_over_p (struct inferior_list_entry *entry, void *dummy)
   if (!lwp->stopped)
     {
       if (debug_threads)
-       fprintf (stderr,
-                "Need step over [LWP %ld]? Ignoring, not stopped\n",
-                lwpid_of (lwp));
+       debug_printf ("Need step over [LWP %ld]? Ignoring, not stopped\n",
+                     lwpid_of (lwp));
       return 0;
     }
 
@@ -3424,9 +3444,9 @@ need_step_over_p (struct inferior_list_entry *entry, void *dummy)
   if (thread->last_resume_kind == resume_stop)
     {
       if (debug_threads)
-       fprintf (stderr,
-                "Need step over [LWP %ld]? Ignoring, should remain stopped\n",
-                lwpid_of (lwp));
+       debug_printf ("Need step over [LWP %ld]? Ignoring, should remain"
+                     " stopped\n",
+                     lwpid_of (lwp));
       return 0;
     }
 
@@ -3435,25 +3455,23 @@ need_step_over_p (struct inferior_list_entry *entry, void *dummy)
   if (lwp->suspended)
     {
       if (debug_threads)
-       fprintf (stderr,
-                "Need step over [LWP %ld]? Ignoring, suspended\n",
-                lwpid_of (lwp));
+       debug_printf ("Need step over [LWP %ld]? Ignoring, suspended\n",
+                     lwpid_of (lwp));
       return 0;
     }
 
   if (!lwp->need_step_over)
     {
       if (debug_threads)
-       fprintf (stderr,
-                "Need step over [LWP %ld]? No\n", lwpid_of (lwp));
+       debug_printf ("Need step over [LWP %ld]? No\n", lwpid_of (lwp));
     }
 
   if (lwp->status_pending_p)
     {
       if (debug_threads)
-       fprintf (stderr,
-                "Need step over [LWP %ld]? Ignoring, has pending status.\n",
-                lwpid_of (lwp));
+       debug_printf ("Need step over [LWP %ld]? Ignoring, has pending"
+                     " status.\n",
+                     lwpid_of (lwp));
       return 0;
     }
 
@@ -3469,10 +3487,9 @@ need_step_over_p (struct inferior_list_entry *entry, void *dummy)
   if (pc != lwp->stop_pc)
     {
       if (debug_threads)
-       fprintf (stderr,
-                "Need step over [LWP %ld]? Cancelling, PC was changed.  "
-                "Old stop_pc was 0x%s, PC is now 0x%s\n",
-                lwpid_of (lwp), paddress (lwp->stop_pc), paddress (pc));
+       debug_printf ("Need step over [LWP %ld]? Cancelling, PC was changed. "
+                     "Old stop_pc was 0x%s, PC is now 0x%s\n",
+                     lwpid_of (lwp), paddress (lwp->stop_pc), paddress (pc));
 
       lwp->need_step_over = 0;
       return 0;
@@ -3492,10 +3509,9 @@ need_step_over_p (struct inferior_list_entry *entry, void *dummy)
          && gdb_no_commands_at_breakpoint (pc))
        {
          if (debug_threads)
-           fprintf (stderr,
-                    "Need step over [LWP %ld]? yes, but found"
-                    " GDB breakpoint at 0x%s; skipping step over\n",
-                    lwpid_of (lwp), paddress (pc));
+           debug_printf ("Need step over [LWP %ld]? yes, but found"
+                         " GDB breakpoint at 0x%s; skipping step over\n",
+                         lwpid_of (lwp), paddress (pc));
 
          current_inferior = saved_inferior;
          return 0;
@@ -3503,10 +3519,9 @@ need_step_over_p (struct inferior_list_entry *entry, void *dummy)
       else
        {
          if (debug_threads)
-           fprintf (stderr,
-                    "Need step over [LWP %ld]? yes, "
-                    "found breakpoint at 0x%s\n",
-                    lwpid_of (lwp), paddress (pc));
+           debug_printf ("Need step over [LWP %ld]? yes, "
+                         "found breakpoint at 0x%s\n",
+                         lwpid_of (lwp), paddress (pc));
 
          /* We've found an lwp that needs stepping over --- return 1 so
             that find_inferior stops looking.  */
@@ -3521,9 +3536,9 @@ need_step_over_p (struct inferior_list_entry *entry, void *dummy)
   current_inferior = saved_inferior;
 
   if (debug_threads)
-    fprintf (stderr,
-            "Need step over [LWP %ld]? No, no breakpoint found at 0x%s\n",
-            lwpid_of (lwp), paddress (pc));
+    debug_printf ("Need step over [LWP %ld]? No, no breakpoint found"
+                 " at 0x%s\n",
+                 lwpid_of (lwp), paddress (pc));
 
   return 0;
 }
@@ -3554,15 +3569,14 @@ start_step_over (struct lwp_info *lwp)
   int step;
 
   if (debug_threads)
-    fprintf (stderr,
-            "Starting step-over on LWP %ld.  Stopping all threads\n",
-            lwpid_of (lwp));
+    debug_printf ("Starting step-over on LWP %ld.  Stopping all threads\n",
+                 lwpid_of (lwp));
 
   stop_all_lwps (1, lwp);
   gdb_assert (lwp->suspended == 0);
 
   if (debug_threads)
-    fprintf (stderr, "Done stopping all threads for step-over.\n");
+    debug_printf ("Done stopping all threads for step-over.\n");
 
   /* Note, we should always reach here with an already adjusted PC,
      either by GDB (if we're resuming due to GDB's request), or by our
@@ -3607,7 +3621,7 @@ finish_step_over (struct lwp_info *lwp)
   if (lwp->bp_reinsert != 0)
     {
       if (debug_threads)
-       fprintf (stderr, "Finished step over.\n");
+       debug_printf ("Finished step over.\n");
 
       /* Reinsert any breakpoint at LWP->BP_REINSERT.  Note that there
         may be no breakpoint to reinsert there by now.  */
@@ -3662,12 +3676,12 @@ linux_resume_one_thread (struct inferior_list_entry *entry, void *arg)
   if (lwp->resume->kind == resume_stop)
     {
       if (debug_threads)
-       fprintf (stderr, "resume_stop request for LWP %ld\n", lwpid_of (lwp));
+       debug_printf ("resume_stop request for LWP %ld\n", lwpid_of (lwp));
 
       if (!lwp->stopped)
        {
          if (debug_threads)
-           fprintf (stderr, "stopping LWP %ld\n", lwpid_of (lwp));
+           debug_printf ("stopping LWP %ld\n", lwpid_of (lwp));
 
          /* Stop the thread, and wait for the event asynchronously,
             through the event loop.  */
@@ -3676,8 +3690,8 @@ linux_resume_one_thread (struct inferior_list_entry *entry, void *arg)
       else
        {
          if (debug_threads)
-           fprintf (stderr, "already stopped LWP %ld\n",
-                    lwpid_of (lwp));
+           debug_printf ("already stopped LWP %ld\n",
+                         lwpid_of (lwp));
 
          /* The LWP may have been stopped in an internal event that
             was not meant to be notified back to GDB (e.g., gdbserver
@@ -3719,7 +3733,7 @@ linux_resume_one_thread (struct inferior_list_entry *entry, void *arg)
   if (!leave_pending)
     {
       if (debug_threads)
-       fprintf (stderr, "resuming LWP %ld\n", lwpid_of (lwp));
+       debug_printf ("resuming LWP %ld\n", lwpid_of (lwp));
 
       step = (lwp->resume->kind == resume_step);
       linux_resume_one_lwp (lwp, step, lwp->resume->sig, NULL);
@@ -3727,7 +3741,7 @@ linux_resume_one_thread (struct inferior_list_entry *entry, void *arg)
   else
     {
       if (debug_threads)
-       fprintf (stderr, "leaving LWP %ld stopped\n", lwpid_of (lwp));
+       debug_printf ("leaving LWP %ld stopped\n", lwpid_of (lwp));
 
       /* If we have a new signal, enqueue the signal.  */
       if (lwp->resume->sig != 0)
@@ -3764,6 +3778,12 @@ linux_resume (struct thread_resume *resume_info, size_t n)
   int any_pending;
   int leave_all_stopped;
 
+  if (debug_threads)
+    {
+      debug_enter ();
+      debug_printf ("linux_resume:\n");
+    }
+
   find_inferior (&all_threads, linux_set_resume_request, &array);
 
   /* If there is a thread which would otherwise be resumed, which has
@@ -3792,13 +3812,12 @@ linux_resume (struct thread_resume *resume_info, size_t n)
   if (debug_threads)
     {
       if (need_step_over != NULL)
-       fprintf (stderr, "Not resuming all, need step over\n");
+       debug_printf ("Not resuming all, need step over\n");
       else if (any_pending)
-       fprintf (stderr,
-                "Not resuming, all-stop and found "
-                "an LWP with pending status\n");
+       debug_printf ("Not resuming, all-stop and found "
+                     "an LWP with pending status\n");
       else
-       fprintf (stderr, "Resuming, no pending status or step over needed\n");
+       debug_printf ("Resuming, no pending status or step over needed\n");
     }
 
   /* Even if we're leaving threads stopped, queue all signals we'd
@@ -3807,6 +3826,12 @@ linux_resume (struct thread_resume *resume_info, size_t n)
 
   if (need_step_over)
     start_step_over (need_step_over);
+
+  if (debug_threads)
+    {
+      debug_printf ("linux_resume done\n");
+      debug_exit ();
+    }
 }
 
 /* This function is called once per thread.  We check the thread's
@@ -3829,13 +3854,12 @@ proceed_one_lwp (struct inferior_list_entry *entry, void *except)
     return 0;
 
   if (debug_threads)
-    fprintf (stderr,
-            "proceed_one_lwp: lwp %ld\n", lwpid_of (lwp));
+    debug_printf ("proceed_one_lwp: lwp %ld\n", lwpid_of (lwp));
 
   if (!lwp->stopped)
     {
       if (debug_threads)
-       fprintf (stderr, "   LWP %ld already running\n", lwpid_of (lwp));
+       debug_printf ("   LWP %ld already running\n", lwpid_of (lwp));
       return 0;
     }
 
@@ -3845,16 +3869,16 @@ proceed_one_lwp (struct inferior_list_entry *entry, void *except)
       && thread->last_status.kind != TARGET_WAITKIND_IGNORE)
     {
       if (debug_threads)
-       fprintf (stderr, "   client wants LWP to remain %ld stopped\n",
-                lwpid_of (lwp));
+       debug_printf ("   client wants LWP to remain %ld stopped\n",
+                     lwpid_of (lwp));
       return 0;
     }
 
   if (lwp->status_pending_p)
     {
       if (debug_threads)
-       fprintf (stderr, "   LWP %ld has pending status, leaving stopped\n",
-                lwpid_of (lwp));
+       debug_printf ("   LWP %ld has pending status, leaving stopped\n",
+                     lwpid_of (lwp));
       return 0;
     }
 
@@ -3863,7 +3887,7 @@ proceed_one_lwp (struct inferior_list_entry *entry, void *except)
   if (lwp->suspended)
     {
       if (debug_threads)
-       fprintf (stderr, "   LWP %ld is suspended\n", lwpid_of (lwp));
+       debug_printf ("   LWP %ld is suspended\n", lwpid_of (lwp));
       return 0;
     }
 
@@ -3882,10 +3906,9 @@ proceed_one_lwp (struct inferior_list_entry *entry, void *except)
         pending, this is a no-op.  */
 
       if (debug_threads)
-       fprintf (stderr,
-                "Client wants LWP %ld to stop. "
-                "Making sure it has a SIGSTOP pending\n",
-                lwpid_of (lwp));
+       debug_printf ("Client wants LWP %ld to stop. "
+                     "Making sure it has a SIGSTOP pending\n",
+                     lwpid_of (lwp));
 
       send_sigstop (lwp);
     }
@@ -3932,9 +3955,9 @@ proceed_all_lwps (void)
       if (need_step_over != NULL)
        {
          if (debug_threads)
-           fprintf (stderr, "proceed_all_lwps: found "
-                    "thread %ld needing a step-over\n",
-                    lwpid_of (need_step_over));
+           debug_printf ("proceed_all_lwps: found "
+                         "thread %ld needing a step-over\n",
+                         lwpid_of (need_step_over));
 
          start_step_over (need_step_over);
          return;
@@ -3942,7 +3965,7 @@ proceed_all_lwps (void)
     }
 
   if (debug_threads)
-    fprintf (stderr, "Proceeding, no step-over needed\n");
+    debug_printf ("Proceeding, no step-over needed\n");
 
   find_inferior (&all_lwps, proceed_one_lwp, NULL);
 }
@@ -3956,18 +3979,24 @@ unstop_all_lwps (int unsuspend, struct lwp_info *except)
 {
   if (debug_threads)
     {
+      debug_enter ();
       if (except)
-       fprintf (stderr,
-                "unstopping all lwps, except=(LWP %ld)\n", lwpid_of (except));
+       debug_printf ("unstopping all lwps, except=(LWP %ld)\n",
+                     lwpid_of (except));
       else
-       fprintf (stderr,
-                "unstopping all lwps\n");
+       debug_printf ("unstopping all lwps\n");
     }
 
   if (unsuspend)
     find_inferior (&all_lwps, unsuspend_and_proceed_one_lwp, except);
   else
     find_inferior (&all_lwps, proceed_one_lwp, except);
+
+  if (debug_threads)
+    {
+      debug_printf ("unstop_all_lwps done\n");
+      debug_exit ();
+    }
 }
 
 
@@ -4533,8 +4562,8 @@ linux_write_memory (CORE_ADDR memaddr, const unsigned char *myaddr, int len)
        val = val & 0xffff;
       else if (len == 3)
        val = val & 0xffffff;
-      fprintf (stderr, "Writing %0*x to 0x%08lx\n", 2 * ((len < 4) ? len : 4),
-              val, (long)memaddr);
+      debug_printf ("Writing %0*x to 0x%08lx\n", 2 * ((len < 4) ? len : 4),
+                   val, (long)memaddr);
     }
 
   /* Fill start and end extra bytes of buffer with existing memory data.  */
@@ -4777,9 +4806,9 @@ linux_xfer_siginfo (const char *annex, unsigned char *readbuf,
   pid = lwpid_of (get_thread_lwp (current_inferior));
 
   if (debug_threads)
-    fprintf (stderr, "%s siginfo for lwp %d.\n",
-            readbuf != NULL ? "Reading" : "Writing",
-            pid);
+    debug_printf ("%s siginfo for lwp %d.\n",
+                 readbuf != NULL ? "Reading" : "Writing",
+                 pid);
 
   if (offset >= sizeof (siginfo))
     return -1;
@@ -4851,8 +4880,8 @@ linux_async (int enable)
   int previous = (linux_event_pipe[0] != -1);
 
   if (debug_threads)
-    fprintf (stderr, "linux_async (%d), previous=%d\n",
-            enable, previous);
+    debug_printf ("linux_async (%d), previous=%d\n",
+                 enable, previous);
 
   if (previous != enable)
     {
index e20852366fefba574c78fa0779e5104e2b478225..477e185f50fb795fac2b9c983fd24131dd370c65 100644 (file)
@@ -59,7 +59,7 @@ m32r_get_pc (struct regcache *regcache)
   unsigned long pc;
   collect_register_by_name (regcache, "pc", &pc);
   if (debug_threads)
-    fprintf (stderr, "stop pc is %08lx\n", pc);
+    debug_printf ("stop pc is %08lx\n", pc);
   return pc;
 }
 
index fcffe05dcec847e41564003a3c2f37bf4ddee72d..75c97ddebef91772a4f51bf00d528ea7bb938989 100644 (file)
@@ -231,7 +231,7 @@ sparc_get_pc (struct regcache *regcache)
   unsigned long pc;
   collect_register_by_name (regcache, "pc", &pc);
   if (debug_threads)
-    fprintf (stderr, "stop pc is %08lx\n", pc);
+    debug_printf ("stop pc is %08lx\n", pc);
   return pc;
 }
 
index e1daa7da7fe582ca62cd52fa66dcf68607cd9c1c..1e590e84ef251506fb8348ac7aed6b1f3ff0f22a 100644 (file)
@@ -2027,8 +2027,8 @@ add_insns (unsigned char *start, int len)
   CORE_ADDR buildaddr = current_insn_ptr;
 
   if (debug_threads)
-    fprintf (stderr, "Adding %d bytes of insn at %s\n",
-            len, paddress (buildaddr));
+    debug_printf ("Adding %d bytes of insn at %s\n",
+                 len, paddress (buildaddr));
 
   append_insns (&buildaddr, len, start);
   current_insn_ptr = buildaddr;
index fd706e90937119121189651fd2949f2a0357aa0b..b77ce100ff499aa249ed8d2391ca4f25c6c5f0b2 100644 (file)
@@ -197,10 +197,9 @@ set_raw_breakpoint_at (CORE_ADDR where)
   if (err != 0)
     {
       if (debug_threads)
-       fprintf (stderr,
-                "Failed to read shadow memory of"
-                " breakpoint at 0x%s (%s).\n",
-                paddress (where), strerror (err));
+       debug_printf ("Failed to read shadow memory of"
+                     " breakpoint at 0x%s (%s).\n",
+                     paddress (where), strerror (err));
       free (bp);
       return NULL;
     }
@@ -211,9 +210,8 @@ set_raw_breakpoint_at (CORE_ADDR where)
   if (err != 0)
     {
       if (debug_threads)
-       fprintf (stderr,
-                "Failed to insert breakpoint at 0x%s (%s).\n",
-                paddress (where), strerror (err));
+       debug_printf ("Failed to insert breakpoint at 0x%s (%s).\n",
+                     paddress (where), strerror (err));
       free (bp);
       return NULL;
     }
@@ -334,10 +332,9 @@ delete_fast_tracepoint_jump (struct fast_tracepoint_jump *todel)
                  *bp_link = prev_bp_link;
 
                  if (debug_threads)
-                   fprintf (stderr,
-                            "Failed to uninsert fast tracepoint jump "
-                            "at 0x%s (%s) while deleting it.\n",
-                            paddress (bp->pc), strerror (ret));
+                   debug_printf ("Failed to uninsert fast tracepoint jump "
+                                 "at 0x%s (%s) while deleting it.\n",
+                                 paddress (bp->pc), strerror (ret));
                  return ret;
                }
 
@@ -398,10 +395,9 @@ set_fast_tracepoint_jump (CORE_ADDR where,
   if (err != 0)
     {
       if (debug_threads)
-       fprintf (stderr,
-                "Failed to read shadow memory of"
-                " fast tracepoint at 0x%s (%s).\n",
-                paddress (where), strerror (err));
+       debug_printf ("Failed to read shadow memory of"
+                     " fast tracepoint at 0x%s (%s).\n",
+                     paddress (where), strerror (err));
       free (jp);
       return NULL;
     }
@@ -424,9 +420,8 @@ set_fast_tracepoint_jump (CORE_ADDR where,
   if (err != 0)
     {
       if (debug_threads)
-       fprintf (stderr,
-                "Failed to insert fast tracepoint jump at 0x%s (%s).\n",
-                paddress (where), strerror (err));
+       debug_printf ("Failed to insert fast tracepoint jump at 0x%s (%s).\n",
+                     paddress (where), strerror (err));
 
       /* Unlink it.  */
       proc->fast_tracepoint_jumps = jp->next;
@@ -450,10 +445,9 @@ uninsert_fast_tracepoint_jumps_at (CORE_ADDR pc)
       /* This can happen when we remove all breakpoints while handling
         a step-over.  */
       if (debug_threads)
-       fprintf (stderr,
-                "Could not find fast tracepoint jump at 0x%s "
-                "in list (uninserting).\n",
-                paddress (pc));
+       debug_printf ("Could not find fast tracepoint jump at 0x%s "
+                     "in list (uninserting).\n",
+                     paddress (pc));
       return;
     }
 
@@ -480,9 +474,9 @@ uninsert_fast_tracepoint_jumps_at (CORE_ADDR pc)
          jp->inserted = 1;
 
          if (debug_threads)
-           fprintf (stderr,
-                    "Failed to uninsert fast tracepoint jump at 0x%s (%s).\n",
-                    paddress (pc), strerror (err));
+           debug_printf ("Failed to uninsert fast tracepoint jump at"
+                         " 0x%s (%s).\n",
+                         paddress (pc), strerror (err));
        }
     }
 }
@@ -500,10 +494,9 @@ reinsert_fast_tracepoint_jumps_at (CORE_ADDR where)
       /* This can happen when we remove breakpoints when a tracepoint
         hit causes a tracing stop, while handling a step-over.  */
       if (debug_threads)
-       fprintf (stderr,
-                "Could not find fast tracepoint jump at 0x%s "
-                "in list (reinserting).\n",
-                paddress (where));
+       debug_printf ("Could not find fast tracepoint jump at 0x%s "
+                     "in list (reinserting).\n",
+                     paddress (where));
       return;
     }
 
@@ -528,9 +521,9 @@ reinsert_fast_tracepoint_jumps_at (CORE_ADDR where)
       jp->inserted = 0;
 
       if (debug_threads)
-       fprintf (stderr,
-                "Failed to reinsert fast tracepoint jump at 0x%s (%s).\n",
-                paddress (where), strerror (err));
+       debug_printf ("Failed to reinsert fast tracepoint jump at"
+                     " 0x%s (%s).\n",
+                     paddress (where), strerror (err));
     }
 }
 
@@ -598,10 +591,9 @@ delete_raw_breakpoint (struct process_info *proc, struct raw_breakpoint *todel)
                  *bp_link = prev_bp_link;
 
                  if (debug_threads)
-                   fprintf (stderr,
-                            "Failed to uninsert raw breakpoint "
-                            "at 0x%s (%s) while deleting it.\n",
-                            paddress (bp->pc), strerror (ret));
+                   debug_printf ("Failed to uninsert raw breakpoint "
+                                 "at 0x%s (%s) while deleting it.\n",
+                                 paddress (bp->pc), strerror (ret));
                  return ret;
                }
 
@@ -932,9 +924,9 @@ gdb_no_commands_at_breakpoint (CORE_ADDR where)
     return 0;
 
   if (debug_threads)
-    fprintf (stderr, "at 0x%s, bp command_list is 0x%s\n",
-            paddress (where),
-            phex_nz ((uintptr_t) bp->command_list, 0));
+    debug_printf ("at 0x%s, bp command_list is 0x%s\n",
+                 paddress (where),
+                 phex_nz ((uintptr_t) bp->command_list, 0));
   return (bp->command_list == NULL);
 }
 
@@ -1034,9 +1026,8 @@ uninsert_raw_breakpoint (struct raw_breakpoint *bp)
          bp->inserted = 1;
 
          if (debug_threads)
-           fprintf (stderr,
-                    "Failed to uninsert raw breakpoint at 0x%s (%s).\n",
-                    paddress (bp->pc), strerror (err));
+           debug_printf ("Failed to uninsert raw breakpoint at 0x%s (%s).\n",
+                         paddress (bp->pc), strerror (err));
        }
     }
 }
@@ -1052,10 +1043,9 @@ uninsert_breakpoints_at (CORE_ADDR pc)
       /* This can happen when we remove all breakpoints while handling
         a step-over.  */
       if (debug_threads)
-       fprintf (stderr,
-                "Could not find breakpoint at 0x%s "
-                "in list (uninserting).\n",
-                paddress (pc));
+       debug_printf ("Could not find breakpoint at 0x%s "
+                     "in list (uninserting).\n",
+                     paddress (pc));
       return;
     }
 
@@ -1087,9 +1077,8 @@ reinsert_raw_breakpoint (struct raw_breakpoint *bp)
   if (err == 0)
     bp->inserted = 1;
   else if (debug_threads)
-    fprintf (stderr,
-            "Failed to reinsert breakpoint at 0x%s (%s).\n",
-            paddress (bp->pc), strerror (err));
+    debug_printf ("Failed to reinsert breakpoint at 0x%s (%s).\n",
+                 paddress (bp->pc), strerror (err));
 }
 
 void
@@ -1103,10 +1092,9 @@ reinsert_breakpoints_at (CORE_ADDR pc)
       /* This can happen when we remove all breakpoints while handling
         a step-over.  */
       if (debug_threads)
-       fprintf (stderr,
-                "Could not find raw breakpoint at 0x%s "
-                "in list (reinserting).\n",
-                paddress (pc));
+       debug_printf ("Could not find raw breakpoint at 0x%s "
+                     "in list (reinserting).\n",
+                     paddress (pc));
       return;
     }
 
index 75ace6ec5d7da711e7b181456ad7e2475517e171..3b88995962daa04efc1cb9b9e63fffc0efdf3c4d 100644 (file)
@@ -161,7 +161,7 @@ handle_accept_event (int err, gdb_client_data client_data)
   socklen_t tmp;
 
   if (debug_threads)
-    fprintf (stderr, "handling possible accept event\n");
+    debug_printf ("handling possible accept event\n");
 
   tmp = sizeof (sockaddr);
   remote_desc = accept (listen_desc, (struct sockaddr *) &sockaddr, &tmp);
@@ -1315,8 +1315,8 @@ prepare_resume_reply (char *buf, ptid_t ptid,
                      struct target_waitstatus *status)
 {
   if (debug_threads)
-    fprintf (stderr, "Writing resume reply for %s:%d\n",
-            target_pid_to_str (ptid), status->kind);
+    debug_printf ("Writing resume reply for %s:%d\n",
+                 target_pid_to_str (ptid), status->kind);
 
   switch (status->kind)
     {
index 28ea0482eaba9ba4856fcf4ed36133dc2cd5cd95..88354be1fbe8343b6f39df4999f0401877172343 100644 (file)
 #include "notif.h"
 #include "tdesc.h"
 
+#include <ctype.h>
 #include <unistd.h>
 #if HAVE_SIGNAL_H
 #include <signal.h>
 #endif
+#include "gdb_vecs.h"
 #include "gdb_wait.h"
 #include "btrace-common.h"
 #include "filestuff.h"
@@ -66,10 +68,6 @@ int disable_randomization = 1;
 
 static char **program_argv, **wrapper_argv;
 
-/* Enable miscellaneous debugging output.  The name is historical - it
-   was originally used to debug LinuxThreads support.  */
-int debug_threads;
-
 /* Enable debugging of h/w breakpoint/watchpoint support.  */
 int debug_hw_points;
 
@@ -222,8 +220,8 @@ start_inferior (char **argv)
     {
       int i;
       for (i = 0; new_argv[i]; ++i)
-       fprintf (stderr, "new_argv[%d] = \"%s\"\n", i, new_argv[i]);
-      fflush (stderr);
+       debug_printf ("new_argv[%d] = \"%s\"\n", i, new_argv[i]);
+      debug_flush ();
     }
 
 #ifdef SIGTTOU
@@ -694,6 +692,13 @@ monitor_show_help (void)
   monitor_output ("    Enable h/w breakpoint/watchpoint debugging messages\n");
   monitor_output ("  set remote-debug <0|1>\n");
   monitor_output ("    Enable remote protocol debugging messages\n");
+  monitor_output ("  set debug-format option1[,option2,...]\n");
+  monitor_output ("    Add additional information to debugging messages\n");
+  monitor_output ("    Options: all, none");
+#ifdef HAVE_GETTIMEOFDAY
+  monitor_output (", timestamp");
+#endif
+  monitor_output ("\n");
   monitor_output ("  exit\n");
   monitor_output ("    Quit GDBserver\n");
 }
@@ -921,6 +926,80 @@ handle_search_memory (char *own_buf, int packet_len)
       return;                                  \
     }
 
+/* Parse options to --debug-format= and "monitor set debug-format".
+   ARG is the text after "--debug-format=" or "monitor set debug-format".
+   IS_MONITOR is non-zero if we're invoked via "monitor set debug-format".
+   This triggers calls to monitor_output.
+   The result is NULL if all options were parsed ok, otherwise an error
+   message which the caller must free.
+
+   N.B. These commands affect all debug format settings, they are not
+   cumulative.  If a format is not specified, it is turned off.
+   However, we don't go to extra trouble with things like
+   "monitor set debug-format all,none,timestamp".
+   Instead we just parse them one at a time, in order.
+
+   The syntax for "monitor set debug" we support here is not identical
+   to gdb's "set debug foo on|off" because we also use this function to
+   parse "--debug-format=foo,bar".  */
+
+static char *
+parse_debug_format_options (const char *arg, int is_monitor)
+{
+  VEC (char_ptr) *options;
+  int ix;
+  char *option;
+
+  /* First turn all debug format options off.  */
+  debug_timestamp = 0;
+
+  /* First remove leading spaces, for "monitor set debug-format".  */
+  while (isspace (*arg))
+    ++arg;
+
+  options = delim_string_to_char_ptr_vec (arg, ',');
+
+  for (ix = 0; VEC_iterate (char_ptr, options, ix, option); ++ix)
+    {
+      if (strcmp (option, "all") == 0)
+       {
+         debug_timestamp = 1;
+         if (is_monitor)
+           monitor_output ("All extra debug format options enabled.\n");
+       }
+      else if (strcmp (option, "none") == 0)
+       {
+         debug_timestamp = 0;
+         if (is_monitor)
+           monitor_output ("All extra debug format options disabled.\n");
+       }
+#ifdef HAVE_GETTIMEOFDAY
+      else if (strcmp (option, "timestamp") == 0)
+       {
+         debug_timestamp = 1;
+         if (is_monitor)
+           monitor_output ("Timestamps will be added to debug output.\n");
+       }
+#endif
+      else if (*option == '\0')
+       {
+         /* An empty option, e.g., "--debug-format=foo,,bar", is ignored.  */
+         continue;
+       }
+      else
+       {
+         char *msg = xstrprintf ("Unknown debug-format argument: \"%s\"\n",
+                                 option);
+
+         free_char_ptr_vec (options);
+         return msg;
+       }
+    }
+
+  free_char_ptr_vec (options);
+  return NULL;
+}
+
 /* Handle monitor commands not handled by target-specific handlers.  */
 
 static void
@@ -956,6 +1035,21 @@ handle_monitor_command (char *mon, char *own_buf)
       remote_debug = 0;
       monitor_output ("Protocol debug output disabled.\n");
     }
+  else if (strncmp (mon, "set debug-format ",
+                   sizeof ("set debug-format ") - 1) == 0)
+    {
+      char *error_msg
+       = parse_debug_format_options (mon + sizeof ("set debug-format ") - 1,
+                                     1);
+
+      if (error_msg != NULL)
+       {
+         monitor_output (error_msg);
+         monitor_show_help ();
+         write_enn (own_buf);
+         xfree (error_msg);
+       }
+    }
   else if (strcmp (mon, "help") == 0)
     monitor_show_help ();
   else if (strcmp (mon, "exit") == 0)
@@ -2535,10 +2629,9 @@ queue_stop_reply_callback (struct inferior_list_entry *entry, void *arg)
              char *status_string
                = target_waitstatus_to_string (&thread->last_status);
 
-             fprintf (stderr,
-                      "Reporting thread %s as already stopped with %s\n",
-                      target_pid_to_str (entry->id),
-                      status_string);
+             debug_printf ("Reporting thread %s as already stopped with %s\n",
+                           target_pid_to_str (entry->id),
+                           status_string);
 
              xfree (status_string);
            }
@@ -2733,6 +2826,14 @@ gdbserver_usage (FILE *stream)
           "\n"
           "Options:\n"
           "  --debug               Enable general debugging output.\n"
+          "  --debug-format=opt1[,opt2,...]\n"
+          "                        Specify extra content in debugging output.\n"
+          "                          Options:\n"
+          "                            all\n"
+          "                            none\n"
+#ifdef HAVE_GETTIMEOFDAY
+          "                            timestamp\n"
+#endif
           "  --remote-debug        Enable remote protocol debugging output.\n"
           "  --version             Display version information and exit.\n"
           "  --wrapper WRAPPER --  Run WRAPPER to start new programs.\n"
@@ -2908,6 +3009,20 @@ main (int argc, char *argv[])
        }
       else if (strcmp (*next_arg, "--debug") == 0)
        debug_threads = 1;
+      else if (strncmp (*next_arg,
+                       "--debug-format=",
+                       sizeof ("--debug-format=") - 1) == 0)
+       {
+         char *error_msg
+           = parse_debug_format_options ((*next_arg)
+                                         + sizeof ("--debug-format=") - 1, 0);
+
+         if (error_msg != NULL)
+           {
+             fprintf (stderr, "%s", error_msg);
+             exit (1);
+           }
+       }
       else if (strcmp (*next_arg, "--remote-debug") == 0)
        remote_debug = 1;
       else if (strcmp (*next_arg, "--disable-packet") == 0)
@@ -3204,14 +3319,14 @@ process_point_options (CORE_ADDR point_addr, char **packet)
        {
          /* Conditional expression.  */
          if (debug_threads)
-           fprintf (stderr, "Found breakpoint condition.\n");
+           debug_printf ("Found breakpoint condition.\n");
          add_breakpoint_condition (point_addr, &dataptr);
        }
       else if (strncmp (dataptr, "cmds:", strlen ("cmds:")) == 0)
        {
          dataptr += strlen ("cmds:");
          if (debug_threads)
-           fprintf (stderr, "Found breakpoint commands %s.\n", dataptr);
+           debug_printf ("Found breakpoint commands %s.\n", dataptr);
          persist = (*dataptr == '1');
          dataptr += 2;
          add_breakpoint_commands (point_addr, &dataptr, persist);
@@ -3316,7 +3431,7 @@ process_serial_event (void)
          if (!non_stop)
            {
              if (debug_threads)
-               fprintf (stderr, "Forcing non-stop mode\n");
+               debug_printf ("Forcing non-stop mode\n");
 
              non_stop = 1;
              start_non_stop (1);
@@ -3709,7 +3824,7 @@ int
 handle_serial_event (int err, gdb_client_data client_data)
 {
   if (debug_threads)
-    fprintf (stderr, "handling possible serial event\n");
+    debug_printf ("handling possible serial event\n");
 
   /* Really handle it.  */
   if (process_serial_event () < 0)
@@ -3728,7 +3843,7 @@ int
 handle_target_event (int err, gdb_client_data client_data)
 {
   if (debug_threads)
-    fprintf (stderr, "handling possible target event\n");
+    debug_printf ("handling possible target event\n");
 
   last_ptid = mywait (minus_one_ptid, &last_status,
                      TARGET_WNOHANG, 1);
@@ -3770,10 +3885,10 @@ handle_target_event (int err, gdb_client_data client_data)
              struct thread_resume resume_info;
 
              if (debug_threads)
-               fprintf (stderr,
-                        "GDB not connected; forwarding event %d for [%s]\n",
-                        (int) last_status.kind,
-                        target_pid_to_str (last_ptid));
+               debug_printf ("GDB not connected; forwarding event %d for"
+                             " [%s]\n",
+                             (int) last_status.kind,
+                             target_pid_to_str (last_ptid));
 
              resume_info.thread = last_ptid;
              resume_info.kind = resume_continue;
@@ -3781,9 +3896,9 @@ handle_target_event (int err, gdb_client_data client_data)
              (*the_target->resume) (&resume_info, 1);
            }
          else if (debug_threads)
-           fprintf (stderr, "GDB not connected; ignoring event %d for [%s]\n",
-                    (int) last_status.kind,
-                    target_pid_to_str (last_ptid));
+           debug_printf ("GDB not connected; ignoring event %d for [%s]\n",
+                         (int) last_status.kind,
+                         target_pid_to_str (last_ptid));
        }
       else
        {
index d82556acd4c095afd66686ad7e5ad9933779280a..6eb1a16bc7a0049cb47ce06a9f24b667a0386953 100644 (file)
@@ -110,7 +110,6 @@ extern ptid_t cont_thread;
 extern ptid_t general_thread;
 
 extern int server_waiting;
-extern int debug_threads;
 extern int debug_hw_points;
 extern int pass_signals[];
 extern int program_signals[];
@@ -146,6 +145,7 @@ extern int handle_target_event (int err, gdb_client_data client_data);
 
 #include "common-utils.h"
 #include "utils.h"
+#include "debug.h"
 
 #include "gdb_assert.h"
 
index 8b8f6ce80ac121e7e740f9f6a618cff485bcbf8e..df06b9dbbb9d77afb98eac5cc7e467499248e836 100644 (file)
@@ -194,7 +194,7 @@ thread_db_create_event (CORE_ADDR where)
     fatal ("unexpected thread_db->td_ta_event_getmsg_p == NULL");
 
   if (debug_threads)
-    fprintf (stderr, "Thread creation event.\n");
+    debug_printf ("Thread creation event.\n");
 
   /* FIXME: This assumes we don't get another event.
      In the LinuxThreads implementation, this is safe,
@@ -289,8 +289,8 @@ find_one_thread (ptid_t ptid)
           lwpid, thread_db_err_str (err));
 
   if (debug_threads)
-    fprintf (stderr, "Found thread %ld (LWP %d)\n",
-            ti.ti_tid, ti.ti_lid);
+    debug_printf ("Found thread %ld (LWP %d)\n",
+                 ti.ti_tid, ti.ti_lid);
 
   if (lwpid != ti.ti_lid)
     {
@@ -326,8 +326,8 @@ attach_thread (const td_thrhandle_t *th_p, td_thrinfo_t *ti_p)
   struct lwp_info *lwp;
 
   if (debug_threads)
-    fprintf (stderr, "Attaching to thread %ld (LWP %d)\n",
-            ti_p->ti_tid, ti_p->ti_lid);
+    debug_printf ("Attaching to thread %ld (LWP %d)\n",
+                 ti_p->ti_tid, ti_p->ti_lid);
   linux_attach_lwp (ti_p->ti_lid);
   lwp = find_lwp_pid (pid_to_ptid (ti_p->ti_lid));
   if (lwp == NULL)
@@ -432,8 +432,8 @@ thread_db_find_new_threads (void)
                                         TD_THR_LOWEST_PRIORITY,
                                         TD_SIGNO_MASK, TD_THR_ANY_USER_FLAGS);
       if (debug_threads)
-       fprintf (stderr, "Found %d threads in iteration %d.\n",
-                new_thread_count, iteration);
+       debug_printf ("Found %d threads in iteration %d.\n",
+                     new_thread_count, iteration);
 
       if (new_thread_count != 0)
        {
@@ -546,7 +546,7 @@ thread_db_load_search (void)
   if (err != TD_OK)
     {
       if (debug_threads)
-       fprintf (stderr, "td_ta_new(): %s\n", thread_db_err_str (err));
+       debug_printf ("td_ta_new(): %s\n", thread_db_err_str (err));
       free (tdb);
       proc->private->thread_db = NULL;
       return 0;
@@ -595,7 +595,7 @@ try_thread_db_load_1 (void *handle)
       if ((a) == NULL)                                         \
        {                                                       \
          if (debug_threads)                                    \
-           fprintf (stderr, "dlsym: %s\n", dlerror ());        \
+           debug_printf ("dlsym: %s\n", dlerror ());           \
          if (required)                                         \
            {                                                   \
              free (tdb);                                       \
@@ -613,7 +613,7 @@ try_thread_db_load_1 (void *handle)
   if (err != TD_OK)
     {
       if (debug_threads)
-       fprintf (stderr, "td_ta_new(): %s\n", thread_db_err_str (err));
+       debug_printf ("td_ta_new(): %s\n", thread_db_err_str (err));
       free (tdb);
       proc->private->thread_db = NULL;
       return 0;
@@ -663,13 +663,13 @@ try_thread_db_load (const char *library)
   void *handle;
 
   if (debug_threads)
-    fprintf (stderr, "Trying host libthread_db library: %s.\n",
-            library);
+    debug_printf ("Trying host libthread_db library: %s.\n",
+                 library);
   handle = dlopen (library, RTLD_NOW);
   if (handle == NULL)
     {
       if (debug_threads)
-       fprintf (stderr, "dlopen failed: %s.\n", dlerror ());
+       debug_printf ("dlopen failed: %s.\n", dlerror ());
       return 0;
     }
 
@@ -786,7 +786,7 @@ thread_db_load_search (void)
 
   free_char_ptr_vec (dir_vec);
   if (debug_threads)
-    fprintf (stderr, "thread_db_load_search returning %d\n", rc);
+    debug_printf ("thread_db_load_search returning %d\n", rc);
   return rc;
 }
 
index 13b3ff60d84751766be40931bfa45b8608575b64..37065777b7b9546fe757bd0ba0f99b3d682b0ba3 100644 (file)
@@ -61,6 +61,8 @@
 
 */
 
+#ifdef IN_PROCESS_AGENT
+
 static void trace_vdebug (const char *, ...) ATTRIBUTE_PRINTF (1, 2);
 
 static void
@@ -81,6 +83,19 @@ trace_vdebug (const char *fmt, ...)
       trace_vdebug ((fmt), ##args);            \
   } while (0)
 
+#else
+
+#define trace_debug_1(level, fmt, args...)     \
+  do {                                         \
+    if (level <= debug_threads)                        \
+      {                                                \
+       debug_printf ((fmt), ##args);           \
+       debug_printf ("\n");                    \
+      }                                                \
+  } while (0)
+
+#endif
+
 #define trace_debug(FMT, args...)              \
   trace_debug_1 (1, FMT, ##args)
 
@@ -338,7 +353,7 @@ tracepoint_look_up_symbols (void)
       if (look_up_one_symbol (symbol_list[i].name, addrp, 1) == 0)
        {
          if (debug_threads)
-           fprintf (stderr, "symbol `%s' not found\n", symbol_list[i].name);
+           debug_printf ("symbol `%s' not found\n", symbol_list[i].name);
          return;
        }
     }
index e45fda3562cc59d23e16f4030c42ef1057b9674e..1cbbfa0c3dfb4f13b19d0ad9a600583541c55acf 100644 (file)
@@ -1,3 +1,7 @@
+2014-01-22  Doug Evans  <dje@google.com>
+
+       * gdb.server/server-mon.exp: Add tests for "set debug-format".
+
 2014-01-22  Andreas Arnez  <arnez@vnet.linux.ibm.com>
 
        * gdb.base/catch-syscall.exp: Activate test on s390*-linux.
index 05f63e7dcda7b35c44e57856dbb87603f492d994..58b1df276eb95ac7849d857dd09e601d35c0796a 100644 (file)
@@ -50,3 +50,7 @@ gdb_test "monitor set debug 1" "Debug output enabled\\."
 gdb_test "monitor set debug 0" "Debug output disabled\\."
 gdb_test "monitor set remote-debug 1" "Protocol debug output enabled\\."
 gdb_test "monitor set remote-debug 0" "Protocol debug output disabled\\."
+gdb_test "monitor set debug-format all" \
+    "All extra debug format options enabled\\."
+gdb_test "monitor set debug-format none" \
+    "All extra debug format options disabled\\."