diff options
| author | Luis Lozano <llozano@chromium.org> | 2015-03-20 18:19:58 -0700 |
|---|---|---|
| committer | ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> | 2015-03-25 01:33:56 +0000 |
| commit | 004bccbd890f184ea873ad6ab8260808e1e79988 (patch) | |
| tree | 581fcf732e9b2a4aa58ae8bddf0faea3d92bdffd /crosperf | |
| parent | f9bb4a966e59736fe6c9eea198faf3613b5110f8 (diff) | |
| download | toolchain-utils-004bccbd890f184ea873ad6ab8260808e1e79988.tar.gz | |
Use original autotest location for telemetry runs.
For telemetry benchmarks, there is not need for a board specific
version of autotest. There are no compiled tests. So it is easier to
use the "source" location of autotest. This avoid having to build
autotest for the specific board which can be time consuming.
This also fixes the problem where after doing a build, sometimes
the telemetry_Crosperf test cannot be found since there is no need
for the copy.
This does nothing regarding autotest compiled tests. These still need
to be under /build/<board>. If using downloaded image, we could also
download the auotest bits but we will probably need a new "mode" or
flag for crosperf to indicate we want to use the autotest bits that
come from the image and not the one in your chroot. So, more work is
required.
BUG=None
TEST=Tested by hand with multiple telemetry benchmarks.
Change-Id: Id669782a9612fad39326b4d820cbc8e78a8a3d0c
Reviewed-on: https://chrome-internal-review.googlesource.com/209980
Reviewed-by: Caroline Tice <cmtice@google.com>
Commit-Queue: Luis Lozano <llozano@chromium.org>
Tested-by: Luis Lozano <llozano@chromium.org>
Diffstat (limited to 'crosperf')
| -rw-r--r-- | crosperf/suite_runner.py | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/crosperf/suite_runner.py b/crosperf/suite_runner.py index b472c9ff..84dc7fd6 100644 --- a/crosperf/suite_runner.py +++ b/crosperf/suite_runner.py @@ -153,6 +153,10 @@ class SuiteRunner(object): self._logger.LogFatal("Cannot find chrome src dir to" " run telemetry: %s" % label.chrome_src) + # For telemetry runs, we can use the autotest copy from the source + # location. No need to have one under /build/<board>. + autotest_dir_arg = '--autotest_dir ~/trunk/src/third_party/autotest/files' + profiler_args = GetProfilerArgs (profiler_args) fast_arg = "" if not profiler_args: @@ -166,8 +170,10 @@ class SuiteRunner(object): if test_args[0] == '"' and test_args[-1] == '"': test_args = test_args[1:-1] args_string = "test_args='%s'" % test_args - cmd = ('{} {} --board={} --args="{} test={} ' + + cmd = ('{} {} {} --board={} --args="{} test={} ' '{}" {} telemetry_Crosperf'.format(TEST_THAT_PATH, + autotest_dir_arg, fast_arg, label.board, args_string, |