Skip to content
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

Improve the density of runtime annotations collected. #242

Open
virajbshah opened this issue Oct 25, 2024 · 0 comments
Open

Improve the density of runtime annotations collected. #242

virajbshah opened this issue Oct 25, 2024 · 0 comments
Assignees

Comments

@virajbshah
Copy link
Collaborator

Gematria models show an improvement in performance on tricky code snippets when runtime annotations are enabled, but not to the degree we would have liked to see.

This is suspected to be a result of poor density of runtime annotations - i.e. very many (pseudo-)basic blocks with instructions that evidently cache miss don't have the corresponding cache miss annotation, because cache misses caused by those instruction are "missed" while sampling.

In attempt to get more samples, so far we've tried to:

  • change the sampling frequency (perf record -F),
  • change the number of events counted before a sample is recorded (perf record -c).

But it might take some more investigating/experimenting with perf to try to record a satisfactory amount of samples. To illustrate, perf record -c 1 ... should record as many samples every single sample, but might not be doing so because we were also counting CYCLES events, and doing so at that frequency might have adverse effects on how many samples we end up collecting.

@virajbshah virajbshah self-assigned this Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant