commit 4a6c942d1d3bae7ba8dfe67e1487b66bc8c311c7
author Luigi Semenzato <semenzato@chromium.org> Mon Jun 30 18:12:28 2014 -0700
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Wed Jul 02 22:11:43 2014 +0000
tree f77b83099430d1a03f16adc5bd6fe7e41e03846a
parent 6a26a0d4c69528ddd827357ecf2dc4bd95ee42d8

metrics_daemon: fix parsing of zram sysfs

The only real change is in ReadFileToUint64.  The other changes
appease the updated pre-commit checks.

BUG=chromium:390334
TEST=ran manually, checked syslog, checked chrome://histograms

Change-Id: I5ca10aab03697aa8fd64c5dd51ca64911e8d2a76
Reviewed-on: https://chromium-review.googlesource.com/206285
Tested-by: Luigi Semenzato <semenzato@chromium.org>
Reviewed-by: Daniel Erat <derat@chromium.org>
Commit-Queue: Luigi Semenzato <semenzato@chromium.org>

metrics/metrics_daemon.cc

diff --git a/metrics/metrics_daemon.cc b/metrics/metrics_daemon.cc
index 9b66877..e8f0c6c 100644
--- a/metrics/metrics_daemon.cc
+++ b/metrics/metrics_daemon.cc
@@ -2,9 +2,14 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "metrics_daemon.h"
+// For PRIu64 in inttypes.h, used by scanf.  TODO(semenzato): replace
+// with libchromeos methods.
+#define __STDC_FORMAT_MACROS
+
+#include "metrics/metrics_daemon.h"
 
 #include <fcntl.h>
+#include <inttypes.h>
 #include <math.h>
 #include <string.h>
 #include <time.h>
@@ -153,7 +158,7 @@
     PLOG(WARNING) << "clock_gettime(CLOCK_MONOTONIC) failed";
     return 0;
   } else {
-    return ts.tv_sec + ((double) ts.tv_nsec) / (1000 * 1000 * 1000);
+    return ts.tv_sec + static_cast<double>(ts.tv_nsec) / (1000 * 1000 * 1000);
   }
 }
 
@@ -456,8 +461,8 @@
   g_timeout_add_seconds(wait, StatsCallbackStatic, this);
 }
 
-bool MetricsDaemon::DiskStatsReadStats(long int* read_sectors,
-                                       long int* write_sectors) {
+bool MetricsDaemon::DiskStatsReadStats(uint64* read_sectors,
+                                       uint64* write_sectors) {
   int nchars;
   int nitems;
   bool success = false;
@@ -478,7 +483,7 @@
     LOG_IF(WARNING, nchars == sizeof(line))
         << "line too long in " << diskstats_path_;
     line[nchars] = '\0';
-    nitems = sscanf(line, "%*d %*d %ld %*d %*d %*d %ld",
+    nitems = sscanf(line, "%*d %*d %" PRIu64 "d %*d %*d %*d %" PRIu64 "d",
                     read_sectors, write_sectors);
     if (nitems == 2) {
       success = true;
@@ -618,7 +623,7 @@
 // Collects disk and vm stats alternating over a short and a long interval.
 
 void MetricsDaemon::StatsCallback() {
-  long int read_sectors_now, write_sectors_now;
+  uint64 read_sectors_now, write_sectors_now;
   struct VmstatRecord vmstats_now;
   double time_now = GetActiveTime();
   double delta_time = time_now - stats_initial_time_;
@@ -757,6 +762,8 @@
     PLOG(WARNING) << "cannot read " << path.MaybeAsASCII();
     return false;
   }
+  // Remove final newline.
+  base::TrimWhitespaceASCII(content, base::TRIM_TRAILING, &content);
   if (!base::StringToUint64(content, value)) {
     LOG(WARNING) << "invalid integer: " << content;
     return false;