gdb/
[deliverable/binutils-gdb.git] / gdb / tracepoint.c
index 8a33bb933c8640777b2a02bf7e570ccd7240c16a..912341af710cf5f7472de926d399ee3e9732d352 100644 (file)
@@ -1,13 +1,12 @@
 /* Tracing functionality for remote targets in custom GDB protocol
 
-   Copyright (C) 1997, 1998, 1999, 2000, 2001, 2002, 2003, 2004, 2005, 2006,
-   2007 Free Software Foundation, Inc.
+   Copyright (C) 1997-2012 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 2 of the License, or
+   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,
    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, write to the Free Software
-   Foundation, Inc., 51 Franklin Street, Fifth Floor,
-   Boston, MA 02110-1301, USA.  */
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
 
 #include "defs.h"
+#include "arch-utils.h"
 #include "symtab.h"
 #include "frame.h"
 #include "gdbtypes.h"
 #include "language.h"
 #include "gdb_string.h"
 #include "inferior.h"
+#include "breakpoint.h"
 #include "tracepoint.h"
-#include "remote.h"
 #include "linespec.h"
 #include "regcache.h"
 #include "completer.h"
-#include "gdb-events.h"
 #include "block.h"
 #include "dictionary.h"
-
+#include "observer.h"
+#include "user-regs.h"
+#include "valprint.h"
+#include "gdbcore.h"
+#include "objfiles.h"
+#include "filenames.h"
+#include "gdbthread.h"
+#include "stack.h"
+#include "gdbcore.h"
+#include "remote.h"
+#include "source.h"
 #include "ax.h"
 #include "ax-gdb.h"
+#include "memrange.h"
+#include "exceptions.h"
+#include "cli/cli-utils.h"
+#include "probe.h"
 
 /* readline include files */
 #include "readline/readline.h"
 #include <unistd.h>
 #endif
 
+#ifndef O_LARGEFILE
+#define O_LARGEFILE 0
+#endif
+
+extern int hex2bin (const char *hex, gdb_byte *bin, int count);
+extern int bin2hex (const gdb_byte *bin, char *hex, int count);
+
 /* Maximum length of an agent aexpression.
    This accounts for the fact that packets are limited to 400 bytes
    (which includes everything -- including the checksum), and assumes
    large.  (400 - 31)/2 == 184 */
 #define MAX_AGENT_EXPR_LEN     184
 
+#define TFILE_PID (1)
+
+/* A hook used to notify the UI of tracepoint operations.  */
+
+void (*deprecated_trace_find_hook) (char *arg, int from_tty);
+void (*deprecated_trace_start_stop_hook) (int start, int from_tty);
 
 extern void (*deprecated_readline_begin_hook) (char *, ...);
 extern char *(*deprecated_readline_hook) (char *);
 extern void (*deprecated_readline_end_hook) (void);
-extern int addressprint;       /* Print machine addresses? */
 
 /* GDB commands implemented in other modules:
  */  
@@ -103,11 +126,26 @@ extern void output_command (char *, int);
 
 /* ======= Important global variables: ======= */
 
-/* Chain of all tracepoints defined.  */
-struct tracepoint *tracepoint_chain;
+/* The list of all trace state variables.  We don't retain pointers to
+   any of these for any reason - API is by name or number only - so it
+   works to have a vector of objects.  */
 
-/* Number of last tracepoint made.  */
-static int tracepoint_count;
+typedef struct trace_state_variable tsv_s;
+DEF_VEC_O(tsv_s);
+
+/* An object describing the contents of a traceframe.  */
+
+struct traceframe_info
+{
+  /* Collected memory.  */
+  VEC(mem_range_s) *memory;
+};
+
+static VEC(tsv_s) *tvariables;
+
+/* The next integer to assign to a variable.  */
+
+static int next_tsv_number = 1;
 
 /* Number of last traceframe collected.  */
 static int traceframe_number;
@@ -115,22 +153,44 @@ static int traceframe_number;
 /* Tracepoint for last traceframe collected.  */
 static int tracepoint_number;
 
-/* Symbol for function for last traceframe collected */
+/* Symbol for function for last traceframe collected */
 static struct symbol *traceframe_fun;
 
-/* Symtab and line for last traceframe collected */
+/* Symtab and line for last traceframe collected */
 static struct symtab_and_line traceframe_sal;
 
-/* Tracing command lists */
+/* The traceframe info of the current traceframe.  NULL if we haven't
+   yet attempted to fetch it, or if the target does not support
+   fetching this object, or if we're not inspecting a traceframe
+   presently.  */
+static struct traceframe_info *traceframe_info;
+
+/* Tracing command lists.  */
 static struct cmd_list_element *tfindlist;
 
+/* List of expressions to collect by default at each tracepoint hit.  */
+char *default_collect = "";
+
+static int disconnected_tracing;
+
+/* This variable controls whether we ask the target for a linear or
+   circular trace buffer.  */
+
+static int circular_trace_buffer;
+
+/* Textual notes applying to the current and/or future trace runs.  */
+
+char *trace_user = NULL;
+
+/* Textual notes applying to the current and/or future trace runs.  */
+
+char *trace_notes = NULL;
+
+/* Textual notes applying to the stopping of a trace.  */
+
+char *trace_stop_notes = NULL;
+
 /* ======= Important command functions: ======= */
-static void trace_command (char *, int);
-static void tracepoints_info (char *, int);
-static void delete_trace_command (char *, int);
-static void enable_trace_command (char *, int);
-static void disable_trace_command (char *, int);
-static void trace_pass_command (char *, int);
 static void trace_actions_command (char *, int);
 static void trace_start_command (char *, int);
 static void trace_stop_command (char *, int);
@@ -141,87 +201,61 @@ static void trace_find_tracepoint_command (char *, int);
 static void trace_find_line_command (char *, int);
 static void trace_find_range_command (char *, int);
 static void trace_find_outside_command (char *, int);
-static void tracepoint_save_command (char *, int);
 static void trace_dump_command (char *, int);
 
 /* support routines */
-static void trace_mention (struct tracepoint *);
 
 struct collection_list;
 static void add_aexpr (struct collection_list *, struct agent_expr *);
 static char *mem2hex (gdb_byte *, char *, int);
 static void add_register (struct collection_list *collection,
                          unsigned int regno);
-static struct cleanup *make_cleanup_free_actions (struct tracepoint *t);
-static void free_actions_list (char **actions_list);
-static void free_actions_list_cleanup_wrapper (void *);
+
+static void free_uploaded_tps (struct uploaded_tp **utpp);
+static void free_uploaded_tsvs (struct uploaded_tsv **utsvp);
+
 
 extern void _initialize_tracepoint (void);
 
-/* Utility: returns true if "target remote" */
-static int
-target_is_remote (void)
+static struct trace_status trace_status;
+
+char *stop_reason_names[] = {
+  "tunknown",
+  "tnotrun",
+  "tstop",
+  "tfull",
+  "tdisconnected",
+  "tpasscount",
+  "terror"
+};
+
+struct trace_status *
+current_trace_status (void)
 {
-  if (current_target.to_shortname &&
-      (strcmp (current_target.to_shortname, "remote") == 0
-       || strcmp (current_target.to_shortname, "extended-remote") == 0))
-    return 1;
-  else
-    return 0;
+  return &trace_status;
 }
 
-/* Utility: generate error from an incoming stub packet.  */
+/* Destroy INFO.  */
+
 static void
-trace_error (char *buf)
+free_traceframe_info (struct traceframe_info *info)
 {
-  if (*buf++ != 'E')
-    return;                    /* not an error msg */
-  switch (*buf)
+  if (info != NULL)
     {
-    case '1':                  /* malformed packet error */
-      if (*++buf == '0')       /*   general case: */
-       error (_("tracepoint.c: error in outgoing packet."));
-      else
-       error (_("tracepoint.c: error in outgoing packet at field #%ld."),
-              strtol (buf, NULL, 16));
-    case '2':
-      error (_("trace API error 0x%s."), ++buf);
-    default:
-      error (_("Target returns error code '%s'."), buf);
-    }
-}
+      VEC_free (mem_range_s, info->memory);
 
-/* Utility: wait for reply from stub, while accepting "O" packets.  */
-static char *
-remote_get_noisy_reply (char **buf_p,
-                       long *sizeof_buf)
-{
-  do                           /* Loop on reply from remote stub.  */
-    {
-      char *buf;
-      QUIT;                    /* allow user to bail out with ^C */
-      getpkt (buf_p, sizeof_buf, 0);
-      buf = *buf_p;
-      if (buf[0] == 0)
-       error (_("Target does not support this command."));
-      else if (buf[0] == 'E')
-       trace_error (buf);
-      else if (buf[0] == 'O' &&
-              buf[1] != 'K')
-       remote_console_output (buf + 1);        /* 'O' message from stub */
-      else
-       return buf;             /* here's the actual reply */
+      xfree (info);
     }
-  while (1);
 }
 
-/* Set tracepoint count to NUM.  */
+/* Free and clear the traceframe info cache of the current
+   traceframe.  */
+
 static void
-set_tracepoint_count (int num)
+clear_traceframe_info (void)
 {
-  tracepoint_count = num;
-  set_internalvar (lookup_internalvar ("tpnum"),
-                  value_from_longest (builtin_type_int, (LONGEST) num));
+  free_traceframe_info (traceframe_info);
+  traceframe_info = NULL;
 }
 
 /* Set traceframe number to NUM.  */
@@ -229,8 +263,7 @@ static void
 set_traceframe_num (int num)
 {
   traceframe_number = num;
-  set_internalvar (lookup_internalvar ("trace_frame"),
-                  value_from_longest (builtin_type_int, (LONGEST) num));
+  set_internalvar_integer (lookup_internalvar ("trace_frame"), num);
 }
 
 /* Set tracepoint number to NUM.  */
@@ -238,525 +271,293 @@ static void
 set_tracepoint_num (int num)
 {
   tracepoint_number = num;
-  set_internalvar (lookup_internalvar ("tracepoint"),
-                  value_from_longest (builtin_type_int, 
-                                      (LONGEST) num));
+  set_internalvar_integer (lookup_internalvar ("tracepoint"), num);
 }
 
 /* Set externally visible debug variables for querying/printing
-   the traceframe context (line, function, file) */
+   the traceframe context (line, function, file) */
 
 static void
-set_traceframe_context (CORE_ADDR trace_pc)
-{
-  static struct type *func_string, *file_string;
-  static struct type *func_range, *file_range;
-  struct value *func_val;
-  struct value *file_val;
-  static struct type *charstar;
-  int len;
-
-  if (charstar == (struct type *) NULL)
-    charstar = lookup_pointer_type (builtin_type_char);
-
-  if (trace_pc == -1)          /* Cease debugging any trace buffers.  */
-    {
-      traceframe_fun = 0;
-      traceframe_sal.pc = traceframe_sal.line = 0;
-      traceframe_sal.symtab = NULL;
-      set_internalvar (lookup_internalvar ("trace_func"),
-                      value_from_pointer (charstar, (LONGEST) 0));
-      set_internalvar (lookup_internalvar ("trace_file"),
-                      value_from_pointer (charstar, (LONGEST) 0));
-      set_internalvar (lookup_internalvar ("trace_line"),
-                      value_from_longest (builtin_type_int, 
-                                          (LONGEST) - 1));
-      return;
-    }
+set_traceframe_context (struct frame_info *trace_frame)
+{
+  CORE_ADDR trace_pc;
 
   /* Save as globals for internal use.  */
-  traceframe_sal = find_pc_line (trace_pc, 0);
-  traceframe_fun = find_pc_function (trace_pc);
+  if (trace_frame != NULL
+      && get_frame_pc_if_available (trace_frame, &trace_pc))
+    {
+      traceframe_sal = find_pc_line (trace_pc, 0);
+      traceframe_fun = find_pc_function (trace_pc);
 
-  /* Save linenumber as "$trace_line", a debugger variable visible to
-     users.  */
-  set_internalvar (lookup_internalvar ("trace_line"),
-                  value_from_longest (builtin_type_int,
-                                      (LONGEST) traceframe_sal.line));
+      /* Save linenumber as "$trace_line", a debugger variable visible to
+        users.  */
+      set_internalvar_integer (lookup_internalvar ("trace_line"),
+                              traceframe_sal.line);
+    }
+  else
+    {
+      init_sal (&traceframe_sal);
+      traceframe_fun = NULL;
+      set_internalvar_integer (lookup_internalvar ("trace_line"), -1);
+    }
 
   /* Save func name as "$trace_func", a debugger variable visible to
      users.  */
-  if (traceframe_fun == NULL ||
-      DEPRECATED_SYMBOL_NAME (traceframe_fun) == NULL)
-    set_internalvar (lookup_internalvar ("trace_func"),
-                    value_from_pointer (charstar, (LONGEST) 0));
+  if (traceframe_fun == NULL
+      || SYMBOL_LINKAGE_NAME (traceframe_fun) == NULL)
+    clear_internalvar (lookup_internalvar ("trace_func"));
   else
-    {
-      len = strlen (DEPRECATED_SYMBOL_NAME (traceframe_fun));
-      func_range = create_range_type (func_range,
-                                     builtin_type_int, 0, len - 1);
-      func_string = create_array_type (func_string,
-                                      builtin_type_char, func_range);
-      func_val = allocate_value (func_string);
-      deprecated_set_value_type (func_val, func_string);
-      memcpy (value_contents_raw (func_val),
-             DEPRECATED_SYMBOL_NAME (traceframe_fun),
-             len);
-      deprecated_set_value_modifiable (func_val, 0);
-      set_internalvar (lookup_internalvar ("trace_func"), func_val);
-    }
+    set_internalvar_string (lookup_internalvar ("trace_func"),
+                           SYMBOL_LINKAGE_NAME (traceframe_fun));
 
   /* Save file name as "$trace_file", a debugger variable visible to
      users.  */
-  if (traceframe_sal.symtab == NULL ||
-      traceframe_sal.symtab->filename == NULL)
-    set_internalvar (lookup_internalvar ("trace_file"),
-                    value_from_pointer (charstar, (LONGEST) 0));
+  if (traceframe_sal.symtab == NULL
+      || traceframe_sal.symtab->filename == NULL)
+    clear_internalvar (lookup_internalvar ("trace_file"));
   else
-    {
-      len = strlen (traceframe_sal.symtab->filename);
-      file_range = create_range_type (file_range,
-                                     builtin_type_int, 0, len - 1);
-      file_string = create_array_type (file_string,
-                                      builtin_type_char, file_range);
-      file_val = allocate_value (file_string);
-      deprecated_set_value_type (file_val, file_string);
-      memcpy (value_contents_raw (file_val),
-             traceframe_sal.symtab->filename,
-             len);
-      deprecated_set_value_modifiable (file_val, 0);
-      set_internalvar (lookup_internalvar ("trace_file"), file_val);
-    }
+    set_internalvar_string (lookup_internalvar ("trace_file"),
+                           traceframe_sal.symtab->filename);
 }
 
-/* Low level routine to set a tracepoint.
-   Returns the tracepoint object so caller can set other things.
-   Does not set the tracepoint number!
-   Does not print anything.
-
-   ==> This routine should not be called if there is a chance of later
-   error(); otherwise it leaves a bogus tracepoint on the chain.
-   Validate your arguments BEFORE calling this routine!  */
+/* Create a new trace state variable with the given name.  */
 
-static struct tracepoint *
-set_raw_tracepoint (struct symtab_and_line sal)
+struct trace_state_variable *
+create_trace_state_variable (const char *name)
 {
-  struct tracepoint *t, *tc;
-  struct cleanup *old_chain;
+  struct trace_state_variable tsv;
 
-  t = (struct tracepoint *) xmalloc (sizeof (struct tracepoint));
-  old_chain = make_cleanup (xfree, t);
-  memset (t, 0, sizeof (*t));
-  t->address = sal.pc;
-  if (sal.symtab == NULL)
-    t->source_file = NULL;
-  else
-    t->source_file = savestring (sal.symtab->filename,
-                                strlen (sal.symtab->filename));
-
-  t->section = sal.section;
-  t->language = current_language->la_language;
-  t->input_radix = input_radix;
-  t->line_number = sal.line;
-  t->enabled_p = 1;
-  t->next = 0;
-  t->step_count = 0;
-  t->pass_count = 0;
-  t->addr_string = NULL;
-
-  /* Add this tracepoint to the end of the chain
-     so that a list of tracepoints will come out in order
-     of increasing numbers.  */
-
-  tc = tracepoint_chain;
-  if (tc == 0)
-    tracepoint_chain = t;
-  else
-    {
-      while (tc->next)
-       tc = tc->next;
-      tc->next = t;
-    }
-  discard_cleanups (old_chain);
-  return t;
+  memset (&tsv, 0, sizeof (tsv));
+  tsv.name = xstrdup (name);
+  tsv.number = next_tsv_number++;
+  return VEC_safe_push (tsv_s, tvariables, &tsv);
 }
 
-/* Set a tracepoint according to ARG (function, linenum or *address).  */
-static void
-trace_command (char *arg, int from_tty)
-{
-  char **canonical = (char **) NULL;
-  struct symtabs_and_lines sals;
-  struct symtab_and_line sal;
-  struct tracepoint *t;
-  char *addr_start = 0, *addr_end = 0;
-  int i;
-
-  if (!arg || !*arg)
-    error (_("trace command requires an argument"));
-
-  if (from_tty && info_verbose)
-    printf_filtered ("TRACE %s\n", arg);
-
-  addr_start = arg;
-  sals = decode_line_1 (&arg, 1, (struct symtab *) NULL, 
-                       0, &canonical, NULL);
-  addr_end = arg;
-  if (!sals.nelts)
-    return;    /* ??? Presumably decode_line_1 has already warned?  */
-
-  /* Resolve all line numbers to PC's */
-  for (i = 0; i < sals.nelts; i++)
-    resolve_sal_pc (&sals.sals[i]);
-
-  /* Now set all the tracepoints.  */
-  for (i = 0; i < sals.nelts; i++)
-    {
-      sal = sals.sals[i];
-
-      t = set_raw_tracepoint (sal);
-      set_tracepoint_count (tracepoint_count + 1);
-      t->number = tracepoint_count;
+/* Look for a trace state variable of the given name.  */
 
-      /* If a canonical line spec is needed use that instead of the
-         command string.  */
-      if (canonical != (char **) NULL && canonical[i] != NULL)
-       t->addr_string = canonical[i];
-      else if (addr_start)
-       t->addr_string = savestring (addr_start, addr_end - addr_start);
+struct trace_state_variable *
+find_trace_state_variable (const char *name)
+{
+  struct trace_state_variable *tsv;
+  int ix;
 
-      trace_mention (t);
-    }
+  for (ix = 0; VEC_iterate (tsv_s, tvariables, ix, tsv); ++ix)
+    if (strcmp (name, tsv->name) == 0)
+      return tsv;
 
-  if (sals.nelts > 1)
-    {
-      printf_filtered ("Multiple tracepoints were set.\n");
-      printf_filtered ("Use 'delete trace' to delete unwanted tracepoints.\n");
-    }
+  return NULL;
 }
 
-/* Tell the user we have just set a tracepoint TP.  */
-
 static void
-trace_mention (struct tracepoint *tp)
+delete_trace_state_variable (const char *name)
 {
-  printf_filtered ("Tracepoint %d", tp->number);
+  struct trace_state_variable *tsv;
+  int ix;
 
-  if (addressprint || (tp->source_file == NULL))
-    {
-      printf_filtered (" at ");
-      deprecated_print_address_numeric (tp->address, 1, gdb_stdout);
-    }
-  if (tp->source_file)
-    printf_filtered (": file %s, line %d.",
-                    tp->source_file, tp->line_number);
+  for (ix = 0; VEC_iterate (tsv_s, tvariables, ix, tsv); ++ix)
+    if (strcmp (name, tsv->name) == 0)
+      {
+       xfree ((void *)tsv->name);
+       VEC_unordered_remove (tsv_s, tvariables, ix);
+
+       observer_notify_tsv_deleted (name);
 
-  printf_filtered ("\n");
+       return;
+      }
+
+  warning (_("No trace variable named \"$%s\", not deleting"), name);
 }
 
-/* Print information on tracepoint number TPNUM_EXP, or all if
-   omitted.  */
+/* The 'tvariable' command collects a name and optional expression to
+   evaluate into an initial value.  */
 
 static void
-tracepoints_info (char *tpnum_exp, int from_tty)
+trace_variable_command (char *args, int from_tty)
 {
-  struct tracepoint *t;
-  struct action_line *action;
-  int found_a_tracepoint = 0;
-  char wrap_indent[80];
-  struct symbol *sym;
-  int tpnum = -1;
-
-  if (tpnum_exp)
-    tpnum = parse_and_eval_long (tpnum_exp);
-
-  ALL_TRACEPOINTS (t)
-    if (tpnum == -1 || tpnum == t->number)
-    {
-      extern int addressprint; /* Print machine addresses?  */
-
-      if (!found_a_tracepoint++)
-       {
-         printf_filtered ("Num Enb ");
-         if (addressprint)
-           {
-             if (TARGET_ADDR_BIT <= 32)
-               printf_filtered ("Address    ");
-             else
-               printf_filtered ("Address            ");
-           }
-         printf_filtered ("PassC StepC What\n");
-       }
-      strcpy (wrap_indent, "                           ");
-      if (addressprint)
-       {
-         if (TARGET_ADDR_BIT <= 32)
-           strcat (wrap_indent, "           ");
-         else
-           strcat (wrap_indent, "                   ");
-       }
-
-      printf_filtered ("%-3d %-3s ", t->number,
-                      t->enabled_p ? "y" : "n");
-      if (addressprint)
-       {
-         char *tmp;
+  struct expression *expr;
+  struct cleanup *old_chain;
+  struct internalvar *intvar = NULL;
+  LONGEST initval = 0;
+  struct trace_state_variable *tsv;
 
-         if (TARGET_ADDR_BIT <= 32)
-           tmp = hex_string_custom (t->address & (CORE_ADDR) 0xffffffff, 
-                                    8);
-         else
-           tmp = hex_string_custom (t->address, 16);
+  if (!args || !*args)
+    error_no_arg (_("trace state variable name"));
 
-         printf_filtered ("%s ", tmp);
-       }
-      printf_filtered ("%-5d %-5ld ", t->pass_count, t->step_count);
+  /* All the possible valid arguments are expressions.  */
+  expr = parse_expression (args);
+  old_chain = make_cleanup (free_current_contents, &expr);
 
-      if (t->source_file)
-       {
-         sym = find_pc_sect_function (t->address, t->section);
-         if (sym)
-           {
-             fputs_filtered ("in ", gdb_stdout);
-             fputs_filtered (SYMBOL_PRINT_NAME (sym), gdb_stdout);
-             wrap_here (wrap_indent);
-             fputs_filtered (" at ", gdb_stdout);
-           }
-         fputs_filtered (t->source_file, gdb_stdout);
-         printf_filtered (":%d", t->line_number);
-       }
-      else
-       print_address_symbolic (t->address, gdb_stdout, demangle, " ");
+  if (expr->nelts == 0)
+    error (_("No expression?"));
 
-      printf_filtered ("\n");
-      if (t->actions)
-       {
-         printf_filtered ("  Actions for tracepoint %d: \n", t->number);
-         for (action = t->actions; action; action = action->next)
-           {
-             printf_filtered ("\t%s\n", action->action);
-           }
-       }
+  /* Only allow two syntaxes; "$name" and "$name=value".  */
+  if (expr->elts[0].opcode == OP_INTERNALVAR)
+    {
+      intvar = expr->elts[1].internalvar;
     }
-  if (!found_a_tracepoint)
+  else if (expr->elts[0].opcode == BINOP_ASSIGN
+          && expr->elts[1].opcode == OP_INTERNALVAR)
     {
-      if (tpnum == -1)
-       printf_filtered ("No tracepoints.\n");
-      else
-       printf_filtered ("No tracepoint number %d.\n", tpnum);
+      intvar = expr->elts[2].internalvar;
+      initval = value_as_long (evaluate_subexpression_type (expr, 4));
     }
-}
-
-/* Optimization: the code to parse an enable, disable, or delete TP
-   command is virtually identical except for whether it performs an
-   enable, disable, or delete.  Therefore I've combined them into one
-   function with an opcode.  */
-enum tracepoint_opcode
-{
-  enable_op,
-  disable_op,
-  delete_op
-};
+  else
+    error (_("Syntax must be $NAME [ = EXPR ]"));
 
-/* This function implements enable, disable and delete commands.  */
-static void
-tracepoint_operation (struct tracepoint *t, int from_tty,
-                     enum tracepoint_opcode opcode)
-{
-  struct tracepoint *t2;
+  if (!intvar)
+    error (_("No name given"));
 
-  if (t == NULL)       /* no tracepoint operand */
-    return;
+  if (strlen (internalvar_name (intvar)) <= 0)
+    error (_("Must supply a non-empty variable name"));
 
-  switch (opcode)
+  /* If the variable already exists, just change its initial value.  */
+  tsv = find_trace_state_variable (internalvar_name (intvar));
+  if (tsv)
     {
-    case enable_op:
-      t->enabled_p = 1;
-      tracepoint_modify_event (t->number);
-      break;
-    case disable_op:
-      t->enabled_p = 0;
-      tracepoint_modify_event (t->number);
-      break;
-    case delete_op:
-      if (tracepoint_chain == t)
-       tracepoint_chain = t->next;
+      tsv->initial_value = initval;
+      printf_filtered (_("Trace state variable $%s "
+                        "now has initial value %s.\n"),
+                      tsv->name, plongest (tsv->initial_value));
+      do_cleanups (old_chain);
+      return;
+    }
 
-      ALL_TRACEPOINTS (t2)
-       if (t2->next == t)
-       {
-         tracepoint_delete_event (t2->number);
-         t2->next = t->next;
-         break;
-       }
+  /* Create a new variable.  */
+  tsv = create_trace_state_variable (internalvar_name (intvar));
+  tsv->initial_value = initval;
 
-      if (t->addr_string)
-       xfree (t->addr_string);
-      if (t->source_file)
-       xfree (t->source_file);
-      if (t->actions)
-       free_actions (t);
+  observer_notify_tsv_created (tsv->name, initval);
 
-      xfree (t);
-      break;
-    }
+  printf_filtered (_("Trace state variable $%s "
+                    "created, with initial value %s.\n"),
+                  tsv->name, plongest (tsv->initial_value));
+
+  do_cleanups (old_chain);
 }
 
-/* Utility: parse a tracepoint number and look it up in the list.
-   If MULTI_P is true, there might be a range of tracepoints in ARG.
-   if OPTIONAL_P is true, then if the argument is missing, the most
-   recent tracepoint (tracepoint_count) is returned.  */
-struct tracepoint *
-get_tracepoint_by_number (char **arg, int multi_p, int optional_p)
+static void
+delete_trace_variable_command (char *args, int from_tty)
 {
-  struct tracepoint *t;
-  int tpnum;
-  char *instring = arg == NULL ? NULL : *arg;
+  int ix;
+  char **argv;
+  struct cleanup *back_to;
 
-  if (arg == NULL || *arg == NULL || ! **arg)
+  if (args == NULL)
     {
-      if (optional_p)
-       tpnum = tracepoint_count;
-      else
-       error_no_arg (_("tracepoint number"));
+      if (query (_("Delete all trace state variables? ")))
+       VEC_free (tsv_s, tvariables);
+      dont_repeat ();
+      observer_notify_tsv_deleted (NULL);
+      return;
     }
-  else
-    tpnum = multi_p ? get_number_or_range (arg) : get_number (arg);
 
-  if (tpnum <= 0)
+  argv = gdb_buildargv (args);
+  back_to = make_cleanup_freeargv (argv);
+
+  for (ix = 0; argv[ix] != NULL; ix++)
     {
-      if (instring && *instring)
-       printf_filtered ("bad tracepoint number at or near '%s'\n", 
-                        instring);
+      if (*argv[ix] == '$')
+       delete_trace_state_variable (argv[ix] + 1);
       else
-       printf_filtered ("Tracepoint argument missing and no previous tracepoint\n");
-      return NULL;
+       warning (_("Name \"%s\" not prefixed with '$', ignoring"), argv[ix]);
     }
 
-  ALL_TRACEPOINTS (t)
-    if (t->number == tpnum)
-    {
-      return t;
-    }
+  do_cleanups (back_to);
 
-  /* FIXME: if we are in the middle of a range we don't want to give
-     a message.  The current interface to get_number_or_range doesn't
-     allow us to discover this.  */
-  printf_unfiltered ("No tracepoint number %d.\n", tpnum);
-  return NULL;
+  dont_repeat ();
 }
 
-/* Utility: 
-   parse a list of tracepoint numbers, and call a func for each.  */
-static void
-map_args_over_tracepoints (char *args, int from_tty,
-                          enum tracepoint_opcode opcode)
+void
+tvariables_info_1 (void)
 {
-  struct tracepoint *t, *tmp;
+  struct trace_state_variable *tsv;
+  int ix;
+  int count = 0;
+  struct cleanup *back_to;
+  struct ui_out *uiout = current_uiout;
 
-  if (args == 0 || *args == 0) /* do them all */
-    ALL_TRACEPOINTS_SAFE (t, tmp)
-      tracepoint_operation (t, from_tty, opcode);
-  else
-    while (*args)
-      {
-       QUIT;           /* Give user option to bail out with ^C.  */
-       t = get_tracepoint_by_number (&args, 1, 0);
-       tracepoint_operation (t, from_tty, opcode);
-       while (*args == ' ' || *args == '\t')
-         args++;
-      }
-}
+  if (VEC_length (tsv_s, tvariables) == 0 && !ui_out_is_mi_like_p (uiout))
+    {
+      printf_filtered (_("No trace state variables.\n"));
+      return;
+    }
 
-/* The 'enable trace' command enables tracepoints.  
-   Not supported by all targets.  */
-static void
-enable_trace_command (char *args, int from_tty)
-{
-  dont_repeat ();
-  map_args_over_tracepoints (args, from_tty, enable_op);
-}
+  /* Try to acquire values from the target.  */
+  for (ix = 0; VEC_iterate (tsv_s, tvariables, ix, tsv); ++ix, ++count)
+    tsv->value_known = target_get_trace_state_variable_value (tsv->number,
+                                                             &(tsv->value));
 
-/* The 'disable trace' command disables tracepoints.  
-   Not supported by all targets.  */
-static void
-disable_trace_command (char *args, int from_tty)
-{
-  dont_repeat ();
-  map_args_over_tracepoints (args, from_tty, disable_op);
-}
+  back_to = make_cleanup_ui_out_table_begin_end (uiout, 3,
+                                                 count, "trace-variables");
+  ui_out_table_header (uiout, 15, ui_left, "name", "Name");
+  ui_out_table_header (uiout, 11, ui_left, "initial", "Initial");
+  ui_out_table_header (uiout, 11, ui_left, "current", "Current");
 
-/* Remove a tracepoint (or all if no argument) */
-static void
-delete_trace_command (char *args, int from_tty)
-{
-  dont_repeat ();
-  if (!args || !*args)         /* No args implies all tracepoints; */
-    if (from_tty)              /* confirm only if from_tty...  */
-      if (tracepoint_chain)    /* and if there are tracepoints to
-                                  delete!  */
-       if (!query ("Delete all tracepoints? "))
-         return;
+  ui_out_table_body (uiout);
 
-  map_args_over_tracepoints (args, from_tty, delete_op);
-}
+  for (ix = 0; VEC_iterate (tsv_s, tvariables, ix, tsv); ++ix)
+    {
+      struct cleanup *back_to2;
+      char *c;
+      char *name;
+
+      back_to2 = make_cleanup_ui_out_tuple_begin_end (uiout, "variable");
+
+      name = concat ("$", tsv->name, (char *) NULL);
+      make_cleanup (xfree, name);
+      ui_out_field_string (uiout, "name", name);
+      ui_out_field_string (uiout, "initial", plongest (tsv->initial_value));
+
+      if (tsv->value_known)
+        c = plongest (tsv->value);
+      else if (ui_out_is_mi_like_p (uiout))
+        /* For MI, we prefer not to use magic string constants, but rather
+           omit the field completely.  The difference between unknown and
+           undefined does not seem important enough to represent.  */
+        c = NULL;
+      else if (current_trace_status ()->running || traceframe_number >= 0)
+       /* The value is/was defined, but we don't have it.  */
+        c = "<unknown>";
+      else
+       /* It is not meaningful to ask about the value.  */
+        c = "<undefined>";
+      if (c)
+        ui_out_field_string (uiout, "current", c);
+      ui_out_text (uiout, "\n");
+
+      do_cleanups (back_to2);
+    }
 
-/* Set passcount for tracepoint.
+  do_cleanups (back_to);
+}
 
-   First command argument is passcount, second is tracepoint number.
-   If tracepoint number omitted, apply to most recently defined.
-   Also accepts special argument "all".  */
+/* List all the trace state variables.  */
 
 static void
-trace_pass_command (char *args, int from_tty)
+tvariables_info (char *args, int from_tty)
 {
-  struct tracepoint *t1 = (struct tracepoint *) -1, *t2;
-  unsigned int count;
-  int all = 0;
-
-  if (args == 0 || *args == 0)
-    error (_("passcount command requires an argument (count + optional TP num)"));
-
-  count = strtoul (args, &args, 10);   /* Count comes first, then TP num. */
+  tvariables_info_1 ();
+}
 
-  while (*args && isspace ((int) *args))
-    args++;
+/* Stash definitions of tsvs into the given file.  */
 
-  if (*args && strncasecmp (args, "all", 3) == 0)
-    {
-      args += 3;                       /* Skip special argument "all".  */
-      all = 1;
-      if (*args)
-       error (_("Junk at end of arguments."));
-    }
-  else
-    t1 = get_tracepoint_by_number (&args, 1, 1);
+void
+save_trace_state_variables (struct ui_file *fp)
+{
+  struct trace_state_variable *tsv;
+  int ix;
 
-  do
+  for (ix = 0; VEC_iterate (tsv_s, tvariables, ix, tsv); ++ix)
     {
-      if (t1)
-       {
-         ALL_TRACEPOINTS (t2)
-           if (t1 == (struct tracepoint *) -1 || t1 == t2)
-             {
-               t2->pass_count = count;
-               tracepoint_modify_event (t2->number);
-               if (from_tty)
-                 printf_filtered ("Setting tracepoint %d's passcount to %d\n",
-                                  t2->number, count);
-             }
-         if (! all && *args)
-           t1 = get_tracepoint_by_number (&args, 1, 0);
-       }
+      fprintf_unfiltered (fp, "tvariable $%s", tsv->name);
+      if (tsv->initial_value)
+       fprintf_unfiltered (fp, " = %s", plongest (tsv->initial_value));
+      fprintf_unfiltered (fp, "\n");
     }
-  while (*args);
 }
 
 /* ACTIONS functions: */
 
-/* Prototypes for action-parsing utility commands  */
-static void read_actions (struct tracepoint *);
-
 /* The three functions:
    collect_pseudocommand, 
    while_stepping_pseudocommand, and 
@@ -766,13 +567,13 @@ static void read_actions (struct tracepoint *);
    it means that somebody issued the "command" at the top level,
    which is always an error.  */
 
-static void
+void
 end_actions_pseudocommand (char *args, int from_tty)
 {
   error (_("This command cannot be used at the top level."));
 }
 
-static void
+void
 while_stepping_pseudocommand (char *args, int from_tty)
 {
   error (_("This command can only be used in a tracepoint actions list."));
@@ -784,290 +585,254 @@ collect_pseudocommand (char *args, int from_tty)
   error (_("This command can only be used in a tracepoint actions list."));
 }
 
-/* Enter a list of actions for a tracepoint.  */
 static void
-trace_actions_command (char *args, int from_tty)
+teval_pseudocommand (char *args, int from_tty)
 {
-  struct tracepoint *t;
-  char tmpbuf[128];
-  char *end_msg = "End with a line saying just \"end\".";
-
-  t = get_tracepoint_by_number (&args, 0, 1);
-  if (t)
-    {
-      sprintf (tmpbuf, "Enter actions for tracepoint %d, one per line.",
-              t->number);
+  error (_("This command can only be used in a tracepoint actions list."));
+}
 
-      if (from_tty)
-       {
-         if (deprecated_readline_begin_hook)
-           (*deprecated_readline_begin_hook) ("%s  %s\n", tmpbuf, end_msg);
-         else if (input_from_terminal_p ())
-           printf_filtered ("%s\n%s\n", tmpbuf, end_msg);
-       }
+/* Parse any collection options, such as /s for strings.  */
 
-      free_actions (t);
-      t->step_count = 0;       /* read_actions may set this */
-      read_actions (t);
+char *
+decode_agent_options (char *exp)
+{
+  struct value_print_options opts;
 
-      if (deprecated_readline_end_hook)
-       (*deprecated_readline_end_hook) ();
-      /* tracepoints_changed () */
-    }
-  /* else just return */
-}
+  if (*exp != '/')
+    return exp;
 
-/* worker function */
-static void
-read_actions (struct tracepoint *t)
-{
-  char *line;
-  char *prompt1 = "> ", *prompt2 = "  > ";
-  char *prompt = prompt1;
-  enum actionline_type linetype;
-  extern FILE *instream;
-  struct action_line *next = NULL, *temp;
-  struct cleanup *old_chain;
+  /* Call this to borrow the print elements default for collection
+     size.  */
+  get_user_print_options (&opts);
 
-  /* Control-C quits instantly if typed while in this loop
-     since it should not wait until the user types a newline.  */
-  immediate_quit++;
-  /* FIXME: kettenis/20010823: Something is wrong here.  In this file
-     STOP_SIGNAL is never defined.  So this code has been left out, at
-     least for quite a while now.  Replacing STOP_SIGNAL with SIGTSTP
-     leads to compilation failures since the variable job_control
-     isn't declared.  Leave this alone for now.  */
-#ifdef STOP_SIGNAL
-  if (job_control)
-    signal (STOP_SIGNAL, handle_stop_sig);
-#endif
-  old_chain = make_cleanup_free_actions (t);
-  while (1)
+  exp++;
+  if (*exp == 's')
     {
-      /* Make sure that all output has been output.  Some machines may
-         let you get away with leaving out some of the gdb_flush, but
-         not all.  */
-      wrap_here ("");
-      gdb_flush (gdb_stdout);
-      gdb_flush (gdb_stderr);
-
-      if (deprecated_readline_hook && instream == NULL)
-       line = (*deprecated_readline_hook) (prompt);
-      else if (instream == stdin && ISATTY (instream))
+      if (target_supports_string_tracing ())
        {
-         line = gdb_readline_wrapper (prompt);
-         if (line && *line)    /* add it to command history */
-           add_history (line);
+         /* Allow an optional decimal number giving an explicit maximum
+            string length, defaulting it to the "print elements" value;
+            so "collect/s80 mystr" gets at most 80 bytes of string.  */
+         trace_string_kludge = opts.print_max;
+         exp++;
+         if (*exp >= '0' && *exp <= '9')
+           trace_string_kludge = atoi (exp);
+         while (*exp >= '0' && *exp <= '9')
+           exp++;
        }
       else
-       line = gdb_readline (0);
+       error (_("Target does not support \"/s\" option for string tracing."));
+    }
+  else
+    error (_("Undefined collection format \"%c\"."), *exp);
 
-      if (!line)
-       line = "end";
-      
-      linetype = validate_actionline (&line, t);
-      if (linetype == BADLINE)
-       continue;               /* already warned -- collect another line */
+  exp = skip_spaces (exp);
 
-      temp = xmalloc (sizeof (struct action_line));
-      temp->next = NULL;
-      temp->action = line;
+  return exp;
+}
 
-      if (next == NULL)                /* first action for this tracepoint? */
-       t->actions = next = temp;
-      else
-       {
-         next->next = temp;
-         next = temp;
-       }
+/* Enter a list of actions for a tracepoint.  */
+static void
+trace_actions_command (char *args, int from_tty)
+{
+  struct tracepoint *t;
+  struct command_line *l;
 
-      if (linetype == STEPPING)        /* begin "while-stepping" */
-       {
-         if (prompt == prompt2)
-           {
-             warning (_("Already processing 'while-stepping'"));
-             continue;
-           }
-         else
-           prompt = prompt2;   /* change prompt for stepping actions */
-       }
-      else if (linetype == END)
-       {
-         if (prompt == prompt2)
-           {
-             prompt = prompt1; /* end of single-stepping actions */
-           }
-         else
-           {                   /* end of actions */
-             if (t->actions->next == NULL)
-               {
-                 /* An "end" all by itself with no other actions
-                    means this tracepoint has no actions.
-                    Discard empty list.  */
-                 free_actions (t);
-               }
-             break;
-           }
-       }
+  t = get_tracepoint_by_number (&args, NULL, 1);
+  if (t)
+    {
+      char *tmpbuf =
+       xstrprintf ("Enter actions for tracepoint %d, one per line.",
+                   t->base.number);
+      struct cleanup *cleanups = make_cleanup (xfree, tmpbuf);
+
+      l = read_command_lines (tmpbuf, from_tty, 1,
+                             check_tracepoint_command, t);
+      do_cleanups (cleanups);
+      breakpoint_set_commands (&t->base, l);
     }
-#ifdef STOP_SIGNAL
-  if (job_control)
-    signal (STOP_SIGNAL, SIG_DFL);
-#endif
-  immediate_quit--;
-  discard_cleanups (old_chain);
+  /* else just return */
+}
+
+/* Report the results of checking the agent expression, as errors or
+   internal errors.  */
+
+static void
+report_agent_reqs_errors (struct agent_expr *aexpr)
+{
+  /* All of the "flaws" are serious bytecode generation issues that
+     should never occur.  */
+  if (aexpr->flaw != agent_flaw_none)
+    internal_error (__FILE__, __LINE__, _("expression is malformed"));
+
+  /* If analysis shows a stack underflow, GDB must have done something
+     badly wrong in its bytecode generation.  */
+  if (aexpr->min_height < 0)
+    internal_error (__FILE__, __LINE__,
+                   _("expression has min height < 0"));
+
+  /* Issue this error if the stack is predicted to get too deep.  The
+     limit is rather arbitrary; a better scheme might be for the
+     target to report how much stack it will have available.  The
+     depth roughly corresponds to parenthesization, so a limit of 20
+     amounts to 20 levels of expression nesting, which is actually
+     a pretty big hairy expression.  */
+  if (aexpr->max_height > 20)
+    error (_("Expression is too complicated."));
 }
 
 /* worker function */
-enum actionline_type
-validate_actionline (char **line, struct tracepoint *t)
+void
+validate_actionline (char **line, struct breakpoint *b)
 {
   struct cmd_list_element *c;
   struct expression *exp = NULL;
   struct cleanup *old_chain = NULL;
-  char *p;
+  char *p, *tmp_p;
+  struct bp_location *loc;
+  struct agent_expr *aexpr;
+  struct tracepoint *t = (struct tracepoint *) b;
 
-  /* if EOF is typed, *line is NULL */
+  /* If EOF is typed, *line is NULL.  */
   if (*line == NULL)
-    return END;
+    return;
 
   for (p = *line; isspace ((int) *p);)
     p++;
 
   /* Symbol lookup etc.  */
   if (*p == '\0')      /* empty line: just prompt for another line.  */
-    return BADLINE;
+    return;
 
   if (*p == '#')               /* comment line */
-    return GENERIC;
+    return;
 
   c = lookup_cmd (&p, cmdlist, "", -1, 1);
   if (c == 0)
-    {
-      warning (_("'%s' is not an action that I know, or is ambiguous."), 
-              p);
-      return BADLINE;
-    }
+    error (_("`%s' is not a tracepoint action, or is ambiguous."), p);
 
   if (cmd_cfunc_eq (c, collect_pseudocommand))
     {
-      struct agent_expr *aexpr;
-      struct agent_reqs areqs;
+      trace_string_kludge = 0;
+      if (*p == '/')
+       p = decode_agent_options (p);
 
       do
-       {                       /* repeat over a comma-separated list */
-         QUIT;                 /* allow user to bail out with ^C */
+       {                       /* Repeat over a comma-separated list.  */
+         QUIT;                 /* Allow user to bail out with ^C.  */
          while (isspace ((int) *p))
            p++;
 
-         if (*p == '$')        /* look for special pseudo-symbols */
+         if (*p == '$')        /* Look for special pseudo-symbols.  */
            {
-             if ((0 == strncasecmp ("reg", p + 1, 3)) ||
-                 (0 == strncasecmp ("arg", p + 1, 3)) ||
-                 (0 == strncasecmp ("loc", p + 1, 3)))
+             if (0 == strncasecmp ("reg", p + 1, 3)
+                 || 0 == strncasecmp ("arg", p + 1, 3)
+                 || 0 == strncasecmp ("loc", p + 1, 3)
+                 || 0 == strncasecmp ("_ret", p + 1, 4)
+                 || 0 == strncasecmp ("_sdata", p + 1, 6))
                {
                  p = strchr (p, ',');
                  continue;
                }
              /* else fall thru, treat p as an expression and parse it!  */
            }
-         exp = parse_exp_1 (&p, block_for_pc (t->address), 1);
-         old_chain = make_cleanup (free_current_contents, &exp);
-
-         if (exp->elts[0].opcode == OP_VAR_VALUE)
+         tmp_p = p;
+         for (loc = t->base.loc; loc; loc = loc->next)
            {
-             if (SYMBOL_CLASS (exp->elts[2].symbol) == LOC_CONST)
-               {
-                 warning (_("constant %s (value %ld) will not be collected."),
-                          DEPRECATED_SYMBOL_NAME (exp->elts[2].symbol),
-                          SYMBOL_VALUE (exp->elts[2].symbol));
-                 return BADLINE;
-               }
-             else if (SYMBOL_CLASS (exp->elts[2].symbol) == LOC_OPTIMIZED_OUT)
+             p = tmp_p;
+             exp = parse_exp_1 (&p, loc->address,
+                                block_for_pc (loc->address), 1);
+             old_chain = make_cleanup (free_current_contents, &exp);
+
+             if (exp->elts[0].opcode == OP_VAR_VALUE)
                {
-                 warning (_("%s is optimized away and cannot be collected."),
-                          DEPRECATED_SYMBOL_NAME (exp->elts[2].symbol));
-                 return BADLINE;
+                 if (SYMBOL_CLASS (exp->elts[2].symbol) == LOC_CONST)
+                   {
+                     error (_("constant `%s' (value %s) "
+                              "will not be collected."),
+                            SYMBOL_PRINT_NAME (exp->elts[2].symbol),
+                            plongest (SYMBOL_VALUE (exp->elts[2].symbol)));
+                   }
+                 else if (SYMBOL_CLASS (exp->elts[2].symbol)
+                          == LOC_OPTIMIZED_OUT)
+                   {
+                     error (_("`%s' is optimized away "
+                              "and cannot be collected."),
+                            SYMBOL_PRINT_NAME (exp->elts[2].symbol));
+                   }
                }
-           }
 
-         /* We have something to collect, make sure that the expr to
-            bytecode translator can handle it and that it's not too
-            long.  */
-         aexpr = gen_trace_for_expr (t->address, exp);
-         make_cleanup_free_agent_expr (aexpr);
+             /* We have something to collect, make sure that the expr to
+                bytecode translator can handle it and that it's not too
+                long.  */
+             aexpr = gen_trace_for_expr (loc->address, exp);
+             make_cleanup_free_agent_expr (aexpr);
+
+             if (aexpr->len > MAX_AGENT_EXPR_LEN)
+               error (_("Expression is too complicated."));
+
+             ax_reqs (aexpr);
+
+             report_agent_reqs_errors (aexpr);
+
+             do_cleanups (old_chain);
+           }
+       }
+      while (p && *p++ == ',');
+    }
 
-         if (aexpr->len > MAX_AGENT_EXPR_LEN)
-           error (_("expression too complicated, try simplifying"));
+  else if (cmd_cfunc_eq (c, teval_pseudocommand))
+    {
+      do
+       {                       /* Repeat over a comma-separated list.  */
+         QUIT;                 /* Allow user to bail out with ^C.  */
+         while (isspace ((int) *p))
+           p++;
 
-         ax_reqs (aexpr, &areqs);
-         (void) make_cleanup (xfree, areqs.reg_mask);
+         tmp_p = p;
+         for (loc = t->base.loc; loc; loc = loc->next)
+           {
+             p = tmp_p;
+             /* Only expressions are allowed for this action.  */
+             exp = parse_exp_1 (&p, loc->address,
+                                block_for_pc (loc->address), 1);
+             old_chain = make_cleanup (free_current_contents, &exp);
 
-         if (areqs.flaw != agent_flaw_none)
-           error (_("malformed expression"));
+             /* We have something to evaluate, make sure that the expr to
+                bytecode translator can handle it and that it's not too
+                long.  */
+             aexpr = gen_eval_for_expr (loc->address, exp);
+             make_cleanup_free_agent_expr (aexpr);
 
-         if (areqs.min_height < 0)
-           error (_("gdb: Internal error: expression has min height < 0"));
+             if (aexpr->len > MAX_AGENT_EXPR_LEN)
+               error (_("Expression is too complicated."));
 
-         if (areqs.max_height > 20)
-           error (_("expression too complicated, try simplifying"));
+             ax_reqs (aexpr);
+             report_agent_reqs_errors (aexpr);
 
-         do_cleanups (old_chain);
+             do_cleanups (old_chain);
+           }
        }
       while (p && *p++ == ',');
-      return GENERIC;
     }
+
   else if (cmd_cfunc_eq (c, while_stepping_pseudocommand))
     {
-      char *steparg;           /* in case warning is necessary */
+      char *steparg;           /* In case warning is necessary.  */
 
       while (isspace ((int) *p))
        p++;
       steparg = p;
 
-      if (*p == '\0' ||
-         (t->step_count = strtol (p, &p, 0)) == 0)
-       {
-         warning (_("'%s': bad step-count; command ignored."), *line);
-         return BADLINE;
-       }
-      return STEPPING;
-    }
-  else if (cmd_cfunc_eq (c, end_actions_pseudocommand))
-    return END;
-  else
-    {
-      warning (_("'%s' is not a supported tracepoint action."), *line);
-      return BADLINE;
-    }
-}
-
-/* worker function */
-void
-free_actions (struct tracepoint *t)
-{
-  struct action_line *line, *next;
-
-  for (line = t->actions; line; line = next)
-    {
-      next = line->next;
-      if (line->action)
-       xfree (line->action);
-      xfree (line);
+      if (*p == '\0' || (t->step_count = strtol (p, &p, 0)) == 0)
+       error (_("while-stepping step count `%s' is malformed."), *line);
     }
-  t->actions = NULL;
-}
 
-static void
-do_free_actions_cleanup (void *t)
-{
-  free_actions (t);
-}
+  else if (cmd_cfunc_eq (c, end_actions_pseudocommand))
+    ;
 
-static struct cleanup *
-make_cleanup_free_actions (struct tracepoint *t)
-{
-  return make_cleanup (do_free_actions_cleanup, t);
+  else
+    error (_("`%s' is not a supported tracepoint action."), *line);
 }
 
 enum {
@@ -1077,7 +842,7 @@ enum {
 struct memrange
 {
   int type;            /* memrange_absolute for absolute memory range,
-                           else basereg number */
+                           else basereg number */
   bfd_signed_vma start;
   bfd_signed_vma end;
 };
@@ -1092,6 +857,9 @@ struct collection_list
     long next_aexpr_elt;
     struct agent_expr **aexpr_list;
 
+    /* True is the user requested a collection of "$_sdata", "static
+       tracepoint data".  */
+    int strace_data;
   }
 tracepoint_list, stepping_list;
 
@@ -1099,7 +867,7 @@ tracepoint_list, stepping_list;
 
 static int memrange_cmp (const void *, const void *);
 
-/* compare memranges for qsort */
+/* Compare memranges for qsort.  */
 static int
 memrange_cmp (const void *va, const void *vb)
 {
@@ -1138,11 +906,11 @@ memrange_sortmerge (struct collection_list *memranges)
     {
       for (a = 0, b = 1; b < memranges->next_memrange; b++)
        {
-         if (memranges->list[a].type == memranges->list[b].type &&
-             memranges->list[b].start - memranges->list[a].end <=
-             MAX_REGISTER_SIZE)
+         /* If memrange b overlaps or is adjacent to memrange a,
+            merge them.  */
+         if (memranges->list[a].type == memranges->list[b].type
+             && memranges->list[b].start <= memranges->list[a].end)
            {
-             /* memrange b starts before memrange a ends; merge them.  */
              if (memranges->list[b].end > memranges->list[a].end)
                memranges->list[a].end = memranges->list[b].end;
              continue;         /* next b, same a */
@@ -1162,13 +930,13 @@ add_register (struct collection_list *collection, unsigned int regno)
 {
   if (info_verbose)
     printf_filtered ("collect register %d\n", regno);
-  if (regno > (8 * sizeof (collection->regs_mask)))
+  if (regno >= (8 * sizeof (collection->regs_mask)))
     error (_("Internal: register number %d too large for tracepoint"),
           regno);
   collection->regs_mask[regno / 8] |= 1 << (regno % 8);
 }
 
-/* Add a memrange to a collection list */
+/* Add a memrange to a collection list */
 static void
 add_memrange (struct collection_list *memranges, 
              int type, bfd_signed_vma base,
@@ -1195,7 +963,7 @@ add_memrange (struct collection_list *memranges,
                                  memranges->listsize);
     }
 
-  if (type != memrange_absolute)               /* Better collect the base register!  */
+  if (type != memrange_absolute)    /* Better collect the base register!  */
     add_register (memranges, type);
 }
 
@@ -1203,23 +971,26 @@ add_memrange (struct collection_list *memranges,
 static void
 collect_symbol (struct collection_list *collect, 
                struct symbol *sym,
-               long frame_regno, long frame_offset)
+               struct gdbarch *gdbarch,
+               long frame_regno, long frame_offset,
+               CORE_ADDR scope)
 {
   unsigned long len;
   unsigned int reg;
   bfd_signed_vma offset;
+  int treat_as_expr = 0;
 
   len = TYPE_LENGTH (check_typedef (SYMBOL_TYPE (sym)));
   switch (SYMBOL_CLASS (sym))
     {
     default:
       printf_filtered ("%s: don't know symbol class %d\n",
-                      DEPRECATED_SYMBOL_NAME (sym), 
+                      SYMBOL_PRINT_NAME (sym),
                       SYMBOL_CLASS (sym));
       break;
     case LOC_CONST:
-      printf_filtered ("constant %s (value %ld) will not be collected.\n",
-                      DEPRECATED_SYMBOL_NAME (sym), SYMBOL_VALUE (sym));
+      printf_filtered ("constant %s (value %s) will not be collected.\n",
+                      SYMBOL_PRINT_NAME (sym), plongest (SYMBOL_VALUE (sym)));
       break;
     case LOC_STATIC:
       offset = SYMBOL_VALUE_ADDRESS (sym);
@@ -1229,28 +1000,32 @@ collect_symbol (struct collection_list *collect,
 
          sprintf_vma (tmp, offset);
          printf_filtered ("LOC_STATIC %s: collect %ld bytes at %s.\n",
-                          DEPRECATED_SYMBOL_NAME (sym), len, 
+                          SYMBOL_PRINT_NAME (sym), len,
                           tmp /* address */);
        }
-      add_memrange (collect, memrange_absolute, offset, len);
+      /* A struct may be a C++ class with static fields, go to general
+        expression handling.  */
+      if (TYPE_CODE (SYMBOL_TYPE (sym)) == TYPE_CODE_STRUCT)
+       treat_as_expr = 1;
+      else
+       add_memrange (collect, memrange_absolute, offset, len);
       break;
     case LOC_REGISTER:
-    case LOC_REGPARM:
-      reg = SYMBOL_VALUE (sym);
+      reg = SYMBOL_REGISTER_OPS (sym)->register_number (sym, gdbarch);
       if (info_verbose)
        printf_filtered ("LOC_REG[parm] %s: ", 
-                        DEPRECATED_SYMBOL_NAME (sym));
+                        SYMBOL_PRINT_NAME (sym));
       add_register (collect, reg);
       /* Check for doubles stored in two registers.  */
       /* FIXME: how about larger types stored in 3 or more regs?  */
       if (TYPE_CODE (SYMBOL_TYPE (sym)) == TYPE_CODE_FLT &&
-         len > register_size (current_gdbarch, reg))
+         len > register_size (gdbarch, reg))
        add_register (collect, reg + 1);
       break;
     case LOC_REF_ARG:
       printf_filtered ("Sorry, don't know how to do LOC_REF_ARG yet.\n");
       printf_filtered ("       (will not collect %s)\n",
-                      DEPRECATED_SYMBOL_NAME (sym));
+                      SYMBOL_PRINT_NAME (sym));
       break;
     case LOC_ARG:
       reg = frame_regno;
@@ -1258,7 +1033,7 @@ collect_symbol (struct collection_list *collect,
       if (info_verbose)
        {
          printf_filtered ("LOC_LOCAL %s: Collect %ld bytes at offset ",
-                          DEPRECATED_SYMBOL_NAME (sym), len);
+                          SYMBOL_PRINT_NAME (sym), len);
          printf_vma (offset);
          printf_filtered (" from frame ptr reg %d\n", reg);
        }
@@ -1270,103 +1045,166 @@ collect_symbol (struct collection_list *collect,
       if (info_verbose)
        {
          printf_filtered ("LOC_REGPARM_ADDR %s: Collect %ld bytes at offset ",
-                          DEPRECATED_SYMBOL_NAME (sym), len);
+                          SYMBOL_PRINT_NAME (sym), len);
          printf_vma (offset);
          printf_filtered (" from reg %d\n", reg);
        }
       add_memrange (collect, reg, offset, len);
       break;
     case LOC_LOCAL:
-    case LOC_LOCAL_ARG:
       reg = frame_regno;
       offset = frame_offset + SYMBOL_VALUE (sym);
       if (info_verbose)
        {
          printf_filtered ("LOC_LOCAL %s: Collect %ld bytes at offset ",
-                          DEPRECATED_SYMBOL_NAME (sym), len);
+                          SYMBOL_PRINT_NAME (sym), len);
          printf_vma (offset);
          printf_filtered (" from frame ptr reg %d\n", reg);
        }
       add_memrange (collect, reg, offset, len);
       break;
-    case LOC_BASEREG:
-    case LOC_BASEREG_ARG:
-      reg = SYMBOL_BASEREG (sym);
-      offset = SYMBOL_VALUE (sym);
-      if (info_verbose)
-       {
-         printf_filtered ("LOC_BASEREG %s: collect %ld bytes at offset ",
-                          DEPRECATED_SYMBOL_NAME (sym), len);
-         printf_vma (offset);
-         printf_filtered (" from basereg %d\n", reg);
-       }
-      add_memrange (collect, reg, offset, len);
-      break;
+
     case LOC_UNRESOLVED:
-      printf_filtered ("Don't know LOC_UNRESOLVED %s\n", 
-                      DEPRECATED_SYMBOL_NAME (sym));
+      treat_as_expr = 1;
       break;
+
     case LOC_OPTIMIZED_OUT:
       printf_filtered ("%s has been optimized out of existence.\n",
-                      DEPRECATED_SYMBOL_NAME (sym));
+                      SYMBOL_PRINT_NAME (sym));
+      break;
+
+    case LOC_COMPUTED:
+      treat_as_expr = 1;
       break;
     }
+
+  /* Expressions are the most general case.  */
+  if (treat_as_expr)
+    {
+      struct agent_expr *aexpr;
+      struct cleanup *old_chain1 = NULL;
+
+      aexpr = gen_trace_for_var (scope, gdbarch, sym);
+
+      /* It can happen that the symbol is recorded as a computed
+        location, but it's been optimized away and doesn't actually
+        have a location expression.  */
+      if (!aexpr)
+       {
+         printf_filtered ("%s has been optimized out of existence.\n",
+                          SYMBOL_PRINT_NAME (sym));
+         return;
+       }
+
+      old_chain1 = make_cleanup_free_agent_expr (aexpr);
+
+      ax_reqs (aexpr);
+
+      report_agent_reqs_errors (aexpr);
+
+      discard_cleanups (old_chain1);
+      add_aexpr (collect, aexpr);
+
+      /* Take care of the registers.  */
+      if (aexpr->reg_mask_len > 0)
+       {
+         int ndx1, ndx2;
+
+         for (ndx1 = 0; ndx1 < aexpr->reg_mask_len; ndx1++)
+           {
+             QUIT;     /* Allow user to bail out with ^C.  */
+             if (aexpr->reg_mask[ndx1] != 0)
+               {
+                 /* Assume chars have 8 bits.  */
+                 for (ndx2 = 0; ndx2 < 8; ndx2++)
+                   if (aexpr->reg_mask[ndx1] & (1 << ndx2))
+                     /* It's used -- record it.  */
+                     add_register (collect, ndx1 * 8 + ndx2);
+               }
+           }
+       }
+    }
 }
 
-/* Add all locals (or args) symbols to collection list */
+/* Data to be passed around in the calls to the locals and args
+   iterators.  */
+
+struct add_local_symbols_data
+{
+  struct collection_list *collect;
+  struct gdbarch *gdbarch;
+  CORE_ADDR pc;
+  long frame_regno;
+  long frame_offset;
+  int count;
+};
+
+/* The callback for the locals and args iterators.  */
+
 static void
-add_local_symbols (struct collection_list *collect, CORE_ADDR pc,
+do_collect_symbol (const char *print_name,
+                  struct symbol *sym,
+                  void *cb_data)
+{
+  struct add_local_symbols_data *p = cb_data;
+
+  collect_symbol (p->collect, sym, p->gdbarch, p->frame_regno,
+                 p->frame_offset, p->pc);
+  p->count++;
+}
+
+/* Add all locals (or args) symbols to collection list.  */
+static void
+add_local_symbols (struct collection_list *collect,
+                  struct gdbarch *gdbarch, CORE_ADDR pc,
                   long frame_regno, long frame_offset, int type)
 {
-  struct symbol *sym;
   struct block *block;
-  struct dict_iterator iter;
-  int count = 0;
+  struct add_local_symbols_data cb_data;
 
-  block = block_for_pc (pc);
-  while (block != 0)
+  cb_data.collect = collect;
+  cb_data.gdbarch = gdbarch;
+  cb_data.pc = pc;
+  cb_data.frame_regno = frame_regno;
+  cb_data.frame_offset = frame_offset;
+  cb_data.count = 0;
+
+  if (type == 'L')
     {
-      QUIT;                    /* allow user to bail out with ^C */
-      ALL_BLOCK_SYMBOLS (block, iter, sym)
+      block = block_for_pc (pc);
+      if (block == NULL)
        {
-         switch (SYMBOL_CLASS (sym))
-           {
-           default:
-             warning (_("don't know how to trace local symbol %s"), 
-                      DEPRECATED_SYMBOL_NAME (sym));
-           case LOC_LOCAL:
-           case LOC_STATIC:
-           case LOC_REGISTER:
-           case LOC_BASEREG:
-             if (type == 'L')  /* collecting Locals */
-               {
-                 count++;
-                 collect_symbol (collect, sym, frame_regno, 
-                                 frame_offset);
-               }
-             break;
-           case LOC_ARG:
-           case LOC_LOCAL_ARG:
-           case LOC_REF_ARG:
-           case LOC_REGPARM:
-           case LOC_REGPARM_ADDR:
-           case LOC_BASEREG_ARG:
-             if (type == 'A')  /* collecting Arguments */
-               {
-                 count++;
-                 collect_symbol (collect, sym, frame_regno, 
-                                 frame_offset);
-               }
-           }
+         warning (_("Can't collect locals; "
+                    "no symbol table info available.\n"));
+         return;
        }
-      if (BLOCK_FUNCTION (block))
-       break;
-      else
-       block = BLOCK_SUPERBLOCK (block);
+
+      iterate_over_block_local_vars (block, do_collect_symbol, &cb_data);
+      if (cb_data.count == 0)
+       warning (_("No locals found in scope."));
+    }
+  else
+    {
+      pc = get_pc_function_start (pc);
+      block = block_for_pc (pc);
+      if (block == NULL)
+       {
+         warning (_("Can't collect args; no symbol table info available."));
+         return;
+       }
+
+      iterate_over_block_arg_vars (block, do_collect_symbol, &cb_data);
+      if (cb_data.count == 0)
+       warning (_("No args found in scope."));
     }
-  if (count == 0)
-    warning (_("No %s found in scope."), 
-            type == 'L' ? "locals" : "args");
+}
+
+static void
+add_static_trace_data (struct collection_list *collection)
+{
+  if (info_verbose)
+    printf_filtered ("collect static trace data\n");
+  collection->strace_data = 1;
 }
 
 /* worker function */
@@ -1383,9 +1221,10 @@ clear_collection_list (struct collection_list *list)
     }
   list->next_aexpr_elt = 0;
   memset (list->regs_mask, 0, sizeof (list->regs_mask));
+  list->strace_data = 0;
 }
 
-/* reduce a collection list to string form (for gdb protocol) */
+/* Reduce a collection list to string form (for gdb protocol).  */
 static char **
 stringify_collection_list (struct collection_list *list, char *string)
 {
@@ -1397,13 +1236,23 @@ stringify_collection_list (struct collection_list *list, char *string)
   char *end;
   long i;
 
-  count = 1 + list->next_memrange + list->next_aexpr_elt + 1;
+  count = 1 + 1 + list->next_memrange + list->next_aexpr_elt + 1;
   str_list = (char *(*)[]) xmalloc (count * sizeof (char *));
 
+  if (list->strace_data)
+    {
+      if (info_verbose)
+       printf_filtered ("\nCollecting static trace data\n");
+      end = temp_buf;
+      *end++ = 'L';
+      (*str_list)[ndx] = savestring (temp_buf, end - temp_buf);
+      ndx++;
+    }
+
   for (i = sizeof (list->regs_mask) - 1; i > 0; i--)
-    if (list->regs_mask[i] != 0)       /* skip leading zeroes in regs_mask */
+    if (list->regs_mask[i] != 0)    /* Skip leading zeroes in regs_mask.  */
       break;
-  if (list->regs_mask[i] != 0) /* prepare to send regs_mask to the stub */
+  if (list->regs_mask[i] != 0) /* Prepare to send regs_mask to the stub.  */
     {
       if (info_verbose)
        printf_filtered ("\nCollecting registers (mask): 0x");
@@ -1411,13 +1260,13 @@ stringify_collection_list (struct collection_list *list, char *string)
       *end++ = 'R';
       for (; i >= 0; i--)
        {
-         QUIT;                 /* allow user to bail out with ^C */
+         QUIT;                 /* Allow user to bail out with ^C.  */
          if (info_verbose)
            printf_filtered ("%02X", list->regs_mask[i]);
          sprintf (end, "%02X", list->regs_mask[i]);
          end += 2;
        }
-      (*str_list)[ndx] = savestring (temp_buf, end - temp_buf);
+      (*str_list)[ndx] = xstrdup (temp_buf);
       ndx++;
     }
   if (info_verbose)
@@ -1426,7 +1275,7 @@ stringify_collection_list (struct collection_list *list, char *string)
     printf_filtered ("Collecting memranges: \n");
   for (i = 0, count = 0, end = temp_buf; i < list->next_memrange; i++)
     {
-      QUIT;                    /* allow user to bail out with ^C */
+      QUIT;                    /* Allow user to bail out with ^C.  */
       sprintf_vma (tmp2, list->list[i].start);
       if (info_verbose)
        {
@@ -1462,7 +1311,7 @@ stringify_collection_list (struct collection_list *list, char *string)
 
   for (i = 0; i < list->next_aexpr_elt; i++)
     {
-      QUIT;                    /* allow user to bail out with ^C */
+      QUIT;                    /* Allow user to bail out with ^C.  */
       if ((count + 10 + 2 * list->aexpr_list[i]->len) > MAX_AGENT_EXPR_LEN)
        {
          (*str_list)[ndx] = savestring (temp_buf, count);
@@ -1489,90 +1338,65 @@ stringify_collection_list (struct collection_list *list, char *string)
   (*str_list)[ndx] = NULL;
 
   if (ndx == 0)
-    return NULL;
+    {
+      xfree (str_list);
+      return NULL;
+    }
   else
     return *str_list;
 }
 
-static void
-free_actions_list_cleanup_wrapper (void *al)
-{
-  free_actions_list (al);
-}
-
-static void
-free_actions_list (char **actions_list)
-{
-  int ndx;
-
-  if (actions_list == 0)
-    return;
-
-  for (ndx = 0; actions_list[ndx]; ndx++)
-    xfree (actions_list[ndx]);
-
-  xfree (actions_list);
-}
 
-/* Render all actions into gdb protocol.  */
 static void
-encode_actions (struct tracepoint *t, char ***tdp_actions,
-               char ***stepping_actions)
+encode_actions_1 (struct command_line *action,
+                 struct breakpoint *t,
+                 struct bp_location *tloc,
+                 int frame_reg,
+                 LONGEST frame_offset,
+                 struct collection_list *collect,
+                 struct collection_list *stepping_list)
 {
-  static char tdp_buff[2048], step_buff[2048];
   char *action_exp;
   struct expression *exp = NULL;
-  struct action_line *action;
   int i;
   struct value *tempval;
-  struct collection_list *collect;
   struct cmd_list_element *cmd;
   struct agent_expr *aexpr;
-  int frame_reg;
-  LONGEST frame_offset;
 
-
-  clear_collection_list (&tracepoint_list);
-  clear_collection_list (&stepping_list);
-  collect = &tracepoint_list;
-
-  *tdp_actions = NULL;
-  *stepping_actions = NULL;
-
-  TARGET_VIRTUAL_FRAME_POINTER (t->address, &frame_reg, &frame_offset);
-
-  for (action = t->actions; action; action = action->next)
+  for (; action; action = action->next)
     {
-      QUIT;                    /* allow user to bail out with ^C */
-      action_exp = action->action;
+      QUIT;                    /* Allow user to bail out with ^C.  */
+      action_exp = action->line;
       while (isspace ((int) *action_exp))
        action_exp++;
 
-      if (*action_exp == '#')  /* comment line */
-       return;
-
       cmd = lookup_cmd (&action_exp, cmdlist, "", -1, 1);
       if (cmd == 0)
        error (_("Bad action list item: %s"), action_exp);
 
       if (cmd_cfunc_eq (cmd, collect_pseudocommand))
        {
+         trace_string_kludge = 0;
+         if (*action_exp == '/')
+           action_exp = decode_agent_options (action_exp);
+
          do
-           {                   /* repeat over a comma-separated list */
-             QUIT;             /* allow user to bail out with ^C */
+           {                   /* Repeat over a comma-separated list.  */
+             QUIT;             /* Allow user to bail out with ^C.  */
              while (isspace ((int) *action_exp))
                action_exp++;
 
              if (0 == strncasecmp ("$reg", action_exp, 4))
                {
-                 for (i = 0; i < gdbarch_num_regs (current_gdbarch); i++)
+                 for (i = 0; i < gdbarch_num_regs (tloc->gdbarch); i++)
                    add_register (collect, i);
                  action_exp = strchr (action_exp, ',');        /* more? */
                }
              else if (0 == strncasecmp ("$arg", action_exp, 4))
                {
                  add_local_symbols (collect,
-                                    t->address,
+                                    tloc->gdbarch,
+                                    tloc->address,
                                     frame_reg,
                                     frame_offset,
                                     'A');
@@ -1581,79 +1405,129 @@ encode_actions (struct tracepoint *t, char ***tdp_actions,
              else if (0 == strncasecmp ("$loc", action_exp, 4))
                {
                  add_local_symbols (collect,
-                                    t->address,
+                                    tloc->gdbarch,
+                                    tloc->address,
                                     frame_reg,
                                     frame_offset,
                                     'L');
                  action_exp = strchr (action_exp, ',');        /* more? */
                }
+             else if (0 == strncasecmp ("$_ret", action_exp, 5))
+               {
+                 struct cleanup *old_chain1 = NULL;
+
+                 aexpr = gen_trace_for_return_address (tloc->address,
+                                                       tloc->gdbarch);
+
+                 old_chain1 = make_cleanup_free_agent_expr (aexpr);
+
+                 ax_reqs (aexpr);
+                 report_agent_reqs_errors (aexpr);
+
+                 discard_cleanups (old_chain1);
+                 add_aexpr (collect, aexpr);
+
+                 /* take care of the registers */
+                 if (aexpr->reg_mask_len > 0)
+                   {
+                     int ndx1, ndx2;
+
+                     for (ndx1 = 0; ndx1 < aexpr->reg_mask_len; ndx1++)
+                       {
+                         QUIT; /* allow user to bail out with ^C */
+                         if (aexpr->reg_mask[ndx1] != 0)
+                           {
+                             /* assume chars have 8 bits */
+                             for (ndx2 = 0; ndx2 < 8; ndx2++)
+                               if (aexpr->reg_mask[ndx1] & (1 << ndx2))
+                                 /* it's used -- record it */
+                                 add_register (collect, 
+                                               ndx1 * 8 + ndx2);
+                           }
+                       }
+                   }
+
+                 action_exp = strchr (action_exp, ',');        /* more? */
+               }
+             else if (0 == strncasecmp ("$_sdata", action_exp, 7))
+               {
+                 add_static_trace_data (collect);
+                 action_exp = strchr (action_exp, ',');        /* more? */
+               }
              else
                {
-                 unsigned long addr, len;
+                 unsigned long addr;
                  struct cleanup *old_chain = NULL;
                  struct cleanup *old_chain1 = NULL;
-                 struct agent_reqs areqs;
 
-                 exp = parse_exp_1 (&action_exp, 
-                                    block_for_pc (t->address), 1);
+                 exp = parse_exp_1 (&action_exp, tloc->address,
+                                    block_for_pc (tloc->address), 1);
                  old_chain = make_cleanup (free_current_contents, &exp);
 
                  switch (exp->elts[0].opcode)
                    {
                    case OP_REGISTER:
-                     i = exp->elts[1].longconst;
-                     if (info_verbose)
-                       printf_filtered ("OP_REGISTER: ");
-                     add_register (collect, i);
-                     break;
+                     {
+                       const char *name = &exp->elts[2].string;
+
+                       i = user_reg_map_name_to_regnum (tloc->gdbarch,
+                                                        name, strlen (name));
+                       if (i == -1)
+                         internal_error (__FILE__, __LINE__,
+                                         _("Register $%s not available"),
+                                         name);
+                       if (info_verbose)
+                         printf_filtered ("OP_REGISTER: ");
+                       add_register (collect, i);
+                       break;
+                     }
 
                    case UNOP_MEMVAL:
-                     /* safe because we know it's a simple expression */
+                     /* Safe because we know it's a simple expression.  */
                      tempval = evaluate_expression (exp);
-                     addr = VALUE_ADDRESS (tempval) + value_offset (tempval);
-                     len = TYPE_LENGTH (check_typedef (exp->elts[1].type));
-                     add_memrange (collect, memrange_absolute, addr, len);
+                     addr = value_address (tempval);
+                     /* Initialize the TYPE_LENGTH if it is a typedef.  */
+                     check_typedef (exp->elts[1].type);
+                     add_memrange (collect, memrange_absolute, addr,
+                                   TYPE_LENGTH (exp->elts[1].type));
                      break;
 
                    case OP_VAR_VALUE:
                      collect_symbol (collect,
                                      exp->elts[2].symbol,
+                                     tloc->gdbarch,
                                      frame_reg,
-                                     frame_offset);
+                                     frame_offset,
+                                     tloc->address);
                      break;
 
-                   default:    /* full-fledged expression */
-                     aexpr = gen_trace_for_expr (t->address, exp);
+                   default:    /* Full-fledged expression.  */
+                     aexpr = gen_trace_for_expr (tloc->address, exp);
 
                      old_chain1 = make_cleanup_free_agent_expr (aexpr);
 
-                     ax_reqs (aexpr, &areqs);
-                     if (areqs.flaw != agent_flaw_none)
-                       error (_("malformed expression"));
+                     ax_reqs (aexpr);
 
-                     if (areqs.min_height < 0)
-                       error (_("gdb: Internal error: expression has min height < 0"));
-                     if (areqs.max_height > 20)
-                       error (_("expression too complicated, try simplifying"));
+                     report_agent_reqs_errors (aexpr);
 
                      discard_cleanups (old_chain1);
                      add_aexpr (collect, aexpr);
 
-                     /* take care of the registers */
-                     if (areqs.reg_mask_len > 0)
+                     /* Take care of the registers.  */
+                     if (aexpr->reg_mask_len > 0)
                        {
                          int ndx1;
                          int ndx2;
 
-                         for (ndx1 = 0; ndx1 < areqs.reg_mask_len; ndx1++)
+                         for (ndx1 = 0; ndx1 < aexpr->reg_mask_len; ndx1++)
                            {
-                             QUIT;     /* allow user to bail out with ^C */
-                             if (areqs.reg_mask[ndx1] != 0)
+                             QUIT;     /* Allow user to bail out with ^C.  */
+                             if (aexpr->reg_mask[ndx1] != 0)
                                {
-                                 /* assume chars have 8 bits */
+                                 /* Assume chars have 8 bits.  */
                                  for (ndx2 = 0; ndx2 < 8; ndx2++)
-                                   if (areqs.reg_mask[ndx1] & (1 << ndx2))
-                                     /* it's used -- record it */
+                                   if (aexpr->reg_mask[ndx1] & (1 << ndx2))
+                                     /* It's used -- record it.  */
                                      add_register (collect, 
                                                    ndx1 * 8 + ndx2);
                                }
@@ -1666,25 +1540,113 @@ encode_actions (struct tracepoint *t, char ***tdp_actions,
            }
          while (action_exp && *action_exp++ == ',');
        }                       /* if */
-      else if (cmd_cfunc_eq (cmd, while_stepping_pseudocommand))
+      else if (cmd_cfunc_eq (cmd, teval_pseudocommand))
        {
-         collect = &stepping_list;
-       }
-      else if (cmd_cfunc_eq (cmd, end_actions_pseudocommand))
+         do
+           {                   /* Repeat over a comma-separated list.  */
+             QUIT;             /* Allow user to bail out with ^C.  */
+             while (isspace ((int) *action_exp))
+               action_exp++;
+
+               {
+                 struct cleanup *old_chain = NULL;
+                 struct cleanup *old_chain1 = NULL;
+
+                 exp = parse_exp_1 (&action_exp, tloc->address,
+                                    block_for_pc (tloc->address), 1);
+                 old_chain = make_cleanup (free_current_contents, &exp);
+
+                 aexpr = gen_eval_for_expr (tloc->address, exp);
+                 old_chain1 = make_cleanup_free_agent_expr (aexpr);
+
+                 ax_reqs (aexpr);
+                 report_agent_reqs_errors (aexpr);
+
+                 discard_cleanups (old_chain1);
+                 /* Even though we're not officially collecting, add
+                    to the collect list anyway.  */
+                 add_aexpr (collect, aexpr);
+
+                 do_cleanups (old_chain);
+               }               /* do */
+           }
+         while (action_exp && *action_exp++ == ',');
+       }                       /* if */
+      else if (cmd_cfunc_eq (cmd, while_stepping_pseudocommand))
        {
-         if (collect == &stepping_list)        /* end stepping actions */
-           collect = &tracepoint_list;
-         else
-           break;              /* end tracepoint actions */
+         /* We check against nested while-stepping when setting
+            breakpoint action, so no way to run into nested
+            here.  */
+         gdb_assert (stepping_list);
+
+         encode_actions_1 (action->body_list[0], t, tloc, frame_reg,
+                           frame_offset, stepping_list, NULL);
        }
+      else
+       error (_("Invalid tracepoint command '%s'"), action->line);
     }                          /* for */
+}
+
+/* Render all actions into gdb protocol.  */
+
+void
+encode_actions (struct breakpoint *t, struct bp_location *tloc,
+               char ***tdp_actions, char ***stepping_actions)
+{
+  static char tdp_buff[2048], step_buff[2048];
+  char *default_collect_line = NULL;
+  struct command_line *actions;
+  struct command_line *default_collect_action = NULL;
+  int frame_reg;
+  LONGEST frame_offset;
+  struct cleanup *back_to;
+
+  back_to = make_cleanup (null_cleanup, NULL);
+
+  clear_collection_list (&tracepoint_list);
+  clear_collection_list (&stepping_list);
+
+  *tdp_actions = NULL;
+  *stepping_actions = NULL;
+
+  gdbarch_virtual_frame_pointer (tloc->gdbarch,
+                                tloc->address, &frame_reg, &frame_offset);
+
+  actions = breakpoint_commands (t);
+
+  /* If there are default expressions to collect, make up a collect
+     action and prepend to the action list to encode.  Note that since
+     validation is per-tracepoint (local var "xyz" might be valid for
+     one tracepoint and not another, etc), we make up the action on
+     the fly, and don't cache it.  */
+  if (*default_collect)
+    {
+      char *line;
+
+      default_collect_line =  xstrprintf ("collect %s", default_collect);
+      make_cleanup (xfree, default_collect_line);
+
+      line = default_collect_line;
+      validate_actionline (&line, t);
+
+      default_collect_action = xmalloc (sizeof (struct command_line));
+      make_cleanup (xfree, default_collect_action);
+      default_collect_action->next = actions;
+      default_collect_action->line = line;
+      actions = default_collect_action;
+    }
+  encode_actions_1 (actions, t, tloc, frame_reg, frame_offset,
+                   &tracepoint_list, &stepping_list);
+
   memrange_sortmerge (&tracepoint_list);
   memrange_sortmerge (&stepping_list);
 
-  *tdp_actions = stringify_collection_list (&tracepoint_list, 
+  *tdp_actions = stringify_collection_list (&tracepoint_list,
                                            tdp_buff);
-  *stepping_actions = stringify_collection_list (&stepping_list, 
+  *stepping_actions = stringify_collection_list (&stepping_list,
                                                 step_buff);
+
+  do_cleanups (back_to);
 }
 
 static void
@@ -1694,310 +1656,700 @@ add_aexpr (struct collection_list *collect, struct agent_expr *aexpr)
     {
       collect->aexpr_list =
        xrealloc (collect->aexpr_list,
-               2 * collect->aexpr_listsize * sizeof (struct agent_expr *));
+                 2 * collect->aexpr_listsize * sizeof (struct agent_expr *));
       collect->aexpr_listsize *= 2;
     }
   collect->aexpr_list[collect->next_aexpr_elt] = aexpr;
   collect->next_aexpr_elt++;
 }
 
-static char *target_buf;
-static long target_buf_size;
-
-/* Set "transparent" memory ranges
-
-   Allow trace mechanism to treat text-like sections
-   (and perhaps all read-only sections) transparently, 
-   i.e. don't reject memory requests from these address ranges
-   just because they haven't been collected.  */
-
 static void
-remote_set_transparent_ranges (void)
+process_tracepoint_on_disconnect (void)
 {
-  extern bfd *exec_bfd;
-  asection *s;
-  bfd_size_type size;
-  bfd_vma lma;
-  int anysecs = 0;
-
-  if (!exec_bfd)
-    return;                    /* No information to give.  */
-
-  strcpy (target_buf, "QTro");
-  for (s = exec_bfd->sections; s; s = s->next)
+  VEC(breakpoint_p) *tp_vec = NULL;
+  int ix;
+  struct breakpoint *b;
+  int has_pending_p = 0;
+
+  /* Check whether we still have pending tracepoint.  If we have, warn the
+     user that pending tracepoint will no longer work.  */
+  tp_vec = all_tracepoints ();
+  for (ix = 0; VEC_iterate (breakpoint_p, tp_vec, ix, b); ix++)
     {
-      char tmp1[40], tmp2[40];
-
-      if ((s->flags & SEC_LOAD) == 0 ||
-      /* (s->flags & SEC_CODE)     == 0 || */
-         (s->flags & SEC_READONLY) == 0)
-       continue;
+      if (b->loc == NULL)
+       {
+         has_pending_p = 1;
+         break;
+       }
+      else
+       {
+         struct bp_location *loc1;
 
-      anysecs = 1;
-      lma = s->lma;
-      size = bfd_get_section_size (s);
-      sprintf_vma (tmp1, lma);
-      sprintf_vma (tmp2, lma + size);
-      sprintf (target_buf + strlen (target_buf), 
-              ":%s,%s", tmp1, tmp2);
-    }
-  if (anysecs)
-    {
-      putpkt (target_buf);
-      getpkt (&target_buf, &target_buf_size, 0);
+         for (loc1 = b->loc; loc1; loc1 = loc1->next)
+           {
+             if (loc1->shlib_disabled)
+               {
+                 has_pending_p = 1;
+                 break;
+               }
+           }
+
+         if (has_pending_p)
+           break;
+       }
     }
+  VEC_free (breakpoint_p, tp_vec);
+
+  if (has_pending_p)
+    warning (_("Pending tracepoints will not be resolved while"
+              " GDB is disconnected\n"));
 }
 
-/* tstart command:
 
-   Tell target to clear any previous trace experiment.
-   Walk the list of tracepoints, and send them (and their actions)
-   to the target.  If no errors, 
-   Tell target to start a new trace experiment.  */
+void
+start_tracing (char *notes)
+{
+  VEC(breakpoint_p) *tp_vec = NULL;
+  int ix;
+  struct breakpoint *b;
+  struct trace_state_variable *tsv;
+  int any_enabled = 0, num_to_download = 0;
+  int ret;
+
+  tp_vec = all_tracepoints ();
+
+  /* No point in tracing without any tracepoints...  */
+  if (VEC_length (breakpoint_p, tp_vec) == 0)
+    {
+      VEC_free (breakpoint_p, tp_vec);
+      error (_("No tracepoints defined, not starting trace"));
+    }
+
+  for (ix = 0; VEC_iterate (breakpoint_p, tp_vec, ix, b); ix++)
+    {
+      struct tracepoint *t = (struct tracepoint *) b;
+      struct bp_location *loc;
+
+      if (b->enable_state == bp_enabled)
+       any_enabled = 1;
+
+      if ((b->type == bp_fast_tracepoint
+          ? may_insert_fast_tracepoints
+          : may_insert_tracepoints))
+       ++num_to_download;
+      else
+       warning (_("May not insert %stracepoints, skipping tracepoint %d"),
+                (b->type == bp_fast_tracepoint ? "fast " : ""), b->number);
+    }
+
+  if (!any_enabled)
+    {
+      if (target_supports_enable_disable_tracepoint ())
+       warning (_("No tracepoints enabled"));
+      else
+       {
+         /* No point in tracing with only disabled tracepoints that
+            cannot be re-enabled.  */
+         VEC_free (breakpoint_p, tp_vec);
+         error (_("No tracepoints enabled, not starting trace"));
+       }
+    }
+
+  if (num_to_download <= 0)
+    {
+      VEC_free (breakpoint_p, tp_vec);
+      error (_("No tracepoints that may be downloaded, not starting trace"));
+    }
+
+  target_trace_init ();
+
+  for (ix = 0; VEC_iterate (breakpoint_p, tp_vec, ix, b); ix++)
+    {
+      struct tracepoint *t = (struct tracepoint *) b;
+      struct bp_location *loc;
+
+      /* Clear `inserted' flag.  */
+      for (loc = b->loc; loc; loc = loc->next)
+       loc->inserted = 0;
+
+      if ((b->type == bp_fast_tracepoint
+          ? !may_insert_fast_tracepoints
+          : !may_insert_tracepoints))
+       continue;
+
+      t->number_on_target = 0;
+
+      for (loc = b->loc; loc; loc = loc->next)
+       {
+         /* Since tracepoint locations are never duplicated, `inserted'
+            flag should be zero.  */
+         gdb_assert (!loc->inserted);
+
+         target_download_tracepoint (loc);
+
+         loc->inserted = 1;
+       }
+
+      t->number_on_target = b->number;
+
+      for (loc = b->loc; loc; loc = loc->next)
+       if (loc->probe != NULL)
+         loc->probe->pops->set_semaphore (loc->probe, loc->gdbarch);
+    }
+  VEC_free (breakpoint_p, tp_vec);
+
+  /* Send down all the trace state variables too.  */
+  for (ix = 0; VEC_iterate (tsv_s, tvariables, ix, tsv); ++ix)
+    {
+      target_download_trace_state_variable (tsv);
+    }
+  
+  /* Tell target to treat text-like sections as transparent.  */
+  target_trace_set_readonly_regions ();
+  /* Set some mode flags.  */
+  target_set_disconnected_tracing (disconnected_tracing);
+  target_set_circular_trace_buffer (circular_trace_buffer);
+
+  if (!notes)
+    notes = trace_notes;
+  ret = target_set_trace_notes (trace_user, notes, NULL);
+
+  if (!ret && (trace_user || notes))
+    warning (_("Target does not support trace user/notes, info ignored"));
+
+  /* Now insert traps and begin collecting data.  */
+  target_trace_start ();
+
+  /* Reset our local state.  */
+  set_traceframe_num (-1);
+  set_tracepoint_num (-1);
+  set_traceframe_context (NULL);
+  current_trace_status()->running = 1;
+  clear_traceframe_info ();
+}
+
+/* The tstart command requests the target to start a new trace run.
+   The command passes any arguments it has to the target verbatim, as
+   an optional "trace note".  This is useful as for instance a warning
+   to other users if the trace runs disconnected, and you don't want
+   anybody else messing with the target.  */
 
 static void
 trace_start_command (char *args, int from_tty)
 {
-  struct tracepoint *t;
-  char buf[2048];
-  char **tdp_actions;
-  char **stepping_actions;
-  int ndx;
-  struct cleanup *old_chain = NULL;
-
   dont_repeat ();      /* Like "run", dangerous to repeat accidentally.  */
 
-  if (target_is_remote ())
+  if (current_trace_status ()->running)
     {
-      putpkt ("QTinit");
-      remote_get_noisy_reply (&target_buf, &target_buf_size);
-      if (strcmp (target_buf, "OK"))
-       error (_("Target does not support this command."));
-
-      ALL_TRACEPOINTS (t)
-      {
-       char tmp[40];
-
-       sprintf_vma (tmp, t->address);
-       sprintf (buf, "QTDP:%x:%s:%c:%lx:%x", t->number, 
-                tmp, /* address */
-                t->enabled_p ? 'E' : 'D',
-                t->step_count, t->pass_count);
-
-       if (t->actions)
-         strcat (buf, "-");
-       putpkt (buf);
-       remote_get_noisy_reply (&target_buf, &target_buf_size);
-       if (strcmp (target_buf, "OK"))
-         error (_("Target does not support tracepoints."));
-
-       if (t->actions)
-         {
-           encode_actions (t, &tdp_actions, &stepping_actions);
-           old_chain = make_cleanup (free_actions_list_cleanup_wrapper,
-                                     tdp_actions);
-           (void) make_cleanup (free_actions_list_cleanup_wrapper,
-                                stepping_actions);
-
-           /* do_single_steps (t); */
-           if (tdp_actions)
-             {
-               for (ndx = 0; tdp_actions[ndx]; ndx++)
-                 {
-                   QUIT;       /* allow user to bail out with ^C */
-                   sprintf (buf, "QTDP:-%x:%s:%s%c",
-                            t->number, tmp, /* address */
-                            tdp_actions[ndx],
-                            ((tdp_actions[ndx + 1] || stepping_actions)
-                             ? '-' : 0));
-                   putpkt (buf);
-                   remote_get_noisy_reply (&target_buf,
-                                           &target_buf_size);
-                   if (strcmp (target_buf, "OK"))
-                     error (_("Error on target while setting tracepoints."));
-                 }
-             }
-           if (stepping_actions)
-             {
-               for (ndx = 0; stepping_actions[ndx]; ndx++)
-                 {
-                   QUIT;       /* allow user to bail out with ^C */
-                   sprintf (buf, "QTDP:-%x:%s:%s%s%s",
-                            t->number, tmp, /* address */
-                            ((ndx == 0) ? "S" : ""),
-                            stepping_actions[ndx],
-                            (stepping_actions[ndx + 1] ? "-" : ""));
-                   putpkt (buf);
-                   remote_get_noisy_reply (&target_buf,
-                                           &target_buf_size);
-                   if (strcmp (target_buf, "OK"))
-                     error (_("Error on target while setting tracepoints."));
-                 }
-             }
-
-           do_cleanups (old_chain);
-         }
-      }
-      /* Tell target to treat text-like sections as transparent.  */
-      remote_set_transparent_ranges ();
-      /* Now insert traps and begin collecting data.  */
-      putpkt ("QTStart");
-      remote_get_noisy_reply (&target_buf, &target_buf_size);
-      if (strcmp (target_buf, "OK"))
-       error (_("Bogus reply from target: %s"), target_buf);
-      set_traceframe_num (-1); /* All old traceframes invalidated.  */
-      set_tracepoint_num (-1);
-      set_traceframe_context (-1);
-      trace_running_p = 1;
-      if (deprecated_trace_start_stop_hook)
-       deprecated_trace_start_stop_hook (1, from_tty);
-
+      if (from_tty
+         && !query (_("A trace is running already.  Start a new run? ")))
+       error (_("New trace run not started."));
     }
-  else
-    error (_("Trace can only be run on remote targets."));
+
+  start_tracing (args);
 }
 
-/* tstop command */
+/* The tstop command stops the tracing run.  The command passes any
+   supplied arguments to the target verbatim as a "stop note"; if the
+   target supports trace notes, then it will be reported back as part
+   of the trace run's status.  */
+
 static void
 trace_stop_command (char *args, int from_tty)
 {
-  if (target_is_remote ())
+  if (!current_trace_status ()->running)
+    error (_("Trace is not running."));
+
+  stop_tracing (args);
+}
+
+void
+stop_tracing (char *note)
+{
+  int ret;
+  VEC(breakpoint_p) *tp_vec = NULL;
+  int ix;
+  struct breakpoint *t;
+
+  target_trace_stop ();
+
+  tp_vec = all_tracepoints ();
+  for (ix = 0; VEC_iterate (breakpoint_p, tp_vec, ix, t); ix++)
     {
-      putpkt ("QTStop");
-      remote_get_noisy_reply (&target_buf, &target_buf_size);
-      if (strcmp (target_buf, "OK"))
-       error (_("Bogus reply from target: %s"), target_buf);
-      trace_running_p = 0;
-      if (deprecated_trace_start_stop_hook)
-       deprecated_trace_start_stop_hook (0, from_tty);
+      struct bp_location *loc;
+
+      if ((t->type == bp_fast_tracepoint
+          ? !may_insert_fast_tracepoints
+          : !may_insert_tracepoints))
+       continue;
+
+      for (loc = t->loc; loc; loc = loc->next)
+       {
+         /* GDB can be totally absent in some disconnected trace scenarios,
+            but we don't really care if this semaphore goes out of sync.
+            That's why we are decrementing it here, but not taking care
+            in other places.  */
+         if (loc->probe != NULL)
+           loc->probe->pops->clear_semaphore (loc->probe, loc->gdbarch);
+       }
     }
-  else
-    error (_("Trace can only be run on remote targets."));
-}
 
-unsigned long trace_running_p;
+  VEC_free (breakpoint_p, tp_vec);
+
+  if (!note)
+    note = trace_stop_notes;
+  ret = target_set_trace_notes (NULL, NULL, note);
+
+  if (!ret && note)
+    warning (_("Target does not support trace notes, note ignored"));
+
+  /* Should change in response to reply?  */
+  current_trace_status ()->running = 0;
+}
 
 /* tstatus command */
 static void
 trace_status_command (char *args, int from_tty)
 {
-  if (target_is_remote ())
+  struct trace_status *ts = current_trace_status ();
+  int status, ix;
+  VEC(breakpoint_p) *tp_vec = NULL;
+  struct breakpoint *t;
+  
+  status = target_get_trace_status (ts);
+
+  if (status == -1)
+    {
+      if (ts->from_file)
+       printf_filtered (_("Using a trace file.\n"));
+      else
+       {
+         printf_filtered (_("Trace can not be run on this target.\n"));
+         return;
+       }
+    }
+
+  if (!ts->running_known)
+    {
+      printf_filtered (_("Run/stop status is unknown.\n"));
+    }
+  else if (ts->running)
+    {
+      printf_filtered (_("Trace is running on the target.\n"));
+    }
+  else
+    {
+      switch (ts->stop_reason)
+       {
+       case trace_never_run:
+         printf_filtered (_("No trace has been run on the target.\n"));
+         break;
+       case tstop_command:
+         if (ts->stop_desc)
+           printf_filtered (_("Trace stopped by a tstop command (%s).\n"),
+                            ts->stop_desc);
+         else
+           printf_filtered (_("Trace stopped by a tstop command.\n"));
+         break;
+       case trace_buffer_full:
+         printf_filtered (_("Trace stopped because the buffer was full.\n"));
+         break;
+       case trace_disconnected:
+         printf_filtered (_("Trace stopped because of disconnection.\n"));
+         break;
+       case tracepoint_passcount:
+         printf_filtered (_("Trace stopped by tracepoint %d.\n"),
+                          ts->stopping_tracepoint);
+         break;
+       case tracepoint_error:
+         if (ts->stopping_tracepoint)
+           printf_filtered (_("Trace stopped by an "
+                              "error (%s, tracepoint %d).\n"),
+                            ts->stop_desc, ts->stopping_tracepoint);
+         else
+           printf_filtered (_("Trace stopped by an error (%s).\n"),
+                            ts->stop_desc);
+         break;
+       case trace_stop_reason_unknown:
+         printf_filtered (_("Trace stopped for an unknown reason.\n"));
+         break;
+       default:
+         printf_filtered (_("Trace stopped for some other reason (%d).\n"),
+                          ts->stop_reason);
+         break;
+       }
+    }
+
+  if (ts->traceframes_created >= 0
+      && ts->traceframe_count != ts->traceframes_created)
+    {
+      printf_filtered (_("Buffer contains %d trace "
+                        "frames (of %d created total).\n"),
+                      ts->traceframe_count, ts->traceframes_created);
+    }
+  else if (ts->traceframe_count >= 0)
+    {
+      printf_filtered (_("Collected %d trace frames.\n"),
+                      ts->traceframe_count);
+    }
+
+  if (ts->buffer_free >= 0)
+    {
+      if (ts->buffer_size >= 0)
+       {
+         printf_filtered (_("Trace buffer has %d bytes of %d bytes free"),
+                          ts->buffer_free, ts->buffer_size);
+         if (ts->buffer_size > 0)
+           printf_filtered (_(" (%d%% full)"),
+                            ((int) ((((long long) (ts->buffer_size
+                                                   - ts->buffer_free)) * 100)
+                                    / ts->buffer_size)));
+         printf_filtered (_(".\n"));
+       }
+      else
+       printf_filtered (_("Trace buffer has %d bytes free.\n"),
+                        ts->buffer_free);
+    }
+
+  if (ts->disconnected_tracing)
+    printf_filtered (_("Trace will continue if GDB disconnects.\n"));
+  else
+    printf_filtered (_("Trace will stop if GDB disconnects.\n"));
+
+  if (ts->circular_buffer)
+    printf_filtered (_("Trace buffer is circular.\n"));
+
+  if (ts->user_name && strlen (ts->user_name) > 0)
+    printf_filtered (_("Trace user is %s.\n"), ts->user_name);
+
+  if (ts->notes && strlen (ts->notes) > 0)
+    printf_filtered (_("Trace notes: %s.\n"), ts->notes);
+
+  /* Now report on what we're doing with tfind.  */
+  if (traceframe_number >= 0)
+    printf_filtered (_("Looking at trace frame %d, tracepoint %d.\n"),
+                    traceframe_number, tracepoint_number);
+  else
+    printf_filtered (_("Not looking at any trace frame.\n"));
+
+  /* Report start/stop times if supplied.  */
+  if (ts->start_time)
+    {
+      if (ts->stop_time)
+       {
+         LONGEST run_time = ts->stop_time - ts->start_time;
+
+         /* Reporting a run time is more readable than two long numbers.  */
+         printf_filtered (_("Trace started at %ld.%06ld secs, stopped %ld.%06ld secs later.\n"),
+                          (long int) ts->start_time / 1000000,
+                          (long int) ts->start_time % 1000000,
+                          (long int) run_time / 1000000,
+                          (long int) run_time % 1000000);
+       }
+      else
+       printf_filtered (_("Trace started at %ld.%06ld secs.\n"),
+                        (long int) ts->start_time / 1000000,
+                        (long int) ts->start_time % 1000000);
+    }
+  else if (ts->stop_time)
+    printf_filtered (_("Trace stopped at %ld.%06ld secs.\n"),
+                    (long int) ts->stop_time / 1000000,
+                    (long int) ts->stop_time % 1000000);
+
+  /* Now report any per-tracepoint status available.  */
+  tp_vec = all_tracepoints ();
+
+  for (ix = 0; VEC_iterate (breakpoint_p, tp_vec, ix, t); ix++)
+    target_get_tracepoint_status (t, NULL);
+
+  VEC_free (breakpoint_p, tp_vec);
+}
+
+/* Report the trace status to uiout, in a way suitable for MI, and not
+   suitable for CLI.  If ON_STOP is true, suppress a few fields that
+   are not meaningful in the -trace-stop response.
+
+   The implementation is essentially parallel to trace_status_command, but
+   merging them will result in unreadable code.  */
+void
+trace_status_mi (int on_stop)
+{
+  struct ui_out *uiout = current_uiout;
+  struct trace_status *ts = current_trace_status ();
+  int status;
+
+  status = target_get_trace_status (ts);
+
+  if (status == -1 && !ts->from_file)
     {
-      putpkt ("qTStatus");
-      remote_get_noisy_reply (&target_buf, &target_buf_size);
+      ui_out_field_string (uiout, "supported", "0");
+      return;
+    }
+
+  if (ts->from_file)
+    ui_out_field_string (uiout, "supported", "file");
+  else if (!on_stop)
+    ui_out_field_string (uiout, "supported", "1");
 
-      if (target_buf[0] != 'T' ||
-         (target_buf[1] != '0' && target_buf[1] != '1'))
-       error (_("Bogus reply from target: %s"), target_buf);
+  gdb_assert (ts->running_known);
 
-      /* exported for use by the GUI */
-      trace_running_p = (target_buf[1] == '1');
+  if (ts->running)
+    {
+      ui_out_field_string (uiout, "running", "1");
+
+      /* Unlike CLI, do not show the state of 'disconnected-tracing' variable.
+        Given that the frontend gets the status either on -trace-stop, or from
+        -trace-status after re-connection, it does not seem like this
+        information is necessary for anything.  It is not necessary for either
+        figuring the vital state of the target nor for navigation of trace
+        frames.  If the frontend wants to show the current state is some
+        configure dialog, it can request the value when such dialog is
+        invoked by the user.  */
     }
   else
-    error (_("Trace can only be run on remote targets."));
+    {
+      char *stop_reason = NULL;
+      int stopping_tracepoint = -1;
+
+      if (!on_stop)
+       ui_out_field_string (uiout, "running", "0");
+
+      if (ts->stop_reason != trace_stop_reason_unknown)
+       {
+         switch (ts->stop_reason)
+           {
+           case tstop_command:
+             stop_reason = "request";
+             break;
+           case trace_buffer_full:
+             stop_reason = "overflow";
+             break;
+           case trace_disconnected:
+             stop_reason = "disconnection";
+             break;
+           case tracepoint_passcount:
+             stop_reason = "passcount";
+             stopping_tracepoint = ts->stopping_tracepoint;
+             break;
+           case tracepoint_error:
+             stop_reason = "error";
+             stopping_tracepoint = ts->stopping_tracepoint;
+             break;
+           }
+         
+         if (stop_reason)
+           {
+             ui_out_field_string (uiout, "stop-reason", stop_reason);
+             if (stopping_tracepoint != -1)
+               ui_out_field_int (uiout, "stopping-tracepoint",
+                                 stopping_tracepoint);
+             if (ts->stop_reason == tracepoint_error)
+               ui_out_field_string (uiout, "error-description",
+                                    ts->stop_desc);
+           }
+       }
+    }
+
+  if (ts->traceframe_count != -1)
+    ui_out_field_int (uiout, "frames", ts->traceframe_count);
+  if (ts->traceframes_created != -1)
+    ui_out_field_int (uiout, "frames-created", ts->traceframes_created);
+  if (ts->buffer_size != -1)
+    ui_out_field_int (uiout, "buffer-size", ts->buffer_size);
+  if (ts->buffer_free != -1)
+    ui_out_field_int (uiout, "buffer-free", ts->buffer_free);
+
+  ui_out_field_int (uiout, "disconnected",  ts->disconnected_tracing);
+  ui_out_field_int (uiout, "circular",  ts->circular_buffer);
+
+  ui_out_field_string (uiout, "user-name", ts->user_name);
+  ui_out_field_string (uiout, "notes", ts->notes);
+
+  {
+    char buf[100];
+
+    xsnprintf (buf, sizeof buf, "%ld.%06ld",
+              (long int) ts->start_time / 1000000,
+              (long int) ts->start_time % 1000000);
+    ui_out_field_string (uiout, "start-time", buf);
+    xsnprintf (buf, sizeof buf, "%ld.%06ld",
+              (long int) ts->stop_time / 1000000,
+              (long int) ts->stop_time % 1000000);
+    ui_out_field_string (uiout, "stop-time", buf);
+  }
+}
+
+/* This function handles the details of what to do about an ongoing
+   tracing run if the user has asked to detach or otherwise disconnect
+   from the target.  */
+void
+disconnect_tracing (int from_tty)
+{
+  /* It can happen that the target that was tracing went away on its
+     own, and we didn't notice.  Get a status update, and if the
+     current target doesn't even do tracing, then assume it's not
+     running anymore.  */
+  if (target_get_trace_status (current_trace_status ()) < 0)
+    current_trace_status ()->running = 0;
+
+  /* If running interactively, give the user the option to cancel and
+     then decide what to do differently with the run.  Scripts are
+     just going to disconnect and let the target deal with it,
+     according to how it's been instructed previously via
+     disconnected-tracing.  */
+  if (current_trace_status ()->running && from_tty)
+    {
+      process_tracepoint_on_disconnect ();
+
+      if (current_trace_status ()->disconnected_tracing)
+       {
+         if (!query (_("Trace is running and will "
+                       "continue after detach; detach anyway? ")))
+           error (_("Not confirmed."));
+       }
+      else
+       {
+         if (!query (_("Trace is running but will "
+                       "stop on detach; detach anyway? ")))
+           error (_("Not confirmed."));
+       }
+    }
+
+  /* Also we want to be out of tfind mode, otherwise things can get
+     confusing upon reconnection.  Just use these calls instead of
+     full tfind_1 behavior because we're in the middle of detaching,
+     and there's no point to updating current stack frame etc.  */
+  set_current_traceframe (-1);
+  set_tracepoint_num (-1);
+  set_traceframe_context (NULL);
 }
 
 /* Worker function for the various flavors of the tfind command.  */
-static void
-finish_tfind_command (char **msg,
-                     long *sizeof_msg,
-                     int from_tty)
+void
+tfind_1 (enum trace_find_type type, int num,
+        ULONGEST addr1, ULONGEST addr2,
+        int from_tty)
 {
   int target_frameno = -1, target_tracept = -1;
-  CORE_ADDR old_frame_addr;
-  struct symbol *old_func;
-  char *reply;
+  struct frame_id old_frame_id = null_frame_id;
+  struct tracepoint *tp;
+  struct ui_out *uiout = current_uiout;
+
+  /* Only try to get the current stack frame if we have a chance of
+     succeeding.  In particular, if we're trying to get a first trace
+     frame while all threads are running, it's not going to succeed,
+     so leave it with a default value and let the frame comparison
+     below (correctly) decide to print out the source location of the
+     trace frame.  */
+  if (!(type == tfind_number && num == -1)
+      && (has_stack_frames () || traceframe_number >= 0))
+    old_frame_id = get_frame_id (get_current_frame ());
+
+  target_frameno = target_trace_find (type, num, addr1, addr2,
+                                     &target_tracept);
+  
+  if (type == tfind_number
+      && num == -1
+      && target_frameno == -1)
+    {
+      /* We told the target to get out of tfind mode, and it did.  */
+    }
+  else if (target_frameno == -1)
+    {
+      /* A request for a non-existent trace frame has failed.
+        Our response will be different, depending on FROM_TTY:
+
+        If FROM_TTY is true, meaning that this command was 
+        typed interactively by the user, then give an error
+        and DO NOT change the state of traceframe_number etc.
+
+        However if FROM_TTY is false, meaning that we're either
+        in a script, a loop, or a user-defined command, then 
+        DON'T give an error, but DO change the state of
+        traceframe_number etc. to invalid.
+
+        The rationalle is that if you typed the command, you
+        might just have committed a typo or something, and you'd
+        like to NOT lose your current debugging state.  However
+        if you're in a user-defined command or especially in a
+        loop, then you need a way to detect that the command
+        failed WITHOUT aborting.  This allows you to write
+        scripts that search thru the trace buffer until the end,
+        and then continue on to do something else.  */
+  
+      if (from_tty)
+       error (_("Target failed to find requested trace frame."));
+      else
+       {
+         if (info_verbose)
+           printf_filtered ("End of trace buffer.\n");
+#if 0 /* dubious now?  */
+         /* The following will not recurse, since it's
+            special-cased.  */
+         trace_find_command ("-1", from_tty);
+#endif
+       }
+    }
+  
+  tp = get_tracepoint_by_number_on_target (target_tracept);
 
-  old_frame_addr = get_frame_base (get_current_frame ());
-  old_func = find_pc_function (read_pc ());
+  reinit_frame_cache ();
+  target_dcache_invalidate ();
 
-  putpkt (*msg);
-  reply = remote_get_noisy_reply (msg, sizeof_msg);
+  set_tracepoint_num (tp ? tp->base.number : target_tracept);
 
-  while (reply && *reply)
-    switch (*reply)
-      {
-      case 'F':
-       if ((target_frameno = (int) strtol (++reply, &reply, 16)) == -1)
-         {
-           /* A request for a non-existant trace frame has failed.
-              Our response will be different, depending on FROM_TTY:
-
-              If FROM_TTY is true, meaning that this command was 
-              typed interactively by the user, then give an error
-              and DO NOT change the state of traceframe_number etc.
-
-              However if FROM_TTY is false, meaning that we're either
-              in a script, a loop, or a user-defined command, then 
-              DON'T give an error, but DO change the state of
-              traceframe_number etc. to invalid.
-
-              The rationalle is that if you typed the command, you
-              might just have committed a typo or something, and you'd
-              like to NOT lose your current debugging state.  However
-              if you're in a user-defined command or especially in a
-              loop, then you need a way to detect that the command
-              failed WITHOUT aborting.  This allows you to write
-              scripts that search thru the trace buffer until the end,
-              and then continue on to do something else.  */
-
-           if (from_tty)
-             error (_("Target failed to find requested trace frame."));
-           else
-             {
-               if (info_verbose)
-                 printf_filtered ("End of trace buffer.\n");
-               /* The following will not recurse, since it's
-                  special-cased.  */
-               trace_find_command ("-1", from_tty);
-               reply = NULL;   /* Break out of loop 
-                                  (avoid recursive nonsense).  */
-             }
-         }
-       break;
-      case 'T':
-       if ((target_tracept = (int) strtol (++reply, &reply, 16)) == -1)
-         error (_("Target failed to find requested trace frame."));
-       break;
-      case 'O':                /* "OK"? */
-       if (reply[1] == 'K' && reply[2] == '\0')
-         reply += 2;
-       else
-         error (_("Bogus reply from target: %s"), reply);
-       break;
-      default:
-       error (_("Bogus reply from target: %s"), reply);
-      }
+  if (target_frameno != get_traceframe_number ())
+    observer_notify_traceframe_changed (target_frameno, tracepoint_number);
+
+  set_current_traceframe (target_frameno);
 
-  reinit_frame_cache ();
-  registers_changed ();
-  set_traceframe_num (target_frameno);
-  set_tracepoint_num (target_tracept);
   if (target_frameno == -1)
-    set_traceframe_context (-1);
+    set_traceframe_context (NULL);
   else
-    set_traceframe_context (read_pc ());
+    set_traceframe_context (get_current_frame ());
 
-  if (from_tty)
+  if (traceframe_number >= 0)
+    {
+      /* Use different branches for MI and CLI to make CLI messages
+        i18n-eable.  */
+      if (ui_out_is_mi_like_p (uiout))
+       {
+         ui_out_field_string (uiout, "found", "1");
+         ui_out_field_int (uiout, "tracepoint", tracepoint_number);
+         ui_out_field_int (uiout, "traceframe", traceframe_number);
+       }
+      else
+       {
+         printf_unfiltered (_("Found trace frame %d, tracepoint %d\n"),
+                            traceframe_number, tracepoint_number);
+       }
+    }
+  else
+    {
+      if (ui_out_is_mi_like_p (uiout))
+       ui_out_field_string (uiout, "found", "0");
+      else if (type == tfind_number && num == -1)
+       printf_unfiltered (_("No longer looking at any trace frame\n"));
+      else /* This case may never occur, check.  */
+       printf_unfiltered (_("No trace frame found\n"));
+    }
+
+  /* If we're in nonstop mode and getting out of looking at trace
+     frames, there won't be any current frame to go back to and
+     display.  */
+  if (from_tty
+      && (has_stack_frames () || traceframe_number >= 0))
     {
       enum print_what print_what;
 
-      /* NOTE: in immitation of the step command, try to determine
+      /* NOTE: in imitation of the step command, try to determine
          whether we have made a transition from one function to
          another.  If so, we'll print the "stack frame" (ie. the new
          function and it's arguments) -- otherwise we'll just show the
-         new source line.
-
-         This determination is made by checking (1) whether the
-         current function has changed, and (2) whether the current FP
-         has changed.  Hack: if the FP wasn't collected, either at the
-         current or the previous frame, assume that the FP has NOT
-         changed.  */
-
-      if (old_func == find_pc_function (read_pc ()) &&
-         (old_frame_addr == 0 ||
-          get_frame_base (get_current_frame ()) == 0 ||
-          old_frame_addr == get_frame_base (get_current_frame ())))
+         new source line.  */
+
+      if (frame_id_eq (old_frame_id,
+                      get_frame_id (get_current_frame ())))
        print_what = SRC_LINE;
       else
        print_what = SRC_AND_LOC;
@@ -2024,41 +2376,38 @@ finish_tfind_command (char **msg,
 /* tfind command */
 static void
 trace_find_command (char *args, int from_tty)
-{ /* this should only be called with a numeric argument */
+{ /* This should only be called with a numeric argument.  */
   int frameno = -1;
 
-  if (target_is_remote ())
+  if (current_trace_status ()->running && !current_trace_status ()->from_file)
+    error (_("May not look at trace frames while trace is running."));
+  
+  if (args == 0 || *args == 0)
+    { /* TFIND with no args means find NEXT trace frame.  */
+      if (traceframe_number == -1)
+       frameno = 0;    /* "next" is first one.  */
+        else
+       frameno = traceframe_number + 1;
+    }
+  else if (0 == strcmp (args, "-"))
     {
-      if (deprecated_trace_find_hook)
-       deprecated_trace_find_hook (args, from_tty);
-
-      if (args == 0 || *args == 0)
-       { /* TFIND with no args means find NEXT trace frame.  */
-         if (traceframe_number == -1)
-           frameno = 0;        /* "next" is first one */
-         else
-           frameno = traceframe_number + 1;
-       }
-      else if (0 == strcmp (args, "-"))
-       {
-         if (traceframe_number == -1)
-           error (_("not debugging trace buffer"));
-         else if (from_tty && traceframe_number == 0)
-           error (_("already at start of trace buffer"));
-
-         frameno = traceframe_number - 1;
-       }
-      else
-       frameno = parse_and_eval_long (args);
+      if (traceframe_number == -1)
+       error (_("not debugging trace buffer"));
+      else if (from_tty && traceframe_number == 0)
+       error (_("already at start of trace buffer"));
+      
+      frameno = traceframe_number - 1;
+      }
+  /* A hack to work around eval's need for fp to have been collected.  */
+  else if (0 == strcmp (args, "-1"))
+    frameno = -1;
+  else
+    frameno = parse_and_eval_long (args);
 
-      if (frameno < -1)
-       error (_("invalid input (%d is less than zero)"), frameno);
+  if (frameno < -1)
+    error (_("invalid input (%d is less than zero)"), frameno);
 
-      sprintf (target_buf, "QTFrame:%x", frameno);
-      finish_tfind_command (&target_buf, &target_buf_size, from_tty);
-    }
-  else
-    error (_("Trace can only be run on remote targets."));
+  tfind_1 (tfind_number, frameno, 0, 0, from_tty);
 }
 
 /* tfind end */
@@ -2068,13 +2417,6 @@ trace_find_end_command (char *args, int from_tty)
   trace_find_command ("-1", from_tty);
 }
 
-/* tfind none */
-static void
-trace_find_none_command (char *args, int from_tty)
-{
-  trace_find_command ("-1", from_tty);
-}
-
 /* tfind start */
 static void
 trace_find_start_command (char *args, int from_tty)
@@ -2087,21 +2429,16 @@ static void
 trace_find_pc_command (char *args, int from_tty)
 {
   CORE_ADDR pc;
-  char tmp[40];
 
-  if (target_is_remote ())
-    {
-      if (args == 0 || *args == 0)
-       pc = read_pc ();        /* default is current pc */
-      else
-       pc = parse_and_eval_address (args);
+  if (current_trace_status ()->running && !current_trace_status ()->from_file)
+    error (_("May not look at trace frames while trace is running."));
 
-      sprintf_vma (tmp, pc);
-      sprintf (target_buf, "QTFrame:pc:%s", tmp);
-      finish_tfind_command (&target_buf, &target_buf_size, from_tty);
-    }
+  if (args == 0 || *args == 0)
+    pc = regcache_read_pc (get_current_regcache ());
   else
-    error (_("Trace can only be run on remote targets."));
+    pc = parse_and_eval_address (args);
+
+  tfind_1 (tfind_pc, 0, pc, 0, from_tty);
 }
 
 /* tfind tracepoint command */
@@ -2109,24 +2446,29 @@ static void
 trace_find_tracepoint_command (char *args, int from_tty)
 {
   int tdp;
+  struct tracepoint *tp;
 
-  if (target_is_remote ())
+  if (current_trace_status ()->running && !current_trace_status ()->from_file)
+    error (_("May not look at trace frames while trace is running."));
+
+  if (args == 0 || *args == 0)
     {
-      if (args == 0 || *args == 0)
-       {
-         if (tracepoint_number == -1)
-           error (_("No current tracepoint -- please supply an argument."));
-         else
-           tdp = tracepoint_number;    /* default is current TDP */
-       }
+      if (tracepoint_number == -1)
+       error (_("No current tracepoint -- please supply an argument."));
       else
-       tdp = parse_and_eval_long (args);
-
-      sprintf (target_buf, "QTFrame:tdp:%x", tdp);
-      finish_tfind_command (&target_buf, &target_buf_size, from_tty);
+       tdp = tracepoint_number;        /* Default is current TDP.  */
     }
   else
-    error (_("Trace can only be run on remote targets."));
+    tdp = parse_and_eval_long (args);
+
+  /* If we have the tracepoint on hand, use the number that the
+     target knows about (which may be different if we disconnected
+     and reconnected).  */
+  tp = get_tracepoint (tdp);
+  if (tp)
+    tdp = tp->number_on_target;
+
+  tfind_1 (tfind_tp, tdp, 0, 0, from_tty);
 }
 
 /* TFIND LINE command:
@@ -2144,545 +2486,2658 @@ trace_find_line_command (char *args, int from_tty)
   struct symtabs_and_lines sals;
   struct symtab_and_line sal;
   struct cleanup *old_chain;
-  char   startpc_str[40], endpc_str[40];
 
-  if (target_is_remote ())
+  if (current_trace_status ()->running && !current_trace_status ()->from_file)
+    error (_("May not look at trace frames while trace is running."));
+
+  if (args == 0 || *args == 0)
+    {
+      sal = find_pc_line (get_frame_pc (get_current_frame ()), 0);
+      sals.nelts = 1;
+      sals.sals = (struct symtab_and_line *)
+       xmalloc (sizeof (struct symtab_and_line));
+      sals.sals[0] = sal;
+    }
+  else
+    {
+      sals = decode_line_with_current_source (args, DECODE_LINE_FUNFIRSTLINE);
+      sal = sals.sals[0];
+    }
+  
+  old_chain = make_cleanup (xfree, sals.sals);
+  if (sal.symtab == 0)
+    error (_("No line number information available."));
+
+  if (sal.line > 0 && find_line_pc_range (sal, &start_pc, &end_pc))
+    {
+      if (start_pc == end_pc)
+       {
+         printf_filtered ("Line %d of \"%s\"",
+                          sal.line, sal.symtab->filename);
+         wrap_here ("  ");
+         printf_filtered (" is at address ");
+         print_address (get_current_arch (), start_pc, gdb_stdout);
+         wrap_here ("  ");
+         printf_filtered (" but contains no code.\n");
+         sal = find_pc_line (start_pc, 0);
+         if (sal.line > 0
+             && find_line_pc_range (sal, &start_pc, &end_pc)
+             && start_pc != end_pc)
+           printf_filtered ("Attempting to find line %d instead.\n",
+                            sal.line);
+         else
+           error (_("Cannot find a good line."));
+       }
+      }
+    else
+    /* Is there any case in which we get here, and have an address
+       which the user would want to see?  If we have debugging
+       symbols and no line numbers?  */
+    error (_("Line number %d is out of range for \"%s\"."),
+          sal.line, sal.symtab->filename);
+
+  /* Find within range of stated line.  */
+  if (args && *args)
+    tfind_1 (tfind_range, 0, start_pc, end_pc - 1, from_tty);
+  else
+    tfind_1 (tfind_outside, 0, start_pc, end_pc - 1, from_tty);
+  do_cleanups (old_chain);
+}
+
+/* tfind range command */
+static void
+trace_find_range_command (char *args, int from_tty)
+{
+  static CORE_ADDR start, stop;
+  char *tmp;
+
+  if (current_trace_status ()->running && !current_trace_status ()->from_file)
+    error (_("May not look at trace frames while trace is running."));
+
+  if (args == 0 || *args == 0)
+    { /* XXX FIXME: what should default behavior be?  */
+      printf_filtered ("Usage: tfind range <startaddr>,<endaddr>\n");
+      return;
+    }
+
+  if (0 != (tmp = strchr (args, ',')))
+    {
+      *tmp++ = '\0';   /* Terminate start address.  */
+      while (isspace ((int) *tmp))
+       tmp++;
+      start = parse_and_eval_address (args);
+      stop = parse_and_eval_address (tmp);
+    }
+  else
+    {                  /* No explicit end address?  */
+      start = parse_and_eval_address (args);
+      stop = start + 1;        /* ??? */
+    }
+
+  tfind_1 (tfind_range, 0, start, stop, from_tty);
+}
+
+/* tfind outside command */
+static void
+trace_find_outside_command (char *args, int from_tty)
+{
+  CORE_ADDR start, stop;
+  char *tmp;
+
+  if (current_trace_status ()->running && !current_trace_status ()->from_file)
+    error (_("May not look at trace frames while trace is running."));
+
+  if (args == 0 || *args == 0)
+    { /* XXX FIXME: what should default behavior be?  */
+      printf_filtered ("Usage: tfind outside <startaddr>,<endaddr>\n");
+      return;
+    }
+
+  if (0 != (tmp = strchr (args, ',')))
+    {
+      *tmp++ = '\0';   /* Terminate start address.  */
+      while (isspace ((int) *tmp))
+       tmp++;
+      start = parse_and_eval_address (args);
+      stop = parse_and_eval_address (tmp);
+    }
+  else
+    {                  /* No explicit end address?  */
+      start = parse_and_eval_address (args);
+      stop = start + 1;        /* ??? */
+    }
+
+  tfind_1 (tfind_outside, 0, start, stop, from_tty);
+}
+
+/* info scope command: list the locals for a scope.  */
+static void
+scope_info (char *args, int from_tty)
+{
+  struct symtabs_and_lines sals;
+  struct symbol *sym;
+  struct minimal_symbol *msym;
+  struct block *block;
+  const char *symname;
+  char *save_args = args;
+  struct block_iterator iter;
+  int j, count = 0;
+  struct gdbarch *gdbarch;
+  int regno;
+
+  if (args == 0 || *args == 0)
+    error (_("requires an argument (function, "
+            "line or *addr) to define a scope"));
+
+  sals = decode_line_1 (&args, DECODE_LINE_FUNFIRSTLINE, NULL, 0);
+  if (sals.nelts == 0)
+    return;            /* Presumably decode_line_1 has already warned.  */
+
+  /* Resolve line numbers to PC.  */
+  resolve_sal_pc (&sals.sals[0]);
+  block = block_for_pc (sals.sals[0].pc);
+
+  while (block != 0)
+    {
+      QUIT;                    /* Allow user to bail out with ^C.  */
+      ALL_BLOCK_SYMBOLS (block, iter, sym)
+       {
+         QUIT;                 /* Allow user to bail out with ^C.  */
+         if (count == 0)
+           printf_filtered ("Scope for %s:\n", save_args);
+         count++;
+
+         symname = SYMBOL_PRINT_NAME (sym);
+         if (symname == NULL || *symname == '\0')
+           continue;           /* Probably botched, certainly useless.  */
+
+         gdbarch = get_objfile_arch (SYMBOL_SYMTAB (sym)->objfile);
+
+         printf_filtered ("Symbol %s is ", symname);
+         switch (SYMBOL_CLASS (sym))
+           {
+           default:
+           case LOC_UNDEF:     /* Messed up symbol?  */
+             printf_filtered ("a bogus symbol, class %d.\n",
+                              SYMBOL_CLASS (sym));
+             count--;          /* Don't count this one.  */
+             continue;
+           case LOC_CONST:
+             printf_filtered ("a constant with value %s (%s)",
+                              plongest (SYMBOL_VALUE (sym)),
+                              hex_string (SYMBOL_VALUE (sym)));
+             break;
+           case LOC_CONST_BYTES:
+             printf_filtered ("constant bytes: ");
+             if (SYMBOL_TYPE (sym))
+               for (j = 0; j < TYPE_LENGTH (SYMBOL_TYPE (sym)); j++)
+                 fprintf_filtered (gdb_stdout, " %02x",
+                                   (unsigned) SYMBOL_VALUE_BYTES (sym)[j]);
+             break;
+           case LOC_STATIC:
+             printf_filtered ("in static storage at address ");
+             printf_filtered ("%s", paddress (gdbarch,
+                                              SYMBOL_VALUE_ADDRESS (sym)));
+             break;
+           case LOC_REGISTER:
+             /* GDBARCH is the architecture associated with the objfile
+                the symbol is defined in; the target architecture may be
+                different, and may provide additional registers.  However,
+                we do not know the target architecture at this point.
+                We assume the objfile architecture will contain all the
+                standard registers that occur in debug info in that
+                objfile.  */
+             regno = SYMBOL_REGISTER_OPS (sym)->register_number (sym,
+                                                                 gdbarch);
+
+             if (SYMBOL_IS_ARGUMENT (sym))
+               printf_filtered ("an argument in register $%s",
+                                gdbarch_register_name (gdbarch, regno));
+             else
+               printf_filtered ("a local variable in register $%s",
+                                gdbarch_register_name (gdbarch, regno));
+             break;
+           case LOC_ARG:
+             printf_filtered ("an argument at stack/frame offset %s",
+                              plongest (SYMBOL_VALUE (sym)));
+             break;
+           case LOC_LOCAL:
+             printf_filtered ("a local variable at frame offset %s",
+                              plongest (SYMBOL_VALUE (sym)));
+             break;
+           case LOC_REF_ARG:
+             printf_filtered ("a reference argument at offset %s",
+                              plongest (SYMBOL_VALUE (sym)));
+             break;
+           case LOC_REGPARM_ADDR:
+             /* Note comment at LOC_REGISTER.  */
+             regno = SYMBOL_REGISTER_OPS (sym)->register_number (sym,
+                                                                 gdbarch);
+             printf_filtered ("the address of an argument, in register $%s",
+                              gdbarch_register_name (gdbarch, regno));
+             break;
+           case LOC_TYPEDEF:
+             printf_filtered ("a typedef.\n");
+             continue;
+           case LOC_LABEL:
+             printf_filtered ("a label at address ");
+             printf_filtered ("%s", paddress (gdbarch,
+                                              SYMBOL_VALUE_ADDRESS (sym)));
+             break;
+           case LOC_BLOCK:
+             printf_filtered ("a function at address ");
+             printf_filtered ("%s",
+               paddress (gdbarch, BLOCK_START (SYMBOL_BLOCK_VALUE (sym))));
+             break;
+           case LOC_UNRESOLVED:
+             msym = lookup_minimal_symbol (SYMBOL_LINKAGE_NAME (sym),
+                                           NULL, NULL);
+             if (msym == NULL)
+               printf_filtered ("Unresolved Static");
+             else
+               {
+                 printf_filtered ("static storage at address ");
+                 printf_filtered ("%s",
+                   paddress (gdbarch, SYMBOL_VALUE_ADDRESS (msym)));
+               }
+             break;
+           case LOC_OPTIMIZED_OUT:
+             printf_filtered ("optimized out.\n");
+             continue;
+           case LOC_COMPUTED:
+             SYMBOL_COMPUTED_OPS (sym)->describe_location (sym,
+                                                           BLOCK_START (block),
+                                                           gdb_stdout);
+             break;
+           }
+         if (SYMBOL_TYPE (sym))
+           printf_filtered (", length %d.\n",
+                            TYPE_LENGTH (check_typedef (SYMBOL_TYPE (sym))));
+       }
+      if (BLOCK_FUNCTION (block))
+       break;
+      else
+       block = BLOCK_SUPERBLOCK (block);
+    }
+  if (count <= 0)
+    printf_filtered ("Scope for %s contains no locals or arguments.\n",
+                    save_args);
+}
+
+/* worker function (cleanup) */
+static void
+replace_comma (void *data)
+{
+  char *comma = data;
+  *comma = ',';
+}
+
+
+/* Helper for trace_dump_command.  Dump the action list starting at
+   ACTION.  STEPPING_ACTIONS is true if we're iterating over the
+   actions of the body of a while-stepping action.  STEPPING_FRAME is
+   set if the current traceframe was determined to be a while-stepping
+   traceframe.  */
+
+static void
+trace_dump_actions (struct command_line *action,
+                   int stepping_actions, int stepping_frame,
+                   int from_tty)
+{
+  char *action_exp, *next_comma;
+
+  for (; action != NULL; action = action->next)
+    {
+      struct cmd_list_element *cmd;
+
+      QUIT;                    /* Allow user to bail out with ^C.  */
+      action_exp = action->line;
+      while (isspace ((int) *action_exp))
+       action_exp++;
+
+      /* The collection actions to be done while stepping are
+         bracketed by the commands "while-stepping" and "end".  */
+
+      if (*action_exp == '#')  /* comment line */
+       continue;
+
+      cmd = lookup_cmd (&action_exp, cmdlist, "", -1, 1);
+      if (cmd == 0)
+       error (_("Bad action list item: %s"), action_exp);
+
+      if (cmd_cfunc_eq (cmd, while_stepping_pseudocommand))
+       {
+         int i;
+
+         for (i = 0; i < action->body_count; ++i)
+           trace_dump_actions (action->body_list[i],
+                               1, stepping_frame, from_tty);
+       }
+      else if (cmd_cfunc_eq (cmd, collect_pseudocommand))
+       {
+         /* Display the collected data.
+            For the trap frame, display only what was collected at
+            the trap.  Likewise for stepping frames, display only
+            what was collected while stepping.  This means that the
+            two boolean variables, STEPPING_FRAME and
+            STEPPING_ACTIONS should be equal.  */
+         if (stepping_frame == stepping_actions)
+           {
+             if (*action_exp == '/')
+               action_exp = decode_agent_options (action_exp);
+
+             do
+               {               /* Repeat over a comma-separated list.  */
+                 QUIT;         /* Allow user to bail out with ^C.  */
+                 if (*action_exp == ',')
+                   action_exp++;
+                 while (isspace ((int) *action_exp))
+                   action_exp++;
+
+                 next_comma = strchr (action_exp, ',');
+
+                 if (0 == strncasecmp (action_exp, "$reg", 4))
+                   registers_info (NULL, from_tty);
+                 else if (0 == strncasecmp (action_exp, "$_ret", 5))
+                   ;
+                 else if (0 == strncasecmp (action_exp, "$loc", 4))
+                   locals_info (NULL, from_tty);
+                 else if (0 == strncasecmp (action_exp, "$arg", 4))
+                   args_info (NULL, from_tty);
+                 else
+                   {           /* variable */
+                     if (next_comma)
+                       {
+                         make_cleanup (replace_comma, next_comma);
+                         *next_comma = '\0';
+                       }
+                     printf_filtered ("%s = ", action_exp);
+                     output_command (action_exp, from_tty);
+                     printf_filtered ("\n");
+                   }
+                 if (next_comma)
+                   *next_comma = ',';
+                 action_exp = next_comma;
+               }
+             while (action_exp && *action_exp == ',');
+           }
+       }
+    }
+}
+
+/* The tdump command.  */
+
+static void
+trace_dump_command (char *args, int from_tty)
+{
+  struct regcache *regcache;
+  struct tracepoint *t;
+  int stepping_frame = 0;
+  struct bp_location *loc;
+  char *line, *default_collect_line = NULL;
+  struct command_line *actions, *default_collect_action = NULL;
+  struct cleanup *old_chain = NULL;
+
+  if (tracepoint_number == -1)
+    {
+      warning (_("No current trace frame."));
+      return;
+    }
+
+  t = get_tracepoint (tracepoint_number);
+
+  if (t == NULL)
+    error (_("No known tracepoint matches 'current' tracepoint #%d."),
+          tracepoint_number);
+
+  printf_filtered ("Data collected at tracepoint %d, trace frame %d:\n",
+                  tracepoint_number, traceframe_number);
+
+  /* The current frame is a trap frame if the frame PC is equal
+     to the tracepoint PC.  If not, then the current frame was
+     collected during single-stepping.  */
+
+  regcache = get_current_regcache ();
+
+  /* If the traceframe's address matches any of the tracepoint's
+     locations, assume it is a direct hit rather than a while-stepping
+     frame.  (FIXME this is not reliable, should record each frame's
+     type.)  */
+  stepping_frame = 1;
+  for (loc = t->base.loc; loc; loc = loc->next)
+    if (loc->address == regcache_read_pc (regcache))
+      stepping_frame = 0;
+
+  actions = breakpoint_commands (&t->base);
+
+  /* If there is a default-collect list, make up a collect command,
+     prepend to the tracepoint's commands, and pass the whole mess to
+     the trace dump scanner.  We need to validate because
+     default-collect might have been junked since the trace run.  */
+  if (*default_collect)
+    {
+      default_collect_line = xstrprintf ("collect %s", default_collect);
+      old_chain = make_cleanup (xfree, default_collect_line);
+      line = default_collect_line;
+      validate_actionline (&line, &t->base);
+      default_collect_action = xmalloc (sizeof (struct command_line));
+      make_cleanup (xfree, default_collect_action);
+      default_collect_action->next = actions;
+      default_collect_action->line = line;
+      actions = default_collect_action;
+    }
+
+  trace_dump_actions (actions, 0, stepping_frame, from_tty);
+
+  if (*default_collect)
+    do_cleanups (old_chain);
+}
+
+/* Encode a piece of a tracepoint's source-level definition in a form
+   that is suitable for both protocol and saving in files.  */
+/* This version does not do multiple encodes for long strings; it should
+   return an offset to the next piece to encode.  FIXME  */
+
+extern int
+encode_source_string (int tpnum, ULONGEST addr,
+                     char *srctype, char *src, char *buf, int buf_size)
+{
+  if (80 + strlen (srctype) > buf_size)
+    error (_("Buffer too small for source encoding"));
+  sprintf (buf, "%x:%s:%s:%x:%x:",
+          tpnum, phex_nz (addr, sizeof (addr)),
+          srctype, 0, (int) strlen (src));
+  if (strlen (buf) + strlen (src) * 2 >= buf_size)
+    error (_("Source string too long for buffer"));
+  bin2hex (src, buf + strlen (buf), 0);
+  return -1;
+}
+
+extern int trace_regblock_size;
+
+/* Save tracepoint data to file named FILENAME.  If TARGET_DOES_SAVE is
+   non-zero, the save is performed on the target, otherwise GDB obtains all
+   trace data and saves it locally.  */
+
+void
+trace_save (const char *filename, int target_does_save)
+{
+  struct cleanup *cleanup;
+  char *pathname;
+  struct trace_status *ts = current_trace_status ();
+  int err, status;
+  FILE *fp;
+  struct uploaded_tp *uploaded_tps = NULL, *utp;
+  struct uploaded_tsv *uploaded_tsvs = NULL, *utsv;
+  int a;
+  char *act;
+  LONGEST gotten = 0;
+  ULONGEST offset = 0;
+#define MAX_TRACE_UPLOAD 2000
+  gdb_byte buf[MAX_TRACE_UPLOAD];
+  int written;
+
+  /* If the target is to save the data to a file on its own, then just
+     send the command and be done with it.  */
+  if (target_does_save)
+    {
+      err = target_save_trace_data (filename);
+      if (err < 0)
+       error (_("Target failed to save trace data to '%s'."),
+              filename);
+      return;
+    }
+
+  /* Get the trace status first before opening the file, so if the
+     target is losing, we can get out without touching files.  */
+  status = target_get_trace_status (ts);
+
+  pathname = tilde_expand (filename);
+  cleanup = make_cleanup (xfree, pathname);
+
+  fp = fopen (pathname, "wb");
+  if (!fp)
+    error (_("Unable to open file '%s' for saving trace data (%s)"),
+          filename, safe_strerror (errno));
+  make_cleanup_fclose (fp);
+
+  /* Write a file header, with a high-bit-set char to indicate a
+     binary file, plus a hint as what this file is, and a version
+     number in case of future needs.  */
+  written = fwrite ("\x7fTRACE0\n", 8, 1, fp);
+  if (written < 1)
+    perror_with_name (pathname);
+
+  /* Write descriptive info.  */
+
+  /* Write out the size of a register block.  */
+  fprintf (fp, "R %x\n", trace_regblock_size);
+
+  /* Write out status of the tracing run (aka "tstatus" info).  */
+  fprintf (fp, "status %c;%s",
+          (ts->running ? '1' : '0'), stop_reason_names[ts->stop_reason]);
+  if (ts->stop_reason == tracepoint_error)
+    {
+      char *buf = (char *) alloca (strlen (ts->stop_desc) * 2 + 1);
+
+      bin2hex ((gdb_byte *) ts->stop_desc, buf, 0);
+      fprintf (fp, ":%s", buf);
+    }
+  fprintf (fp, ":%x", ts->stopping_tracepoint);
+  if (ts->traceframe_count >= 0)
+    fprintf (fp, ";tframes:%x", ts->traceframe_count);
+  if (ts->traceframes_created >= 0)
+    fprintf (fp, ";tcreated:%x", ts->traceframes_created);
+  if (ts->buffer_free >= 0)
+    fprintf (fp, ";tfree:%x", ts->buffer_free);
+  if (ts->buffer_size >= 0)
+    fprintf (fp, ";tsize:%x", ts->buffer_size);
+  if (ts->disconnected_tracing)
+    fprintf (fp, ";disconn:%x", ts->disconnected_tracing);
+  if (ts->circular_buffer)
+    fprintf (fp, ";circular:%x", ts->circular_buffer);
+  fprintf (fp, "\n");
+
+  /* Note that we want to upload tracepoints and save those, rather
+     than simply writing out the local ones, because the user may have
+     changed tracepoints in GDB in preparation for a future tracing
+     run, or maybe just mass-deleted all types of breakpoints as part
+     of cleaning up.  So as not to contaminate the session, leave the
+     data in its uploaded form, don't make into real tracepoints.  */
+
+  /* Get trace state variables first, they may be checked when parsing
+     uploaded commands.  */
+
+  target_upload_trace_state_variables (&uploaded_tsvs);
+
+  for (utsv = uploaded_tsvs; utsv; utsv = utsv->next)
+    {
+      char *buf = "";
+
+      if (utsv->name)
+       {
+         buf = (char *) xmalloc (strlen (utsv->name) * 2 + 1);
+         bin2hex ((gdb_byte *) (utsv->name), buf, 0);
+       }
+
+      fprintf (fp, "tsv %x:%s:%x:%s\n",
+              utsv->number, phex_nz (utsv->initial_value, 8),
+              utsv->builtin, buf);
+
+      if (utsv->name)
+       xfree (buf);
+    }
+
+  free_uploaded_tsvs (&uploaded_tsvs);
+
+  target_upload_tracepoints (&uploaded_tps);
+
+  for (utp = uploaded_tps; utp; utp = utp->next)
+    target_get_tracepoint_status (NULL, utp);
+
+  for (utp = uploaded_tps; utp; utp = utp->next)
+    {
+      fprintf (fp, "tp T%x:%s:%c:%x:%x",
+              utp->number, phex_nz (utp->addr, sizeof (utp->addr)),
+              (utp->enabled ? 'E' : 'D'), utp->step, utp->pass);
+      if (utp->type == bp_fast_tracepoint)
+       fprintf (fp, ":F%x", utp->orig_size);
+      if (utp->cond)
+       fprintf (fp, ":X%x,%s", (unsigned int) strlen (utp->cond) / 2,
+                utp->cond);
+      fprintf (fp, "\n");
+      for (a = 0; VEC_iterate (char_ptr, utp->actions, a, act); ++a)
+       fprintf (fp, "tp A%x:%s:%s\n",
+                utp->number, phex_nz (utp->addr, sizeof (utp->addr)), act);
+      for (a = 0; VEC_iterate (char_ptr, utp->step_actions, a, act); ++a)
+       fprintf (fp, "tp S%x:%s:%s\n",
+                utp->number, phex_nz (utp->addr, sizeof (utp->addr)), act);
+      if (utp->at_string)
+       {
+         encode_source_string (utp->number, utp->addr,
+                               "at", utp->at_string, buf, MAX_TRACE_UPLOAD);
+         fprintf (fp, "tp Z%s\n", buf);
+       }
+      if (utp->cond_string)
+       {
+         encode_source_string (utp->number, utp->addr,
+                               "cond", utp->cond_string,
+                               buf, MAX_TRACE_UPLOAD);
+         fprintf (fp, "tp Z%s\n", buf);
+       }
+      for (a = 0; VEC_iterate (char_ptr, utp->cmd_strings, a, act); ++a)
+       {
+         encode_source_string (utp->number, utp->addr, "cmd", act,
+                               buf, MAX_TRACE_UPLOAD);
+         fprintf (fp, "tp Z%s\n", buf);
+       }
+      fprintf (fp, "tp V%x:%s:%x:%s\n",
+              utp->number, phex_nz (utp->addr, sizeof (utp->addr)),
+              utp->hit_count,
+              phex_nz (utp->traceframe_usage,
+                       sizeof (utp->traceframe_usage)));
+    }
+
+  free_uploaded_tps (&uploaded_tps);
+
+  /* Mark the end of the definition section.  */
+  fprintf (fp, "\n");
+
+  /* Get and write the trace data proper.  We ask for big blocks, in
+     the hopes of efficiency, but will take less if the target has
+     packet size limitations or some such.  */
+  while (1)
+    {
+      gotten = target_get_raw_trace_data (buf, offset, MAX_TRACE_UPLOAD);
+      if (gotten < 0)
+       error (_("Failure to get requested trace buffer data"));
+      /* No more data is forthcoming, we're done.  */
+      if (gotten == 0)
+       break;
+      written = fwrite (buf, gotten, 1, fp);
+      if (written < 1)
+       perror_with_name (pathname);
+      offset += gotten;
+    }
+
+  /* Mark the end of trace data.  (We know that gotten is 0 at this point.)  */
+  written = fwrite (&gotten, 4, 1, fp);
+  if (written < 1)
+    perror_with_name (pathname);
+
+  do_cleanups (cleanup);
+}
+
+static void
+trace_save_command (char *args, int from_tty)
+{
+  int target_does_save = 0;
+  char **argv;
+  char *filename = NULL;
+  struct cleanup *back_to;
+
+  if (args == NULL)
+    error_no_arg (_("file in which to save trace data"));
+
+  argv = gdb_buildargv (args);
+  back_to = make_cleanup_freeargv (argv);
+
+  for (; *argv; ++argv)
+    {
+      if (strcmp (*argv, "-r") == 0)
+       target_does_save = 1;
+      else if (**argv == '-')
+       error (_("unknown option `%s'"), *argv);
+      else
+       filename = *argv;
+    }
+
+  if (!filename)
+    error_no_arg (_("file in which to save trace data"));
+
+  trace_save (filename, target_does_save);
+
+  if (from_tty)
+    printf_filtered (_("Trace data saved to file '%s'.\n"), filename);
+
+  do_cleanups (back_to);
+}
+
+/* Tell the target what to do with an ongoing tracing run if GDB
+   disconnects for some reason.  */
+
+static void
+set_disconnected_tracing (char *args, int from_tty,
+                         struct cmd_list_element *c)
+{
+  target_set_disconnected_tracing (disconnected_tracing);
+}
+
+static void
+set_circular_trace_buffer (char *args, int from_tty,
+                          struct cmd_list_element *c)
+{
+  target_set_circular_trace_buffer (circular_trace_buffer);
+}
+
+static void
+set_trace_user (char *args, int from_tty,
+               struct cmd_list_element *c)
+{
+  int ret;
+
+  ret = target_set_trace_notes (trace_user, NULL, NULL);
+
+  if (!ret)
+    warning (_("Target does not support trace notes, user ignored"));
+}
+
+static void
+set_trace_notes (char *args, int from_tty,
+                struct cmd_list_element *c)
+{
+  int ret;
+
+  ret = target_set_trace_notes (NULL, trace_notes, NULL);
+
+  if (!ret)
+    warning (_("Target does not support trace notes, note ignored"));
+}
+
+static void
+set_trace_stop_notes (char *args, int from_tty,
+                     struct cmd_list_element *c)
+{
+  int ret;
+
+  ret = target_set_trace_notes (NULL, NULL, trace_stop_notes);
+
+  if (!ret)
+    warning (_("Target does not support trace notes, stop note ignored"));
+}
+
+/* Convert the memory pointed to by mem into hex, placing result in buf.
+ * Return a pointer to the last char put in buf (null)
+ * "stolen" from sparc-stub.c
+ */
+
+static const char hexchars[] = "0123456789abcdef";
+
+static char *
+mem2hex (gdb_byte *mem, char *buf, int count)
+{
+  gdb_byte ch;
+
+  while (count-- > 0)
+    {
+      ch = *mem++;
+
+      *buf++ = hexchars[ch >> 4];
+      *buf++ = hexchars[ch & 0xf];
+    }
+
+  *buf = 0;
+
+  return buf;
+}
+
+int
+get_traceframe_number (void)
+{
+  return traceframe_number;
+}
+
+/* Make the traceframe NUM be the current trace frame.  Does nothing
+   if NUM is already current.  */
+
+void
+set_current_traceframe (int num)
+{
+  int newnum;
+
+  if (traceframe_number == num)
+    {
+      /* Nothing to do.  */
+      return;
+    }
+
+  newnum = target_trace_find (tfind_number, num, 0, 0, NULL);
+
+  if (newnum != num)
+    warning (_("could not change traceframe"));
+
+  set_traceframe_num (newnum);
+
+  /* Changing the traceframe changes our view of registers and of the
+     frame chain.  */
+  registers_changed ();
+
+  clear_traceframe_info ();
+}
+
+/* Make the traceframe NUM be the current trace frame, and do nothing
+   more.  */
+
+void
+set_traceframe_number (int num)
+{
+  traceframe_number = num;
+}
+
+/* A cleanup used when switching away and back from tfind mode.  */
+
+struct current_traceframe_cleanup
+{
+  /* The traceframe we were inspecting.  */
+  int traceframe_number;
+};
+
+static void
+do_restore_current_traceframe_cleanup (void *arg)
+{
+  struct current_traceframe_cleanup *old = arg;
+
+  set_current_traceframe (old->traceframe_number);
+}
+
+static void
+restore_current_traceframe_cleanup_dtor (void *arg)
+{
+  struct current_traceframe_cleanup *old = arg;
+
+  xfree (old);
+}
+
+struct cleanup *
+make_cleanup_restore_current_traceframe (void)
+{
+  struct current_traceframe_cleanup *old;
+
+  old = xmalloc (sizeof (struct current_traceframe_cleanup));
+  old->traceframe_number = traceframe_number;
+
+  return make_cleanup_dtor (do_restore_current_traceframe_cleanup, old,
+                           restore_current_traceframe_cleanup_dtor);
+}
+
+struct cleanup *
+make_cleanup_restore_traceframe_number (void)
+{
+  return make_cleanup_restore_integer (&traceframe_number);
+}
+
+/* Given a number and address, return an uploaded tracepoint with that
+   number, creating if necessary.  */
+
+struct uploaded_tp *
+get_uploaded_tp (int num, ULONGEST addr, struct uploaded_tp **utpp)
+{
+  struct uploaded_tp *utp;
+
+  for (utp = *utpp; utp; utp = utp->next)
+    if (utp->number == num && utp->addr == addr)
+      return utp;
+  utp = (struct uploaded_tp *) xmalloc (sizeof (struct uploaded_tp));
+  memset (utp, 0, sizeof (struct uploaded_tp));
+  utp->number = num;
+  utp->addr = addr;
+  utp->actions = NULL;
+  utp->step_actions = NULL;
+  utp->cmd_strings = NULL;
+  utp->next = *utpp;
+  *utpp = utp;
+  return utp;
+}
+
+static void
+free_uploaded_tps (struct uploaded_tp **utpp)
+{
+  struct uploaded_tp *next_one;
+
+  while (*utpp)
+    {
+      next_one = (*utpp)->next;
+      xfree (*utpp);
+      *utpp = next_one;
+    }
+}
+
+/* Given a number and address, return an uploaded tracepoint with that
+   number, creating if necessary.  */
+
+static struct uploaded_tsv *
+get_uploaded_tsv (int num, struct uploaded_tsv **utsvp)
+{
+  struct uploaded_tsv *utsv;
+
+  for (utsv = *utsvp; utsv; utsv = utsv->next)
+    if (utsv->number == num)
+      return utsv;
+  utsv = (struct uploaded_tsv *) xmalloc (sizeof (struct uploaded_tsv));
+  memset (utsv, 0, sizeof (struct uploaded_tsv));
+  utsv->number = num;
+  utsv->next = *utsvp;
+  *utsvp = utsv;
+  return utsv;
+}
+
+static void
+free_uploaded_tsvs (struct uploaded_tsv **utsvp)
+{
+  struct uploaded_tsv *next_one;
+
+  while (*utsvp)
+    {
+      next_one = (*utsvp)->next;
+      xfree (*utsvp);
+      *utsvp = next_one;
+    }
+}
+
+/* FIXME this function is heuristic and will miss the cases where the
+   conditional is semantically identical but differs in whitespace,
+   such as "x == 0" vs "x==0".  */
+
+static int
+cond_string_is_same (char *str1, char *str2)
+{
+  if (str1 == NULL || str2 == NULL)
+    return (str1 == str2);
+
+  return (strcmp (str1, str2) == 0);
+}
+
+/* Look for an existing tracepoint that seems similar enough to the
+   uploaded one.  Enablement isn't compared, because the user can
+   toggle that freely, and may have done so in anticipation of the
+   next trace run.  Return the location of matched tracepoint.  */
+
+static struct bp_location *
+find_matching_tracepoint_location (struct uploaded_tp *utp)
+{
+  VEC(breakpoint_p) *tp_vec = all_tracepoints ();
+  int ix;
+  struct breakpoint *b;
+  struct bp_location *loc;
+
+  for (ix = 0; VEC_iterate (breakpoint_p, tp_vec, ix, b); ix++)
+    {
+      struct tracepoint *t = (struct tracepoint *) b;
+
+      if (b->type == utp->type
+         && t->step_count == utp->step
+         && t->pass_count == utp->pass
+         && cond_string_is_same (t->base.cond_string, utp->cond_string)
+         /* FIXME also test actions.  */
+         )
+       {
+         /* Scan the locations for an address match.  */
+         for (loc = b->loc; loc; loc = loc->next)
+           {
+             if (loc->address == utp->addr)
+               return loc;
+           }
+       }
+    }
+  return NULL;
+}
+
+/* Given a list of tracepoints uploaded from a target, attempt to
+   match them up with existing tracepoints, and create new ones if not
+   found.  */
+
+void
+merge_uploaded_tracepoints (struct uploaded_tp **uploaded_tps)
+{
+  struct uploaded_tp *utp;
+
+  /* Look for GDB tracepoints that match up with our uploaded versions.  */
+  for (utp = *uploaded_tps; utp; utp = utp->next)
+    {
+      struct bp_location *loc;
+      struct tracepoint *t;
+
+      loc = find_matching_tracepoint_location (utp);
+      if (loc)
+       {
+         /* Mark this location as already inserted.  */
+         loc->inserted = 1;
+         t = (struct tracepoint *) loc->owner;
+         printf_filtered (_("Assuming tracepoint %d is same "
+                            "as target's tracepoint %d at %s.\n"),
+                          loc->owner->number, utp->number,
+                          paddress (loc->gdbarch, utp->addr));
+       }
+      else
+       {
+         t = create_tracepoint_from_upload (utp);
+         if (t)
+           printf_filtered (_("Created tracepoint %d for "
+                              "target's tracepoint %d at %s.\n"),
+                            t->base.number, utp->number,
+                            paddress (get_current_arch (), utp->addr));
+         else
+           printf_filtered (_("Failed to create tracepoint for target's "
+                              "tracepoint %d at %s, skipping it.\n"),
+                            utp->number,
+                            paddress (get_current_arch (), utp->addr));
+       }
+      /* Whether found or created, record the number used by the
+        target, to help with mapping target tracepoints back to their
+        counterparts here.  */
+      if (t)
+       t->number_on_target = utp->number;
+    }
+
+  free_uploaded_tps (uploaded_tps);
+}
+
+/* Trace state variables don't have much to identify them beyond their
+   name, so just use that to detect matches.  */
+
+static struct trace_state_variable *
+find_matching_tsv (struct uploaded_tsv *utsv)
+{
+  if (!utsv->name)
+    return NULL;
+
+  return find_trace_state_variable (utsv->name);
+}
+
+static struct trace_state_variable *
+create_tsv_from_upload (struct uploaded_tsv *utsv)
+{
+  const char *namebase;
+  char *buf;
+  int try_num = 0;
+  struct trace_state_variable *tsv;
+  struct cleanup *old_chain;
+
+  if (utsv->name)
+    {
+      namebase = utsv->name;
+      buf = xstrprintf ("%s", namebase);
+    }
+  else
+    {
+      namebase = "__tsv";
+      buf = xstrprintf ("%s_%d", namebase, try_num++);
+    }
+
+  /* Fish for a name that is not in use.  */
+  /* (should check against all internal vars?)  */
+  while (find_trace_state_variable (buf))
+    {
+      xfree (buf);
+      buf = xstrprintf ("%s_%d", namebase, try_num++);
+    }
+
+  old_chain = make_cleanup (xfree, buf);
+
+  /* We have an available name, create the variable.  */
+  tsv = create_trace_state_variable (buf);
+  tsv->initial_value = utsv->initial_value;
+  tsv->builtin = utsv->builtin;
+
+  observer_notify_tsv_created (tsv->name, tsv->initial_value);
+
+  do_cleanups (old_chain);
+
+  return tsv;
+}
+
+/* Given a list of uploaded trace state variables, try to match them
+   up with existing variables, or create additional ones.  */
+
+void
+merge_uploaded_trace_state_variables (struct uploaded_tsv **uploaded_tsvs)
+{
+  int ix;
+  struct uploaded_tsv *utsv;
+  struct trace_state_variable *tsv;
+  int highest;
+
+  /* Most likely some numbers will have to be reassigned as part of
+     the merge, so clear them all in anticipation.  */
+  for (ix = 0; VEC_iterate (tsv_s, tvariables, ix, tsv); ++ix)
+    tsv->number = 0;
+
+  for (utsv = *uploaded_tsvs; utsv; utsv = utsv->next)
+    {
+      tsv = find_matching_tsv (utsv);
+      if (tsv)
+       {
+         if (info_verbose)
+           printf_filtered (_("Assuming trace state variable $%s "
+                              "is same as target's variable %d.\n"),
+                            tsv->name, utsv->number);
+       }
+      else
+       {
+         tsv = create_tsv_from_upload (utsv);
+         if (info_verbose)
+           printf_filtered (_("Created trace state variable "
+                              "$%s for target's variable %d.\n"),
+                            tsv->name, utsv->number);
+       }
+      /* Give precedence to numberings that come from the target.  */
+      if (tsv)
+       tsv->number = utsv->number;
+    }
+
+  /* Renumber everything that didn't get a target-assigned number.  */
+  highest = 0;
+  for (ix = 0; VEC_iterate (tsv_s, tvariables, ix, tsv); ++ix)
+    if (tsv->number > highest)
+      highest = tsv->number;
+
+  ++highest;
+  for (ix = 0; VEC_iterate (tsv_s, tvariables, ix, tsv); ++ix)
+    if (tsv->number == 0)
+      tsv->number = highest++;
+
+  free_uploaded_tsvs (uploaded_tsvs);
+}
+
+/* target tfile command */
+
+static struct target_ops tfile_ops;
+
+/* Fill in tfile_ops with its defined operations and properties.  */
+
+#define TRACE_HEADER_SIZE 8
+
+static char *trace_filename;
+static int trace_fd = -1;
+static off_t trace_frames_offset;
+static off_t cur_offset;
+static int cur_data_size;
+int trace_regblock_size;
+
+static void tfile_interp_line (char *line,
+                              struct uploaded_tp **utpp,
+                              struct uploaded_tsv **utsvp);
+
+/* Read SIZE bytes into READBUF from the trace frame, starting at
+   TRACE_FD's current position.  Note that this call `read'
+   underneath, hence it advances the file's seek position.  Throws an
+   error if the `read' syscall fails, or less than SIZE bytes are
+   read.  */
+
+static void
+tfile_read (gdb_byte *readbuf, int size)
+{
+  int gotten;
+
+  gotten = read (trace_fd, readbuf, size);
+  if (gotten < 0)
+    perror_with_name (trace_filename);
+  else if (gotten < size)
+    error (_("Premature end of file while reading trace file"));
+}
+
+static void
+tfile_open (char *filename, int from_tty)
+{
+  volatile struct gdb_exception ex;
+  char *temp;
+  struct cleanup *old_chain;
+  int flags;
+  int scratch_chan;
+  char header[TRACE_HEADER_SIZE];
+  char linebuf[1000]; /* Should be max remote packet size or so.  */
+  char byte;
+  int bytes, i;
+  struct trace_status *ts;
+  struct uploaded_tp *uploaded_tps = NULL;
+  struct uploaded_tsv *uploaded_tsvs = NULL;
+
+  target_preopen (from_tty);
+  if (!filename)
+    error (_("No trace file specified."));
+
+  filename = tilde_expand (filename);
+  if (!IS_ABSOLUTE_PATH(filename))
+    {
+      temp = concat (current_directory, "/", filename, (char *) NULL);
+      xfree (filename);
+      filename = temp;
+    }
+
+  old_chain = make_cleanup (xfree, filename);
+
+  flags = O_BINARY | O_LARGEFILE;
+  flags |= O_RDONLY;
+  scratch_chan = open (filename, flags, 0);
+  if (scratch_chan < 0)
+    perror_with_name (filename);
+
+  /* Looks semi-reasonable.  Toss the old trace file and work on the new.  */
+
+  discard_cleanups (old_chain);        /* Don't free filename any more.  */
+  unpush_target (&tfile_ops);
+
+  trace_filename = xstrdup (filename);
+  trace_fd = scratch_chan;
+
+  bytes = 0;
+  /* Read the file header and test for validity.  */
+  tfile_read ((gdb_byte *) &header, TRACE_HEADER_SIZE);
+
+  bytes += TRACE_HEADER_SIZE;
+  if (!(header[0] == 0x7f
+       && (strncmp (header + 1, "TRACE0\n", 7) == 0)))
+    error (_("File is not a valid trace file."));
+
+  push_target (&tfile_ops);
+
+  trace_regblock_size = 0;
+  ts = current_trace_status ();
+  /* We know we're working with a file.  */
+  ts->from_file = 1;
+  /* Set defaults in case there is no status line.  */
+  ts->running_known = 0;
+  ts->stop_reason = trace_stop_reason_unknown;
+  ts->traceframe_count = -1;
+  ts->buffer_free = 0;
+  ts->disconnected_tracing = 0;
+  ts->circular_buffer = 0;
+
+  TRY_CATCH (ex, RETURN_MASK_ALL)
+    {
+      /* Read through a section of newline-terminated lines that
+        define things like tracepoints.  */
+      i = 0;
+      while (1)
+       {
+         tfile_read (&byte, 1);
+
+         ++bytes;
+         if (byte == '\n')
+           {
+             /* Empty line marks end of the definition section.  */
+             if (i == 0)
+               break;
+             linebuf[i] = '\0';
+             i = 0;
+             tfile_interp_line (linebuf, &uploaded_tps, &uploaded_tsvs);
+           }
+         else
+           linebuf[i++] = byte;
+         if (i >= 1000)
+           error (_("Excessively long lines in trace file"));
+       }
+
+      /* Record the starting offset of the binary trace data.  */
+      trace_frames_offset = bytes;
+
+      /* If we don't have a blocksize, we can't interpret the
+        traceframes.  */
+      if (trace_regblock_size == 0)
+       error (_("No register block size recorded in trace file"));
+    }
+  if (ex.reason < 0)
+    {
+      /* Pop the partially set up target.  */
+      pop_target ();
+      throw_exception (ex);
+    }
+
+  inferior_appeared (current_inferior (), TFILE_PID);
+  inferior_ptid = pid_to_ptid (TFILE_PID);
+  add_thread_silent (inferior_ptid);
+
+  if (ts->traceframe_count <= 0)
+    warning (_("No traceframes present in this file."));
+
+  /* Add the file's tracepoints and variables into the current mix.  */
+
+  /* Get trace state variables first, they may be checked when parsing
+     uploaded commands.  */
+  merge_uploaded_trace_state_variables (&uploaded_tsvs);
+
+  merge_uploaded_tracepoints (&uploaded_tps);
+
+  post_create_inferior (&tfile_ops, from_tty);
+}
+
+/* Interpret the given line from the definitions part of the trace
+   file.  */
+
+static void
+tfile_interp_line (char *line,
+                  struct uploaded_tp **utpp, struct uploaded_tsv **utsvp)
+{
+  char *p = line;
+
+  if (strncmp (p, "R ", strlen ("R ")) == 0)
+    {
+      p += strlen ("R ");
+      trace_regblock_size = strtol (p, &p, 16);
+    }
+  else if (strncmp (p, "status ", strlen ("status ")) == 0)
+    {
+      p += strlen ("status ");
+      parse_trace_status (p, current_trace_status ());
+    }
+  else if (strncmp (p, "tp ", strlen ("tp ")) == 0)
+    {
+      p += strlen ("tp ");
+      parse_tracepoint_definition (p, utpp);
+    }
+  else if (strncmp (p, "tsv ", strlen ("tsv ")) == 0)
+    {
+      p += strlen ("tsv ");
+      parse_tsv_definition (p, utsvp);
+    }
+  else
+    warning (_("Ignoring trace file definition \"%s\""), line);
+}
+
+/* Parse the part of trace status syntax that is shared between
+   the remote protocol and the trace file reader.  */
+
+void
+parse_trace_status (char *line, struct trace_status *ts)
+{
+  char *p = line, *p1, *p2, *p3, *p_temp;
+  int end;
+  ULONGEST val;
+
+  ts->running_known = 1;
+  ts->running = (*p++ == '1');
+  ts->stop_reason = trace_stop_reason_unknown;
+  xfree (ts->stop_desc);
+  ts->stop_desc = NULL;
+  ts->traceframe_count = -1;
+  ts->traceframes_created = -1;
+  ts->buffer_free = -1;
+  ts->buffer_size = -1;
+  ts->disconnected_tracing = 0;
+  ts->circular_buffer = 0;
+  xfree (ts->user_name);
+  ts->user_name = NULL;
+  xfree (ts->notes);
+  ts->notes = NULL;
+  ts->start_time = ts->stop_time = 0;
+
+  while (*p++)
+    {
+      p1 = strchr (p, ':');
+      if (p1 == NULL)
+       error (_("Malformed trace status, at %s\n\
+Status line: '%s'\n"), p, line);
+      p3 = strchr (p, ';');
+      if (p3 == NULL)
+       p3 = p + strlen (p);
+      if (strncmp (p, stop_reason_names[trace_buffer_full], p1 - p) == 0)
+       {
+         p = unpack_varlen_hex (++p1, &val);
+         ts->stop_reason = trace_buffer_full;
+       }
+      else if (strncmp (p, stop_reason_names[trace_never_run], p1 - p) == 0)
+       {
+         p = unpack_varlen_hex (++p1, &val);
+         ts->stop_reason = trace_never_run;
+       }
+      else if (strncmp (p, stop_reason_names[tracepoint_passcount],
+                       p1 - p) == 0)
+       {
+         p = unpack_varlen_hex (++p1, &val);
+         ts->stop_reason = tracepoint_passcount;
+         ts->stopping_tracepoint = val;
+       }
+      else if (strncmp (p, stop_reason_names[tstop_command], p1 - p) == 0)
+       {
+         p2 = strchr (++p1, ':');
+         if (!p2 || p2 > p3)
+           {
+             /*older style*/
+             p2 = p1;
+           }
+         else if (p2 != p1)
+           {
+             ts->stop_desc = xmalloc (strlen (line));
+             end = hex2bin (p1, ts->stop_desc, (p2 - p1) / 2);
+             ts->stop_desc[end] = '\0';
+           }
+         else
+           ts->stop_desc = xstrdup ("");
+
+         p = unpack_varlen_hex (++p2, &val);
+         ts->stop_reason = tstop_command;
+       }
+      else if (strncmp (p, stop_reason_names[trace_disconnected], p1 - p) == 0)
+       {
+         p = unpack_varlen_hex (++p1, &val);
+         ts->stop_reason = trace_disconnected;
+       }
+      else if (strncmp (p, stop_reason_names[tracepoint_error], p1 - p) == 0)
+       {
+         p2 = strchr (++p1, ':');
+         if (p2 != p1)
+           {
+             ts->stop_desc = xmalloc ((p2 - p1) / 2 + 1);
+             end = hex2bin (p1, ts->stop_desc, (p2 - p1) / 2);
+             ts->stop_desc[end] = '\0';
+           }
+         else
+           ts->stop_desc = xstrdup ("");
+
+         p = unpack_varlen_hex (++p2, &val);
+         ts->stopping_tracepoint = val;
+         ts->stop_reason = tracepoint_error;
+       }
+      else if (strncmp (p, "tframes", p1 - p) == 0)
+       {
+         p = unpack_varlen_hex (++p1, &val);
+         ts->traceframe_count = val;
+       }
+      else if (strncmp (p, "tcreated", p1 - p) == 0)
+       {
+         p = unpack_varlen_hex (++p1, &val);
+         ts->traceframes_created = val;
+       }
+      else if (strncmp (p, "tfree", p1 - p) == 0)
+       {
+         p = unpack_varlen_hex (++p1, &val);
+         ts->buffer_free = val;
+       }
+      else if (strncmp (p, "tsize", p1 - p) == 0)
+       {
+         p = unpack_varlen_hex (++p1, &val);
+         ts->buffer_size = val;
+       }
+      else if (strncmp (p, "disconn", p1 - p) == 0)
+       {
+         p = unpack_varlen_hex (++p1, &val);
+         ts->disconnected_tracing = val;
+       }
+      else if (strncmp (p, "circular", p1 - p) == 0)
+       {
+         p = unpack_varlen_hex (++p1, &val);
+         ts->circular_buffer = val;
+       }
+      else if (strncmp (p, "starttime", p1 - p) == 0)
+       {
+         p = unpack_varlen_hex (++p1, &val);
+         ts->start_time = val;
+       }
+      else if (strncmp (p, "stoptime", p1 - p) == 0)
+       {
+         p = unpack_varlen_hex (++p1, &val);
+         ts->stop_time = val;
+       }
+      else if (strncmp (p, "username", p1 - p) == 0)
+       {
+         ++p1;
+         ts->user_name = xmalloc (strlen (p) / 2);
+         end = hex2bin (p1, ts->user_name, (p3 - p1)  / 2);
+         ts->user_name[end] = '\0';
+         p = p3;
+       }
+      else if (strncmp (p, "notes", p1 - p) == 0)
+       {
+         ++p1;
+         ts->notes = xmalloc (strlen (p) / 2);
+         end = hex2bin (p1, ts->notes, (p3 - p1) / 2);
+         ts->notes[end] = '\0';
+         p = p3;
+       }
+      else
+       {
+         /* Silently skip unknown optional info.  */
+         p_temp = strchr (p1 + 1, ';');
+         if (p_temp)
+           p = p_temp;
+         else
+           /* Must be at the end.  */
+           break;
+       }
+    }
+}
+
+void
+parse_tracepoint_status (char *p, struct breakpoint *bp,
+                        struct uploaded_tp *utp)
+{
+  ULONGEST uval;
+  struct tracepoint *tp = (struct tracepoint *) bp;
+
+  p = unpack_varlen_hex (p, &uval);
+  if (tp)
+    tp->base.hit_count += uval;
+  else
+    utp->hit_count += uval;
+  p = unpack_varlen_hex (p + 1, &uval);
+  if (tp)
+    tp->traceframe_usage += uval;
+  else
+    utp->traceframe_usage += uval;
+  /* Ignore any extra, allowing for future extensions.  */
+}
+
+/* Given a line of text defining a part of a tracepoint, parse it into
+   an "uploaded tracepoint".  */
+
+void
+parse_tracepoint_definition (char *line, struct uploaded_tp **utpp)
+{
+  char *p;
+  char piece;
+  ULONGEST num, addr, step, pass, orig_size, xlen, start;
+  int enabled, end;
+  enum bptype type;
+  char *cond, *srctype, *buf;
+  struct uploaded_tp *utp = NULL;
+
+  p = line;
+  /* Both tracepoint and action definitions start with the same number
+     and address sequence.  */
+  piece = *p++;
+  p = unpack_varlen_hex (p, &num);
+  p++;  /* skip a colon */
+  p = unpack_varlen_hex (p, &addr);
+  p++;  /* skip a colon */
+  if (piece == 'T')
+    {
+      enabled = (*p++ == 'E');
+      p++;  /* skip a colon */
+      p = unpack_varlen_hex (p, &step);
+      p++;  /* skip a colon */
+      p = unpack_varlen_hex (p, &pass);
+      type = bp_tracepoint;
+      cond = NULL;
+      /* Thumb through optional fields.  */
+      while (*p == ':')
+       {
+         p++;  /* skip a colon */
+         if (*p == 'F')
+           {
+             type = bp_fast_tracepoint;
+             p++;
+             p = unpack_varlen_hex (p, &orig_size);
+           }
+         else if (*p == 'S')
+           {
+             type = bp_static_tracepoint;
+             p++;
+           }
+         else if (*p == 'X')
+           {
+             p++;
+             p = unpack_varlen_hex (p, &xlen);
+             p++;  /* skip a comma */
+             cond = (char *) xmalloc (2 * xlen + 1);
+             strncpy (cond, p, 2 * xlen);
+             cond[2 * xlen] = '\0';
+             p += 2 * xlen;
+           }
+         else
+           warning (_("Unrecognized char '%c' in tracepoint "
+                      "definition, skipping rest"), *p);
+       }
+      utp = get_uploaded_tp (num, addr, utpp);
+      utp->type = type;
+      utp->enabled = enabled;
+      utp->step = step;
+      utp->pass = pass;
+      utp->cond = cond;
+    }
+  else if (piece == 'A')
+    {
+      utp = get_uploaded_tp (num, addr, utpp);
+      VEC_safe_push (char_ptr, utp->actions, xstrdup (p));
+    }
+  else if (piece == 'S')
+    {
+      utp = get_uploaded_tp (num, addr, utpp);
+      VEC_safe_push (char_ptr, utp->step_actions, xstrdup (p));
+    }
+  else if (piece == 'Z')
+    {
+      /* Parse a chunk of source form definition.  */
+      utp = get_uploaded_tp (num, addr, utpp);
+      srctype = p;
+      p = strchr (p, ':');
+      p++;  /* skip a colon */
+      p = unpack_varlen_hex (p, &start);
+      p++;  /* skip a colon */
+      p = unpack_varlen_hex (p, &xlen);
+      p++;  /* skip a colon */
+
+      buf = alloca (strlen (line));
+
+      end = hex2bin (p, (gdb_byte *) buf, strlen (p) / 2);
+      buf[end] = '\0';
+
+      if (strncmp (srctype, "at:", strlen ("at:")) == 0)
+       utp->at_string = xstrdup (buf);
+      else if (strncmp (srctype, "cond:", strlen ("cond:")) == 0)
+       utp->cond_string = xstrdup (buf);
+      else if (strncmp (srctype, "cmd:", strlen ("cmd:")) == 0)
+       VEC_safe_push (char_ptr, utp->cmd_strings, xstrdup (buf));
+    }
+  else if (piece == 'V')
+    {
+      utp = get_uploaded_tp (num, addr, utpp);
+
+      parse_tracepoint_status (p, NULL, utp);
+    }
+  else
+    {
+      /* Don't error out, the target might be sending us optional
+        info that we don't care about.  */
+      warning (_("Unrecognized tracepoint piece '%c', ignoring"), piece);
+    }
+}
+
+/* Convert a textual description of a trace state variable into an
+   uploaded object.  */
+
+void
+parse_tsv_definition (char *line, struct uploaded_tsv **utsvp)
+{
+  char *p, *buf;
+  ULONGEST num, initval, builtin;
+  int end;
+  struct uploaded_tsv *utsv = NULL;
+
+  buf = alloca (strlen (line));
+
+  p = line;
+  p = unpack_varlen_hex (p, &num);
+  p++; /* skip a colon */
+  p = unpack_varlen_hex (p, &initval);
+  p++; /* skip a colon */
+  p = unpack_varlen_hex (p, &builtin);
+  p++; /* skip a colon */
+  end = hex2bin (p, (gdb_byte *) buf, strlen (p) / 2);
+  buf[end] = '\0';
+
+  utsv = get_uploaded_tsv (num, utsvp);
+  utsv->initial_value = initval;
+  utsv->builtin = builtin;
+  utsv->name = xstrdup (buf);
+}
+
+/* Close the trace file and generally clean up.  */
+
+static void
+tfile_close (int quitting)
+{
+  int pid;
+
+  if (trace_fd < 0)
+    return;
+
+  pid = ptid_get_pid (inferior_ptid);
+  inferior_ptid = null_ptid;   /* Avoid confusion from thread stuff.  */
+  exit_inferior_silent (pid);
+
+  close (trace_fd);
+  trace_fd = -1;
+  xfree (trace_filename);
+  trace_filename = NULL;
+}
+
+static void
+tfile_files_info (struct target_ops *t)
+{
+  /* (it would be useful to mention the name of the file).  */
+  printf_filtered ("Looking at a trace file.\n");
+}
+
+/* The trace status for a file is that tracing can never be run.  */
+
+static int
+tfile_get_trace_status (struct trace_status *ts)
+{
+  /* Other bits of trace status were collected as part of opening the
+     trace files, so nothing to do here.  */
+
+  return -1;
+}
+
+static void
+tfile_get_tracepoint_status (struct breakpoint *tp, struct uploaded_tp *utp)
+{
+  /* Other bits of trace status were collected as part of opening the
+     trace files, so nothing to do here.  */
+}
+
+/* Given the position of a traceframe in the file, figure out what
+   address the frame was collected at.  This would normally be the
+   value of a collected PC register, but if not available, we
+   improvise.  */
+
+static ULONGEST
+tfile_get_traceframe_address (off_t tframe_offset)
+{
+  ULONGEST addr = 0;
+  short tpnum;
+  struct tracepoint *tp;
+  off_t saved_offset = cur_offset;
+
+  /* FIXME dig pc out of collected registers.  */
+
+  /* Fall back to using tracepoint address.  */
+  lseek (trace_fd, tframe_offset, SEEK_SET);
+  tfile_read ((gdb_byte *) &tpnum, 2);
+  tpnum = (short) extract_signed_integer ((gdb_byte *) &tpnum, 2,
+                                         gdbarch_byte_order
+                                             (target_gdbarch ()));
+
+  tp = get_tracepoint_by_number_on_target (tpnum);
+  /* FIXME this is a poor heuristic if multiple locations.  */
+  if (tp && tp->base.loc)
+    addr = tp->base.loc->address;
+
+  /* Restore our seek position.  */
+  cur_offset = saved_offset;
+  lseek (trace_fd, cur_offset, SEEK_SET);
+  return addr;
+}
+
+/* Given a type of search and some parameters, scan the collection of
+   traceframes in the file looking for a match.  When found, return
+   both the traceframe and tracepoint number, otherwise -1 for
+   each.  */
+
+static int
+tfile_trace_find (enum trace_find_type type, int num,
+                 ULONGEST addr1, ULONGEST addr2, int *tpp)
+{
+  short tpnum;
+  int tfnum = 0, found = 0;
+  unsigned int data_size;
+  struct tracepoint *tp;
+  off_t offset, tframe_offset;
+  ULONGEST tfaddr;
+
+  if (num == -1)
+    {
+      if (tpp)
+        *tpp = -1;
+      return -1;
+    }
+
+  lseek (trace_fd, trace_frames_offset, SEEK_SET);
+  offset = trace_frames_offset;
+  while (1)
     {
-      if (args == 0 || *args == 0)
+      tframe_offset = offset;
+      tfile_read ((gdb_byte *) &tpnum, 2);
+      tpnum = (short) extract_signed_integer ((gdb_byte *) &tpnum, 2,
+                                             gdbarch_byte_order
+                                                 (target_gdbarch ()));
+      offset += 2;
+      if (tpnum == 0)
+       break;
+      tfile_read ((gdb_byte *) &data_size, 4);
+      data_size = (unsigned int) extract_unsigned_integer
+                                     ((gdb_byte *) &data_size, 4,
+                                     gdbarch_byte_order (target_gdbarch ()));
+      offset += 4;
+      switch (type)
        {
-         sal = find_pc_line (get_frame_pc (get_current_frame ()), 0);
-         sals.nelts = 1;
-         sals.sals = (struct symtab_and_line *)
-           xmalloc (sizeof (struct symtab_and_line));
-         sals.sals[0] = sal;
+       case tfind_number:
+         if (tfnum == num)
+           found = 1;
+         break;
+       case tfind_pc:
+         tfaddr = tfile_get_traceframe_address (tframe_offset);
+         if (tfaddr == addr1)
+           found = 1;
+         break;
+       case tfind_tp:
+         tp = get_tracepoint (num);
+         if (tp && tpnum == tp->number_on_target)
+           found = 1;
+         break;
+       case tfind_range:
+         tfaddr = tfile_get_traceframe_address (tframe_offset);
+         if (addr1 <= tfaddr && tfaddr <= addr2)
+           found = 1;
+         break;
+       case tfind_outside:
+         tfaddr = tfile_get_traceframe_address (tframe_offset);
+         if (!(addr1 <= tfaddr && tfaddr <= addr2))
+           found = 1;
+         break;
+       default:
+         internal_error (__FILE__, __LINE__, _("unknown tfind type"));
        }
-      else
+      if (found)
        {
-         sals = decode_line_spec (args, 1);
-         sal = sals.sals[0];
+         if (tpp)
+           *tpp = tpnum;
+         cur_offset = offset;
+         cur_data_size = data_size;
+
+         return tfnum;
        }
+      /* Skip past the traceframe's data.  */
+      lseek (trace_fd, data_size, SEEK_CUR);
+      offset += data_size;
+      /* Update our own count of traceframes.  */
+      ++tfnum;
+    }
+  /* Did not find what we were looking for.  */
+  if (tpp)
+    *tpp = -1;
+  return -1;
+}
+
+/* Prototype of the callback passed to tframe_walk_blocks.  */
+typedef int (*walk_blocks_callback_func) (char blocktype, void *data);
 
-      old_chain = make_cleanup (xfree, sals.sals);
-      if (sal.symtab == 0)
+/* Callback for traceframe_walk_blocks, used to find a given block
+   type in a traceframe.  */
+
+static int
+match_blocktype (char blocktype, void *data)
+{
+  char *wantedp = data;
+
+  if (*wantedp == blocktype)
+    return 1;
+
+  return 0;
+}
+
+/* Walk over all traceframe block starting at POS offset from
+   CUR_OFFSET, and call CALLBACK for each block found, passing in DATA
+   unmodified.  If CALLBACK returns true, this returns the position in
+   the traceframe where the block is found, relative to the start of
+   the traceframe (cur_offset).  Returns -1 if no callback call
+   returned true, indicating that all blocks have been walked.  */
+
+static int
+traceframe_walk_blocks (walk_blocks_callback_func callback,
+                       int pos, void *data)
+{
+  /* Iterate through a traceframe's blocks, looking for a block of the
+     requested type.  */
+
+  lseek (trace_fd, cur_offset + pos, SEEK_SET);
+  while (pos < cur_data_size)
+    {
+      unsigned short mlen;
+      char block_type;
+
+      tfile_read (&block_type, 1);
+
+      ++pos;
+
+      if ((*callback) (block_type, data))
+       return pos;
+
+      switch (block_type)
        {
-         printf_filtered ("TFIND: No line number information available");
-         if (sal.pc != 0)
-           {
-             /* This is useful for "info line *0x7f34".  If we can't
-                tell the user about a source line, at least let them
-                have the symbolic address.  */
-             printf_filtered (" for address ");
-             wrap_here ("  ");
-             print_address (sal.pc, gdb_stdout);
-             printf_filtered (";\n -- will attempt to find by PC. \n");
-           }
-         else
+       case 'R':
+         lseek (trace_fd, cur_offset + pos + trace_regblock_size, SEEK_SET);
+         pos += trace_regblock_size;
+         break;
+       case 'M':
+         lseek (trace_fd, cur_offset + pos + 8, SEEK_SET);
+         tfile_read ((gdb_byte *) &mlen, 2);
+          mlen = (unsigned short)
+                extract_unsigned_integer ((gdb_byte *) &mlen, 2,
+                                          gdbarch_byte_order
+                                              (target_gdbarch ()));
+         lseek (trace_fd, mlen, SEEK_CUR);
+         pos += (8 + 2 + mlen);
+         break;
+       case 'V':
+         lseek (trace_fd, cur_offset + pos + 4 + 8, SEEK_SET);
+         pos += (4 + 8);
+         break;
+       default:
+         error (_("Unknown block type '%c' (0x%x) in trace frame"),
+                block_type, block_type);
+         break;
+       }
+    }
+
+  return -1;
+}
+
+/* Convenience wrapper around traceframe_walk_blocks.  Looks for the
+   position offset of a block of type TYPE_WANTED in the current trace
+   frame, starting at POS.  Returns -1 if no such block was found.  */
+
+static int
+traceframe_find_block_type (char type_wanted, int pos)
+{
+  return traceframe_walk_blocks (match_blocktype, pos, &type_wanted);
+}
+
+/* Look for a block of saved registers in the traceframe, and get the
+   requested register from it.  */
+
+static void
+tfile_fetch_registers (struct target_ops *ops,
+                      struct regcache *regcache, int regno)
+{
+  struct gdbarch *gdbarch = get_regcache_arch (regcache);
+  int offset, regn, regsize, pc_regno;
+  char *regs;
+
+  /* An uninitialized reg size says we're not going to be
+     successful at getting register blocks.  */
+  if (!trace_regblock_size)
+    return;
+
+  regs = alloca (trace_regblock_size);
+
+  if (traceframe_find_block_type ('R', 0) >= 0)
+    {
+      tfile_read (regs, trace_regblock_size);
+
+      /* Assume the block is laid out in GDB register number order,
+        each register with the size that it has in GDB.  */
+      offset = 0;
+      for (regn = 0; regn < gdbarch_num_regs (gdbarch); regn++)
+       {
+         regsize = register_size (gdbarch, regn);
+         /* Make sure we stay within block bounds.  */
+         if (offset + regsize >= trace_regblock_size)
+           break;
+         if (regcache_register_status (regcache, regn) == REG_UNKNOWN)
            {
-             printf_filtered (".\n");
-             return;           /* No line, no PC; what can we do?  */
+             if (regno == regn)
+               {
+                 regcache_raw_supply (regcache, regno, regs + offset);
+                 break;
+               }
+             else if (regno == -1)
+               {
+                 regcache_raw_supply (regcache, regn, regs + offset);
+               }
            }
+         offset += regsize;
        }
-      else if (sal.line > 0
-              && find_line_pc_range (sal, &start_pc, &end_pc))
+      return;
+    }
+
+  /* We get here if no register data has been found.  Mark registers
+     as unavailable.  */
+  for (regn = 0; regn < gdbarch_num_regs (gdbarch); regn++)
+    regcache_raw_supply (regcache, regn, NULL);
+
+  /* We can often usefully guess that the PC is going to be the same
+     as the address of the tracepoint.  */
+  pc_regno = gdbarch_pc_regnum (gdbarch);
+  if (pc_regno >= 0 && (regno == -1 || regno == pc_regno))
+    {
+      struct tracepoint *tp = get_tracepoint (tracepoint_number);
+
+      if (tp && tp->base.loc)
        {
-         if (start_pc == end_pc)
+         /* But don't try to guess if tracepoint is multi-location...  */
+         if (tp->base.loc->next)
            {
-             printf_filtered ("Line %d of \"%s\"",
-                              sal.line, sal.symtab->filename);
-             wrap_here ("  ");
-             printf_filtered (" is at address ");
-             print_address (start_pc, gdb_stdout);
-             wrap_here ("  ");
-             printf_filtered (" but contains no code.\n");
-             sal = find_pc_line (start_pc, 0);
-             if (sal.line > 0 &&
-                 find_line_pc_range (sal, &start_pc, &end_pc) &&
-                 start_pc != end_pc)
-               printf_filtered ("Attempting to find line %d instead.\n",
-                                sal.line);
-             else
-               error (_("Cannot find a good line."));
+             warning (_("Tracepoint %d has multiple "
+                        "locations, cannot infer $pc"),
+                      tp->base.number);
+             return;
+           }
+         /* ... or does while-stepping.  */
+         if (tp->step_count > 0)
+           {
+             warning (_("Tracepoint %d does while-stepping, "
+                        "cannot infer $pc"),
+                      tp->base.number);
+             return;
            }
+
+         store_unsigned_integer (regs, register_size (gdbarch, pc_regno),
+                                 gdbarch_byte_order (gdbarch),
+                                 tp->base.loc->address);
+         regcache_raw_supply (regcache, pc_regno, regs);
        }
-      else
-       /* Is there any case in which we get here, and have an address
-          which the user would want to see?  If we have debugging
-          symbols and no line numbers?  */
-       error (_("Line number %d is out of range for \"%s\"."),
-              sal.line, sal.symtab->filename);
-
-      sprintf_vma (startpc_str, start_pc);
-      sprintf_vma (endpc_str, end_pc - 1);
-      /* Find within range of stated line.  */
-      if (args && *args)
-       sprintf (target_buf, "QTFrame:range:%s:%s", 
-                startpc_str, endpc_str);
-      /* Find OUTSIDE OF range of CURRENT line.  */
-      else
-       sprintf (target_buf, "QTFrame:outside:%s:%s", 
-                startpc_str, endpc_str);
-      finish_tfind_command (&target_buf, &target_buf_size,
-                           from_tty);
-      do_cleanups (old_chain);
     }
-  else
-    error (_("Trace can only be run on remote targets."));
 }
 
-/* tfind range command */
-static void
-trace_find_range_command (char *args, int from_tty)
+static LONGEST
+tfile_xfer_partial (struct target_ops *ops, enum target_object object,
+                   const char *annex, gdb_byte *readbuf,
+                   const gdb_byte *writebuf, ULONGEST offset, LONGEST len)
 {
-  static CORE_ADDR start, stop;
-  char start_str[40], stop_str[40];
-  char *tmp;
+  /* We're only doing regular memory for now.  */
+  if (object != TARGET_OBJECT_MEMORY)
+    return -1;
 
-  if (target_is_remote ())
+  if (readbuf == NULL)
+    error (_("tfile_xfer_partial: trace file is read-only"));
+
+ if (traceframe_number != -1)
     {
-      if (args == 0 || *args == 0)
-       { /* XXX FIXME: what should default behavior be?  */
-         printf_filtered ("Usage: tfind range <startaddr>,<endaddr>\n");
-         return;
+      int pos = 0;
+
+      /* Iterate through the traceframe's blocks, looking for
+        memory.  */
+      while ((pos = traceframe_find_block_type ('M', pos)) >= 0)
+       {
+         ULONGEST maddr, amt;
+         unsigned short mlen;
+         enum bfd_endian byte_order = gdbarch_byte_order (target_gdbarch ());
+
+         tfile_read ((gdb_byte *) &maddr, 8);
+         maddr = extract_unsigned_integer ((gdb_byte *) &maddr, 8,
+                                           byte_order);
+         tfile_read ((gdb_byte *) &mlen, 2);
+         mlen = (unsigned short)
+           extract_unsigned_integer ((gdb_byte *) &mlen, 2, byte_order);
+
+         /* If the block includes the first part of the desired
+            range, return as much it has; GDB will re-request the
+            remainder, which might be in a different block of this
+            trace frame.  */
+         if (maddr <= offset && offset < (maddr + mlen))
+           {
+             amt = (maddr + mlen) - offset;
+             if (amt > len)
+               amt = len;
+
+             if (maddr != offset)
+               lseek (trace_fd, offset - maddr, SEEK_CUR);
+             tfile_read (readbuf, amt);
+             return amt;
+           }
+
+         /* Skip over this block.  */
+         pos += (8 + 2 + mlen);
        }
+    }
+
+  /* It's unduly pedantic to refuse to look at the executable for
+     read-only pieces; so do the equivalent of readonly regions aka
+     QTro packet.  */
+  /* FIXME account for relocation at some point.  */
+  if (exec_bfd)
+    {
+      asection *s;
+      bfd_size_type size;
+      bfd_vma vma;
 
-      if (0 != (tmp = strchr (args, ',')))
+      for (s = exec_bfd->sections; s; s = s->next)
        {
-         *tmp++ = '\0';        /* terminate start address */
-         while (isspace ((int) *tmp))
-           tmp++;
-         start = parse_and_eval_address (args);
-         stop = parse_and_eval_address (tmp);
+         if ((s->flags & SEC_LOAD) == 0
+             || (s->flags & SEC_READONLY) == 0)
+           continue;
+
+         vma = s->vma;
+         size = bfd_get_section_size (s);
+         if (vma <= offset && offset < (vma + size))
+           {
+             ULONGEST amt;
+
+             amt = (vma + size) - offset;
+             if (amt > len)
+               amt = len;
+
+             amt = bfd_get_section_contents (exec_bfd, s,
+                                             readbuf, offset - vma, amt);
+             return amt;
+           }
        }
-      else
-       {                       /* no explicit end address? */
-         start = parse_and_eval_address (args);
-         stop = start + 1;     /* ??? */
+    }
+
+  /* Indicate failure to find the requested memory block.  */
+  return -1;
+}
+
+/* Iterate through the blocks of a trace frame, looking for a 'V'
+   block with a matching tsv number.  */
+
+static int
+tfile_get_trace_state_variable_value (int tsvnum, LONGEST *val)
+{
+  int pos;
+
+  pos = 0;
+  while ((pos = traceframe_find_block_type ('V', pos)) >= 0)
+    {
+      int vnum;
+
+      tfile_read ((gdb_byte *) &vnum, 4);
+      vnum = (int) extract_signed_integer ((gdb_byte *) &vnum, 4,
+                                          gdbarch_byte_order
+                                          (target_gdbarch ()));
+      if (tsvnum == vnum)
+       {
+         tfile_read ((gdb_byte *) val, 8);
+         *val = extract_signed_integer ((gdb_byte *) val, 8,
+                                        gdbarch_byte_order
+                                        (target_gdbarch ()));
+         return 1;
        }
+      pos += (4 + 8);
+    }
+
+  /* Didn't find anything.  */
+  return 0;
+}
+
+static int
+tfile_has_all_memory (struct target_ops *ops)
+{
+  return 1;
+}
+
+static int
+tfile_has_memory (struct target_ops *ops)
+{
+  return 1;
+}
+
+static int
+tfile_has_stack (struct target_ops *ops)
+{
+  return traceframe_number != -1;
+}
+
+static int
+tfile_has_registers (struct target_ops *ops)
+{
+  return traceframe_number != -1;
+}
+
+static int
+tfile_thread_alive (struct target_ops *ops, ptid_t ptid)
+{
+  return 1;
+}
+
+/* Callback for traceframe_walk_blocks.  Builds a traceframe_info
+   object for the tfile target's current traceframe.  */
+
+static int
+build_traceframe_info (char blocktype, void *data)
+{
+  struct traceframe_info *info = data;
+
+  switch (blocktype)
+    {
+    case 'M':
+      {
+       struct mem_range *r;
+       ULONGEST maddr;
+       unsigned short mlen;
 
-      sprintf_vma (start_str, start);
-      sprintf_vma (stop_str, stop);
-      sprintf (target_buf, "QTFrame:range:%s:%s", start_str, stop_str);
-      finish_tfind_command (&target_buf, &target_buf_size, from_tty);
+       tfile_read ((gdb_byte *) &maddr, 8);
+       tfile_read ((gdb_byte *) &mlen, 2);
+
+       r = VEC_safe_push (mem_range_s, info->memory, NULL);
+
+       r->start = maddr;
+       r->length = mlen;
+       break;
+      }
+    case 'V':
+    case 'R':
+    case 'S':
+      {
+       break;
+      }
+    default:
+      warning (_("Unhandled trace block type (%d) '%c ' "
+                "while building trace frame info."),
+              blocktype, blocktype);
+      break;
+    }
+
+  return 0;
+}
+
+static struct traceframe_info *
+tfile_traceframe_info (void)
+{
+  struct traceframe_info *info = XCNEW (struct traceframe_info);
+
+  traceframe_walk_blocks (build_traceframe_info, 0, info);
+  return info;
+}
+
+static void
+init_tfile_ops (void)
+{
+  tfile_ops.to_shortname = "tfile";
+  tfile_ops.to_longname = "Local trace dump file";
+  tfile_ops.to_doc
+    = "Use a trace file as a target.  Specify the filename of the trace file.";
+  tfile_ops.to_open = tfile_open;
+  tfile_ops.to_close = tfile_close;
+  tfile_ops.to_fetch_registers = tfile_fetch_registers;
+  tfile_ops.to_xfer_partial = tfile_xfer_partial;
+  tfile_ops.to_files_info = tfile_files_info;
+  tfile_ops.to_get_trace_status = tfile_get_trace_status;
+  tfile_ops.to_get_tracepoint_status = tfile_get_tracepoint_status;
+  tfile_ops.to_trace_find = tfile_trace_find;
+  tfile_ops.to_get_trace_state_variable_value
+    = tfile_get_trace_state_variable_value;
+  tfile_ops.to_stratum = process_stratum;
+  tfile_ops.to_has_all_memory = tfile_has_all_memory;
+  tfile_ops.to_has_memory = tfile_has_memory;
+  tfile_ops.to_has_stack = tfile_has_stack;
+  tfile_ops.to_has_registers = tfile_has_registers;
+  tfile_ops.to_traceframe_info = tfile_traceframe_info;
+  tfile_ops.to_thread_alive = tfile_thread_alive;
+  tfile_ops.to_magic = OPS_MAGIC;
+}
+
+void
+free_current_marker (void *arg)
+{
+  struct static_tracepoint_marker **marker_p = arg;
+
+  if (*marker_p != NULL)
+    {
+      release_static_tracepoint_marker (*marker_p);
+      xfree (*marker_p);
     }
   else
-    error (_("Trace can only be run on remote targets."));
+    *marker_p = NULL;
 }
 
-/* tfind outside command */
+/* Given a line of text defining a static tracepoint marker, parse it
+   into a "static tracepoint marker" object.  Throws an error is
+   parsing fails.  If PP is non-null, it points to one past the end of
+   the parsed marker definition.  */
+
+void
+parse_static_tracepoint_marker_definition (char *line, char **pp,
+                                          struct static_tracepoint_marker *marker)
+{
+  char *p, *endp;
+  ULONGEST addr;
+  int end;
+
+  p = line;
+  p = unpack_varlen_hex (p, &addr);
+  p++;  /* skip a colon */
+
+  marker->gdbarch = target_gdbarch ();
+  marker->address = (CORE_ADDR) addr;
+
+  endp = strchr (p, ':');
+  if (endp == NULL)
+    error (_("bad marker definition: %s"), line);
+
+  marker->str_id = xmalloc (endp - p + 1);
+  end = hex2bin (p, (gdb_byte *) marker->str_id, (endp - p + 1) / 2);
+  marker->str_id[end] = '\0';
+
+  p += 2 * end;
+  p++;  /* skip a colon */
+
+  marker->extra = xmalloc (strlen (p) + 1);
+  end = hex2bin (p, (gdb_byte *) marker->extra, strlen (p) / 2);
+  marker->extra[end] = '\0';
+
+  if (pp)
+    *pp = p;
+}
+
+/* Release a static tracepoint marker's contents.  Note that the
+   object itself isn't released here.  There objects are usually on
+   the stack.  */
+
+void
+release_static_tracepoint_marker (struct static_tracepoint_marker *marker)
+{
+  xfree (marker->str_id);
+  marker->str_id = NULL;
+}
+
+/* Print MARKER to gdb_stdout.  */
+
 static void
-trace_find_outside_command (char *args, int from_tty)
+print_one_static_tracepoint_marker (int count,
+                                   struct static_tracepoint_marker *marker)
 {
-  CORE_ADDR start, stop;
-  char start_str[40], stop_str[40];
-  char *tmp;
+  struct command_line *l;
+  struct symbol *sym;
+
+  char wrap_indent[80];
+  char extra_field_indent[80];
+  struct ui_out *uiout = current_uiout;
+  struct cleanup *bkpt_chain;
+  VEC(breakpoint_p) *tracepoints;
+
+  struct symtab_and_line sal;
+
+  init_sal (&sal);
+
+  sal.pc = marker->address;
+
+  tracepoints = static_tracepoints_here (marker->address);
+
+  bkpt_chain = make_cleanup_ui_out_tuple_begin_end (uiout, "marker");
+
+  /* A counter field to help readability.  This is not a stable
+     identifier!  */
+  ui_out_field_int (uiout, "count", count);
+
+  ui_out_field_string (uiout, "marker-id", marker->str_id);
+
+  ui_out_field_fmt (uiout, "enabled", "%c",
+                   !VEC_empty (breakpoint_p, tracepoints) ? 'y' : 'n');
+  ui_out_spaces (uiout, 2);
+
+  strcpy (wrap_indent, "                                   ");
+
+  if (gdbarch_addr_bit (marker->gdbarch) <= 32)
+    strcat (wrap_indent, "           ");
+  else
+    strcat (wrap_indent, "                   ");
+
+  strcpy (extra_field_indent, "         ");
 
-  if (target_is_remote ())
+  ui_out_field_core_addr (uiout, "addr", marker->gdbarch, marker->address);
+
+  sal = find_pc_line (marker->address, 0);
+  sym = find_pc_sect_function (marker->address, NULL);
+  if (sym)
     {
-      if (args == 0 || *args == 0)
-       { /* XXX FIXME: what should default behavior be? */
-         printf_filtered ("Usage: tfind outside <startaddr>,<endaddr>\n");
-         return;
-       }
+      ui_out_text (uiout, "in ");
+      ui_out_field_string (uiout, "func",
+                          SYMBOL_PRINT_NAME (sym));
+      ui_out_wrap_hint (uiout, wrap_indent);
+      ui_out_text (uiout, " at ");
+    }
+  else
+    ui_out_field_skip (uiout, "func");
+
+  if (sal.symtab != NULL)
+    {
+      ui_out_field_string (uiout, "file", sal.symtab->filename);
+      ui_out_text (uiout, ":");
 
-      if (0 != (tmp = strchr (args, ',')))
+      if (ui_out_is_mi_like_p (uiout))
        {
-         *tmp++ = '\0';        /* terminate start address */
-         while (isspace ((int) *tmp))
-           tmp++;
-         start = parse_and_eval_address (args);
-         stop = parse_and_eval_address (tmp);
+         const char *fullname = symtab_to_fullname (sal.symtab);
+
+         if (fullname)
+           ui_out_field_string (uiout, "fullname", fullname);
        }
       else
-       {                       /* no explicit end address? */
-         start = parse_and_eval_address (args);
-         stop = start + 1;     /* ??? */
+       ui_out_field_skip (uiout, "fullname");
+
+      ui_out_field_int (uiout, "line", sal.line);
+    }
+  else
+    {
+      ui_out_field_skip (uiout, "fullname");
+      ui_out_field_skip (uiout, "line");
+    }
+
+  ui_out_text (uiout, "\n");
+  ui_out_text (uiout, extra_field_indent);
+  ui_out_text (uiout, _("Data: \""));
+  ui_out_field_string (uiout, "extra-data", marker->extra);
+  ui_out_text (uiout, "\"\n");
+
+  if (!VEC_empty (breakpoint_p, tracepoints))
+    {
+      struct cleanup *cleanup_chain;
+      int ix;
+      struct breakpoint *b;
+
+      cleanup_chain = make_cleanup_ui_out_tuple_begin_end (uiout,
+                                                          "tracepoints-at");
+
+      ui_out_text (uiout, extra_field_indent);
+      ui_out_text (uiout, _("Probed by static tracepoints: "));
+      for (ix = 0; VEC_iterate(breakpoint_p, tracepoints, ix, b); ix++)
+       {
+         if (ix > 0)
+           ui_out_text (uiout, ", ");
+         ui_out_text (uiout, "#");
+         ui_out_field_int (uiout, "tracepoint-id", b->number);
        }
 
-      sprintf_vma (start_str, start);
-      sprintf_vma (stop_str, stop);
-      sprintf (target_buf, "QTFrame:outside:%s:%s", start_str, stop_str);
-      finish_tfind_command (&target_buf, &target_buf_size, from_tty);
-    }
+      do_cleanups (cleanup_chain);
+
+      if (ui_out_is_mi_like_p (uiout))
+       ui_out_field_int (uiout, "number-of-tracepoints",
+                         VEC_length(breakpoint_p, tracepoints));
+      else
+       ui_out_text (uiout, "\n");
+    }
+  VEC_free (breakpoint_p, tracepoints);
+
+  do_cleanups (bkpt_chain);
+}
+
+static void
+info_static_tracepoint_markers_command (char *arg, int from_tty)
+{
+  VEC(static_tracepoint_marker_p) *markers;
+  struct cleanup *old_chain;
+  struct static_tracepoint_marker *marker;
+  struct ui_out *uiout = current_uiout;
+  int i;
+
+  /* We don't have to check target_can_use_agent and agent's capability on
+     static tracepoint here, in order to be compatible with older GDBserver.
+     We don't check USE_AGENT is true or not, because static tracepoints
+     don't work without in-process agent, so we don't bother users to type
+     `set agent on' when to use static tracepoint.  */
+
+  old_chain
+    = make_cleanup_ui_out_table_begin_end (uiout, 5, -1,
+                                          "StaticTracepointMarkersTable");
+
+  ui_out_table_header (uiout, 7, ui_left, "counter", "Cnt");
+
+  ui_out_table_header (uiout, 40, ui_left, "marker-id", "ID");
+
+  ui_out_table_header (uiout, 3, ui_left, "enabled", "Enb");
+  if (gdbarch_addr_bit (target_gdbarch ()) <= 32)
+    ui_out_table_header (uiout, 10, ui_left, "addr", "Address");
   else
-    error (_("Trace can only be run on remote targets."));
-}
+    ui_out_table_header (uiout, 18, ui_left, "addr", "Address");
+  ui_out_table_header (uiout, 40, ui_noalign, "what", "What");
 
-/* save-tracepoints command */
-static void
-tracepoint_save_command (char *args, int from_tty)
-{
-  struct tracepoint *tp;
-  struct action_line *line;
-  FILE *fp;
-  char *i1 = "    ", *i2 = "      ";
-  char *indent, *actionline, *pathname;
-  char tmp[40];
+  ui_out_table_body (uiout);
 
-  if (args == 0 || *args == 0)
-    error (_("Argument required (file name in which to save tracepoints)"));
+  markers = target_static_tracepoint_markers_by_strid (NULL);
+  make_cleanup (VEC_cleanup (static_tracepoint_marker_p), &markers);
 
-  if (tracepoint_chain == 0)
+  for (i = 0;
+       VEC_iterate (static_tracepoint_marker_p,
+                   markers, i, marker);
+       i++)
     {
-      warning (_("save-tracepoints: no tracepoints to save."));
-      return;
+      print_one_static_tracepoint_marker (i + 1, marker);
+      release_static_tracepoint_marker (marker);
     }
 
-  pathname = tilde_expand (args);
-  if (!(fp = fopen (pathname, "w")))
-    error (_("Unable to open file '%s' for saving tracepoints (%s)"),
-          args, safe_strerror (errno));
-  xfree (pathname);
-  
-  ALL_TRACEPOINTS (tp)
-  {
-    if (tp->addr_string)
-      fprintf (fp, "trace %s\n", tp->addr_string);
-    else
-      {
-       sprintf_vma (tmp, tp->address);
-       fprintf (fp, "trace *0x%s\n", tmp);
-      }
+  do_cleanups (old_chain);
+}
 
-    if (tp->pass_count)
-      fprintf (fp, "  passcount %d\n", tp->pass_count);
+/* The $_sdata convenience variable is a bit special.  We don't know
+   for sure type of the value until we actually have a chance to fetch
+   the data --- the size of the object depends on what has been
+   collected.  We solve this by making $_sdata be an internalvar that
+   creates a new value on access.  */
 
-    if (tp->actions)
-      {
-       fprintf (fp, "  actions\n");
-       indent = i1;
-       for (line = tp->actions; line; line = line->next)
-         {
-           struct cmd_list_element *cmd;
-
-           QUIT;               /* allow user to bail out with ^C */
-           actionline = line->action;
-           while (isspace ((int) *actionline))
-             actionline++;
-
-           fprintf (fp, "%s%s\n", indent, actionline);
-           if (*actionline != '#')     /* skip for comment lines */
-             {
-               cmd = lookup_cmd (&actionline, cmdlist, "", -1, 1);
-               if (cmd == 0)
-                 error (_("Bad action list item: %s"), actionline);
-               if (cmd_cfunc_eq (cmd, while_stepping_pseudocommand))
-                 indent = i2;
-               else if (cmd_cfunc_eq (cmd, end_actions_pseudocommand))
-                 indent = i1;
-             }
-         }
-      }
-  }
-  fclose (fp);
-  if (from_tty)
-    printf_filtered ("Tracepoints saved to file '%s'.\n", args);
-  return;
-}
+/* Return a new value with the correct type for the sdata object of
+   the current trace frame.  Return a void value if there's no object
+   available.  */
 
-/* info scope command: list the locals for a scope.  */
-static void
-scope_info (char *args, int from_tty)
+static struct value *
+sdata_make_value (struct gdbarch *gdbarch, struct internalvar *var,
+                 void *ignore)
 {
-  struct symtabs_and_lines sals;
-  struct symbol *sym;
-  struct minimal_symbol *msym;
-  struct block *block;
-  char **canonical, *symname, *save_args = args;
-  struct dict_iterator iter;
-  int j, count = 0;
-
-  if (args == 0 || *args == 0)
-    error (_("requires an argument (function, line or *addr) to define a scope"));
+  LONGEST size;
+  gdb_byte *buf;
+
+  /* We need to read the whole object before we know its size.  */
+  size = target_read_alloc (&current_target,
+                           TARGET_OBJECT_STATIC_TRACE_DATA,
+                           NULL, &buf);
+  if (size >= 0)
+    {
+      struct value *v;
+      struct type *type;
+
+      type = init_vector_type (builtin_type (gdbarch)->builtin_true_char,
+                              size);
+      v = allocate_value (type);
+      memcpy (value_contents_raw (v), buf, size);
+      xfree (buf);
+      return v;
+    }
+  else
+    return allocate_value (builtin_type (gdbarch)->builtin_void);
+}
 
-  sals = decode_line_1 (&args, 1, NULL, 0, &canonical, NULL);
-  if (sals.nelts == 0)
-    return;            /* presumably decode_line_1 has already warned */
+#if !defined(HAVE_LIBEXPAT)
 
-  /* Resolve line numbers to PC */
-  resolve_sal_pc (&sals.sals[0]);
-  block = block_for_pc (sals.sals[0].pc);
+struct traceframe_info *
+parse_traceframe_info (const char *tframe_info)
+{
+  static int have_warned;
 
-  while (block != 0)
+  if (!have_warned)
     {
-      QUIT;                    /* allow user to bail out with ^C */
-      ALL_BLOCK_SYMBOLS (block, iter, sym)
-       {
-         QUIT;                 /* allow user to bail out with ^C */
-         if (count == 0)
-           printf_filtered ("Scope for %s:\n", save_args);
-         count++;
-
-         symname = DEPRECATED_SYMBOL_NAME (sym);
-         if (symname == NULL || *symname == '\0')
-           continue;           /* probably botched, certainly useless */
-
-         printf_filtered ("Symbol %s is ", symname);
-         switch (SYMBOL_CLASS (sym))
-           {
-           default:
-           case LOC_UNDEF:     /* messed up symbol? */
-             printf_filtered ("a bogus symbol, class %d.\n",
-                              SYMBOL_CLASS (sym));
-             count--;          /* don't count this one */
-             continue;
-           case LOC_CONST:
-             printf_filtered ("a constant with value %ld (0x%lx)",
-                              SYMBOL_VALUE (sym), SYMBOL_VALUE (sym));
-             break;
-           case LOC_CONST_BYTES:
-             printf_filtered ("constant bytes: ");
-             if (SYMBOL_TYPE (sym))
-               for (j = 0; j < TYPE_LENGTH (SYMBOL_TYPE (sym)); j++)
-                 fprintf_filtered (gdb_stdout, " %02x",
-                                   (unsigned) SYMBOL_VALUE_BYTES (sym)[j]);
-             break;
-           case LOC_STATIC:
-             printf_filtered ("in static storage at address ");
-             deprecated_print_address_numeric (SYMBOL_VALUE_ADDRESS (sym), 
-                                    1, gdb_stdout);
-             break;
-           case LOC_REGISTER:
-             printf_filtered ("a local variable in register $%s",
-                              REGISTER_NAME (SYMBOL_VALUE (sym)));
-             break;
-           case LOC_ARG:
-           case LOC_LOCAL_ARG:
-             printf_filtered ("an argument at stack/frame offset %ld",
-                              SYMBOL_VALUE (sym));
-             break;
-           case LOC_LOCAL:
-             printf_filtered ("a local variable at frame offset %ld",
-                              SYMBOL_VALUE (sym));
-             break;
-           case LOC_REF_ARG:
-             printf_filtered ("a reference argument at offset %ld",
-                              SYMBOL_VALUE (sym));
-             break;
-           case LOC_REGPARM:
-             printf_filtered ("an argument in register $%s",
-                              REGISTER_NAME (SYMBOL_VALUE (sym)));
-             break;
-           case LOC_REGPARM_ADDR:
-             printf_filtered ("the address of an argument, in register $%s",
-                              REGISTER_NAME (SYMBOL_VALUE (sym)));
-             break;
-           case LOC_TYPEDEF:
-             printf_filtered ("a typedef.\n");
-             continue;
-           case LOC_LABEL:
-             printf_filtered ("a label at address ");
-             deprecated_print_address_numeric (SYMBOL_VALUE_ADDRESS (sym), 
-                                    1, gdb_stdout);
-             break;
-           case LOC_BLOCK:
-             printf_filtered ("a function at address ");
-             deprecated_print_address_numeric (BLOCK_START (SYMBOL_BLOCK_VALUE (sym)),
-                                    1, gdb_stdout);
-             break;
-           case LOC_BASEREG:
-             printf_filtered ("a variable at offset %ld from register $%s",
-                              SYMBOL_VALUE (sym),
-                              REGISTER_NAME (SYMBOL_BASEREG (sym)));
-             break;
-           case LOC_BASEREG_ARG:
-             printf_filtered ("an argument at offset %ld from register $%s",
-                              SYMBOL_VALUE (sym),
-                              REGISTER_NAME (SYMBOL_BASEREG (sym)));
-             break;
-           case LOC_UNRESOLVED:
-             msym = lookup_minimal_symbol (DEPRECATED_SYMBOL_NAME (sym), 
-                                           NULL, NULL);
-             if (msym == NULL)
-               printf_filtered ("Unresolved Static");
-             else
-               {
-                 printf_filtered ("static storage at address ");
-                 deprecated_print_address_numeric (SYMBOL_VALUE_ADDRESS (msym), 1,
-                                        gdb_stdout);
-               }
-             break;
-           case LOC_OPTIMIZED_OUT:
-             printf_filtered ("optimized out.\n");
-             continue;
-           case LOC_HP_THREAD_LOCAL_STATIC:
-             printf_filtered ("HP thread local static ");
-             break;
-           case LOC_INDIRECT:
-             printf_filtered ("extern (local indirect) at address ");
-             deprecated_print_address_numeric (SYMBOL_VALUE_ADDRESS (sym), 
-                                    1, gdb_stdout);
-             break;
-           case LOC_COMPUTED:
-           case LOC_COMPUTED_ARG:
-             SYMBOL_OPS (sym)->describe_location (sym, gdb_stdout);
-             break;
-           }
-         if (SYMBOL_TYPE (sym))
-           printf_filtered (", length %d.\n",
-                            TYPE_LENGTH (check_typedef (SYMBOL_TYPE (sym))));
-       }
-      if (BLOCK_FUNCTION (block))
-       break;
-      else
-       block = BLOCK_SUPERBLOCK (block);
+      have_warned = 1;
+      warning (_("Can not parse XML trace frame info; XML support "
+                "was disabled at compile time"));
     }
-  if (count <= 0)
-    printf_filtered ("Scope for %s contains no locals or arguments.\n",
-                    save_args);
+
+  return NULL;
 }
 
-/* worker function (cleanup) */
+#else /* HAVE_LIBEXPAT */
+
+#include "xml-support.h"
+
+/* Handle the start of a <memory> element.  */
+
 static void
-replace_comma (void *data)
+traceframe_info_start_memory (struct gdb_xml_parser *parser,
+                             const struct gdb_xml_element *element,
+                             void *user_data, VEC(gdb_xml_value_s) *attributes)
 {
-  char *comma = data;
-  *comma = ',';
+  struct traceframe_info *info = user_data;
+  struct mem_range *r = VEC_safe_push (mem_range_s, info->memory, NULL);
+  ULONGEST *start_p, *length_p;
+
+  start_p = xml_find_attribute (attributes, "start")->value;
+  length_p = xml_find_attribute (attributes, "length")->value;
+
+  r->start = *start_p;
+  r->length = *length_p;
 }
 
-/* tdump command */
+/* Discard the constructed trace frame info (if an error occurs).  */
+
 static void
-trace_dump_command (char *args, int from_tty)
+free_result (void *p)
 {
-  struct tracepoint *t;
-  struct action_line *action;
-  char *action_exp, *next_comma;
-  struct cleanup *old_cleanups;
-  int stepping_actions = 0;
-  int stepping_frame = 0;
+  struct traceframe_info *result = p;
 
-  if (!target_is_remote ())
-    {
-      error (_("Trace can only be run on remote targets."));
-      return;
-    }
+  free_traceframe_info (result);
+}
 
-  if (tracepoint_number == -1)
-    {
-      warning (_("No current trace frame."));
-      return;
-    }
+/* The allowed elements and attributes for an XML memory map.  */
 
-  ALL_TRACEPOINTS (t)
-    if (t->number == tracepoint_number)
-    break;
+static const struct gdb_xml_attribute memory_attributes[] = {
+  { "start", GDB_XML_AF_NONE, gdb_xml_parse_attr_ulongest, NULL },
+  { "length", GDB_XML_AF_NONE, gdb_xml_parse_attr_ulongest, NULL },
+  { NULL, GDB_XML_AF_NONE, NULL, NULL }
+};
 
-  if (t == NULL)
-    error (_("No known tracepoint matches 'current' tracepoint #%d."),
-          tracepoint_number);
+static const struct gdb_xml_element traceframe_info_children[] = {
+  { "memory", memory_attributes, NULL,
+    GDB_XML_EF_REPEATABLE | GDB_XML_EF_OPTIONAL,
+    traceframe_info_start_memory, NULL },
+  { NULL, NULL, NULL, GDB_XML_EF_NONE, NULL, NULL }
+};
 
-  old_cleanups = make_cleanup (null_cleanup, NULL);
+static const struct gdb_xml_element traceframe_info_elements[] = {
+  { "traceframe-info", NULL, traceframe_info_children, GDB_XML_EF_NONE,
+    NULL, NULL },
+  { NULL, NULL, NULL, GDB_XML_EF_NONE, NULL, NULL }
+};
 
-  printf_filtered ("Data collected at tracepoint %d, trace frame %d:\n",
-                  tracepoint_number, traceframe_number);
+/* Parse a traceframe-info XML document.  */
 
-  /* The current frame is a trap frame if the frame PC is equal
-     to the tracepoint PC.  If not, then the current frame was
-     collected during single-stepping.  */
+struct traceframe_info *
+parse_traceframe_info (const char *tframe_info)
+{
+  struct traceframe_info *result;
+  struct cleanup *back_to;
 
-  stepping_frame = (t->address != (read_pc () - DECR_PC_AFTER_BREAK));
+  result = XCNEW (struct traceframe_info);
+  back_to = make_cleanup (free_result, result);
 
-  for (action = t->actions; action; action = action->next)
+  if (gdb_xml_parse_quick (_("trace frame info"),
+                          "traceframe-info.dtd", traceframe_info_elements,
+                          tframe_info, result) == 0)
     {
-      struct cmd_list_element *cmd;
-
-      QUIT;                    /* allow user to bail out with ^C */
-      action_exp = action->action;
-      while (isspace ((int) *action_exp))
-       action_exp++;
+      /* Parsed successfully, keep the result.  */
+      discard_cleanups (back_to);
 
-      /* The collection actions to be done while stepping are
-         bracketed by the commands "while-stepping" and "end".  */
+      return result;
+    }
 
-      if (*action_exp == '#')  /* comment line */
-       continue;
+  do_cleanups (back_to);
+  return NULL;
+}
 
-      cmd = lookup_cmd (&action_exp, cmdlist, "", -1, 1);
-      if (cmd == 0)
-       error (_("Bad action list item: %s"), action_exp);
+#endif /* HAVE_LIBEXPAT */
 
-      if (cmd_cfunc_eq (cmd, while_stepping_pseudocommand))
-       stepping_actions = 1;
-      else if (cmd_cfunc_eq (cmd, end_actions_pseudocommand))
-       stepping_actions = 0;
-      else if (cmd_cfunc_eq (cmd, collect_pseudocommand))
-       {
-         /* Display the collected data.
-            For the trap frame, display only what was collected at
-            the trap.  Likewise for stepping frames, display only
-            what was collected while stepping.  This means that the
-            two boolean variables, STEPPING_FRAME and
-            STEPPING_ACTIONS should be equal.  */
-         if (stepping_frame == stepping_actions)
-           {
-             do
-               {               /* repeat over a comma-separated list */
-                 QUIT;         /* allow user to bail out with ^C */
-                 if (*action_exp == ',')
-                   action_exp++;
-                 while (isspace ((int) *action_exp))
-                   action_exp++;
+/* Returns the traceframe_info object for the current traceframe.
+   This is where we avoid re-fetching the object from the target if we
+   already have it cached.  */
 
-                 next_comma = strchr (action_exp, ',');
+static struct traceframe_info *
+get_traceframe_info (void)
+{
+  if (traceframe_info == NULL)
+    traceframe_info = target_traceframe_info ();
 
-                 if (0 == strncasecmp (action_exp, "$reg", 4))
-                   registers_info (NULL, from_tty);
-                 else if (0 == strncasecmp (action_exp, "$loc", 4))
-                   locals_info (NULL, from_tty);
-                 else if (0 == strncasecmp (action_exp, "$arg", 4))
-                   args_info (NULL, from_tty);
-                 else
-                   {           /* variable */
-                     if (next_comma)
-                       {
-                         make_cleanup (replace_comma, next_comma);
-                         *next_comma = '\0';
-                       }
-                     printf_filtered ("%s = ", action_exp);
-                     output_command (action_exp, from_tty);
-                     printf_filtered ("\n");
-                   }
-                 if (next_comma)
-                   *next_comma = ',';
-                 action_exp = next_comma;
-               }
-             while (action_exp && *action_exp == ',');
-           }
-       }
-    }
-  discard_cleanups (old_cleanups);
+  return traceframe_info;
 }
 
-/* Convert the memory pointed to by mem into hex, placing result in buf.
- * Return a pointer to the last char put in buf (null)
- * "stolen" from sparc-stub.c
- */
-
-static const char hexchars[] = "0123456789abcdef";
+/* If the target supports the query, return in RESULT the set of
+   collected memory in the current traceframe, found within the LEN
+   bytes range starting at MEMADDR.  Returns true if the target
+   supports the query, otherwise returns false, and RESULT is left
+   undefined.  */
 
-static char *
-mem2hex (gdb_byte *mem, char *buf, int count)
+int
+traceframe_available_memory (VEC(mem_range_s) **result,
+                            CORE_ADDR memaddr, ULONGEST len)
 {
-  gdb_byte ch;
+  struct traceframe_info *info = get_traceframe_info ();
 
-  while (count-- > 0)
+  if (info != NULL)
     {
-      ch = *mem++;
+      struct mem_range *r;
+      int i;
 
-      *buf++ = hexchars[ch >> 4];
-      *buf++ = hexchars[ch & 0xf];
-    }
+      *result = NULL;
 
-  *buf = 0;
+      for (i = 0; VEC_iterate (mem_range_s, info->memory, i, r); i++)
+       if (mem_ranges_overlap (r->start, r->length, memaddr, len))
+         {
+           ULONGEST lo1, hi1, lo2, hi2;
+           struct mem_range *nr;
 
-  return buf;
-}
+           lo1 = memaddr;
+           hi1 = memaddr + len;
 
-int
-get_traceframe_number (void)
-{
-  return traceframe_number;
+           lo2 = r->start;
+           hi2 = r->start + r->length;
+
+           nr = VEC_safe_push (mem_range_s, *result, NULL);
+
+           nr->start = max (lo1, lo2);
+           nr->length = min (hi1, hi2) - nr->start;
+         }
+
+      normalize_mem_ranges (*result);
+      return 1;
+    }
+
+  return 0;
 }
 
+/* Implementation of `sdata' variable.  */
+
+static const struct internalvar_funcs sdata_funcs =
+{
+  sdata_make_value,
+  NULL,
+  NULL
+};
 
 /* module initialization */
 void
@@ -2690,8 +5145,12 @@ _initialize_tracepoint (void)
 {
   struct cmd_list_element *c;
 
-  tracepoint_chain = 0;
-  tracepoint_count = 0;
+  /* Explicitly create without lookup, since that tries to create a
+     value with a void typed value, and when we get here, gdbarch
+     isn't initialized yet.  At this point, we're quite sure there
+     isn't another convenience variable of the same name.  */
+  create_internalvar_type_lazy ("_sdata", &sdata_funcs, NULL);
+
   traceframe_number = -1;
   tracepoint_number = -1;
 
@@ -2729,39 +5188,54 @@ _initialize_tracepoint (void)
           _("Tracing of program execution without stopping the program."),
           &cmdlist);
 
-  add_info ("tracepoints", tracepoints_info, _("\
-Status of tracepoints, or tracepoint number NUMBER.\n\
-Convenience variable \"$tpnum\" contains the number of the\n\
-last tracepoint set."));
-
-  add_info_alias ("tp", "tracepoints", 1);
-
-  c = add_com ("save-tracepoints", class_trace, tracepoint_save_command, _("\
-Save current tracepoint definitions as a script.\n\
-Use the 'source' command in another debug session to restore them."));
-  set_cmd_completer (c, filename_completer);
-
   add_com ("tdump", class_trace, trace_dump_command,
           _("Print everything collected at the current tracepoint."));
 
+  add_com ("tsave", class_trace, trace_save_command, _("\
+Save the trace data to a file.\n\
+Use the '-r' option to direct the target to save directly to the file,\n\
+using its own filesystem."));
+
+  c = add_com ("tvariable", class_trace, trace_variable_command,_("\
+Define a trace state variable.\n\
+Argument is a $-prefixed name, optionally followed\n\
+by '=' and an expression that sets the initial value\n\
+at the start of tracing."));
+  set_cmd_completer (c, expression_completer);
+
+  add_cmd ("tvariable", class_trace, delete_trace_variable_command, _("\
+Delete one or more trace state variables.\n\
+Arguments are the names of the variables to delete.\n\
+If no arguments are supplied, delete all variables."), &deletelist);
+  /* FIXME add a trace variable completer.  */
+
+  add_info ("tvariables", tvariables_info, _("\
+Status of trace state variables and their values.\n\
+"));
+
+  add_info ("static-tracepoint-markers",
+           info_static_tracepoint_markers_command, _("\
+List target static tracepoints markers.\n\
+"));
+
   add_prefix_cmd ("tfind", class_trace, trace_find_command, _("\
 Select a trace frame;\n\
 No argument means forward by one frame; '-' means backward by one frame."),
                  &tfindlist, "tfind ", 1, &cmdlist);
 
   add_cmd ("outside", class_trace, trace_find_outside_command, _("\
-Select a trace frame whose PC is outside the given range.\n\
+Select a trace frame whose PC is outside the given range (exclusive).\n\
 Usage: tfind outside addr1, addr2"),
           &tfindlist);
 
   add_cmd ("range", class_trace, trace_find_range_command, _("\
-Select a trace frame whose PC is in the given range.\n\
+Select a trace frame whose PC is in the given range (inclusive).\n\
 Usage: tfind range addr1,addr2"),
           &tfindlist);
 
   add_cmd ("line", class_trace, trace_find_line_command, _("\
 Select a trace frame by source line.\n\
-Argument can be a line number (with optional source file), \n\
+Argument can be a line number (with optional source file),\n\
 a function name, or '*' followed by an address.\n\
 Default argument is 'the next source line that was traced'."),
           &tfindlist);
@@ -2777,13 +5251,10 @@ Default is the current PC, or the PC of the current trace frame."),
           &tfindlist);
 
   add_cmd ("end", class_trace, trace_find_end_command, _("\
-Synonym for 'none'.\n\
 De-select any trace frame and resume 'live' debugging."),
           &tfindlist);
 
-  add_cmd ("none", class_trace, trace_find_none_command,
-          _("De-select any trace frame and resume 'live' debugging."),
-          &tfindlist);
+  add_alias_cmd ("none", "end", class_trace, 0, &tfindlist);
 
   add_cmd ("start", class_trace, trace_find_start_command,
           _("Select the first trace frame in the trace buffer."),
@@ -2792,17 +5263,17 @@ De-select any trace frame and resume 'live' debugging."),
   add_com ("tstatus", class_trace, trace_status_command,
           _("Display the status of the current trace data collection."));
 
-  add_com ("tstop", class_trace, trace_stop_command,
-          _("Stop trace data collection."));
-
-  add_com ("tstart", class_trace, trace_start_command,
-          _("Start trace data collection."));
+  add_com ("tstop", class_trace, trace_stop_command, _("\
+Stop trace data collection.\n\
+Usage: tstop [ <notes> ... ]\n\
+Any arguments supplied are recorded with the trace as a stop reason and\n\
+reported by tstatus (if the target supports trace notes)."));
 
-  add_com ("passcount", class_trace, trace_pass_command, _("\
-Set the passcount for a tracepoint.\n\
-The trace will end when the tracepoint has been passed 'count' times.\n\
-Usage: passcount COUNT TPNUM, where TPNUM may also be \"all\";\n\
-if TPNUM is omitted, passcount refers to the last tracepoint defined."));
+  add_com ("tstart", class_trace, trace_start_command, _("\
+Start trace data collection.\n\
+Usage: tstart [ <notes> ... ]\n\
+Any arguments supplied are recorded with the trace as a note and\n\
+reported by tstatus (if the target supports trace notes)."));
 
   add_com ("end", class_trace, end_actions_pseudocommand, _("\
 Ends a list of commands or actions.\n\
@@ -2830,45 +5301,75 @@ Also accepts the following special arguments:\n\
     $regs   -- all registers.\n\
     $args   -- all function arguments.\n\
     $locals -- all variables local to the block/function scope.\n\
+    $_sdata -- static tracepoint data (ignored for non-static tracepoints).\n\
+Note: this command can only be used in a tracepoint \"actions\" list."));
+
+  add_com ("teval", class_trace, teval_pseudocommand, _("\
+Specify one or more expressions to be evaluated at a tracepoint.\n\
+Accepts a comma-separated list of (one or more) expressions.\n\
+The result of each evaluation will be discarded.\n\
 Note: this command can only be used in a tracepoint \"actions\" list."));
 
   add_com ("actions", class_trace, trace_actions_command, _("\
 Specify the actions to be taken at a tracepoint.\n\
-Tracepoint actions may include collecting of specified data, \n\
-single-stepping, or enabling/disabling other tracepoints, \n\
+Tracepoint actions may include collecting of specified data,\n\
+single-stepping, or enabling/disabling other tracepoints,\n\
 depending on target's capabilities."));
 
-  add_cmd ("tracepoints", class_trace, delete_trace_command, _("\
-Delete specified tracepoints.\n\
-Arguments are tracepoint numbers, separated by spaces.\n\
-No argument means delete all tracepoints."),
-          &deletelist);
-
-  add_cmd ("tracepoints", class_trace, disable_trace_command, _("\
-Disable specified tracepoints.\n\
-Arguments are tracepoint numbers, separated by spaces.\n\
-No argument means disable all tracepoints."),
-          &disablelist);
-
-  add_cmd ("tracepoints", class_trace, enable_trace_command, _("\
-Enable specified tracepoints.\n\
-Arguments are tracepoint numbers, separated by spaces.\n\
-No argument means enable all tracepoints."),
-          &enablelist);
-
-  c = add_com ("trace", class_trace, trace_command, _("\
-Set a tracepoint at a specified line or function or address.\n\
-Argument may be a line number, function name, or '*' plus an address.\n\
-For a line number or function, trace at the start of its code.\n\
-If an address is specified, trace at that exact address.\n\n\
-Do \"help tracepoints\" for info on other tracepoint commands."));
-  set_cmd_completer (c, location_completer);
-
-  add_com_alias ("tp", "trace", class_alias, 0);
-  add_com_alias ("tr", "trace", class_alias, 1);
-  add_com_alias ("tra", "trace", class_alias, 1);
-  add_com_alias ("trac", "trace", class_alias, 1);
-
-  target_buf_size = 2048;
-  target_buf = xmalloc (target_buf_size);
+  default_collect = xstrdup ("");
+  add_setshow_string_cmd ("default-collect", class_trace,
+                         &default_collect, _("\
+Set the list of expressions to collect by default"), _("\
+Show the list of expressions to collect by default"), NULL,
+                         NULL, NULL,
+                         &setlist, &showlist);
+
+  add_setshow_boolean_cmd ("disconnected-tracing", no_class,
+                          &disconnected_tracing, _("\
+Set whether tracing continues after GDB disconnects."), _("\
+Show whether tracing continues after GDB disconnects."), _("\
+Use this to continue a tracing run even if GDB disconnects\n\
+or detaches from the target.  You can reconnect later and look at\n\
+trace data collected in the meantime."),
+                          set_disconnected_tracing,
+                          NULL,
+                          &setlist,
+                          &showlist);
+
+  add_setshow_boolean_cmd ("circular-trace-buffer", no_class,
+                          &circular_trace_buffer, _("\
+Set target's use of circular trace buffer."), _("\
+Show target's use of circular trace buffer."), _("\
+Use this to make the trace buffer into a circular buffer,\n\
+which will discard traceframes (oldest first) instead of filling\n\
+up and stopping the trace run."),
+                          set_circular_trace_buffer,
+                          NULL,
+                          &setlist,
+                          &showlist);
+
+  add_setshow_string_cmd ("trace-user", class_trace,
+                         &trace_user, _("\
+Set the user name to use for current and future trace runs"), _("\
+Show the user name to use for current and future trace runs"), NULL,
+                         set_trace_user, NULL,
+                         &setlist, &showlist);
+
+  add_setshow_string_cmd ("trace-notes", class_trace,
+                         &trace_notes, _("\
+Set notes string to use for current and future trace runs"), _("\
+Show the notes string to use for current and future trace runs"), NULL,
+                         set_trace_notes, NULL,
+                         &setlist, &showlist);
+
+  add_setshow_string_cmd ("trace-stop-notes", class_trace,
+                         &trace_stop_notes, _("\
+Set notes string to use for future tstop commands"), _("\
+Show the notes string to use for future tstop commands"), NULL,
+                         set_trace_stop_notes, NULL,
+                         &setlist, &showlist);
+
+  init_tfile_ops ();
+
+  add_target (&tfile_ops);
 }
This page took 0.116111 seconds and 4 git commands to generate.