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.

Sign up for GitHub

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

Jump to bottom

Feature flag to force memory_stats to use the statm code-path #2

Closed
jleibs opened this issue Nov 23, 2022 · 0 comments · Fixed by #3
Closed

Feature flag to force memory_stats to use the statm code-path #2

jleibs opened this issue Nov 23, 2022 · 0 comments · Fixed by #3

Comments

@jleibs
Copy link

jleibs commented Nov 23, 2022

As noted in the readme /proc/self/smaps can be quite slow on linux. In some projects the speed-benefits of statm likely outweigh the accuracy benefits of smaps. Given the code exists anyways, it would be great if we could force it to always use statm in cases where we are sensitive to overhead.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

add always_use_statm feature Arc-blroth/memory-stats
1 participant
@jleibs