Re: [Qemu-devel] [PATCH 0/6] trace: Add support for trace events grouping

[Mark Wu <wudxw@vnet.linux.ibm.com>]

On 10/13/2011 01:14 AM, 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".
>
> 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      |   88 +++++++++++++++++++++++++++++++++++++++++++++++++
>   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, 359 insertions(+), 5 deletions(-)
>
Sorry, there're some coding style problems in the patches. I have fixed 
them and will send out later in order to see if there's any other 
problem coming up. :)