This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[PATCH 0/6 V5] MI notification on trace started/stopped


Hi,
This is the V5 of this patch series, and the changes in V5 are:

 - Move 'notif_queue' to remote_state, which is done by patch 1/6.
 - Store the supported status of each annex in remote_state too.  It
   is the major change in patch 3/6 compared with V4.
 - some small fixes in code and comments.

Patch 1/6 can be regarded as a refactor patch, so it can be approved
separately.  Patch 5/6 adds MI notifications when command 'tstart'
and 'tstop' is typed in MI, it is a typical MI patch, and can be
approved separately too.

Regression tested them on x86_64-linux with
{unix, native-gdbserver} x {sync, async}.  Is it OK?

Below is the introduction of this series.  People who are
familiar with this series, please skip it.

This patch series adds the MI notifications of 'trace-started' and
'trace-stopped', which are emitted when

  1) trace is started or stopped by commands in GDB,
  2) trace is stopped due to some reasons in the remote stub, such as
trace buffer full.

With these notifications, MI front-end can show the status of trace
up to date.

V4 can be found here

  [PATCH v4 0/5] MI notification on trace started/stopped
  http://sourceware.org/ml/gdb-patches/2013-04/msg00019.html

*** BLURB HERE ***

Yao Qi (6):
  Move notif_queue to remote_state
  Add annex in an async remote notification.
  Query supported notifications by qSupported
  async remote notification 'Trace'.
  MI notification on trace started/stopped:basic
  MI notification on trace stop: triggered by remote

 gdb/Makefile.in                              |   12 ++-
 gdb/NEWS                                     |    2 +
 gdb/common/notif-base.c                      |  198 ++++++++++++++++++++++++++
 gdb/common/notif-base.h                      |  111 ++++++++++++++
 gdb/doc/gdb.texinfo                          |   48 ++++++-
 gdb/doc/observer.texi                        |    6 +
 gdb/gdbserver/Makefile.in                    |    7 +-
 gdb/gdbserver/notif.c                        |   63 +++++++-
 gdb/gdbserver/notif.h                        |   23 ++--
 gdb/gdbserver/server.c                       |   20 +++-
 gdb/gdbserver/tracepoint.c                   |   36 +++++
 gdb/mi/mi-cmds.c                             |    6 +-
 gdb/mi/mi-interp.c                           |   22 +++
 gdb/mi/mi-main.h                             |    2 +
 gdb/remote-notif-trace.c                     |   86 +++++++++++
 gdb/remote-notif.c                           |  192 ++++++++++++++++++++++----
 gdb/remote-notif.h                           |   48 +++++--
 gdb/remote.c                                 |   56 ++++++--
 gdb/testsuite/gdb.trace/mi-trace-changed.exp |  149 +++++++++++++++++++
 gdb/tracepoint.c                             |    4 +
 20 files changed, 1009 insertions(+), 82 deletions(-)
 create mode 100644 gdb/common/notif-base.c
 create mode 100644 gdb/common/notif-base.h
 create mode 100644 gdb/remote-notif-trace.c
 create mode 100644 gdb/testsuite/gdb.trace/mi-trace-changed.exp

-- 
1.7.7.6


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]