diff options
| author | Denis Nikitin <denik@google.com> | 2019-09-26 09:37:51 -0700 |
|---|---|---|
| committer | Denis Nikitin <denik@chromium.org> | 2019-09-26 18:31:41 +0000 |
| commit | 52bf1077e4117f4497d0cff09443d1e843058e7f (patch) | |
| tree | 0c7279678751569e2271424a147dc2609a3a7fb4 | |
| parent | c60ec013fcb88a82089f198464230c456dbbd888 (diff) | |
| download | toolchain-utils-52bf1077e4117f4497d0cff09443d1e843058e7f.tar.gz | |
crosperf: Add cpu temp/freq in cwp report
Result organizer with CWP has a different control flow and didn't
include cpu frequency and temperature data.
Current change adds missing data into the Full table of the report
file.
BUG=None
TEST=tested on veyron with cwp experiment file.
Change-Id: I67447ab016cee8265c1937b236cc5c38330576e9
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1827918
Tested-by: Denis Nikitin <denik@chromium.org>
Reviewed-by: Zhizhou Yang <zhizhouy@google.com>
| -rw-r--r-- | crosperf/results_organizer.py | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/crosperf/results_organizer.py b/crosperf/results_organizer.py index 15d495c7..4879caeb 100644 --- a/crosperf/results_organizer.py +++ b/crosperf/results_organizer.py @@ -200,6 +200,11 @@ def OrganizeResults(benchmark_runs, labels, benchmarks=None, json_report=False): if 'samples' in benchmark_run.result.keyvals: cur_dict['samples'] = benchmark_run.result.keyvals['samples'] cur_dict['retval'] = benchmark_run.result.keyvals['retval'] + for key, value in benchmark_run.result.keyvals.items(): + if any( + key.startswith(cpustat_keyword) + for cpustat_keyword in ['cpufreq', 'cputemp']): + cur_dict[key] = value else: for test_key in benchmark_run.result.keyvals: if show_all_results or test_key in summary_list: |