Revert "[test] Remove longer timeout on bots"
This reverts commit e632f8f45b
.
Reason for revert: Seems to cause some timeouts to trigger failures: https://ci.chromium.org/p/v8/builders/ci/V8%20Linux64%20TSAN/26129
Original change's description:
> [test] Remove longer timeout on bots
>
> CQ_INCLUDE_TRYBOTS=luci.v8.try:v8_linux_gc_stress_dbg
> CQ_INCLUDE_TRYBOTS=luci.v8.try:v8_linux_gcc_rel
> CQ_INCLUDE_TRYBOTS=luci.v8.try:v8_linux64_msan_rel
> CQ_INCLUDE_TRYBOTS=luci.v8.try:v8_linux64_tsan_rel
> CQ_INCLUDE_TRYBOTS=luci.v8.try:v8_linux_arm64_dbg
> CQ_INCLUDE_TRYBOTS=luci.v8.try:v8_mac64_asan_rel
> CQ_INCLUDE_TRYBOTS=luci.v8.try:v8_win64_dbg
> CQ_INCLUDE_TRYBOTS=luci.v8.try:v8_android_arm64_n5x_rel_ng
> CQ_INCLUDE_TRYBOTS=luci.v8.try:v8_linux64_cfi_rel_ng
> CQ_INCLUDE_TRYBOTS=luci.v8.try:v8_linux64_ubsan_rel_ng
> CQ_INCLUDE_TRYBOTS=luci.v8.try:v8_linux_arm_lite_rel_ng
> CQ_INCLUDE_TRYBOTS=luci.v8.try:v8_linux_noi18n_rel_ng
> CQ_INCLUDE_TRYBOTS=luci.v8.try:v8_mac64_dbg_ng
>
> Bug: v8:9145
> Change-Id: I6efee8579d9d9e0aad0431f6b87c152141d4ec7f
> Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1581261
> Commit-Queue: Michael Achenbach <machenbach@chromium.org>
> Reviewed-by: Tamer Tas <tmrts@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#61107}
TBR=machenbach@chromium.org,tmrts@chromium.org
Change-Id: I5e23aebc41e6ab15baa57093f7f056cdabd58f21
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: v8:9145
Cq-Include-Trybots: luci.v8.try:v8_linux_gc_stress_dbg, luci.v8.try:v8_linux_gcc_rel, luci.v8.try:v8_linux64_msan_rel, luci.v8.try:v8_linux64_tsan_rel, luci.v8.try:v8_linux_arm64_dbg, luci.v8.try:v8_mac64_asan_rel, luci.v8.try:v8_win64_dbg, luci.v8.try:v8_android_arm64_n5x_rel_ng, luci.v8.try:v8_linux64_cfi_rel_ng, luci.v8.try:v8_linux64_ubsan_rel_ng, luci.v8.try:v8_linux_arm_lite_rel_ng, luci.v8.try:v8_linux_noi18n_rel_ng, luci.v8.try:v8_mac64_dbg_ng
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1588425
Reviewed-by: Yang Guo <yangguo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#61111}
This commit is contained in:
parent
1c4482ec3d
commit
2e6b9f576c
@ -309,7 +309,6 @@
|
||||
# Slow tests.
|
||||
'array-constructor': [PASS, SLOW],
|
||||
'json': [PASS, SLOW],
|
||||
'large-object-literal-slow-elements': [PASS, SLOW],
|
||||
|
||||
# BUG(v8:4779): Crashes flakily with stress mode on arm64.
|
||||
'array-splice': [PASS, SLOW, ['arch == arm64', NO_VARIANTS]],
|
||||
|
@ -190,7 +190,6 @@ class BuildConfig(object):
|
||||
self.no_snap = not build_config['v8_use_snapshot']
|
||||
self.predictable = build_config['v8_enable_verify_predictable']
|
||||
self.tsan = build_config['is_tsan']
|
||||
# TODO(machenbach): We only have ubsan not ubsan_vptr.
|
||||
self.ubsan_vptr = build_config['is_ubsan_vptr']
|
||||
self.embedded_builtins = build_config['v8_enable_embedded_builtins']
|
||||
self.verify_csa = build_config['v8_enable_verify_csa']
|
||||
@ -201,11 +200,6 @@ class BuildConfig(object):
|
||||
self.mips_arch_variant = build_config['mips_arch_variant']
|
||||
self.mips_use_msa = build_config['mips_use_msa']
|
||||
|
||||
@property
|
||||
def use_sanitizer(self):
|
||||
return (self.asan or self.cfi_vptr or self.msan or self.tsan or
|
||||
self.ubsan_vptr)
|
||||
|
||||
def __str__(self):
|
||||
detected_options = []
|
||||
|
||||
@ -699,6 +693,10 @@ class BaseTestRunner(object):
|
||||
}
|
||||
|
||||
def _create_test_config(self, options):
|
||||
# TODO(machenbach): Remove temporary hard-coded timeout when infra side is
|
||||
# removed.
|
||||
if options.buildbot:
|
||||
options.timeout = 200
|
||||
timeout = options.timeout * self._timeout_scalefactor(options)
|
||||
return TestConfig(
|
||||
command_prefix=options.command_prefix,
|
||||
@ -715,16 +713,15 @@ class BaseTestRunner(object):
|
||||
)
|
||||
|
||||
def _timeout_scalefactor(self, options):
|
||||
"""Increases timeout for slow build configurations."""
|
||||
factor = self.mode_options.timeout_scalefactor
|
||||
|
||||
# Simulators are slow, therefore allow a longer timeout.
|
||||
if self.build_config.arch in SLOW_ARCHS:
|
||||
factor *= 4
|
||||
if self.build_config.lite_mode:
|
||||
factor *= 2
|
||||
|
||||
# Predictable mode is slower.
|
||||
if self.build_config.predictable:
|
||||
factor *= 2
|
||||
if self.build_config.use_sanitizer:
|
||||
factor *= 1.5
|
||||
|
||||
return factor
|
||||
|
||||
|
@ -244,16 +244,13 @@ class TestCase(object):
|
||||
timeout = self._test_config.timeout
|
||||
if "--stress-opt" in params:
|
||||
timeout *= 4
|
||||
if "--jitless" in params:
|
||||
timeout *= 2
|
||||
if "--no-opt" in params:
|
||||
timeout *= 2
|
||||
if "--noenable-vfp3" in params:
|
||||
timeout *= 2
|
||||
if self._get_timeout_param() == TIMEOUT_LONG:
|
||||
timeout *= 10
|
||||
if self.is_slow:
|
||||
timeout *= 4
|
||||
|
||||
# TODO(majeski): make it slow outcome dependent.
|
||||
timeout *= 2
|
||||
return timeout
|
||||
|
||||
def get_shell(self):
|
||||
|
Loading…
Reference in New Issue
Block a user