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

Unified Diff: tools/perf/metrics/power.py

Issue 220963003: [Telemetry] Output average GPU frequency from power metric (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 9 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/perf/metrics/power.py
diff --git a/tools/perf/metrics/power.py b/tools/perf/metrics/power.py
index c115d02e4de4480557823f54a8386791ad6da817..3b8ed2730d6e90980b38db413339a6a478fd455f 100644
--- a/tools/perf/metrics/power.py
+++ b/tools/perf/metrics/power.py
@@ -85,6 +85,10 @@ class PowerMetric(Metric):
energy_consumption_mwh = self._results['energy_consumption_mwh']
results.Add('energy_consumption_mwh', 'mWh', energy_consumption_mwh)
+ gpu_freq_hz = (
+ self._results['component_utilization']['gpu']['average_frequency_mhz'])
+ results.Add('gpu_average_frequency_hz', 'hz', gpu_freq_hz)
tonyg 2014/04/01 16:19:43 Seems like we're missing the multiplication from m
+
# Add idle wakeup numbers for all processes.
for process_type in self._results['cpu_stats']:
trace_name_for_process = 'idle_wakeups_%s' % (process_type.lower())
« 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