From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:38768) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RGmcS-0007TP-UA for qemu-devel@nongnu.org; Thu, 20 Oct 2011 03:00:49 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RGmcR-0002bt-E0 for qemu-devel@nongnu.org; Thu, 20 Oct 2011 03:00:48 -0400 Received: from e23smtp02.au.ibm.com ([202.81.31.144]:55499) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RGmcQ-0002bf-RK for qemu-devel@nongnu.org; Thu, 20 Oct 2011 03:00:47 -0400 Received: from d23relay05.au.ibm.com (d23relay05.au.ibm.com [202.81.31.247]) by e23smtp02.au.ibm.com (8.14.4/8.13.1) with ESMTP id p9K6rssJ001015 for ; Thu, 20 Oct 2011 17:53:54 +1100 Received: from d23av03.au.ibm.com (d23av03.au.ibm.com [9.190.234.97]) by d23relay05.au.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id p9K6vx9E2129926 for ; Thu, 20 Oct 2011 17:58:02 +1100 Received: from d23av03.au.ibm.com (loopback [127.0.0.1]) by d23av03.au.ibm.com (8.14.4/8.13.1/NCO v10.0 AVout) with ESMTP id p9K70X3J016445 for ; Thu, 20 Oct 2011 18:00:33 +1100 Received: from oc4654482034.ibm.com ([9.115.122.78]) by d23av03.au.ibm.com (8.14.4/8.13.1/NCO v10.0 AVin) with ESMTP id p9K70W5j014208 for ; Thu, 20 Oct 2011 18:00:32 +1100 Message-ID: <4E9FC6DE.5040909@vnet.linux.ibm.com> Date: Thu, 20 Oct 2011 14:59:42 +0800 From: Mark Wu MIME-Version: 1.0 References: <1319092911-23711-1-git-send-email-wudxw@linux.vnet.ibm.com> In-Reply-To: <1319092911-23711-1-git-send-email-wudxw@linux.vnet.ibm.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2 0/6] trace: Add support for trace events grouping List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org On 10/20/2011 02:41 PM, Mark Wu wrote: > This series add support for trace events grouping. The state of a given group > of trace events can be queried or changed in bulk by the following monitor > commands: > > * info trace-groups > View available trace event groups and their state. State 1 means enabled, > state 0 means disabled. > > * trace-group NAME on|off > Enable/disable a given trace event group. > > A group of trace events can also be enabled in early running stage through > adding its group name prefixed with "group:" to trace events list file > which is passed to "-trace events". > > Change from V1: > * fix coding style problems > * rebase against latest master > > Mark Wu (6): > trace: Make "tracetool" generate a group list > trace: Add HMP monitor commands for trace events group > trace: Add trace events group implementation in the backend "simple" > trace: Add trace events group implementation in the backend "stderr" > trace: Enable "-trace events" argument to control initial state of > groups > trace: Update doc for trace events group > > docs/tracing.txt | 29 ++++++++++++++-- > hmp-commands.hx | 14 ++++++++ > monitor.c | 22 ++++++++++++ > scripts/tracetool | 94 +++++++++++++++++++++++++++++++++++++++++++++++++++- > trace-events | 94 +++++++++++++++++++++++++++++++++++++++++++++++++++++ > trace/control.c | 17 +++++++++ > trace/control.h | 9 +++++ > trace/default.c | 15 ++++++++ > trace/simple.c | 30 +++++++++++++++++ > trace/simple.h | 7 ++++ > trace/stderr.c | 32 ++++++++++++++++++ > trace/stderr.h | 7 ++++ > 12 files changed, 365 insertions(+), 5 deletions(-) > > Please ignore this series of patch. I am going to make a new patch according to Stefan's suggestion instead of grouping.