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

Side by Side Diff: tools/perf/metrics/cpu.py

Issue 809393002: Added support for improvement_direction to relevant values, which is propogated to chartjson. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years 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 unified diff | Download patch
OLDNEW
1 # Copyright 2013 The Chromium Authors. All rights reserved. 1 # Copyright 2013 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 from metrics import Metric 5 from metrics import Metric
6 from telemetry.value import scalar 6 from telemetry.value import scalar
7 7
8 8
9 class CpuMetric(Metric): 9 class CpuMetric(Metric):
10 """Calulates CPU load over a span of time.""" 10 """Calulates CPU load over a span of time."""
(...skipping 19 matching lines...) Expand all
30 # pylint: disable=W0221 30 # pylint: disable=W0221
31 def AddResults(self, tab, results, trace_name='cpu_utilization'): 31 def AddResults(self, tab, results, trace_name='cpu_utilization'):
32 assert self._stop_cpu, 'Must call Stop() first' 32 assert self._stop_cpu, 'Must call Stop() first'
33 cpu_stats = _SubtractCpuStats(self._stop_cpu, self._start_cpu) 33 cpu_stats = _SubtractCpuStats(self._stop_cpu, self._start_cpu)
34 # Add a result for each process type. 34 # Add a result for each process type.
35 for process_type in cpu_stats: 35 for process_type in cpu_stats:
36 trace_name_for_process = '%s_%s' % (trace_name, process_type.lower()) 36 trace_name_for_process = '%s_%s' % (trace_name, process_type.lower())
37 cpu_percent = 100 * cpu_stats[process_type] 37 cpu_percent = 100 * cpu_stats[process_type]
38 results.AddValue(scalar.ScalarValue( 38 results.AddValue(scalar.ScalarValue(
39 results.current_page, 'cpu_utilization.%s' % trace_name_for_process, 39 results.current_page, 'cpu_utilization.%s' % trace_name_for_process,
40 '%', cpu_percent, important=False)) 40 '%', cpu_percent, important=False, higher_is_better=False))
41 41
42 42
43 def _SubtractCpuStats(cpu_stats, start_cpu_stats): 43 def _SubtractCpuStats(cpu_stats, start_cpu_stats):
44 """Computes average cpu usage over a time period for different process types. 44 """Computes average cpu usage over a time period for different process types.
45 45
46 Each of the two cpu_stats arguments is a dict with the following format: 46 Each of the two cpu_stats arguments is a dict with the following format:
47 {'Browser': {'CpuProcessTime': ..., 'TotalTime': ...}, 47 {'Browser': {'CpuProcessTime': ..., 'TotalTime': ...},
48 'Renderer': {'CpuProcessTime': ..., 'TotalTime': ...} 48 'Renderer': {'CpuProcessTime': ..., 'TotalTime': ...}
49 'Gpu': {'CpuProcessTime': ..., 'TotalTime': ...}} 49 'Gpu': {'CpuProcessTime': ..., 'TotalTime': ...}}
50 50
(...skipping 19 matching lines...) Expand all
70 # Linux kernel starts with a value close to an overflow, so correction is 70 # Linux kernel starts with a value close to an overflow, so correction is
71 # necessary. 71 # necessary.
72 if total_time < 0: 72 if total_time < 0:
73 total_time += 2**32 73 total_time += 2**32
74 # Assert that the arguments were given in the correct order. 74 # Assert that the arguments were given in the correct order.
75 assert total_time > 0 and total_time < 2**31, ( 75 assert total_time > 0 and total_time < 2**31, (
76 'Expected total_time > 0, was: %d' % total_time) 76 'Expected total_time > 0, was: %d' % total_time)
77 cpu_usage[process_type] = float(cpu_process_time) / total_time 77 cpu_usage[process_type] = float(cpu_process_time) / total_time
78 return cpu_usage 78 return cpu_usage
79 79
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698