-
Notifications
You must be signed in to change notification settings - Fork 95
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Handle tr end async
trace events
#288
Conversation
@pikrzysztof, @wbthomason: this should fix #257 (comment), could you please give it a try? You can get a prebuilt binary from the GitHub action run: https://github.com/janestreet/magic-trace/actions/runs/7449992981 |
Thanks for a quick contribution! This still crashes for me but with a different error
|
Kernel commit https://lore.kernel.org/lkml/[email protected]/ set `PERF_IP_FLAG_ASYNC` on trace end events. As per https://github.com/torvalds/linux/blob/7ee022567bf9e2e0b3cd92461a2f4986ecc99673/tools/perf/builtin-script.c#L1546 This means that where `perf` would have previously outputted `tr end jmp`, it now outputs `tr end async`. Adjust our regex to allow `async`, and treat it as a branch. Signed-off-by: Tudor Brindus <[email protected]>
Thanks for testing. I've pushed some changes that might resolve this--unfortunately I don't have an appropriate version of https://github.com/janestreet/magic-trace/actions/runs/7453308572/job/20278398604 |
Seems to do the job - not quite sure if everything is there as needed but nothing seems to crash hard despite a warning.
I'll probably dig deeper into usefulness of this trace later and report any other issues I find. Thank you! |
Thanks for testing! Your trace has some overflow packets recorded, those map to data being lost due to the system under test running out of memory bandwidth. |
Kernel commit
https://lore.kernel.org/lkml/[email protected]/
set
PERF_IP_FLAG_ASYNC
on trace end events. As perhttps://github.com/torvalds/linux/blob/7ee022567bf9e2e0b3cd92461a2f4986ecc99673/tools/perf/builtin-script.c#L1546
I think this means that where
perf
would have previously outputtedtr end jmp
, it now outputstr end async
.Adjust our regex to allow
async
, and treat it as a branch.