[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 01/13] trace: count number of enabled events
From: |
Stefan Hajnoczi |
Subject: |
[Qemu-devel] [PULL 01/13] trace: count number of enabled events |
Date: |
Wed, 3 Feb 2016 15:47:33 +0000 |
From: Paolo Bonzini <address@hidden>
This lets trace_event_get_state_dynamic quickly return false. Right
now there is hardly any benefit because there are also many assertions
and indirections, but the next patch will streamline all of this.
Signed-off-by: Paolo Bonzini <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>
---
trace/control-internal.h | 4 +++-
trace/control.c | 2 ++
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/trace/control-internal.h b/trace/control-internal.h
index 5a8df28..271bddb 100644
--- a/trace/control-internal.h
+++ b/trace/control-internal.h
@@ -14,6 +14,7 @@
extern TraceEvent trace_events[];
+extern int trace_events_enabled_count;
static inline TraceEventID trace_event_count(void)
@@ -54,13 +55,14 @@ static inline bool trace_event_get_state_static(TraceEvent
*ev)
static inline bool trace_event_get_state_dynamic(TraceEvent *ev)
{
assert(ev != NULL);
- return ev->dstate;
+ return unlikely(trace_events_enabled_count) && ev->dstate;
}
static inline void trace_event_set_state_dynamic(TraceEvent *ev, bool state)
{
assert(ev != NULL);
assert(trace_event_get_state_static(ev));
+ trace_events_enabled_count += state - ev->dstate;
ev->dstate = state;
}
diff --git a/trace/control.c b/trace/control.c
index 995beb3..95fbc07 100644
--- a/trace/control.c
+++ b/trace/control.c
@@ -16,6 +16,8 @@
#endif
#include "qemu/error-report.h"
+int trace_events_enabled_count;
+
TraceEvent *trace_event_name(const char *name)
{
assert(name != NULL);
--
2.5.0
- [Qemu-devel] [PULL 00/13] Tracing patches, Stefan Hajnoczi, 2016/02/03
- [Qemu-devel] [PULL 01/13] trace: count number of enabled events,
Stefan Hajnoczi <=
- [Qemu-devel] [PULL 03/13] trace: fix documentation, Stefan Hajnoczi, 2016/02/03
- [Qemu-devel] [PULL 02/13] trace: track enabled events in a separate array, Stefan Hajnoczi, 2016/02/03
- [Qemu-devel] [PULL 04/13] trace: split trace_init_events out of trace_init_backends, Stefan Hajnoczi, 2016/02/03
- [Qemu-devel] [PULL 06/13] trace: no need to call trace_backend_init in different branches now, Stefan Hajnoczi, 2016/02/03
- [Qemu-devel] [PULL 05/13] trace: split trace_init_file out of trace_init_backends, Stefan Hajnoczi, 2016/02/03
[Qemu-devel] [PULL 07/13] trace: add "-trace enable=...", Stefan Hajnoczi, 2016/02/03