Re: [Qemu-devel] [PATCH v5 0/2] trace: Simplify late initialization

[Stefan Hajnoczi]

On Tue, Aug 23, 2016 at 10:58:45AM +0200, Lluís Vilanova wrote:
> Removes the need for 'trace_events_dstate_init' and does a little cleanup in 
> how
> state values are modified (to avoid implicit conversions from bool).
> 
> Changes in v2
> =============
> 
> * Fix late-init state value [Daniel P. Berrange].
> 
> Changes in v3
> =============
> 
> * Avoid implicit conversions from bool to integers (second patch) [Daniel
>   P. Berrange].
> 
> Changes in v4
> =============
> 
> * Correctly implement idempotent state changes.
> * Clarify when state changes are idempotent [Daniel P. Berrange].
> 
> Changes in v5
> =============
> 
> * Always check first if event is disabled in trace_init_vcpu_events().
> 
> Signed-off-by: Lluís Vilanova <address@hidden>
> ---
> 
> Lluís Vilanova (2):
>       trace: Remove 'trace_events_dstate_init'
>       trace: Avoid implicit bool->integer conversions
> 
> 
>  stubs/trace-control.c  |   22 ++++++++++++++++++++--
>  trace/control-target.c |   34 ++++++++++++++++++++++++++++++++--
>  trace/control.c        |   21 ++++++++++-----------
>  trace/control.h        |    3 +++
>  trace/event-internal.h |    1 +
>  5 files changed, 66 insertions(+), 15 deletions(-)
> 
> 
> To: address@hidden
> Cc: Stefan Hajnoczi <address@hidden>
> Cc: 'Daniel P. Berrange' <address@hidden>
> Cc: Paolo Bonzini <address@hidden>

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

signature.asc
Description: PGP signature