Skip to content

Conversation

@maflcko
Copy link
Member

@maflcko maflcko commented Sep 23, 2019

The feature_block test causes the travis machine to OOM, when run with the thread sanitizer.

The stderr says:

==27237==ERROR: ThreadSanitizer failed to allocate 0xf6000 (1007616) bytes of LargeMmapAllocator (error code: 12)
...
FATAL: ThreadSanitizer CHECK failed: /build/llvm-toolchain-3.8-_PD09B/llvm-toolchain-3.8-3.8/projects/compiler-rt/lib/sanitizer_common/sanitizer_common.cc:183 "((0 && "unable to mmap")) != (0)" (0x0, 0x0)

ERROR: Failed to mmap

(from https://travis-ci.org/bitcoin/bitcoin/jobs/588194563#L10505)

Fix this by disabling feature_block on travis. Longer term, I'd like to move away from travis, but I'll leave this for a follow-up.

@DrahtBot DrahtBot added the Tests label Sep 23, 2019
@promag
Copy link
Contributor

promag commented Sep 23, 2019

Concept ACK, thanks, didn't know how to disable 😕

Copy link
Member

@fanquake fanquake left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK faa079d

Longer term, I'd like to move away from travis, but I'll leave this for a follow-up.

👀

@laanwj
Copy link
Member

laanwj commented Sep 24, 2019

ACK faa079d, anything to make the CI more stable …

laanwj added a commit that referenced this pull request Sep 24, 2019
faa079d travis: Disable feature_block in tsan run (MarcoFalke)

Pull request description:

  The `feature_block` test causes the travis machine to OOM, when run with the thread sanitizer.

  The stderr says:

  ```
  ==27237==ERROR: ThreadSanitizer failed to allocate 0xf6000 (1007616) bytes of LargeMmapAllocator (error code: 12)
  ...
  FATAL: ThreadSanitizer CHECK failed: /build/llvm-toolchain-3.8-_PD09B/llvm-toolchain-3.8-3.8/projects/compiler-rt/lib/sanitizer_common/sanitizer_common.cc:183 "((0 && "unable to mmap")) != (0)" (0x0, 0x0)

  ERROR: Failed to mmap
  ```
  (from https://travis-ci.org/bitcoin/bitcoin/jobs/588194563#L10505)

  Fix this by disabling `feature_block` on travis. Longer term, I'd like to move away from travis, but I'll leave this for a follow-up.

ACKs for top commit:
  fanquake:
    ACK faa079d

Tree-SHA512: c0dc2272853aac53f68eb9e110c8500c4a92211ba89d856660bacdf6e959d875477e422b3280b743d85fc8a65e083bf9153911f12039d026e2501f426540dac4
@laanwj laanwj merged commit faa079d into bitcoin:master Sep 24, 2019
@maflcko maflcko deleted the 1909-travisTsanOOM branch September 24, 2019 11:32
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Dec 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants