From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 24695 invoked by alias); 4 Jan 2012 12:02:34 -0000 Received: (qmail 24685 invoked by uid 22791); 4 Jan 2012 12:02:33 -0000 X-SWARE-Spam-Status: No, hits=-2.6 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,TW_DB X-Spam-Check-By: sourceware.org Received: from mail-we0-f169.google.com (HELO mail-we0-f169.google.com) (74.125.82.169) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Wed, 04 Jan 2012 12:02:18 +0000 Received: by werf1 with SMTP id f1so11182297wer.0 for ; Wed, 04 Jan 2012 04:02:17 -0800 (PST) Received: by 10.216.136.20 with SMTP id v20mr31188782wei.14.1325678537456; Wed, 04 Jan 2012 04:02:17 -0800 (PST) Received: from [192.168.0.103] (bl22-165-22.dsl.telepac.pt. [2.83.165.22]) by mx.google.com with ESMTPS id gf8sm10471746wbb.11.2012.01.04.04.02.15 (version=SSLv3 cipher=OTHER); Wed, 04 Jan 2012 04:02:16 -0800 (PST) Message-ID: <4F043FC6.3070201@gmail.com> Date: Wed, 04 Jan 2012 12:02:00 -0000 From: Pedro Alves User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:9.0) Gecko/20111222 Thunderbird/9.0 MIME-Version: 1.0 To: Yao Qi CC: gdb-patches@sourceware.org Subject: Re: [patch] Turn on dbg msg of IPA along with gdbserver's References: <4F03CA12.7050304@codesourcery.com> In-Reply-To: <4F03CA12.7050304@codesourcery.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit 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-01/txt/msg00125.txt.bz2 On 01/04/2012 03:40 AM, Yao Qi wrote: > Hi, > We are still unable to change IPA's debug flag `debug_threads', because > it is within inferior's address space. Since it's in the inferior, I just do "print debug_threads = 1" from gdb. It won't work if the ipa is stripped though, so I won't object to something like this. > Since gdbserver has been a good > mechanism to access gdb_agent_XXX variables in inferior's space, we can > do similar thing to turn on IPA's debug flag. > > This patch renames debug flag `debug_threads' to `gdb_agent_debug', and > let gdbserver to set it if `debug_threads' (in gdbserver's space) is > true. For the user's perspective, if gdbserver option --debug is on, > IPA's debugging message comes into the gdbserver's debug message. > > OK for mainline? Sorry, not in this form. It's quite useful to be able to turn gdbserver's and the IPA's debug output independently. I'd like to retain that possibility. -- Pedro Alves