@@ -7727,6 +7727,20 @@ fi
if have_backend "log"; then
echo "CONFIG_TRACE_LOG=y" >> $config_host_mak
fi
+if have_backend "recorder"; then
+ recorder_minver="1.0.10"
+ if $pkg_config --atleast-version=$recorder_minver recorder ; then
+ recorder_cflags="$($pkg_config --cflags recorder)"
+ recorder_libs="$($pkg_config --libs recorder)"
+ LIBS="$recorder_libs $LIBS"
+ libs_qga="$recorder_libs $libs_qga"
+ QEMU_CFLAGS="$QEMU_CFLAGS $recorder_cflags"
+ zstd="yes"
+ echo "CONFIG_TRACE_RECORDER=y" >> $config_host_mak
+ else
+ feature_not_found "recorder" "Install recorder-devel package"
+ fi
+fi
if have_backend "ust"; then
echo "CONFIG_TRACE_UST=y" >> $config_host_mak
fi
@@ -302,6 +302,28 @@ SRST
Open, close, or flush the trace file. If no argument is given, the
status of the trace file is displayed.
ERST
+#endif
+
+#if defined(CONFIG_TRACE_RECORDER)
+ {
+ .name = "recorder",
+ .args_type = "op:s?,arg:s?",
+ .params = "trace|dump [arg]",
+ .help = "trace selected recorders or print recorder dump",
+ .cmd = hmp_recorder,
+ },
+
+SRST
+``trace`` *cmds*
+ Activate or deactivate tracing for individual recorder traces.
+ See recorder_trace_set(3) for the syntax of *cmds*
+ For example, to activate trace ``foo`` and disable alll traces
+ ending in ``_warning``, use ``foo:.*_warning=0``.
+ Using ``help`` will list available traces and their current setting.
+``dump`` [*name*]
+ Dump the recorder. If *name* is given, only the specific named recorder
+ will be dumped.
+ERST
#endif
{
@@ -1828,4 +1850,3 @@ ERST
.sub_table = hmp_info_cmds,
.flags = "p",
},
-
@@ -61,6 +61,9 @@
#ifdef CONFIG_TRACE_SIMPLE
#include "trace/simple.h"
#endif
+#ifdef CONFIG_TRACE_RECORDER
+#include "trace/recorder.h"
+#endif
#include "exec/memory.h"
#include "exec/exec-all.h"
#include "qemu/option.h"
@@ -227,6 +230,30 @@ static void hmp_trace_file(Monitor *mon, const QDict *qdict)
}
#endif
+#ifdef CONFIG_TRACE_RECORDER
+static void hmp_recorder(Monitor *mon, const QDict *qdict)
+{
+ const char *op = qdict_get_try_str(qdict, "op");
+ const char *arg = qdict_get_try_str(qdict, "arg");
+
+ if (!op) {
+ monitor_printf(mon, "missing recorder command\"%s\"\n", op);
+ help_cmd(mon, "recorder");
+ } else if (!strcmp(op, "trace")) {
+ recorder_trace_set(arg);
+ } else if (!strcmp(op, "dump")) {
+ if (!arg || !*arg) {
+ recorder_dump();
+ } else {
+ recorder_dump_for(arg);
+ }
+ } else {
+ monitor_printf(mon, "unexpected recorder command \"%s\"\n", op);
+ help_cmd(mon, "recorder");
+ }
+}
+#endif
+
static void hmp_info_help(Monitor *mon, const QDict *qdict)
{
help_cmd(mon, "info");
new file mode 100644
@@ -0,0 +1,52 @@
+# -*- coding: utf-8 -*-
+
+"""
+Trace back-end for recorder library
+"""
+
+__author__ = "Christophe de Dinechin <christophe@dinechin.org>"
+__copyright__ = "Copyright 2020, Christophe de Dinechin and Red Hat"
+__license__ = "GPL version 2 or (at your option) any later version"
+
+__maintainer__ = "Christophe de Dinechin"
+__email__ = "christophe@dinechin.org"
+
+
+from tracetool import out
+
+PUBLIC = True
+
+def generate_h_begin(events, group):
+ out('#include <recorder/recorder.h>', '')
+
+ for event in events:
+ out('RECORDER_DECLARE(%(name)s);', name=event.name)
+
+
+def generate_h(event, group):
+ argnames = ", ".join(event.args.names())
+ if len(event.args) > 0:
+ argnames = ", " + argnames
+
+ out(' record(%(event)s, %(fmt)s %(argnames)s);',
+ event=event.name,
+ fmt=event.fmt.rstrip("\n"),
+ argnames=argnames)
+
+
+def generate_h_backend_dstate(event, group):
+ out(' RECORDER_TWEAK(%(event_id)s) || \\', event_id=event.name)
+
+def generate_c_begin(events, group):
+ out('#include "qemu/osdep.h"',
+ '#include "trace/control.h"',
+ '#include "trace/simple.h"',
+ '#include <recorder/recorder.h>',
+ '')
+
+ for event in events:
+ out('RECORDER_DEFINE(%(name)s, 8,',
+ ' "Tracetool recorder for %(api)s(%(args)s)");',
+ name=event.name,
+ api=event.api(),
+ args=event.args)
@@ -54,6 +54,7 @@ $(obj)/generated-tcg-tracers.h-timestamp: $(SRC_PATH)/trace-events $(BUILD_DIR)/
util-obj-$(CONFIG_TRACE_SIMPLE) += simple.o
util-obj-$(CONFIG_TRACE_FTRACE) += ftrace.o
+util-obj-$(CONFIG_TRACE_RECORDER) += recorder.o
util-obj-y += control.o
obj-y += control-target.o
util-obj-y += qmp.o
@@ -23,6 +23,9 @@
#ifdef CONFIG_TRACE_SYSLOG
#include <syslog.h>
#endif
+#ifdef CONFIG_TRACE_RECORDER
+#include "trace/recorder.h"
+#endif
#include "qapi/error.h"
#include "qemu/error-report.h"
#include "qemu/config-file.h"
@@ -282,6 +285,10 @@ bool trace_init_backends(void)
openlog(NULL, LOG_PID, LOG_DAEMON);
#endif
+#ifdef CONFIG_TRACE_RECORDER
+ recorder_trace_init();
+#endif
+
return true;
}
new file mode 100644
@@ -0,0 +1,25 @@
+/*
+ * Recorder-based trace backend
+ *
+ * Copyright Red Hat 2020
+ *
+ * This work is licensed under the terms of the GNU GPL, version 2. See
+ * the COPYING file in the top-level directory.
+ *
+ */
+
+#include "qemu/osdep.h"
+#include "trace/recorder.h"
+
+RECORDER_CONSTRUCTOR
+void recorder_trace_init(void)
+{
+ const char *traces = getenv("RECORDER_TRACES");
+ recorder_trace_set(traces);
+
+ /* Allow a dump in case we receive some unhandled signal
+ For example, send USR2 to a hung process to get a dump */
+ if (traces) {
+ recorder_dump_on_common_signals(0, 0);
+ }
+}
new file mode 100644
@@ -0,0 +1,32 @@
+/*
+ * Recorder-based trace backend
+ *
+ * Copyright Red Hat 2020
+ *
+ * This work is licensed under the terms of the GNU GPL, version 2. See
+ * the COPYING file in the top-level directory.
+ *
+ */
+
+#ifndef TRACE_RECORDER_H
+#define TRACE_RECORDER_H
+
+#ifdef CONFIG_TRACE_RECORDER
+
+#include <recorder/recorder.h>
+
+extern void recorder_trace_init(void);
+
+#else
+
+/* Disable recorder macros */
+#define RECORDER(Name, Size, Description)
+#define RECORDER_DEFINE(Name, Size, Description)
+#define RECORDER_DECLARE(Name)
+#define RECORD(Name, ...)
+#define record(Name, ...)
+#define recorder_trace_init()
+
+#endif /* CONFIG_TRACE_RECORDER */
+
+#endif // TRACE_RECORDER_H
@@ -22,6 +22,10 @@
#ifdef CONFIG_MODULE_UPGRADES
#include "qemu-version.h"
#endif
+#ifdef CONFIG_TRACE_RECORDER
+#include "trace/recorder.h"
+#endif
+
typedef struct ModuleEntry
{
@@ -150,6 +154,10 @@ static int module_load_file(const char *fname)
g_module_close(g_module);
ret = -EINVAL;
} else {
+#ifdef CONFIG_TRACE_RECORDER
+ // New recorders may have been pulled in, activate them if necessary
+ recorder_trace_init();
+#endif
QTAILQ_FOREACH(e, &dso_init_list, node) {
e->init();
register_module_init(e->init, e->type);
The recorder library provides support for low-cost continuous recording of events, which can then be replayed. This makes it possible to collect data after the fact, for example to show the events that led to a crash or unexpected condition. In this series, minimal recorder support in qemu is implemented using the existing tracing interface. For each trace, a corresponding recorder is created with a generic name and a default size of 8 entries. In addition, it is possible to explicitly enable recorders that are not qemu traces, for example in order to use actually record events rather than trace them, or to use the real-time graphing capabilities. For that reason, a limited set of recorder-related macros are defined as no-ops even if the recorder trace backend is not configured. Recorder-specific features, notably the ability to perform a post-mortem dump and to group traces by topic, are not integrated in this series, as doing so would require modifying the trace infrastructure, which is a non-objective here. This may be the topic of later series if there is any interest for it. HMP COMMAND: The 'recorder' hmp command has been added, which supports two sub-commands: * recorder dump: Dump the current state of the recorder. You can give that command a recorder name, to only dump that recorder. * recorder trace: Set traces using the recorder_trace_set() syntax. You can use "recorder trace help" to list all available recorders. Signed-off-by: Christophe de Dinechin <dinechin@redhat.com> --- configure | 14 ++++++++ hmp-commands.hx | 23 +++++++++++- monitor/misc.c | 27 ++++++++++++++ scripts/tracetool/backend/recorder.py | 52 +++++++++++++++++++++++++++ trace/Makefile.objs | 1 + trace/control.c | 7 ++++ trace/recorder.c | 25 +++++++++++++ trace/recorder.h | 32 +++++++++++++++++ util/module.c | 8 +++++ 9 files changed, 188 insertions(+), 1 deletion(-) create mode 100644 scripts/tracetool/backend/recorder.py create mode 100644 trace/recorder.c create mode 100644 trace/recorder.h