On Sat, Sep 26, 2026 at 10:39:39PM +0000, Kugan Vivekanandarajah wrote: > > > >> + case perf_record_type::COMM: > >> + note_process_lifetime_record ( > >> + record_header, payload_ptr, payload_size); > >> + break; > >> + default: > >> + break; > > > > It would probably be useful to handle FINISHED_ROUND at least, > > I think those can happen with aio writes etc. > > perf emits one after every sweep of the ring buffers too. > > With the timestamp based mapping validity windows above, we compare absolute > timestamps instead, so the round boundaries would not add anything.So I would > document why we skip it rather than handle it.
The time stamps of perf records might be unordered over multiple different CPUs flushing. So normally any tool needing the time stamp has to sort. Supporting FINISHED_ROUND allows the sort to be much more localized instead of a global sort. > Does that seem reasonable? The question is if out of sequence streams are handled correctly. If sorting is needed I would support it. -Andi
