commit | 9351dcecd2a462c9ff908dc915994b746c0697ee | [log] [tgz] |
---|---|---|
author | Kevin Jeon <[email protected]> | Fri Apr 28 19:50:54 2023 +0000 |
committer | Automerger Merge Worker <[email protected]> | Fri Apr 28 19:50:54 2023 +0000 |
tree | 21fb760c14f3daab76e4685b0eedf73475af5778 | |
parent | c112052aeb5c3a90ef68153787a54f82c9477494 [diff] | |
parent | bf0319a41a547d0e66733d310a799e11aea04c3f [diff] |
Temporarily remove Smaps() call in ProcessRecord am: bf0319a41a Original change: https://googleplex-android-review.googlesource.com/c/platform/system/memory/libmeminfo/+/22889144 Change-Id: I95d0307fb4bbaab49e88fa5d6b6968783a872ae0 Signed-off-by: Automerger Merge Worker <[email protected]>
diff --git a/libsmapinfo/processrecord.cpp b/libsmapinfo/processrecord.cpp index e859310..615cfb8 100644 --- a/libsmapinfo/processrecord.cpp +++ b/libsmapinfo/processrecord.cpp
@@ -93,7 +93,13 @@ // We want to use Smaps() to populate procmem_'s maps before calling Wss() or Usage(), as // these will fall back on the slower ReadMaps(). - procmem_.Smaps("", true); + // + // This Smaps() call is temporarily disabled because it results in + // procmem_'s swap_offsets_ not being populated, causing procrank to not + // report PSwap/USwap/ZSwap. Wss() and Usage() both fall back to ReadMaps(), + // which will populate swap_offsets_. + // + // procmem_.Smaps("", true); usage_or_wss_ = get_wss ? procmem_.Wss() : procmem_.Usage(); swap_offsets_ = procmem_.SwapOffsets(); pid_ = pid;