Skip to content

Conversation

briancoutinho
Copy link
Contributor

Summary:
Adds thread level counter for hbt

Differential Revision: D44764690

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported labels Apr 11, 2023
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D44764690

briancoutinho added a commit to briancoutinho/dynolog-1 that referenced this pull request Apr 11, 2023
Summary:
Pull Request resolved: facebookincubator#128

Adds thread level counter for hbt
* Set cpu = -1 to make this thread based perf even
https://man7.org/linux/man-pages/man2/perf_event_open.2.html
* Create Thread counter class specializing `CpuEventGroupBase`
* Avoid verbose prints on PMU sysfs discovery reading?

Differential Revision: D44764690

fbshipit-source-id: 5b932c06f3e6cbed25db3868137c0da2d7f9406e
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D44764690

Summary:
Pull Request resolved: facebookincubator#128

Adds thread level counter for hbt
* Set cpu = -1 to make this thread based perf even
https://man7.org/linux/man-pages/man2/perf_event_open.2.html
* Create Thread counter class specializing `CpuEventGroupBase`
* Avoid verbose prints on PMU sysfs discovery reading?

Reviewed By: haowangludx

Differential Revision: D44764690

fbshipit-source-id: d5dbba59bfb52c8e7324ee1512e1db8545b22b2f
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D44764690

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 23c47e8.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants