-
Notifications
You must be signed in to change notification settings - Fork 317
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
Poll SwapPss in /proc/smaps_rollup #777
Comments
I think this should be fine to add if we're already polling the file and it's just a different field we're not reading. |
@LalitMaganti I noticed there is a CI failure in my change: https://android-review.googlesource.com/c/platform/external/perfetto/+/3087006 and I couldn't repro this issue by building the UI. Any suggestion on resolving the CI failure? |
It was a preexisting failure, I rebased your CL and reran the CI. |
Bug: #777 (comment) Change-Id: Id2f72de10d700ef122b37c02f2c0fd459a6a3d89
Following up on a previous issue: #490. There is option to poll
/proc/smaps_rollup
in the process stats probe, but a few fields were left untracked. I'd like to include all available fields especiallySwapPss
whilescan_smaps_rollup
is set to true.Do you think it is a acceptable feature? And if so, if you don't have bandwidth with this can I make a patch for this?
The text was updated successfully, but these errors were encountered: