New issue
Advanced search Search tips

Issue 809660 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Feb 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Fuchsia
Pri: 3
Type: Bug

Blocking:
issue 738275



Sign in to add a comment

AddressSpaceRandomizationTest.Random times out in base_unittests on Fuchsia ARM64

Project Member Reported by scottmg@chromium.org, Feb 6 2018

Issue description

https://build.chromium.org/p/chromium.fyi/builders/Fuchsia%20ARM64/builds/5325

[00161.977] 03712.03738> [ RUN      ] AddressSpaceRandomizationTest.Random
[00161.977] 03712.03738> [       OK ] AddressSpaceRandomizationTest.Random (48605 ms)
[00161.977] 03712.03738> [----------] 5 tests from AddressSpaceRandomizationTest (48633 ms total)
[00161.977] 03712.03738>
[00161.977] 03712.03738> [----------] 2 tests from PageAllocatorTest
[00161.977] 03712.03738> [ RUN      ] PageAllocatorTest.AllocFailure
[00161.977] 03712.03738> [       OK ] PageAllocatorTest.AllocFailure (1 ms)
[00161.977] 03712.03738> [ RUN      ] PageAllocatorTest.ReserveAddressSpace
[00161.977] 03712.03738> [       OK ] PageAllocatorTest.ReserveAddressSpace (1 ms)
[00161.977] 03712.03738> [----------] 2 tests from PageAllocatorTest (6 ms total)
[00161.977] 03712.03738>
[00161.977] 03712.03738> [----------] 2 tests from PartitionAllocTest
[00161.977] 03712.03738> [ RUN      ] PartitionAllocTest.Basic
[00161.977] 03712.03738> [       OK ] PartitionAllocTest.Basic (7 ms)
[00161.977] 03712.03738> [ RUN      ] PartitionAllocTest.MultiAlloc
[00161.978] 03712.03738> [       OK ] PartitionAllocTest.MultiAlloc (2 ms)
[00161.978] 03712.03738> [----------] 2 tests from PartitionAllocTest (13 ms total)
[00161.978] 03712.03738>
[00161.978] 03712.03738> [----------] Global test environment tear-down
[00161.978] 03712.03738> [==========] 10 tests from 4 test cases ran. (48701 ms total)
[00161.978] 03712.03738> [  PASSED  ] 10 tests.
[00161.978] 03712.03738> [2888/2892] AddressSpaceRandomizationTest.Random (TIMED OUT)
[00161.978] 03712.03738> [2889/2892] PageAllocatorTest.AllocFailure (1 ms)
[00161.978] 03712.03738> [2890/2892] PageAllocatorTest.ReserveAddressSpace (1 ms)
[00161.979] 03712.03738> [2891/2892] PartitionAllocTest.Basic (7 ms)
[00161.979] 03712.03738> [2892/2892] PartitionAllocTest.MultiAlloc (2 ms)
[00162.014] 03712.03738> 1 test timed out:
[00162.014] 03712.03738>     AddressSpaceRandomizationTest.Random (../../base/allocator/partition_allocator/address_space_randomization_unittest.cc:196)
 

Comment 1 by w...@chromium.org, Feb 6 2018

Looks like this was introduced in https://chromium-review.googlesource.com/c/chromium/src/+/894268

Given the name "Random", may be related to  issue 801353 , which is caused by global lock contention, in conjunction with IPI overhead under virtualization.

Comment 2 by w...@chromium.org, Feb 6 2018

Owner: w...@chromium.org
Status: Assigned (was: Untriaged)
Project Member

Comment 3 by bugdroid1@chromium.org, Feb 6 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a741314eb2e0a75f00f4162bfb0ca3ee7a1da1c5

commit a741314eb2e0a75f00f4162bfb0ca3ee7a1da1c5
Author: Scott Graham <scottmg@chromium.org>
Date: Tue Feb 06 21:01:53 2018

fuchsia: Disable two flakes, one in base_unittests one in net_unittests

NetworkQualityEstimatorTest.RecordAccuracy: https://build.chromium.org/p/chromium.fyi/builders/Fuchsia/builds/13666

AddressSpaceRandomizationTest.Random: https://build.chromium.org/p/chromium.fyi/builders/Fuchsia%20ARM64/builds/5325

Bug:  793176 ,  809660 
Change-Id: Ie4c01c4aa7889cc71ed7cec91b8c2f5d358f865c
Reviewed-on: https://chromium-review.googlesource.com/905203
Commit-Queue: Scott Graham <scottmg@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Reviewed-by: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534790}
[modify] https://crrev.com/a741314eb2e0a75f00f4162bfb0ca3ee7a1da1c5/testing/buildbot/filters/fuchsia.base_unittests.filter
[modify] https://crrev.com/a741314eb2e0a75f00f4162bfb0ca3ee7a1da1c5/testing/buildbot/filters/fuchsia.net_unittests.filter

Project Member

Comment 4 by bugdroid1@chromium.org, Feb 27 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/193277bcdc3b5c0527eec7f5e36d90588bdddc3d

commit 193277bcdc3b5c0527eec7f5e36d90588bdddc3d
Author: Wez <wez@chromium.org>
Date: Tue Feb 27 03:44:41 2018

Roll Fuchsia SDK to 739d1fd10565f97bacae8215903403b2677d8f48.

- Removes sys/resource.h, requiring some additional conditionals around
  includes of that header under OS_POSIX.
- Improves the system PRNG implementation to reduce the potential for
  lock contention.

We also clean up some test filter entries that either no longer refer
to an existing test, or are no longer required.

Bug: 707030,  801353 ,  809660 , 738275
Change-Id: Ic1f49625f20f2efc6c2509cf0f1fa8265d4e9f7f
Reviewed-on: https://chromium-review.googlesource.com/932822
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Eric Roman <eroman@chromium.org>
Reviewed-by: Scott Graham <scottmg@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539393}
[modify] https://crrev.com/193277bcdc3b5c0527eec7f5e36d90588bdddc3d/base/allocator/partition_allocator/partition_alloc_unittest.cc
[modify] https://crrev.com/193277bcdc3b5c0527eec7f5e36d90588bdddc3d/base/process/process_metrics_posix.cc
[modify] https://crrev.com/193277bcdc3b5c0527eec7f5e36d90588bdddc3d/base/process/process_util_unittest.cc
[modify] https://crrev.com/193277bcdc3b5c0527eec7f5e36d90588bdddc3d/base/sys_info_posix.cc
[modify] https://crrev.com/193277bcdc3b5c0527eec7f5e36d90588bdddc3d/base/threading/platform_thread_posix.cc
[modify] https://crrev.com/193277bcdc3b5c0527eec7f5e36d90588bdddc3d/build/fuchsia/update_sdk.py
[modify] https://crrev.com/193277bcdc3b5c0527eec7f5e36d90588bdddc3d/net/disk_cache/simple/simple_backend_impl.cc
[modify] https://crrev.com/193277bcdc3b5c0527eec7f5e36d90588bdddc3d/testing/buildbot/filters/fuchsia.base_unittests.filter

Comment 5 by w...@chromium.org, Feb 27 2018

Status: WontFix (was: Assigned)
Project Member

Comment 6 by bugdroid1@chromium.org, Mar 1 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/26f4cdcf5fea1eb642e3cabd35c1bd1aaf502303

commit 26f4cdcf5fea1eb642e3cabd35c1bd1aaf502303
Author: Wez <wez@chromium.org>
Date: Thu Mar 01 07:39:27 2018

Roll Fuchsia SDK to 9d4016533477903c796470e7ab46c2e1dad31761.

- Removes sys/resource.h, requiring some additional conditionals around
  includes of that header under OS_POSIX.
- Improves the system PRNG implementation to reduce the potential for
  lock contention.

TBR: scottmg
Bug: 707030,  801353 ,  809660 , 738275,  817241 ,  817586 
Change-Id: I4e01a0b23ad66c060ac7a6776a45329775117886
Reviewed-on: https://chromium-review.googlesource.com/942685
Reviewed-by: Wez <wez@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540093}
[modify] https://crrev.com/26f4cdcf5fea1eb642e3cabd35c1bd1aaf502303/build/fuchsia/update_sdk.py

Sign in to add a comment