aboutsummaryrefslogtreecommitdiffhomepage
path: root/infra/bots/recipes/perf.py
diff options
context:
space:
mode:
authorGravatar Kevin Lubick <kjlubick@google.com>2017-12-12 07:33:48 -0500
committerGravatar Skia Commit-Bot <skia-commit-bot@chromium.org>2017-12-12 12:36:09 +0000
commitf1585aabeb23cf770c62dd9a1147df885f918671 (patch)
tree3cbdf960820678bf309be56b4508d4a31b991220 /infra/bots/recipes/perf.py
parent376fd31ad485c3df35d934c56364ff0c34eacdfa (diff)
Scale extra CPUs to powersave during nanobench.
This scaling logic correctly accounts for some devices which have multiple CPUs. Previously, we were scaling the smaller of these CPUs, which likely had a negative impact on nanobench, given nanobench was single threaded and the CPUs weren't allowed to idle much (because we set the CPU). This CL sets those additional CPUs to powersave when we run nanobench and then correctly scales down the beefier CPU we want to run nanobench on. For DM, we just run it in ondemand mode, which will hopefully be "as fast as possible", but allow the CPU governor to scale down if overheating becomes a problem. Bug: skia:7378 notry=TRUE Change-Id: I45ca5d9fb32182233d1b2d094842c879f2b84da4 Reviewed-on: https://skia-review.googlesource.com/83240 Commit-Queue: Kevin Lubick <kjlubick@google.com> Reviewed-by: Mike Klein <mtklein@chromium.org>
Diffstat (limited to 'infra/bots/recipes/perf.py')
-rw-r--r--infra/bots/recipes/perf.py8
1 files changed, 4 insertions, 4 deletions
diff --git a/infra/bots/recipes/perf.py b/infra/bots/recipes/perf.py
index 6737387a0b..17c73557cb 100644
--- a/infra/bots/recipes/perf.py
+++ b/infra/bots/recipes/perf.py
@@ -461,7 +461,7 @@ def GenTests(api):
'svg', 'VERSION'),
api.path['start_dir'].join('tmp', 'uninteresting_hashes.txt')
) +
- api.step_data('Scale CPU to 0.600000', retcode=1)
+ api.step_data('Scale CPU 2 to 0.600000', retcode=1)
)
yield (
@@ -480,7 +480,7 @@ def GenTests(api):
'svg', 'VERSION'),
api.path['start_dir'].join('tmp', 'uninteresting_hashes.txt')
) +
- api.step_data('Scale CPU to 0.600000', retcode=1)+
- api.step_data('Scale CPU to 0.600000 (attempt 2)', retcode=1)+
- api.step_data('Scale CPU to 0.600000 (attempt 3)', retcode=1)
+ api.step_data('Scale CPU 2 to 0.600000', retcode=1)+
+ api.step_data('Scale CPU 2 to 0.600000 (attempt 2)', retcode=1)+
+ api.step_data('Scale CPU 2 to 0.600000 (attempt 3)', retcode=1)
)