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

Optional perfetto annotations #206

Merged
merged 7 commits into from
Nov 16, 2022

Conversation

jrmadsen
Copy link
Collaborator

  • new configuration option OMNITRACE_PERFETTO_ANNOTATIONS which when set to OFF will suppress the addition of debug annotations
    • the benefit of disabling annotations is the reduction in the size of the perfetto trace file

- C API function print with warning colors
- split region/trace start/stop functions into regions.cpp file
@jrmadsen jrmadsen added perfetto Issue affects/involves perfetto features/capabilities libomnitrace Involves omnitrace library configuration Changes/involves configuration options labels Nov 13, 2022
@jrmadsen jrmadsen merged commit 2f16e2e into ROCm:main Nov 16, 2022
@jrmadsen jrmadsen deleted the optional-perfetto-annotations branch November 16, 2022 15:48
jrmadsen added a commit to jrmadsen/omnitrace that referenced this pull request Feb 2, 2023
* Misc tweaks

- C API function print with warning colors
- split region/trace start/stop functions into regions.cpp file

* Config option for disabling perfetto annotations

* Missing checks in roctracer.cpp and sampling.cpp

* Verbose makefile in CI

* run-ci uses -VV

* Fix gcc-7 maybe-uninitialized warning

* Fix push/pop perfetto

- moving perfetto::EventContext was causing errors
jrmadsen added a commit to jrmadsen/omnitrace that referenced this pull request Feb 2, 2023
* Misc tweaks

- C API function print with warning colors
- split region/trace start/stop functions into regions.cpp file

* Config option for disabling perfetto annotations

* Missing checks in roctracer.cpp and sampling.cpp

* Verbose makefile in CI

* run-ci uses -VV

* Fix gcc-7 maybe-uninitialized warning

* Fix push/pop perfetto

- moving perfetto::EventContext was causing errors
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
configuration Changes/involves configuration options libomnitrace Involves omnitrace library perfetto Issue affects/involves perfetto features/capabilities
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant