aboutsummaryrefslogtreecommitdiffhomepage
path: root/infra/bots/recipes/perf.py
Commit message (Collapse)AuthorAge
* Revert "Reland "Revert "Update GPU DM/nanobench configs for color testing"""Gravatar Brian Osman2018-06-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit d5161b9a62b6912105e7a49d06e5126bf4fa1f75. Reason for revert: This should be okay again. Original change's description: > Reland "Revert "Update GPU DM/nanobench configs for color testing"" > > This reverts commit c0fb3c8b9fea2f60062819e92a1fa7f0a1966fba. > > Reason for revert: Had to revert dependent CL. > Original change's description: > > Revert "Revert "Update GPU DM/nanobench configs for color testing"" > > > > This reverts commit 2a284de7a51c198096a6f34ec1cbfe9118db554c. > > > > Re-landing without esrgb configs. Going to enable those separately, > > as they require some blacklisting. > > > > Bug: skia: > > Change-Id: I2315849d4f5c45c2e5512f84f9f867d02ff6fe30 > > Reviewed-on: https://skia-review.googlesource.com/136589 > > Reviewed-by: Brian Osman <brianosman@google.com> > > Commit-Queue: Brian Osman <brianosman@google.com> > > TBR=mtklein@google.com,bsalomon@google.com,brianosman@google.com > > Change-Id: I942d4235c17100fd933bf9cbfe923e3ae80657b1 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: skia: > Reviewed-on: https://skia-review.googlesource.com/137721 > Reviewed-by: Brian Osman <brianosman@google.com> > Commit-Queue: Brian Osman <brianosman@google.com> TBR=mtklein@google.com,bsalomon@google.com,brianosman@google.com # Not skipping CQ checks because original CL landed > 1 day ago. Bug: skia: Change-Id: I280dac49eb74d6c1cbf7c2f52a3f6a68fa2510ae Reviewed-on: https://skia-review.googlesource.com/138301 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
* rename Chromecast bots to ClangGravatar Mike Klein2018-06-27
| | | | | | | | | We updated them to use Clang a while ago but forgot to rename the bots. Change-Id: I01c353b48d5aff7a30d63d382bdf6bdfb3e0b196 Reviewed-on: https://skia-review.googlesource.com/137887 Auto-Submit: Mike Klein <mtklein@chromium.org> Reviewed-by: Kevin Lubick <kjlubick@google.com>
* Reland "Revert "Update GPU DM/nanobench configs for color testing""Gravatar Brian Osman2018-06-26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit c0fb3c8b9fea2f60062819e92a1fa7f0a1966fba. Reason for revert: Had to revert dependent CL. Original change's description: > Revert "Revert "Update GPU DM/nanobench configs for color testing"" > > This reverts commit 2a284de7a51c198096a6f34ec1cbfe9118db554c. > > Re-landing without esrgb configs. Going to enable those separately, > as they require some blacklisting. > > Bug: skia: > Change-Id: I2315849d4f5c45c2e5512f84f9f867d02ff6fe30 > Reviewed-on: https://skia-review.googlesource.com/136589 > Reviewed-by: Brian Osman <brianosman@google.com> > Commit-Queue: Brian Osman <brianosman@google.com> TBR=mtklein@google.com,bsalomon@google.com,brianosman@google.com Change-Id: I942d4235c17100fd933bf9cbfe923e3ae80657b1 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: skia: Reviewed-on: https://skia-review.googlesource.com/137721 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
* Revert "Revert "Update GPU DM/nanobench configs for color testing""Gravatar Brian Osman2018-06-26
| | | | | | | | | | | | | This reverts commit 2a284de7a51c198096a6f34ec1cbfe9118db554c. Re-landing without esrgb configs. Going to enable those separately, as they require some blacklisting. Bug: skia: Change-Id: I2315849d4f5c45c2e5512f84f9f867d02ff6fe30 Reviewed-on: https://skia-review.googlesource.com/136589 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
* Add Perf NoGPUThreads jobs.Gravatar Ben Wagner2018-06-21
| | | | | | | | | | | See discussion in https://skia-review.googlesource.com/c/skia/+/136220 No-Try: true Change-Id: I02b0b838379f0b371e8611c637349f55bfb506cf Reviewed-on: https://skia-review.googlesource.com/136598 Commit-Queue: Brian Osman <brianosman@google.com> Auto-Submit: Ben Wagner <benjaminwagner@google.com> Reviewed-by: Brian Osman <brianosman@google.com>
* [infra] Remove PixelC botGravatar Stephan Altmueller2018-06-21
| | | | | | | | | No-Tree-Checks: true Bug: skia: Change-Id: Ib0a5ad3df31f4fca9ab0085a5ba1d174419eee57 Reviewed-on: https://skia-review.googlesource.com/136220 Commit-Queue: Stephan Altmueller <stephana@google.com> Reviewed-by: Ben Wagner <benjaminwagner@google.com>
* Revert "Update GPU DM/nanobench configs for color testing"Gravatar Brian Osman2018-06-20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 5e11777a06d591c1e6ab0523e9c7f4739e164609. Reason for revert: Bots with no F16 support failing (as expected). Original change's description: > Update GPU DM/nanobench configs for color testing > > - Rename '*srgbnl' to '*srgb'. > - Add '*narrow' and '*enarrow' for testing a gamut narrower than sRGB. > - Stopped blacklisting glessrgb. It no longer uses sRGB pixel configs, > so should work fine on all GPUs. > - Also runs 'esrgb' (F16 sRGB) everywhere. This is likely to fail on > some devices that have poor (or no) F16 support. > > Change-Id: I941d0758adb3b0c895c7a8a85ff0ae4aa5bff6e7 > Reviewed-on: https://skia-review.googlesource.com/136171 > Commit-Queue: Brian Osman <brianosman@google.com> > Reviewed-by: Brian Salomon <bsalomon@google.com> > Reviewed-by: Mike Klein <mtklein@google.com> TBR=mtklein@google.com,bsalomon@google.com,brianosman@google.com Change-Id: I091269b93bc9e76c9d42f701d2018610b9c33a89 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://skia-review.googlesource.com/136300 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
* Update GPU DM/nanobench configs for color testingGravatar Brian Osman2018-06-20
| | | | | | | | | | | | | | | - Rename '*srgbnl' to '*srgb'. - Add '*narrow' and '*enarrow' for testing a gamut narrower than sRGB. - Stopped blacklisting glessrgb. It no longer uses sRGB pixel configs, so should work fine on all GPUs. - Also runs 'esrgb' (F16 sRGB) everywhere. This is likely to fail on some devices that have poor (or no) F16 support. Change-Id: I941d0758adb3b0c895c7a8a85ff0ae4aa5bff6e7 Reviewed-on: https://skia-review.googlesource.com/136171 Commit-Queue: Brian Osman <brianosman@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com> Reviewed-by: Mike Klein <mtklein@google.com>
* [recipes] Simplify test/perf dirsGravatar Eric Boren2018-06-20
| | | | | | | | Bug: skia:6473 Change-Id: Iee2f195ddf4bbcdabc1580f2a021d2e9a07ff0b2 Reviewed-on: https://skia-review.googlesource.com/133441 Commit-Queue: Eric Boren <borenet@google.com> Reviewed-by: Ben Wagner <benjaminwagner@google.com>
* update software DM/nanobench configs for color testingGravatar Mike Klein2018-06-19
| | | | | | | | | | | | | | - Rename 'srgbnl' to just 'srgb'. - Add 'narrow' and 'enarrow' for testing a gamut narrower than sRGB. Tested by running xfermodes2 in DM... all look different, what a mess. I also ran a few nanobenches and they seemed somewhat sane. Change-Id: Iacdc391dc0eef4153a76f5b4f78d72c57a4371ee Reviewed-on: https://skia-review.googlesource.com/135871 Commit-Queue: Mike Klein <mtklein@chromium.org> Auto-Submit: Mike Klein <mtklein@chromium.org> Reviewed-by: Brian Osman <brianosman@google.com>
* remove "srgb" config from DM,nanobenchGravatar Mike Klein2018-06-07
| | | | | | | | | | | | | | | | | | Now that "srgb" is broken I don't want to accidentally run it. "srgbnl" if of course identical, and not broken so much as simply not yet working. :) While here, simplify the configs we run in nanobench too, eliminating 565 and moving F16 to GCE-only (i.e. fast, abundant machines). Similarly, remove "adobe" VIA that doesn't use Adobe RGB correctly... Change-Id: Ic295dec97a2caadadbe8500655243db36dd2c43d Reviewed-on: https://skia-review.googlesource.com/132932 Commit-Queue: Mike Klein <mtklein@chromium.org> Commit-Queue: Brian Osman <brianosman@google.com> Auto-Submit: Mike Klein <mtklein@chromium.org> Reviewed-by: Brian Osman <brianosman@google.com>
* Update MoltenVK to 43130ab27a8b2e9eaf9e28b70b64f14b20c3b7bbGravatar Chris Dalton2018-05-29
| | | | | | | | | | Bring in the stencil buffer fix that just landed. Bug: skia:7961 Change-Id: I9eb12836678dcb2e25b58f88e0bf27b40d77c2b1 Reviewed-on: https://skia-review.googlesource.com/130454 Reviewed-by: Ben Wagner <benjaminwagner@google.com> Commit-Queue: Chris Dalton <csmartdalton@google.com>
* [recipes] Rename some modules and filesGravatar Eric Boren2018-05-24
| | | | | | | | | | | | | | | - Move doxygen and binary size out of core and into their own modules. - Rename core -> checkout since that's all it does, shorten method names. - Rename flavors: - Everything is GN, so remove GN/gn_ everywhere. - Merge gn_flavor into default. - Shorten file / module names. Bug: skia:6473 Change-Id: I8ac9ff9c9a267f366206b9991adfa5eb37126ca7 Reviewed-on: https://skia-review.googlesource.com/129176 Commit-Queue: Eric Boren <borenet@google.com> Reviewed-by: Ben Wagner <benjaminwagner@google.com>
* Add more blacklists for intel 405 vulkan perf bot.Gravatar Greg Daniel2018-05-22
| | | | | | | | | Bug: skia: Change-Id: I39cae17623e1b186607258cdd6929d797fcd6e15 Reviewed-on: https://skia-review.googlesource.com/129511 Reviewed-by: Greg Daniel <egdaniel@google.com> Reviewed-by: Mike Klein <mtklein@google.com> Commit-Queue: Greg Daniel <egdaniel@google.com>
* [recipes] Remove a bunch of unnecessary expectationsGravatar Eric Boren2018-05-18
| | | | | | | | Bug: skia:6473 Change-Id: Ic2eee8074476c449cc9bed4701105609e91ea12b Reviewed-on: https://skia-review.googlesource.com/128929 Commit-Queue: Eric Boren <borenet@google.com> Reviewed-by: Ben Wagner <benjaminwagner@google.com>
* [recipes] Move a lot of logic out of vars moduleGravatar Eric Boren2018-05-18
| | | | | | | | | | | | | In general, vars should only contain variables which are the same for all tasks. Variables specific to compilation belong in the build module (or compile recipe), and those specific to running tests belong in the flavor module, or the individual recipe which uses them. Bug: skia:6473 Change-Id: Ifd55a57118c5801e6f4934a6b5de9d1567415b9a Reviewed-on: https://skia-review.googlesource.com/128545 Commit-Queue: Eric Boren <borenet@google.com> Reviewed-by: Ben Wagner <benjaminwagner@google.com>
* Add MoltenVK_Vulkan botsGravatar Chris Dalton2018-05-17
| | | | | | | | | | | | | | Adds a prebuilt MoltenVK framework asset from: KhronosGroup/MoltenVK.git:0279543e0bb561f8d0166d4030481136bc914fec And creates bots that use it. Bug: skia: Change-Id: I081b2745c41a25de9ecd94a5363f4fb13b120215 Reviewed-on: https://skia-review.googlesource.com/128646 Commit-Queue: Chris Dalton <csmartdalton@google.com> Reviewed-by: Ben Wagner <benjaminwagner@google.com>
* [recipes] Remove core.setup()Gravatar Eric Boren2018-05-04
| | | | | | | | | | Each recipe now includes its own setup steps. Bug: skia:6473 Change-Id: I97eab2cd63f5004c07408e844cbbed735aac355f Reviewed-on: https://skia-review.googlesource.com/125660 Commit-Queue: Eric Boren <borenet@google.com> Reviewed-by: Ben Wagner <benjaminwagner@google.com>
* Update test expectations for Debian9Gravatar Kevin Lubick2018-04-26
| | | | | | | | | | | The removal of the blacklists is really a no-op because of https://skia-review.googlesource.com/c/skia/+/9339 Bug: skia:7840 Change-Id: I72e2e03bbbc813737655a3d5f3a1e874d3a41909 Reviewed-on: https://skia-review.googlesource.com/123925 Reviewed-by: Ben Wagner <benjaminwagner@google.com> Commit-Queue: Kevin Lubick <kjlubick@google.com>
* Add more angle_gl_* configs for NVIDIA.Gravatar Ben Wagner2018-04-19
| | | | | | | | | Bug: skia:7823 Change-Id: I9fbf0875696661ca87b37cf8949bd5f0b48b314d Reviewed-on: https://skia-review.googlesource.com/122600 Commit-Queue: Brian Salomon <bsalomon@google.com> Auto-Submit: Ben Wagner <benjaminwagner@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
* Disable vulkan tests to fix botsGravatar Stephan Altmueller2018-03-14
| | | | | | | | Bug: skia:7322 Change-Id: I8a26353134aee2c0972a3cb579040eb1c5c6dacd Reviewed-on: https://skia-review.googlesource.com/114281 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Stephan Altmueller <stephana@google.com>
* Omit path_text_clipped_uncached for RadeonHD7770.Gravatar Ben Wagner2018-03-01
| | | | | | | | | | | | I should have done this in https://skia-review.googlesource.com/c/skia/+/109460 Bug: skia:7677 Change-Id: I2597cbcdfefd9ebad6c1762266e49e72a8e910c3 No-Try: true Reviewed-on: https://skia-review.googlesource.com/111581 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* Change force_quarantine to use bot idGravatar Kevin Lubick2018-03-01
| | | | | | | | Bug: skia:7639 Change-Id: I4b8337bac996062c41fb9ae7460cf214b7524a46 Reviewed-on: https://skia-review.googlesource.com/111300 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Kevin Lubick <kjlubick@google.com>
* Upgrade the Radeon GPU drivers.Gravatar Ben Wagner2018-02-28
| | | | | | | | | | | This affects RadeonR9M470X and RadeonHD7770. Bug: skia:7677 No-Try: true Change-Id: Ic961e73b72ad9625a5cff9e08617281b1f4b2530 Reviewed-on: https://skia-review.googlesource.com/109460 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* Remove nvprdit* configsGravatar Robert Phillips2018-02-08
| | | | | | | | | | Having removed the NVPR text renderer, the nvprdit* configs are no longer interesting/useful. Change-Id: Ic4b9d6507d3e3595723a27636cb58b5e811fb3a3 Reviewed-on: https://skia-review.googlesource.com/105563 Reviewed-by: Brian Salomon <bsalomon@google.com> Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
* Blacklist additional perf skp on win Intel VulkanGravatar Greg Daniel2018-01-29
| | | | | | | | Bug: skia:6863 Change-Id: Iafdf7cd75d287f4d1f4a639e768b3d9418544959 Reviewed-on: https://skia-review.googlesource.com/101125 Reviewed-by: Ethan Nicholas <ethannicholas@google.com> Commit-Queue: Greg Daniel <egdaniel@google.com>
* Disable flaky perf benchmarkGravatar Kevin Lubick2018-01-29
| | | | | | | | Bug: skia:7551 Change-Id: Id1e4a345849f0e596d38278e5e9d88dfa476d678 Reviewed-on: https://skia-review.googlesource.com/101127 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Kevin Lubick <kjlubick@google.com>
* Update ChromecastsGravatar Kevin Lubick2018-01-24
| | | | | | | | | | I'm increasing capacity, might as well update them. Bug: skia: Change-Id: I4a535dbce1b3b586b5be82cb5b9bc0290ad7c6eb Reviewed-on: https://skia-review.googlesource.com/99480 Reviewed-by: Ben Wagner <benjaminwagner@google.com> Commit-Queue: Kevin Lubick <kjlubick@google.com>
* Blacklist changed skpGravatar Kevin Lubick2018-01-22
| | | | | | | | | Bug: skia:6687 Change-Id: I1562d7e9ded7f1be8a7ddc0c2341e54d5abbc0ae Reviewed-on: https://skia-review.googlesource.com/97901 Reviewed-by: Brian Salomon <bsalomon@google.com> Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Kevin Lubick <kjlubick@google.com>
* Add Win UBSAN jobs.Gravatar Ben Wagner2018-01-03
| | | | | | | | Bug: skia:2679 Change-Id: Ia462af01b9832da90206b9e9be2278cb48c6c502 Reviewed-on: https://skia-review.googlesource.com/86401 Reviewed-by: Mike Klein <mtklein@chromium.org> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* Remove Linux IntelIris540 jobs.Gravatar Ben Wagner2017-12-21
| | | | | | | | | | | | | | Replacing IntelIris540 with IntelIris640, added in https://skia-review.googlesource.com/c/skia/+/87202. I verified that Gold results are still identical (modulo flakes). See also https://skia-review.googlesource.com/c/skia/+/87721 and https://skia-review.googlesource.com/c/skia/+/78640. No-Try: true Change-Id: Ifc7e80e92d6203d922e00b59127b136632af7afc Reviewed-on: https://skia-review.googlesource.com/88427 Reviewed-by: Kevin Lubick <kjlubick@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* Re-add IntelIris640 Linux bots.Gravatar Ben Wagner2017-12-19
| | | | | | | | | | | | | These were removed in https://skia-review.googlesource.com/c/skia/+/78640 However, we've now decided on the 640 rather than the 540 due to inventory. No-Try: true Change-Id: Icf6db636287e151d3dd3ac74cfddf8f6bd3bea6a Reviewed-on: https://skia-review.googlesource.com/87202 Reviewed-by: Kevin Lubick <kjlubick@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* Revert "Revert "Disable extra CPUs during Nanobench""Gravatar Kevin Lubick2017-12-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 43307c09b97c5a5c5174f0c4ccb79bdaf3ebf557. Reason for revert: Fixed this time. Echo 1 > already online cpu returns exit code 1, which makes python over-react. Original change's description: > Revert "Disable extra CPUs during Nanobench" > > This reverts commit 32af335e7abbf562dad8f1bd21bb013610b8b021. > > Reason for revert: many unhappy android bots > > Original change's description: > > Disable extra CPUs during Nanobench > > > > The previous experiment revealed that nanobench can > > run on any of the online CPUs, so rather than put > > the ones we don't need/want into powersave mode, just > > disable them. > > > > Maybe in the future we can run CPU tests on the big > > or LITTLE cpus to get perf data on higher end or > > lower end cpus, but only if we get very stable > > results from this. > > > > Bug: skia:7378 > > Change-Id: I057513a691093e7f73c0f5790e17fab1a5ec0bc4 > > Reviewed-on: https://skia-review.googlesource.com/84820 > > Reviewed-by: Kevin Lubick <kjlubick@google.com> > > Commit-Queue: Kevin Lubick <kjlubick@google.com> > > TBR=borenet@google.com,mtklein@google.com,kjlubick@google.com > > Change-Id: I23c37a6bde631e95f0b4ae7277ec8fcf325a00e9 > Bug: skia:7378 > Reviewed-on: https://skia-review.googlesource.com/84921 > Reviewed-by: Mike Klein <mtklein@google.com> > Commit-Queue: Mike Klein <mtklein@google.com> No-Tree-Checks: true Change-Id: Ie7f0a3dc6ba55c124c796aba16a0f0497f285f3a Bug: skia:7378 Reviewed-on: https://skia-review.googlesource.com/84865 Commit-Queue: Kevin Lubick <kjlubick@google.com> Reviewed-by: Kevin Lubick <kjlubick@google.com> Reviewed-by: Mike Klein <mtklein@google.com>
* [recipes] Provide tokenized extra_config and use it most places.Gravatar Ben Wagner2017-12-13
| | | | | | | Change-Id: I7baae5f90b2e510b66443cda449071c7c6ec9ec7 Reviewed-on: https://skia-review.googlesource.com/83520 Commit-Queue: Ben Wagner <benjaminwagner@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Experiment with turning CPUs off during perfGravatar Kevin Lubick2017-12-13
| | | | | | | | | | | | | | | | | | | | | | | Perf was nice and flat after https://skia-review.googlesource.com/c/skia/+/83240 but there was a 4-5x slowdown on the benchmarks themselves, indicating that perhaps we were running nanobench on the LITTLE cores (now on powersave) instead of the big cores we thought (which were recently scaled down). This experiment will show us FOR CERTAIN that we are running on core 0 or 1 which is at .6 max frequency. We expect to see a speedup from the previous results. This is leading to turning of CPUs we don't need to make sure nanobench is running on the ones we expect. Bug: skia: NOTRY=true Change-Id: Ida65181e4d90e778e65e3f22d761288b9ade64f6 Reviewed-on: https://skia-review.googlesource.com/84201 Reviewed-by: Mike Klein <mtklein@chromium.org> Commit-Queue: Kevin Lubick <kjlubick@google.com>
* Scale extra CPUs to powersave during nanobench.Gravatar Kevin Lubick2017-12-12
| | | | | | | | | | | | | | | | | | | | | | | | 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>
* Remove instanced renderingGravatar Brian Salomon2017-12-08
| | | | | | | Change-Id: I4219b1d23a647b849ee41fe71b53e1c45edfc3f4 Reviewed-on: https://skia-review.googlesource.com/82241 Reviewed-by: Chris Dalton <csmartdalton@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
* Add Nexus7 CPU jobs.Gravatar Ben Wagner2017-12-07
| | | | | | | | | | Bug: skia:7381 Change-Id: I2dd5443c81cd895eb1d68b0cd90221a7e2e07d46 No-Try: true Reviewed-on: https://skia-review.googlesource.com/80843 Reviewed-by: Mike Klein <mtklein@chromium.org> Reviewed-by: Kevin Lubick <kjlubick@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* remove ThermalManagerGravatar Mike Klein2017-12-06
| | | | | | | | | | | | It throttles 1000x more than intended, and I suspect that some of the trip points it uses to decide when to throttle make no sense. We've already turned it off on the Nexus 5x. Change-Id: Idf556a83fe61ccc5f63c7bede3eecbe80087e28b Reviewed-on: https://skia-review.googlesource.com/81303 Reviewed-by: Kevin Lubick <kjlubick@google.com> Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Mike Klein <mtklein@chromium.org>
* Disable ThermalManager for Nexus5xGravatar Kevin Lubick2017-12-06
| | | | | | | | | | | | It's causing a lot of sleep for being too hot. ThermalManager assumes the trip points are static, when in fact, they are dynamic. Bug: skia:7378 NOTRY=true Change-Id: I907a42986831b7072a03a0423afd5a36bb2dfa74 Reviewed-on: https://skia-review.googlesource.com/80981 Reviewed-by: Kevin Lubick <kjlubick@google.com> Commit-Queue: Kevin Lubick <kjlubick@google.com>
* Fix recipe comments for ASAN.Gravatar Ben Wagner2017-12-04
| | | | | | | | No-Try: true Change-Id: Ifd97ba438fd30d2d9197857c86bfb010a97827a3 Reviewed-on: https://skia-review.googlesource.com/79702 Reviewed-by: Ben Wagner <benjaminwagner@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* Merge UBSAN_float_cast_overflow into ASAN.Gravatar Ben Wagner2017-12-04
| | | | | | | | | | | | The new check was a different config while we fixed the errors. Most errors are now fixed, and merging these will help with running both on the CQ. No-Try: true Change-Id: I5804ecea84a8dbbaacf6a4ea96e2af9505641d49 Reviewed-on: https://skia-review.googlesource.com/79323 Commit-Queue: Ben Wagner <benjaminwagner@google.com> Reviewed-by: Mike Klein <mtklein@chromium.org>
* Remove IntelIris640 bot.Gravatar Ben Wagner2017-11-30
| | | | | | | | | | IntelIris640 is 100% identical to IntelIris540 on Gold. No-Try: true Change-Id: I0e5342b182267a7d6ee510329b7c8ab7cb3a479e Reviewed-on: https://skia-review.googlesource.com/78640 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* Make cpu scaling more robustGravatar Kevin Lubick2017-11-30
| | | | | | | | | | | | | | | | | | Some bots, especially the Nexus 7s, seemed to occasionally fail when setting the CPU frequency. I was unable to repro this behavior, so this is a shotgun approach. We add a 5 second delay between setting and checking, checking frequency using scaling_cur_freq instead of scaling_setspeed, set the min_freq as well as max_freq, and retry up to 3 times if setting cpu frequency fails. NOTRY=true Bug: skia: Change-Id: Id4d85d8d509c9dba8e3a0e06b5992f5adadf36d2 Reviewed-on: https://skia-review.googlesource.com/78140 Commit-Queue: Kevin Lubick <kjlubick@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Revert "Temporarily add Ubuntu IntelHD4400 jobs."Gravatar Ben Wagner2017-11-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 373588426b9d02576152e820eb7f870d930f1998. Reason for revert: Have enough digests on Gold now. Original change's description: > Temporarily add Ubuntu IntelHD4400 jobs. > > I want to compare this with the IntelBayTrail and if the results are > similar, replace those bots. > > No-Try: true > Change-Id: Ib5476fe91dc446182cd1b37e93fe17962dcf961a > Reviewed-on: https://skia-review.googlesource.com/76900 > Reviewed-by: Kevin Lubick <kjlubick@google.com> > Commit-Queue: Ben Wagner <benjaminwagner@google.com> TBR=benjaminwagner@google.com,kjlubick@google.com Change-Id: Ica07d1ee635e59e3d3da51ee73591ffe08310e34 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://skia-review.googlesource.com/77860 Commit-Queue: Ben Wagner <benjaminwagner@google.com> Reviewed-by: Ben Wagner <benjaminwagner@google.com>
* Temporarily add Ubuntu IntelHD4400 jobs.Gravatar Ben Wagner2017-11-29
| | | | | | | | | | | I want to compare this with the IntelBayTrail and if the results are similar, replace those bots. No-Try: true Change-Id: Ib5476fe91dc446182cd1b37e93fe17962dcf961a Reviewed-on: https://skia-review.googlesource.com/76900 Reviewed-by: Kevin Lubick <kjlubick@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* Make CPU scaling code cleanerGravatar Kevin Lubick2017-11-21
| | | | | | | | | | | | | | | | | This also makes *sure* the CPU frequency we set the device to actually "takes". Along the way, I learned if scaling_max_freq is < the frequency we set, the scaling_max_freq will be used instead, which was happening to the PixelCs and AndroidOnes. As a result, this may make those two Test- configs faster. Bug: skia: Change-Id: I10c98d37e296a19e1cf67bfe7269bb59cdd912d5 Reviewed-on: https://skia-review.googlesource.com/74360 Commit-Queue: Kevin Lubick <kjlubick@google.com> Reviewed-by: Ben Wagner <benjaminwagner@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Put CPU frequency scaling in recipesGravatar Kevin Lubick2017-11-20
| | | | | | | | Bug: skia: Change-Id: I994f67c3043306d7fa612feb03f8fbe8d7bf4c91 Reviewed-on: https://skia-review.googlesource.com/73760 Commit-Queue: Kevin Lubick <kjlubick@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Add bug number to comment.Gravatar Ben Wagner2017-11-20
| | | | | | | | No-Try: true Change-Id: I9fe2b133264eb74b066364d5a490aae2de232bd3 Reviewed-on: https://skia-review.googlesource.com/73820 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* Add Linux IntelHD405 Vulkan jobs.Gravatar Ben Wagner2017-11-20
| | | | | | | | No-Try: true Change-Id: Id2631fc8df97b2a76775b01db212085d94c82906 Reviewed-on: https://skia-review.googlesource.com/72902 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
generated by cgit v1.2.3 (git 2.39.1) at 2024-06-26 03:41:45 +0000