diff options
author | Denis Nikitin <denik@google.com> | 2019-08-13 16:36:44 -0700 |
---|---|---|
committer | Denis Nikitin <denik@chromium.org> | 2019-08-19 03:30:29 +0000 |
commit | 1c95e747af51c784a7bdbf9a584f67e90bab32df (patch) | |
tree | b9651ffe6d6ace570662928873941ad239c5b289 /crosperf/results_organizer.py | |
parent | fcda3b3e33d46699bdfd08a81d897dd903c5d526 (diff) | |
download | toolchain-utils-1c95e747af51c784a7bdbf9a584f67e90bab32df.tar.gz |
crosperf: Add ARM CPU stats in crosperf report
Check for cpustats.log file in results_dir and extract
data with CPU frequncy and temperature.
Calculate avg/min/max values from the measurements and
add to result keyvals.
The parser attempts to read turbostat.log first (which comes from
Intel devices along with cpustats.log) and only if it doesn't exist
or empty switches to cpustats.log (usually ARM devices).
BUG=chromium:966514
TEST=tested locally on veyron, scarlet, kevin64 (arm), eve (intel)
Change-Id: I12991b39a0f1acb6197ca663a6876b0f013d2026
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1753164
Reviewed-by: George Burgess <gbiv@chromium.org>
Tested-by: Denis Nikitin <denik@chromium.org>
Diffstat (limited to 'crosperf/results_organizer.py')
-rw-r--r-- | crosperf/results_organizer.py | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/crosperf/results_organizer.py b/crosperf/results_organizer.py index 8f183cc9..15d495c7 100644 --- a/crosperf/results_organizer.py +++ b/crosperf/results_organizer.py @@ -187,9 +187,11 @@ def OrganizeResults(benchmark_runs, labels, benchmarks=None, json_report=False): if not show_all_results: summary_list = summary_file.get(benchmark.name) if summary_list: - summary_list += [ - 'retval', 'cpufreq_avg', 'cpufreq_min', 'cpufreq_max', 'cputemp' - ] + for key in benchmark_run.result.keyvals.keys(): + if any( + key.startswith(added_key) + for added_key in ['retval', 'cpufreq', 'cputemp']): + summary_list.append(key) else: # Did not find test_name in json file; show everything. show_all_results = True |