Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(47)

Unified Diff: chromeos/system/cpu_temperature_reader.cc

Issue 2823583002: chromeos: Add CPU temperature reader (Closed)
Patch Set: one last fix Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chromeos/system/cpu_temperature_reader.h ('k') | chromeos/system/cpu_temperature_reader_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chromeos/system/cpu_temperature_reader.cc
diff --git a/chromeos/system/cpu_temperature_reader.cc b/chromeos/system/cpu_temperature_reader.cc
new file mode 100644
index 0000000000000000000000000000000000000000..8a3e5c09cbf3aa0010ac85239bff3d41faab543a
--- /dev/null
+++ b/chromeos/system/cpu_temperature_reader.cc
@@ -0,0 +1,117 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chromeos/system/cpu_temperature_reader.h"
+
+#include "base/files/file_enumerator.h"
+#include "base/files/file_path.h"
+#include "base/files/file_util.h"
+#include "base/location.h"
+#include "base/strings/string_number_conversions.h"
+#include "base/strings/string_util.h"
+#include "base/task_scheduler/post_task.h"
+#include "base/task_scheduler/task_traits.h"
+
+namespace chromeos {
+namespace system {
+
+using CPUTemperatureInfo = CPUTemperatureReader::CPUTemperatureInfo;
+
+namespace {
+
+// The location we read our CPU temperature and channel label from.
+constexpr char kDefaultHwmonDir[] = "/sys/class/hwmon/";
+constexpr char kDeviceDir[] = "device";
+constexpr char kHwmonDirectoryPattern[] = "hwmon*";
+constexpr char kCPUTempFilePattern[] = "temp*_input";
+
+// The contents of sysfs files might contain a newline at the end. Use this
+// function to read from a sysfs file and remove the newline.
+bool ReadFileContentsAndTrimWhitespace(const base::FilePath& path,
+ std::string* contents_out) {
+ if (!base::ReadFileToString(path, contents_out))
+ return false;
+ base::TrimWhitespaceASCII(*contents_out, base::TRIM_TRAILING, contents_out);
+ return true;
+}
+
+// Reads the temperature as degrees Celsius from the file at |path|, which is
+// expected to contain the temperature in millidegrees Celsius. Converts the
+// value into degrees and then stores it in |*temp_celsius_out|. returns true if
+// the value was successfully read from file, parsed as an int, and converted.
+bool ReadTemperatureFromPath(const base::FilePath& path,
+ double* temp_celsius_out) {
+ std::string temperature_string;
+ if (!ReadFileContentsAndTrimWhitespace(path, &temperature_string))
+ return false;
+ uint32_t temperature = 0;
+ if (!base::StringToUint(temperature_string, &temperature))
+ return false;
+ *temp_celsius_out = temperature / 1000.0;
+ return true;
+}
+
+// Gets the label describing this temperature. Use the file "temp*_label" if it
+// is present, or fall back on the file "name" or |label_path|.
+std::string GetLabelFromPath(const base::FilePath& label_path) {
+ std::string label;
+ if (base::PathExists(label_path) &&
+ ReadFileContentsAndTrimWhitespace(base::FilePath(label_path), &label) &&
+ !label.empty()) {
+ return label;
+ }
+
+ base::FilePath name_path = label_path.DirName().Append("name");
+ if (base::PathExists(name_path) &&
+ ReadFileContentsAndTrimWhitespace(name_path, &label) && !label.empty()) {
+ return label;
+ }
+ return label_path.value();
+}
+
+} // namespace
+
+CPUTemperatureReader::CPUTemperatureInfo::CPUTemperatureInfo() {}
+
+CPUTemperatureReader::CPUTemperatureInfo::~CPUTemperatureInfo() = default;
+
+CPUTemperatureReader::CPUTemperatureReader() : hwmon_dir_(kDefaultHwmonDir) {}
+
+CPUTemperatureReader::~CPUTemperatureReader() = default;
+
+std::vector<CPUTemperatureInfo> CPUTemperatureReader::GetCPUTemperatures() {
+ std::vector<CPUTemperatureInfo> result;
+
+ // Get directories /sys/class/hwmon/hwmon*.
+ base::FileEnumerator hwmon_enumerator(hwmon_dir_, false,
+ base::FileEnumerator::DIRECTORIES,
+ kHwmonDirectoryPattern);
+ for (base::FilePath hwmon_path = hwmon_enumerator.Next(); !hwmon_path.empty();
+ hwmon_path = hwmon_enumerator.Next()) {
+ // Get temp*_input files in hwmon*/ and hwmon*/device/.
+ if (base::PathExists(hwmon_path.Append(kDeviceDir))) {
+ hwmon_path = hwmon_path.Append(kDeviceDir);
+ }
+ base::FileEnumerator enumerator(
+ hwmon_path, false, base::FileEnumerator::FILES, kCPUTempFilePattern);
+ for (base::FilePath temperature_path = enumerator.Next();
+ !temperature_path.empty(); temperature_path = enumerator.Next()) {
+ CPUTemperatureInfo info;
+ if (!ReadTemperatureFromPath(temperature_path, &info.temp_celsius)) {
+ LOG(WARNING) << "Unable to read CPU temperature from "
+ << temperature_path.value();
+ continue;
+ }
+ // Get appropriate temp*_label file.
+ std::string label_path = temperature_path.value();
+ base::ReplaceSubstringsAfterOffset(&label_path, 0, "input", "label");
+ info.label = GetLabelFromPath(base::FilePath(label_path));
+ result.push_back(info);
+ }
+ }
+ return result;
+}
+
+} // namespace system
+} // namespace chromeos
« no previous file with comments | « chromeos/system/cpu_temperature_reader.h ('k') | chromeos/system/cpu_temperature_reader_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698