From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 11996 invoked by alias); 19 Aug 2013 01:56:07 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 11986 invoked by uid 89); 19 Aug 2013 01:56:07 -0000 X-Spam-SWARE-Status: No, score=-3.8 required=5.0 tests=AWL,BAYES_00,FROM_12LTRDOM,KHOP_RCVD_UNTRUST,RCVD_IN_HOSTKARMA_W,RCVD_IN_HOSTKARMA_WL autolearn=ham version=3.3.2 Received: from relay1.mentorg.com (HELO relay1.mentorg.com) (192.94.38.131) by sourceware.org (qpsmtpd/0.84/v0.84-167-ge50287c) with ESMTP; Mon, 19 Aug 2013 01:56:05 +0000 Received: from svr-orw-fem-01.mgc.mentorg.com ([147.34.98.93]) by relay1.mentorg.com with esmtp id 1VBEhP-0002w1-TD from Yao_Qi@mentor.com for gdb-patches@sourceware.org; Sun, 18 Aug 2013 18:56:03 -0700 Received: from SVR-ORW-FEM-04.mgc.mentorg.com ([147.34.97.41]) by svr-orw-fem-01.mgc.mentorg.com over TLS secured channel with Microsoft SMTPSVC(6.0.3790.4675); Sun, 18 Aug 2013 18:56:03 -0700 Received: from qiyao.dyndns.org (147.34.91.1) by svr-orw-fem-04.mgc.mentorg.com (147.34.97.41) with Microsoft SMTP Server id 14.2.247.3; Sun, 18 Aug 2013 18:56:03 -0700 From: Yao Qi To: Subject: [PATCH 0/6 V5] MI notification on trace started/stopped Date: Mon, 19 Aug 2013 01:56:00 -0000 Message-ID: <1376877311-4135-1-git-send-email-yao@codesourcery.com> MIME-Version: 1.0 Content-Type: text/plain X-SW-Source: 2013-08/txt/msg00484.txt.bz2 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