commit 56c414e872ccafe2c40cd6969b08262ee96dd873
author Sandeep Patil <sspatil@google.com> Sat Jan 12 21:29:19 2019 -0800
committer Sandeep Patil <sspatil@google.com> Mon Jan 14 09:42:50 2019 -0800
tree 477b0a97cafd9b73a9f8398e2e63dd1724a6d881
parent 002f02ea496b1d61af6b063e76ccf374beb37427

meminfo: Remove unnecessary working set stats

The Vma and ProcMemInfo objects do not need separate stats
objects for storing working set. The Vma either has working set
or memory usage information and never both.

Bug: 111694435
Test: libmeminfo_test 1

Change-Id: I2df05f7e750bbba4325474633e705d6d68dd2ccb
Merged-In: I2df05f7e750bbba4325474633e705d6d68dd2ccb
Signed-off-by: Sandeep Patil <sspatil@google.com>

libmeminfo/tools/procrank.cpp

diff --git a/libmeminfo/tools/procrank.cpp b/libmeminfo/tools/procrank.cpp
index a751722..21a684c 100644
--- a/libmeminfo/tools/procrank.cpp
+++ b/libmeminfo/tools/procrank.cpp
@@ -465,7 +465,7 @@
         }
 
         // Skip processes with no memory mappings
-        uint64_t vss = show_wss ? proc.Wss().vss : proc.Usage().vss;
+        uint64_t vss = proc.Usage().vss;
         if (vss == 0) return true;
 
         // collect swap_offset counts from all processes in 1st pass