commit 2998fdb56daa7a6d4437c1d42d05c3592d31726e
author Daniel Colascione <dancol@google.com> Tue Sep 03 18:17:19 2019 -0700
committer Daniel Colascione <dancol@google.com> Wed Sep 04 15:50:29 2019 -0700
tree 0a7400d96cf1e305231ab83bdc6e3b03e9f91db8
parent 1a4c48f252df2f17548bffaf7a2bd21ec672cfa9

Remove static constructors from libmeminfo

This CL changes libmeminfo's API to be less picky about the types of
its inputs and outputs and exposes the list of default memory types as
a constexpr std::array that doesn't need to be constructed as
dlopen time.

Test: tests pass; device boots
Bug: 140456977
Change-Id: Ice45a5400cc77540fb13352bba1adb7737877bcb

libmeminfo/libmeminfo_benchmark.cpp

diff --git a/libmeminfo/libmeminfo_benchmark.cpp b/libmeminfo/libmeminfo_benchmark.cpp
index d88919b..7450c6c 100644
--- a/libmeminfo/libmeminfo_benchmark.cpp
+++ b/libmeminfo/libmeminfo_benchmark.cpp
@@ -218,8 +218,8 @@
     android::base::WriteStringToFd(meminfo, tf.fd);
 
     std::string file = std::string(tf.path);
-    std::vector<uint64_t> mem(MEMINFO_COUNT);
-    const std::vector<std::string> tags = {
+    std::vector<uint64_t> mem;
+    const std::vector<std::string_view> tags = {
             SysMemInfo::kMemTotal,      SysMemInfo::kMemFree,        SysMemInfo::kMemBuffers,
             SysMemInfo::kMemCached,     SysMemInfo::kMemShmem,       SysMemInfo::kMemSlab,
             SysMemInfo::kMemSReclaim,   SysMemInfo::kMemSUnreclaim,  SysMemInfo::kMemSwapTotal,
@@ -229,7 +229,8 @@
 
     SysMemInfo smi;
     for (auto _ : state) {
-        smi.ReadMemInfo(tags, &mem, file);
+        mem.resize(tags.size());
+        smi.ReadMemInfo(tags.size(), tags.data(), mem.data(), file.c_str());
     }
 }
 BENCHMARK(BM_ReadMemInfo_new);
@@ -276,7 +277,7 @@
     std::string zram_mmstat_dir = exec_dir + "/testdata1/";
     SysMemInfo smi;
     for (auto _ : state) {
-        uint64_t zram_total __attribute__((unused)) = smi.mem_zram_kb(zram_mmstat_dir);
+        uint64_t zram_total __attribute__((unused)) = smi.mem_zram_kb(zram_mmstat_dir.c_str());
     }
 }
 BENCHMARK(BM_ZramTotal_new);
@@ -390,14 +391,16 @@
     android::base::WriteStringToFd(meminfo, tf.fd);
 
     std::string file = std::string(tf.path);
-    std::vector<uint64_t> mem(MEMINFO_COUNT);
-    std::vector<std::string> tags(SysMemInfo::kDefaultSysMemInfoTags);
+    std::vector<uint64_t> mem;
+    std::vector<std::string_view> tags(SysMemInfo::kDefaultSysMemInfoTags.begin(),
+                                       SysMemInfo::kDefaultSysMemInfoTags.end());
     auto it = tags.begin();
     tags.insert(it + MEMINFO_ZRAM_TOTAL, "Zram:");
     SysMemInfo smi;
 
     for (auto _ : state) {
-        smi.ReadMemInfo(tags, &mem, file);
+        mem.resize(tags.size());
+        smi.ReadMemInfo(tags.size(), tags.data(), mem.data(), file.c_str());
         CHECK_EQ(mem[MEMINFO_KERNEL_STACK], 4880u);
     }
 }
@@ -455,7 +458,7 @@
     std::string vmallocinfo =
             ::android::base::StringPrintf("%s/testdata1/vmallocinfo", exec_dir.c_str());
     for (auto _ : state) {
-        CHECK_EQ(::android::meminfo::ReadVmallocInfo(vmallocinfo), 29884416);
+        CHECK_EQ(::android::meminfo::ReadVmallocInfo(vmallocinfo.c_str()), 29884416);
     }
 }
 BENCHMARK(BM_VmallocInfo_new);