diff options
| author | Tanish Desai <tanishdesai37@gmail.com> | 2025-09-29 17:49:29 +0200 |
|---|---|---|
| committer | Stefan Hajnoczi <stefanha@redhat.com> | 2025-10-01 11:22:07 -0400 |
| commit | 494492c5c5a145e04f3ebc9eb40fd34ffbcc2061 (patch) | |
| tree | f20e479c6e21468c95537d7c5415904e14482e0c /scripts/tracetool/backend/simple.py | |
| parent | 809caa0d40a853144c02456737ed1e47859b0308 (diff) | |
| download | focaccia-qemu-494492c5c5a145e04f3ebc9eb40fd34ffbcc2061.tar.gz focaccia-qemu-494492c5c5a145e04f3ebc9eb40fd34ffbcc2061.zip | |
tracetool/backend: remove redundant trace event checks
Use CHECK_TRACE_EVENT_GET_STATE in log, syslog, dtrace and simple backend, so that the "if (trace_event_get_state)" is created from common code and unified when multiple backends are active. When a single backend is active there is no code change (except for the log backend, as shown in tests/tracetool/log.h), but the code in the backends is simpler. Signed-off-by: Tanish Desai <tanishdesai37@gmail.com> Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Reviewed-by: Daniel P. Berrangé <berrange@redhat.com> Message-ID: <20250929154938.594389-8-pbonzini@redhat.com> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Diffstat (limited to 'scripts/tracetool/backend/simple.py')
| -rw-r--r-- | scripts/tracetool/backend/simple.py | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/scripts/tracetool/backend/simple.py b/scripts/tracetool/backend/simple.py index 97e40495ee..b5a6b7205a 100644 --- a/scripts/tracetool/backend/simple.py +++ b/scripts/tracetool/backend/simple.py @@ -16,6 +16,7 @@ from tracetool import out PUBLIC = True +CHECK_TRACE_EVENT_GET_STATE = True def is_string(arg): @@ -36,13 +37,8 @@ def generate_h_begin(events, group): def generate_h(event, group): - event_id = 'TRACE_' + event.name.upper() - cond = "trace_event_get_state(%s)" % event_id - out(' if (%(cond)s) {', - ' _simple_%(api)s(%(args)s);', - ' }', + out(' _simple_%(api)s(%(args)s);', api=event.api(), - cond=cond, args=", ".join(event.args.names())) |