[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/6] trace: Add trace events group implementatio
From: |
Ryan Harper |
Subject: |
Re: [Qemu-devel] [PATCH 3/6] trace: Add trace events group implementation in the backend "simple" |
Date: |
Wed, 12 Oct 2011 15:46:30 -0500 |
User-agent: |
Mutt/1.5.6+20040907i |
* Mark Wu <address@hidden> [2011-10-12 12:26]:
>
> Signed-off-by: Mark Wu <address@hidden>
> ---
> trace/simple.c | 30 ++++++++++++++++++++++++++++++
> trace/simple.h | 7 +++++++
> 2 files changed, 37 insertions(+), 0 deletions(-)
>
> diff --git a/trace/simple.c b/trace/simple.c
> index b639dda..7aa4c0b 100644
> --- a/trace/simple.c
> +++ b/trace/simple.c
> @@ -321,6 +321,16 @@ void trace_print_events(FILE *stream, fprintf_function
> stream_printf)
> }
> }
>
> +void trace_print_groups(FILE *stream, fprintf_function stream_printf)
> +{
> + unsigned int i;
> +
> + for (i = 0; i < NR_TRACE_EVENT_GROUPS; i++) {
> + stream_printf(stream, "%s [GROUP ID %u] : state %u\n",
> + trace_group_list[i].gp_name, i,
> + trace_group_list[i].state);
You've got a mix of space and tabs in this file. Remove the tabs and
adjust the spacing to CODING_STYLE rules.
> + }
> +}
> bool trace_event_set_state(const char *name, bool state)
> {
> unsigned int i;
> @@ -334,6 +344,26 @@ bool trace_event_set_state(const char *name, bool state)
> return false;
> }
>
> +bool trace_event_group_set_state(const char *gp_name, bool state)
> +{
> + unsigned int i;
> + unsigned int j;
> + TraceEventGroup *group;
> +
> + for (i = 0; i < NR_TRACE_EVENT_GROUPS; i++) {
> + group = &trace_group_list[i];
here
> + if (!strcmp(group->gp_name, gp_name)) {
> + group->state = state;
> +
> + for (j = group->start; j <= group->end; j++) {
here
> + trace_list[j].state = state;
> + }
> + return true;
> + }
> + }
> + return false;
> +}
> +
> /* Helper function to create a thread with signals blocked. Use glib's
> * portable threads since QEMU abstractions cannot be used due to reentrancy
> in
> * the tracer. Also note the signal masking on POSIX hosts so that the
> thread
> diff --git a/trace/simple.h b/trace/simple.h
> index 466e75b..cf119f3 100644
> --- a/trace/simple.h
> +++ b/trace/simple.h
> @@ -22,6 +22,13 @@ typedef struct {
> bool state;
> } TraceEvent;
>
> +typedef struct {
> + const char *gp_name;
> + bool state;
> + int start;
> + int end;
> +} TraceEventGroup;
> +
> void trace0(TraceEventID event);
> void trace1(TraceEventID event, uint64_t x1);
> void trace2(TraceEventID event, uint64_t x1, uint64_t x2);
> --
> 1.7.1
>
--
Ryan Harper
Software Engineer; Linux Technology Center
IBM Corp., Austin, Tx
address@hidden
- [Qemu-devel] [PATCH 0/6] trace: Add support for trace events grouping, Mark Wu, 2011/10/12
- [Qemu-devel] [PATCH 2/6] trace: Add HMP monitor commands for trace events group, Mark Wu, 2011/10/12
- [Qemu-devel] [PATCH 1/6] trace: Make "tracetool" generate a group list, Mark Wu, 2011/10/12
- [Qemu-devel] [PATCH 3/6] trace: Add trace events group implementation in the backend "simple", Mark Wu, 2011/10/12
- Re: [Qemu-devel] [PATCH 3/6] trace: Add trace events group implementation in the backend "simple",
Ryan Harper <=
- [Qemu-devel] [PATCH 5/6] trace: Enable "-trace events" argument to control initial state of groups, Mark Wu, 2011/10/12
- [Qemu-devel] [PATCH 4/6] trace: Add trace events group implementation in the backend "stderr", Mark Wu, 2011/10/12
- [Qemu-devel] [PATCH 6/6] trace: Update doc for trace events group, Mark Wu, 2011/10/12
- Re: [Qemu-devel] [PATCH 0/6] trace: Add support for trace events grouping, Mark Wu, 2011/10/13