Re: [Qemu-devel] [PATCH 0/8] Tracetool cleanup

[Stefan Hajnoczi]

On Sun, Feb 23, 2014 at 08:36:57PM +0100, Lluís Vilanova wrote:
> Minimizes the amount of backend code, making it simpler to add new/different
> backends.
> 
> Also performs other cleanups all around.
> 
> Signed-off-by: Lluís Vilanova <address@hidden>
> ---
> 
> Changes in v2:
> 
> * Rebase on 6f0b22d.
> * Remove "API" variable in forntend modules.
> * Further split changes.
> * Bump up trace format version for the simple backend.
> 
> 
> Lluís Vilanova (8):
>       trace: [tracetool] Add method 'Event.api' to build event names
>       trace: [tracetool] Add methods 'Event.copy' and 'Arguments.copy'
>       trace: [tracetool] Spacing changes
>       trace: [tracetool] Cosmetic changes
>       trace: [tracetool] Show list of frontends and backends sorted by name
>       trace: [tracetool] Change format docs to point to the generated file
>       trace: [simple] Bump up log version number
>       trace: [tracetool] Minimize the amount of per-backend code
> 
> 
>  scripts/simpletrace.py                   |   16 +---
>  scripts/tracetool/__init__.py            |   53 ++++++------
>  scripts/tracetool/backend/__init__.py    |   74 ++++++++---------
>  scripts/tracetool/backend/dtrace.py      |   81 ++----------------
>  scripts/tracetool/backend/events.py      |   23 -----
>  scripts/tracetool/backend/ftrace.py      |   56 ++++++-------
>  scripts/tracetool/backend/simple.py      |  132 
> +++++++++++++++---------------
>  scripts/tracetool/backend/stderr.py      |   43 ++++------
>  scripts/tracetool/backend/ust.py         |   65 ++-------------
>  scripts/tracetool/format/__init__.py     |   50 ++++-------
>  scripts/tracetool/format/c.py            |   16 +++-
>  scripts/tracetool/format/d.py            |   30 ++++++-
>  scripts/tracetool/format/events_c.py     |   15 +--
>  scripts/tracetool/format/events_h.py     |   15 +--
>  scripts/tracetool/format/h.py            |   30 ++++---
>  scripts/tracetool/format/stap.py         |   42 +++++++++-
>  scripts/tracetool/format/ust_events_c.py |    7 +-
>  scripts/tracetool/format/ust_events_h.py |   42 +++++++++-
>  trace/Makefile.objs                      |    4 -
>  trace/simple.c                           |    2 
>  20 files changed, 362 insertions(+), 434 deletions(-)
>  delete mode 100644 scripts/tracetool/backend/events.py
> 
> 
> To: address@hidden
> Cc: Stefan Hajnoczi <address@hidden>
> Cc: Kazuya Saito <address@hidden>
> 

Thanks, applied to my tracing-next tree:
https://github.com/stefanha/qemu/commits/tracing-next

Stefan