Here are the outstanding tasks for a tracing patchset that can be proposed for merge:
Contents |
The first patchset that brings basic tracing features to QEMU.
Owner: Stefan
User documentation that explains how to collect traces and add new trace events. I have committed documentation to the tracing branch here:
Owner: Stefan<Sent a patch>
It would be good to have trace-events looged with timestamps, for debugging purposes.
Owner: Prerna
LINK i386-linux-user/qemu-i386 ../simpletrace.o: In function `do_info_all_trace_events': /home/stefanha/qemu/simpletrace.c:88: undefined reference to `monitor_printf' ../simpletrace.o: In function `do_info_trace': /home/stefanha/qemu/simpletrace.c:77: undefined reference to `monitor_printf'
Owner: Prerna
Traces are written out to hardcoded /tmp/trace.log. This must be configurable. Tracing at startup time should still be possible so configuration needs to happen early. It should also be possible to flush the trace buffer to the file.
In addition to a config option, we have implemented an optional '-trace' switch that allows users to override defaults at runtime. This can also be changed for a running QEMU instance via the monitor command 'trace-file set FILENAME'
Owner: Stefan
We should leave room for extension. I suggest partitioning the Event ID namespace into normal events and special events. The __trace_begin special event is defined to contain the file format version and/or trace record size in bytes as the first trace record in the file.
This way, post-processing tools can check the format of the binary trace file. http://wiki.qemu.org/Features/Tracing/Roadmap
Owner: Prerna
Switching trace backends using ./configure does not trigger a rebuild as expected. The makefile needs to be able to detect that the trace backend setting has changed and trace.c/trace.h need to be rebuilt, along with anything that depends on the files.
Owner: ?
More tracepoints need to be added for instrumenting other qemu components such as virtio drivers, etc.
QEMU must come with a useful set of trace events that allows people to get up and running quickly. Focus areas include:
Owner: Stefan
Trace buffers are written out to file synchronously. The vcpu thread should not be blocked so an async write-out mechanism is needed.
Owner: ?
The clock_gettime() call used by simpletrace.c is not available under Windows. The simple trace backend should work on all platforms. It would be nice to call QEMU's timer code but there are two issues:
Owner: Prerna
Currently the trace commands are available from the monitor but a QMP interface is needed.
Owner: Prerna, Stefan
Are we happy to support the trace commands in the longer term (i.e. did we get the commands and syntax right)? Is input being validated properly?
Owner: Prerna
In its present format, the tracing infrastructure causes at least two function calls even for trace events that are disabled. Ideally, there should be minimal performance overhead for disabled trace events. Investigating scope for further optimization.
Owner: Prerna
Enabling trace events individually is tedious. A mechanism for selecting trace events in bulk would be useful.