Re: [Qemu-devel] [PATCH v3 0/3] qxl trace events

[Stefan Hajnoczi]

On Wed, Mar 14, 2012 at 1:09 PM, Alon Levy <address@hidden> wrote:
> Changes from v2:
>  split 6+ trace points to X + X_rest (leave qid as first argument in both) 
> (Stefan)
>  (cleanup as requested by Gerd)
>  no trace event for pipe write failure, left dprint there anyway.
>  indentation in a separate patch
>  qxl-render.c: qxl_render prefix
>  spice-display.c: qemu_spice functions trace names prefixed with qemu_spice
>
> Changes from v1:
>  add qid prefix to all tracepoints (except vga update which makes no sense)
>  rename to ring where suggested (Gerd)
>  add some trace points to spice-display
>  drop enter/exit in favor of a single command (Gerd)
>  some sorting of the trace event list
>
> Alon Levy (3):
>  qxl: switch qxl.c to trace-events
>  qxl/qxl_render.c: add trace events
>  qxl: init_pipe_signaling: exit on failure
>
>  hw/qxl-render.c    |   13 ++---
>  hw/qxl.c           |  160 
> ++++++++++++++++++++++++++--------------------------
>  trace-events       |   69 ++++++++++++++++++++++
>  ui/spice-display.c |   13 ++++-
>  4 files changed, 163 insertions(+), 92 deletions(-)

Thanks Alon,

Reviewed-by: Stefan Hajnoczi <address@hidden>