[binutils-gdb] btrace: async

[Markus Metzger <mmetzger at sourceware dot org>]

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=a6b5be76f94664406def040d79ac8bc4d4aa8bd0

commit a6b5be76f94664406def040d79ac8bc4d4aa8bd0
Author: Markus Metzger <markus.t.metzger@intel.com>
Date:   Thu Aug 27 16:24:38 2015 +0200

    btrace: async
    
    The record btrace target runs synchronous with GDB.  That is, GDB steps
    resumed threads in record btrace's to_wait method.  Without GDB calling
    to_wait, nothing happens 'on the target'.
    
    Check for further expected events in to_wait before reporting the current
    event and mark record btrace's async event handler in async mode.
    
    gdb/
    	* record-btrace.c (record_btrace_maybe_mark_async_event): New.
    	(record_btrace_wait): Call record_btrace_maybe_mark_async_event.

Diff:
---
 gdb/ChangeLog       |  5 +++++
 gdb/record-btrace.c | 27 +++++++++++++++++++++++++++
 2 files changed, 32 insertions(+)

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 4c6d0e1..be86243 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,10 @@
 2015-09-18  Markus Metzger  <markus.t.metzger@intel.com>
 
+	* record-btrace.c (record_btrace_maybe_mark_async_event): New.
+	(record_btrace_wait): Call record_btrace_maybe_mark_async_event.
+
+2015-09-18  Markus Metzger  <markus.t.metzger@intel.com>
+
 	* record-btrace.c (get_thread_current_frame): New.
 	(record_btrace_start_replaying): Call get_thread_current_frame.
 
diff --git a/gdb/record-btrace.c b/gdb/record-btrace.c
index 0514471..764b349 100644
--- a/gdb/record-btrace.c
+++ b/gdb/record-btrace.c
@@ -2240,6 +2240,29 @@ record_btrace_step_thread (struct thread_info *tp)
 typedef struct thread_info * tp_t;
 DEF_VEC_P (tp_t);
 
+/* Announce further events if necessary.  */
+
+static void
+record_btrace_maybe_mark_async_event (const VEC (tp_t) *moving,
+				      const VEC (tp_t) *no_history)
+{
+  int more_moving, more_no_history;
+
+  more_moving = !VEC_empty (tp_t, moving);
+  more_no_history = !VEC_empty (tp_t, no_history);
+
+  if (!more_moving && !more_no_history)
+    return;
+
+  if (more_moving)
+    DEBUG ("movers pending");
+
+  if (more_no_history)
+    DEBUG ("no-history pending");
+
+  mark_async_event_handler (record_btrace_async_inferior_event_handler);
+}
+
 /* The to_wait method of target record-btrace.  */
 
 static ptid_t
@@ -2355,6 +2378,10 @@ record_btrace_wait (struct target_ops *ops, ptid_t ptid,
     ALL_NON_EXITED_THREADS (tp)
       record_btrace_cancel_resume (tp);
 
+  /* In async mode, we need to announce further events.  */
+  if (target_is_async_p ())
+    record_btrace_maybe_mark_async_event (moving, no_history);
+
   /* Start record histories anew from the current position.  */
   record_btrace_clear_histories (&eventing->btrace);