record-btrace: provide target_find_new_threads method

The "info threads" command tries to read memory, which is not possible during
replay.  This results in an error message and aborts the command without showing
the existing threads.

Provide a to_find_new_threads target method to skip the search while replaying.

2014-01-16  Markus Metzger  <markus.t.metzger@intel.com>

	* record-btrace.c (record_btrace_find_new_threads)
	(record_btrace_thread_alive): New.
	(init_record_btrace_ops): Initialize to_find_new_threads and
	to_thread_alive.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 7ca7fb32162c835ab1af615dd2c8eec2727473e6..95b59b668b61d26817a91fd7b912bfe5655fee38 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,10 @@
+2014-01-16  Markus Metzger  <markus.t.metzger@intel.com>
+
+       * record-btrace.c (record_btrace_find_new_threads)
+       (record_btrace_thread_alive): New.
+       (init_record_btrace_ops): Initialize to_find_new_threads and
+       to_thread_alive.
+
 2014-01-16  Markus Metzger  <markus.t.metzger@intel.com>
 
        * record-btrace.c (record_btrace_resume): New.

diff --git a/gdb/record-btrace.c b/gdb/record-btrace.c
index 1951dce4eef199007c0ef36c98deca94c35e9040..acea2d636901288b490d7ffb7dd0456e0c6eb51e 100644 (file)
--- a/gdb/record-btrace.c
+++ b/gdb/record-btrace.c
@@ -1050,6 +1050,41 @@ record_btrace_wait (struct target_ops *ops, ptid_t ptid,
   error (_("You can't do this from here.  Do 'record goto end', first."));
 }
 
+/* The to_find_new_threads method of target record-btrace.  */
+
+static void
+record_btrace_find_new_threads (struct target_ops *ops)
+{
+  /* Don't expect new threads if we're replaying.  */
+  if (record_btrace_is_replaying ())
+    return;
+
+  /* Forward the request.  */
+  for (ops = ops->beneath; ops != NULL; ops = ops->beneath)
+    if (ops->to_find_new_threads != NULL)
+      {
+       ops->to_find_new_threads (ops);
+       break;
+      }
+}
+
+/* The to_thread_alive method of target record-btrace.  */
+
+static int
+record_btrace_thread_alive (struct target_ops *ops, ptid_t ptid)
+{
+  /* We don't add or remove threads during replay.  */
+  if (record_btrace_is_replaying ())
+    return find_thread_ptid (ptid) != NULL;
+
+  /* Forward the request.  */
+  for (ops = ops->beneath; ops != NULL; ops = ops->beneath)
+    if (ops->to_thread_alive != NULL)
+      return ops->to_thread_alive (ops, ptid);
+
+  return 0;
+}
+
 /* Initialize the record-btrace target ops.  */
 
 static void
@@ -1086,6 +1121,8 @@ init_record_btrace_ops (void)
   ops->to_get_unwinder = &record_btrace_frame_unwind;
   ops->to_resume = record_btrace_resume;
   ops->to_wait = record_btrace_wait;
+  ops->to_find_new_threads = record_btrace_find_new_threads;
+  ops->to_thread_alive = record_btrace_thread_alive;
   ops->to_stratum = record_stratum;
   ops->to_magic = OPS_MAGIC;
 }