diff options
| author | Luis Lozano <llozano@chromium.org> | 2015-12-07 15:09:33 -0800 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2015-12-08 01:42:57 +0000 |
| commit | d0b46129b0a60c84258a81d3dcce935cff6ba9c3 (patch) | |
| tree | d0d1426baa63d10207efb1871fe6e6aeb42736e4 /crosperf/suite_runner.py | |
| parent | c152cdb3fa9aca8e1c9c9699913ca17d50f1e42b (diff) | |
| download | toolchain-utils-d0b46129b0a60c84258a81d3dcce935cff6ba9c3.tar.gz | |
Sort CPU frequencies before picking the highest one.
We need to sort the available frequencies before selecting
the highest one. The frequency list order changes from
board to board
BUG=chromium:564887
TEST=Verified by hand and on Jaq machine
Change-Id: Ief9ca216489fcf84b4c34688309b89be07d52c1c
Reviewed-on: https://chrome-internal-review.googlesource.com/240946
Commit-Ready: Luis Lozano <llozano@chromium.org>
Tested-by: Luis Lozano <llozano@chromium.org>
Reviewed-by: Yunlian Jiang <yunlian@google.com>
Reviewed-by: Luis Lozano <llozano@chromium.org>
Diffstat (limited to 'crosperf/suite_runner.py')
| -rw-r--r-- | crosperf/suite_runner.py | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/crosperf/suite_runner.py b/crosperf/suite_runner.py index 779784c5..12237d33 100644 --- a/crosperf/suite_runner.py +++ b/crosperf/suite_runner.py @@ -90,6 +90,10 @@ class SuiteRunner(object): self._logger.LogFatalIf(ret, "Could not get available frequencies " "from machine: %s" % machine_name) freqs = freqs_str.split() + # We need to make sure that the frequencies are sorted in decreasing + # order + freqs.sort(key=int, reverse=True) + ## When there is no scaling_available_frequencies file, ## we have only 1 choice. if len(freqs) == 1: |