Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Yao Qi <yao@codesourcery.com>
To: <gdb-patches@sourceware.org>
Subject: [PATCH 0/6 V5] MI notification on trace started/stopped
Date: Mon, 19 Aug 2013 01:56:00 -0000	[thread overview]
Message-ID: <1376877311-4135-1-git-send-email-yao@codesourcery.com> (raw)

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


             reply	other threads:[~2013-08-19  1:56 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-19  1:56 Yao Qi [this message]
2013-08-19  1:56 ` [PATCH 5/6] MI notification on trace started/stopped:basic Yao Qi
2013-08-19  1:56 ` [PATCH 2/6] Add annex in an async remote notification Yao Qi
2013-09-26 18:43   ` Pedro Alves
2013-09-27  1:44     ` Yao Qi
2013-10-18  1:05       ` Yao Qi
2013-08-19  1:56 ` [PATCH 1/6] Move notif_queue to remote_state Yao Qi
2013-09-25 16:12   ` Pedro Alves
2013-09-30  7:34     ` Yao Qi
2013-09-30  7:58       ` Move pending_event to remote_notif_state ([PATCH 1/6] Move notif_queue to remote_state) Yao Qi
2013-09-30 19:34         ` Pedro Alves
2013-10-04  7:42           ` Yao Qi
2013-09-30 17:08       ` [PATCH 1/6] Move notif_queue to remote_state Pedro Alves
2013-10-01 14:08         ` Yao Qi
2013-10-02  1:54         ` Yao Qi
2013-10-02 10:48           ` Pedro Alves
2013-10-04  7:36             ` Yao Qi
2013-08-19  1:56 ` [PATCH 4/6] async remote notification 'Trace' Yao Qi
2013-08-19  1:56 ` [PATCH 6/6] MI notification on trace stop: triggered by remote Yao Qi
2013-08-19  1:56 ` [PATCH 3/6] Query supported notifications by qSupported Yao Qi
2013-09-02  0:14 ` [PATCH 0/6 V5] MI notification on trace started/stopped Yao Qi
2013-09-18 13:24 ` [ping 2]: " Yao Qi
2013-09-18 13:25   ` Pedro Alves

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1376877311-4135-1-git-send-email-yao@codesourcery.com \
    --to=yao@codesourcery.com \
    --cc=gdb-patches@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox