[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/10] trace: [doc] Document event properties on
From: |
Lluís Vilanova |
Subject: |
Re: [Qemu-devel] [PATCH 01/10] trace: [doc] Document event properties on a separate section |
Date: |
Fri, 09 Dec 2011 21:08:39 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.0.92 (gnu/linux) |
This series must be updated to apply on top of the patch recently accepted by
Stefan into the tracing tree [1].
[1] http://lists.gnu.org/archive/html/qemu-devel/2011-12/msg00763.html
Lluis
Lluís Vilanova writes:
> Signed-off-by: Lluís Vilanova <address@hidden>
> ---
> docs/tracing.txt | 22 ++++++++++++++++------
> 1 files changed, 16 insertions(+), 6 deletions(-)
> diff --git a/docs/tracing.txt b/docs/tracing.txt
> index ea29f2c..853cbf8 100644
> --- a/docs/tracing.txt
> +++ b/docs/tracing.txt
> @@ -98,12 +98,6 @@ respectively. This ensures portability between 32- and
> 64-bit platforms.
> 4. Name trace events after their function. If there are multiple trace
> events
> in one function, append a unique distinguisher at the end of the name.
> -5. If specific trace events are going to be called a huge number of times,
> this
> - might have a noticeable performance impact even when the trace events are
> - programmatically disabled. In this case you should declare the trace event
> - with the "disable" property, which will effectively disable it at compile
> - time (using the "nop" backend).
> -
> == Generic interface and monitor commands ==
> You can programmatically query and control the dynamic state of trace events
> @@ -234,3 +228,19 @@ probes:
> --target-type system \
> --target-arch x86_64 \
> <trace-events >qemu.stp
> +
> +== Trace event properties ==
> +
> +Each event in the "trace-events" file can be prefixed with a space-separated
> +list of zero or more of the following event properties.
> +
> +=== "disable" ===
> +
> +If a specific trace event is going to be invoked a huge number of times, this
> +might have a noticeable performance impact even when the event is
> +programmatically disabled.
> +
> +In this case you should declare such event with the "disable" property. This
> +will effectively disable the event at compile time (by using the "nop"
> backend),
> +thus having no performance impact at all on regular builds (i.e., unless you
> +edit the "trace-events" file).
--
"And it's much the same thing with knowledge, for whenever you learn
something new, the whole world becomes that much richer."
-- The Princess of Pure Reason, as told by Norton Juster in The Phantom
Tollbooth
- [Qemu-devel] [RFC][PATCH 00/10] trace-tcg: Allow tracing guest events in TCG-generated code, Lluís Vilanova, 2011/12/08
- [Qemu-devel] [PATCH 01/10] trace: [doc] Document event properties on a separate section, Lluís Vilanova, 2011/12/08
- Re: [Qemu-devel] [PATCH 01/10] trace: [doc] Document event properties on a separate section,
Lluís Vilanova <=
- [Qemu-devel] [PATCH 02/10] trace-tcg: Add documentation, Lluís Vilanova, 2011/12/08
- [Qemu-devel] [PATCH 03/10] Trivial changes to eliminate auto-generated files, Lluís Vilanova, 2011/12/08
- [Qemu-devel] [PATCH 04/10] [m68k, s390, xtensa] Move helpers.h to helper.h, Lluís Vilanova, 2011/12/08
- [Qemu-devel] [PATCH 05/10] trace: [tracetool] Common functions to manage event arguments, Lluís Vilanova, 2011/12/08
- [Qemu-devel] [PATCH 06/10] trace: [tracetool] Add 'get_api_name' to construct the name of tracing routines, Lluís Vilanova, 2011/12/08
- [Qemu-devel] [PATCH 07/10] trace-tcg: [tracetool] Allow TCG types in trace event declarations, Lluís Vilanova, 2011/12/08
- [Qemu-devel] [PATCH 08/10] trace-tcg: [tracetool] Declare TCG tracing helper routines, Lluís Vilanova, 2011/12/08
- [Qemu-devel] [PATCH 09/10] trace-tcg: [tracetool] Define TCG tracing helper routines, Lluís Vilanova, 2011/12/08
- [Qemu-devel] [PATCH 10/10] trace-tcg: [all] Include TCG-tracing helpers, Lluís Vilanova, 2011/12/08