On Fri, 11 Jul 2025 13:26:28 -0700
Namhyung Kim [off-list ref] wrote:
quoted
- Removed use of timestamp. The deferred unwind has gone back to using
cookies, and perf doesn't use the cookie. This means the
struct perf_callchain_deferred_event is not modified.
What about adding the cookies in the records to handle lost data? Even
if it's not necessary to match callchains to samples, it still needs to
reject invalid callchains across the losts. Maybe it can just flush
pending samples when it sees LOST records and not try to match them but
having the cookies will handle it more accurately as some callchains may
be valid after the LOST.
Sure, I can add that back, with the added comments needed that Jens
suggested.
Thanks,
-- Steve