> > From my reply to v1 of this patch series: > > This is fragile since this information will be outdated if the C source > code changes (e.g. renaming files or variables). > > Instead I would add the following comment: > > # This is the binary format that the QEMU "simple" trace backend > # emits. There is no specification documentation because the format is > # not guaranteed to be stable. Trace files must be parsed with the > # same trace-events-all file and the same simpletrace.py file that > # QEMU was built with. > > I hope that clarifies the scope of the binary format and someone wishing > to look into the format would then know to look at the "simple" trace > backend. > > Stefan
I must have missed that. I'll add this too.