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

Unified Diff: tools/metrics/histograms/histograms.xml

Issue 851503005: Add histograms for temperatures and thermal states. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add thermald histograms Created 5 years, 11 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:
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index f879a89c876967194b6b5b4b2422d07978306e67..dedcf310c5365d9f8e34595b0c5e2a6b482f5a2f 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -24621,6 +24621,86 @@ Therefore, the affected-histogram name has to have at least one dot in it.
<summary>Temperature reading at sensor 9 (PECI) taken every 30s.</summary>
</histogram>
+<histogram name="Platform.Thermal.Temperature.Cpu.0" units="Celsius">
+ <owner>mka@chromium.org</owner>
+ <summary>
+ Temperature reading at sensor 0 of the CPU collected every few seconds (may
+ vary between devices).
+
+ This metric is specific to ChromeOS.
+ </summary>
+</histogram>
+
+<histogram name="Platform.Thermal.Temperature.Wifi0" units="Celsius">
Ilya Sherman 2015/01/22 02:53:47 nit: You use ".0" for CPU readings, but just "0" f
mka 2015/01/23 19:03:32 In the case of the Wifi interface the 0 is the ind
+ <owner>mka@chromium.org</owner>
+ <summary>
+ Temperature reading at wireless interface 0 collected every few seconds (may
+ vary between devices).
+
+ This metric is specific to ChromeOS.
+ </summary>
+</histogram>
+
+<histogram name="Platform.Thermal.Temperature.Wifi1" units="Celsius">
+ <owner>mka@chromium.org</owner>
+ <summary>
+ Temperature reading at wireless interface 1 collected every few seconds (may
+ vary between devices).
+
+ This metric is specific to ChromeOS.
+ </summary>
+</histogram>
+
+<histogram name="Platform.Thermal.Temperature.Wifi2" units="Celsius">
+ <owner>mka@chromium.org</owner>
+ <summary>
+ Temperature reading at wireless interface 2 collected every few seconds (may
Ilya Sherman 2015/01/22 02:53:47 What defines wireless interface "1" vs. "2", etc.?
mka 2015/01/23 19:03:32 It's the interface number/name assigned by the OS.
+ vary between devices).
+
+ This metric is specific to ChromeOS.
+ </summary>
+</histogram>
+
+<histogram name="Platform.Thermal.Zone.Cpu.States" units="Thermal state">
Ilya Sherman 2015/01/22 02:53:47 Would it make sense for the state to be an enum?
mka 2015/01/23 19:03:32 I considered initially to use labels like "warm",
+ <owner>mka@chromium.org</owner>
+ <summary>
+ State of the thermal zone of the CPU collected every second. State depends
+ on device, higher state corresponds to a higher temperature.
+
+ This metric is specific to ChromeOS.
+ </summary>
+</histogram>
+
+<histogram name="Platform.Thermal.Zone.Wifi0.States" units="Thermal state">
+ <owner>mka@chromium.org</owner>
+ <summary>
+ State of the thermal zone of wireless interface 0 collected every second.
+ State depends on device, higher state corresponds to a higher temperature.
+
+ This metric is specific to ChromeOS.
+ </summary>
+</histogram>
+
+<histogram name="Platform.Thermal.Zone.Wifi1.States" units="Thermal state">
+ <owner>mka@chromium.org</owner>
+ <summary>
+ State of the thermal zone of wireless interface 1 collected every second.
+ State depends on device, higher state corresponds to a higher temperature.
+
+ This metric is specific to ChromeOS.
+ </summary>
+</histogram>
+
+<histogram name="Platform.Thermal.Zone.Wifi2.States" units="Thermal state">
+ <owner>mka@chromium.org</owner>
+ <summary>
+ State of the thermal zone of wireless interface 2 collected every second.
+ State depends on device, higher state corresponds to a higher temperature.
+
+ This metric is specific to ChromeOS.
+ </summary>
+</histogram>
+
<histogram name="Platform.TPM.DictionaryAttackCounter">
<owner>dkrahn@chromium.org</owner>
<summary>
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698