From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 28347 invoked by alias); 5 May 2011 23:47:00 -0000 Received: (qmail 28338 invoked by uid 22791); 5 May 2011 23:47:00 -0000 X-SWARE-Spam-Status: No, hits=-2.0 required=5.0 tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,RCVD_IN_DNSWL_NONE,T_RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from elasmtp-mealy.atl.sa.earthlink.net (HELO elasmtp-mealy.atl.sa.earthlink.net) (209.86.89.69) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Thu, 05 May 2011 23:46:46 +0000 Received: from [70.170.59.51] (helo=macbook-2.local) by elasmtp-mealy.atl.sa.earthlink.net with esmtpa (Exim 4.67) (envelope-from ) id 1QI8Fp-0005wG-E7 for gdb-patches@sourceware.org; Thu, 05 May 2011 19:46:45 -0400 Message-ID: <4DC336E4.6090203@earthlink.net> Date: Thu, 05 May 2011 23:47:00 -0000 From: Stan Shebs User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.2.17) Gecko/20110414 Thunderbird/3.1.10 MIME-Version: 1.0 To: gdb-patches@sourceware.org Subject: Re: [PATCH] Support for enabling/disabling tracepoints while a trace experiment is running References: <4DC2DDEF.10600@codesourcery.com> <83hb99m14x.fsf@gnu.org> <4DC32F21.9090400@codesourcery.com> In-Reply-To: <4DC32F21.9090400@codesourcery.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-ELNK-Trace: ae6f8838ff913eba0cc1426638a40ef67e972de0d01da940503c050821587910fff6a9899455797f350badd9bab72f9c350badd9bab72f9c350badd9bab72f9c 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: 2011-05/txt/msg00166.txt.bz2 On 5/5/11 4:13 PM, Kwok Cheung Yeung wrote: > On 05/05/2011 19:51, Eli Zaretskii wrote: >>> Date: Thu, 05 May 2011 18:27:11 +0100 >>> From: Kwok Cheung Yeung >>> >>> + If a trace experiment is started with no enabled >>> + tracepoints, then a warning will be printed but the experiment will >>> + proceed anyway. >> You mean, if the experiment started with no enabled tracepoints, and >> the user says "disable", right? Otherwise, why the warning? >> > Originally, it was an error to try to start an experiment with no enabled > tracepoints because the resulting experiment would never do anything. However, > with the patch, tracepoints can be re-enabled after the experiment starts, so I > downgraded the error to a warning. I suppose we could do away with it > altogether, but then a user who inadvertently disabled all tracepoints might sit > and wonder why nothing seems to be happening. Definitely need at least a warning - all too easy to disable everything, including all the tracepoints, with a plain "dis", then be well into the run before realizing that none of the tracepoints are getting hits recorded. Of course, that does go against the Unix tradition of putting the snakepit in the middle of the living room floor... :-) "Eh?" Stan stan@codesourcery.com