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/4 V7] MI notification on trace started/stopped


Hi,
I examined all previous versions of this series and corresponding
reviews.  I find that V2 is the closet one to acceptance.  After
v2, I pursued "notification annex" and supported query on it,
which is a blind alley. (I don't recall why I choose this way then).

I update my patches to follow V2 and address comments I got during
the review.  The major comment raised by Pedro is that "GDB and
the remote negotiate the set of supported notifications".

  https://sourceware.org/ml/gdb-patches/2013-01/msg00245.html

Patch 1/4 does it.  Patch 2/4 is to add new notification on
trace.  Patch 3/4 and 4/4 are to use this notification and add
tests.

Patch series is regression tested on x86_64-linux with board file
{unix, native-gdbserver} x {sync, async}.

*** BLURB HERE ***

Yao Qi (4):
  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                              |    5 +-
 gdb/NEWS                                     |    2 +
 gdb/doc/gdb.texinfo                          |   31 ++++++
 gdb/doc/observer.texi                        |    6 +
 gdb/gdbserver/notif.c                        |   84 +++++++++++++++
 gdb/gdbserver/notif.h                        |    6 +
 gdb/gdbserver/server.c                       |   16 +++-
 gdb/gdbserver/tracepoint.c                   |   27 +++++
 gdb/mi/mi-cmds.c                             |    6 +-
 gdb/mi/mi-interp.c                           |   22 ++++
 gdb/mi/mi-main.h                             |    2 +
 gdb/remote-notif-trace.c                     |   88 +++++++++++++++
 gdb/remote-notif.c                           |  110 ++++++++++++++++++-
 gdb/remote-notif.h                           |   16 +++-
 gdb/remote.c                                 |   29 +++++-
 gdb/testsuite/gdb.trace/mi-trace-changed.exp |  149 ++++++++++++++++++++++++++
 gdb/tracepoint.c                             |    4 +
 17 files changed, 590 insertions(+), 13 deletions(-)
 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]