Skip to content

Perfetto updates #211

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

Merged
merged 7 commits into from
Nov 17, 2022
Merged

Perfetto updates #211

merged 7 commits into from
Nov 17, 2022

Conversation

jrmadsen
Copy link
Collaborator

  • critical trace update
    • use ::perfetto::Track instead of threads to create rows
    • refactor call_chain::generate_perfetto
  • fix backtrace_metrics for perfetto
  • refactor sampling::post_process_perfetto
    • include HW counter delta in sample debug annotations
    • reduce the amount debug annotation data stored in the call-stack
      • if the data is common to the entire stack, it is only annotated in the first and the last call-stack entry

- use ::perfetto::Track instead of threads to create rows
- refactor call_chain::generate_perfetto
- get_papi_labels is now properly populated
- include HW counter delta in sample debug annotations
- reduce the amount debug annotation data stored in the call-stack
  - if the data is common to the entire stack, it is only annotated in the first and the last call-stack entry
@jrmadsen jrmadsen added enhancement New feature or request bug fix Fixes a bug perfetto Issue affects/involves perfetto features/capabilities libomnitrace Involves omnitrace library sampling Statistical sampling via interrupts PAPI Hardware counters via PAPI labels Nov 17, 2022
@jrmadsen jrmadsen merged commit 2ebfe3f into ROCm:main Nov 17, 2022
@jrmadsen jrmadsen deleted the perfetto-updates branch November 17, 2022 14:43
jrmadsen added a commit to jrmadsen/omnitrace that referenced this pull request Feb 2, 2023
* Retry support in build-docker-release.sh

* critical-trace perfetto update

- use ::perfetto::Track instead of threads to create rows
- refactor call_chain::generate_perfetto

* Fix backtrace_metrics for perfetto

- get_papi_labels is now properly populated

* Refactor sampling::post_process_perfetto

- include HW counter delta in sample debug annotations
- reduce the amount debug annotation data stored in the call-stack
  - if the data is common to the entire stack, it is only annotated in the first and the last call-stack entry

* exit_gotcha::exit_info

* Improve OMPT shutdown

- cause spurious test failures

* Update source/lib/omnitrace/library/ompt.cpp
jrmadsen added a commit to jrmadsen/omnitrace that referenced this pull request Feb 2, 2023
* Retry support in build-docker-release.sh

* critical-trace perfetto update

- use ::perfetto::Track instead of threads to create rows
- refactor call_chain::generate_perfetto

* Fix backtrace_metrics for perfetto

- get_papi_labels is now properly populated

* Refactor sampling::post_process_perfetto

- include HW counter delta in sample debug annotations
- reduce the amount debug annotation data stored in the call-stack
  - if the data is common to the entire stack, it is only annotated in the first and the last call-stack entry

* exit_gotcha::exit_info

* Improve OMPT shutdown

- cause spurious test failures

* Update source/lib/omnitrace/library/ompt.cpp
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug fix Fixes a bug enhancement New feature or request libomnitrace Involves omnitrace library PAPI Hardware counters via PAPI perfetto Issue affects/involves perfetto features/capabilities sampling Statistical sampling via interrupts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing CPU HW counters in perfetto traces
1 participant