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 <vilan...@ac.upc.edu> > --- > > 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: qemu-devel@nongnu.org > Cc: Stefan Hajnoczi <stefa...@redhat.com> > Cc: 'Daniel P. Berrange' <berra...@redhat.com> > Cc: Paolo Bonzini <pbonz...@redhat.com>
Thanks, applied to my tracing-next tree: https://github.com/stefanha/qemu/commits/tracing-next
signature.asc
Description: PGP signature
