[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 3/6] trace-state: always use the "nop" backend o
From: |
Stefan Hajnoczi |
Subject: |
[Qemu-devel] Re: [PATCH 3/6] trace-state: always use the "nop" backend on events with the "disable" keyword |
Date: |
Wed, 6 Apr 2011 12:04:32 +0100 |
On Mon, Apr 4, 2011 at 10:49 PM, Lluís <address@hidden> wrote:
> Any event with the keyword/property "disable" generates an empty trace event
> using the "nop" backend, regardless of the current backend.
>
> Signed-off-by: Lluís Vilanova <address@hidden>
> ---
> docs/tracing.txt | 3 +++
> scripts/tracetool | 15 ++-------------
> 2 files changed, 5 insertions(+), 13 deletions(-)
>
> diff --git a/docs/tracing.txt b/docs/tracing.txt
> index 5dbd3c0..49e030e 100644
> --- a/docs/tracing.txt
> +++ b/docs/tracing.txt
> @@ -126,6 +126,9 @@ The "nop" backend generates empty trace event functions
> so that the compiler
> can optimize out trace events completely. This is the default and imposes no
> performance penalty.
>
> +Note that regardless of the selected trace backend, events with the "disable"
> +property will be generated with the "nop" backend.
> +
> === Stderr ===
>
> The "stderr" backend sends trace events directly to standard error. This
Please also fix up docs/tracing.txt:
2. Enable trace events you are interested in:
$EDITOR trace-events # remove "disable" from events you want
The comment is now obsolete.
Stefan
- [Qemu-devel] [RFC][PATCH 0/6] trace-state: make the behaviour of "disable" consistent across all backends, Lluís, 2011/04/04
- [Qemu-devel] [PATCH 1/6] trace: [ust] fix generation of 'trace.c' on events without args, Lluís, 2011/04/04
- [Qemu-devel] [PATCH 2/6] trace: generalize the "property" concept in the trace-events file, Lluís, 2011/04/04
- [Qemu-devel] [PATCH 3/6] trace-state: always use the "nop" backend on events with the "disable" keyword, Lluís, 2011/04/04
- [Qemu-devel] Re: [PATCH 3/6] trace-state: always use the "nop" backend on events with the "disable" keyword,
Stefan Hajnoczi <=
- [Qemu-devel] [PATCH 4/6] trace-state: [simple] disable all trace points by default, Lluís, 2011/04/04
- [Qemu-devel] [PATCH 5/6] trace-state: [simple] add "-trace events" argument to control initial state, Lluís, 2011/04/04
[Qemu-devel] [PATCH 6/6] trace: enable all events, Lluís, 2011/04/04
[Qemu-devel] Re: [RFC][PATCH 0/6] trace-state: make the behaviour of "disable" consistent across all backends, Stefan Hajnoczi, 2011/04/05