diff options
| author | Zhizhou Yang <zhizhouy@google.com> | 2019-12-02 16:41:48 -0800 |
|---|---|---|
| committer | Zhizhou Yang <zhizhouy@google.com> | 2019-12-04 17:39:56 +0000 |
| commit | 3089224e44d5d687ced2e96c7d090dfb36d7cc26 (patch) | |
| tree | ee9738b4cd2e56a1639ae3f10fa7caca7323913c /crosperf/benchmark_run.py | |
| parent | 455d84697c1ed30dfa5064a2d024811c9c6892ac (diff) | |
| download | toolchain-utils-3089224e44d5d687ced2e96c7d090dfb36d7cc26.tar.gz | |
crosperf: move enable_aslr and turbostat option into dut_config
This patch moves enable_aslr and turbostat into dut_config dictionary,
since they are all device setup steps, and it make code cleaner and
easier to migrate to telemetry_Crosperf.
TEST=Passed unittests.
BUG=chromium:1020655
Change-Id: I29a649e76591e206efc19bb2b8a8df9aead16575
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1947903
Reviewed-by: Denis Nikitin <denik@chromium.org>
Commit-Queue: Zhizhou Yang <zhizhouy@google.com>
Tested-by: Zhizhou Yang <zhizhouy@google.com>
Auto-Submit: Zhizhou Yang <zhizhouy@google.com>
Diffstat (limited to 'crosperf/benchmark_run.py')
| -rw-r--r-- | crosperf/benchmark_run.py | 18 |
1 files changed, 4 insertions, 14 deletions
diff --git a/crosperf/benchmark_run.py b/crosperf/benchmark_run.py index a7c3b7d1..f17de1be 100644 --- a/crosperf/benchmark_run.py +++ b/crosperf/benchmark_run.py @@ -31,18 +31,9 @@ STATUS_PENDING = 'PENDING' class BenchmarkRun(threading.Thread): """The benchmarkrun class.""" - def __init__(self, - name, - benchmark, - label, - iteration, - cache_conditions, - machine_manager, - logger_to_use, - log_level, - share_cache, - dut_config, - enable_aslr=False): + def __init__(self, name, benchmark, label, iteration, cache_conditions, + machine_manager, logger_to_use, log_level, share_cache, + dut_config): threading.Thread.__init__(self) self.name = name self._logger = logger_to_use @@ -55,8 +46,7 @@ class BenchmarkRun(threading.Thread): self.retval = None self.run_completed = False self.machine_manager = machine_manager - self.suite_runner = SuiteRunner( - dut_config, self._logger, self.log_level, enable_aslr=enable_aslr) + self.suite_runner = SuiteRunner(dut_config, self._logger, self.log_level) self.machine = None self.cache_conditions = cache_conditions self.runs_complete = 0 |