mirror of
https://gitlab.com/qemu-project/qemu
synced 2024-09-06 09:55:55 +00:00
![Daniel P. Berrange](/assets/img/avatar_default.png)
Instead of having a global dstate array, declare a single 'uint16 TRACE_${EVENT_NAME}_DSTATE' variable for each trace event. Record a pointer to this variable in the TraceEvent struct too. By turning trace_event_get_state_dynamic_by_id into a macro, this still hits the fast path, and cache affinity is ensured by declaring all the uint16 vars adjacent to each other. Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com> Reviewed-by: Lluís Vilanova <vilanova@ac.upc.edu> Signed-off-by: Daniel P. Berrange <berrange@redhat.com> Message-id: 1475588159-30598-7-git-send-email-berrange@redhat.com Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
52 lines
1.3 KiB
C
52 lines
1.3 KiB
C
/*
|
|
* Interface for configuring and controlling the state of tracing events.
|
|
*
|
|
* Copyright (C) 2014-2016 Lluís Vilanova <vilanova@ac.upc.edu>
|
|
*
|
|
* This work is licensed under the terms of the GNU GPL, version 2 or later.
|
|
* See the COPYING file in the top-level directory.
|
|
*/
|
|
|
|
#include "qemu/osdep.h"
|
|
#include "trace/control.h"
|
|
|
|
|
|
void trace_event_set_state_dynamic_init(TraceEvent *ev, bool state)
|
|
{
|
|
trace_event_set_state_dynamic(ev, state);
|
|
}
|
|
|
|
void trace_event_set_state_dynamic(TraceEvent *ev, bool state)
|
|
{
|
|
bool state_pre;
|
|
assert(trace_event_get_state_static(ev));
|
|
|
|
/*
|
|
* We ignore the "vcpu" property here, since there's no target code. Then
|
|
* dstate can only be 1 or 0.
|
|
*/
|
|
state_pre = *(ev->dstate);
|
|
if (state_pre != state) {
|
|
if (state) {
|
|
trace_events_enabled_count++;
|
|
*(ev->dstate) = 1;
|
|
} else {
|
|
trace_events_enabled_count--;
|
|
*(ev->dstate) = 0;
|
|
}
|
|
}
|
|
}
|
|
|
|
void trace_event_set_vcpu_state_dynamic(CPUState *vcpu,
|
|
TraceEvent *ev, bool state)
|
|
{
|
|
/* should never be called on non-target binaries */
|
|
abort();
|
|
}
|
|
|
|
void trace_init_vcpu(CPUState *vcpu)
|
|
{
|
|
/* should never be called on non-target binaries */
|
|
abort();
|
|
}
|