From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 12274 invoked by alias); 24 Oct 2012 17:23:10 -0000 Received: (qmail 12266 invoked by uid 22791); 24 Oct 2012 17:23:09 -0000 X-SWARE-Spam-Status: No, hits=-4.7 required=5.0 tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,KHOP_RCVD_TRUST,KHOP_THREADED,RCVD_IN_DNSWL_LOW,RCVD_IN_HOSTKARMA_YE X-Spam-Check-By: sourceware.org Received: from mail-vc0-f169.google.com (HELO mail-vc0-f169.google.com) (209.85.220.169) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Wed, 24 Oct 2012 17:23:03 +0000 Received: by mail-vc0-f169.google.com with SMTP id fl17so930163vcb.0 for ; Wed, 24 Oct 2012 10:23:02 -0700 (PDT) Received: by 10.52.19.109 with SMTP id d13mr22686740vde.27.1351099382215; Wed, 24 Oct 2012 10:23:02 -0700 (PDT) MIME-Version: 1.0 Received: by 10.220.159.147 with HTTP; Wed, 24 Oct 2012 10:22:22 -0700 (PDT) In-Reply-To: <50881981.7070308@codesourcery.com> References: <50880350.1040309@mentor.com> <50881981.7070308@codesourcery.com> From: Hui Zhu Date: Wed, 24 Oct 2012 17:23:00 -0000 Message-ID: Subject: Re: [RFC] Target-defined breakpoints [3/9] notification async To: Yao Qi Cc: Hui Zhu , gdb-patches@sourceware.org Content-Type: text/plain; charset=ISO-8859-1 X-IsSubscribed: yes 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 X-SW-Source: 2012-10/txt/msg00465.txt.bz2 On Thu, Oct 25, 2012 at 12:38 AM, Yao Qi wrote: > On 10/24/2012 11:03 PM, Hui Zhu wrote: >> >> This patch is the extend for the notification function. >> Current notification function cannot handle the packet when GDB doesn't >> want to send or receive packet from the remote target. >> This patch do the extend make GDB can handle the notification even if it >> doesn't send or receive packet. > > > Looks we have two set of 'async notification' stuff here :) I am trying to > set up an infrastructure of 'async notification' in gdb, so that other > components can use it. Patches are here, > > http://sourceware.org/ml/gdb-patches/2012-10/msg00404.html > [RFC 0/7 V3]A general notification in GDB RSP > > It is general, not specific to any notifications, and scalable. We've > probably have two internal cases of using it, 'trace status change' and > 'stub/probe status change', so "Target-defined breakpoint" can use it easily > as well. > > Do you mind if I suggest maintainers to review my 'async notification' > patches first? Once they go in, we can rebase your patch on top of it. I > am sure your patch will be shorter dramatically then. WDYT? Or if you find > my 'async notification' doesn't meet your needs, let me know. > > -- > Yao I don't mind change my patch according to your change if you can check in. PS. I think target-defind xxx patches still have some work (format and handler, please read the 0/9) not done. So please don't worry about it. Best, Hui