diff options
author | Denis Nikitin <denik@google.com> | 2019-09-20 13:04:31 -0700 |
---|---|---|
committer | Denis Nikitin <denik@chromium.org> | 2019-09-24 00:38:48 +0000 |
commit | 73a0d2ad68df4db8ba08bf15faece66d5a98aa44 (patch) | |
tree | 82a120dc374db8de0be3222f46943c5c13c9a72c /crosperf/settings_factory.py | |
parent | 05ee05dc9b9f8d87cb94be112f0b1dde939ed181 (diff) | |
download | toolchain-utils-73a0d2ad68df4db8ba08bf15faece66d5a98aa44.tar.gz |
crosperf: Add top_interval argument to experiment
Added the following optional argument in the global settings:
"top_interval" - Run top command in the background of a benchmark with
interval of sampling specified in seconds (float type).
With zero value don't run top (default value).
Top output is stored in <resultsdir>/top.log.
BUG=chromium:966514
TEST=unittests and HW tests on eve, kevin64, scarlet passed.
Change-Id: I87b187c85912140fc0c16e224580b60061c8bd94
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1817121
Tested-by: Denis Nikitin <denik@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
Diffstat (limited to 'crosperf/settings_factory.py')
-rw-r--r-- | crosperf/settings_factory.py | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/crosperf/settings_factory.py b/crosperf/settings_factory.py index 1fbc8e14..1f2693c6 100644 --- a/crosperf/settings_factory.py +++ b/crosperf/settings_factory.py @@ -347,6 +347,19 @@ class GlobalSettings(Settings): required=False, default=True)) self.AddField( + FloatField( + 'top_interval', + description='Run top command in the background of a benchmark with' + ' interval of sampling specified in seconds.\n' + 'Recommended values 1-5. Lower number provides more accurate' + ' data.\n' + 'With 0 - do not run top.\n' + 'NOTE: Running top with interval 1-5 sec has insignificant' + ' performance impact (performance degradation does not exceed 0.3%,' + ' measured on x86_64, ARM32, and ARM64).', + required=False, + default=0)) + self.AddField( IntegerField( 'cooldown_temp', required=False, |