Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tsan: blacklist mdb #1637

Merged
merged 1 commit into from Jan 26, 2019

Conversation

Projects
2 participants
@cryptocode
Copy link
Collaborator

commented Jan 25, 2019

Thread sanitizer is currently too noisy to be practical due to lmdb (which looks like false positives) - this PR blacklists the two offending lmdb files.

Except for a potential race on startup (#1636), tsan is now mute on my system.

@cryptocode cryptocode self-assigned this Jan 25, 2019

@cryptocode cryptocode requested a review from rkeene Jan 25, 2019

@cryptocode cryptocode added this to the V18.0 milestone Jan 25, 2019

@cryptocode cryptocode added this to During RC in V18 Jan 25, 2019

@clemahieu clemahieu merged commit 23d895a into nanocurrency:master Jan 26, 2019

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@cryptocode cryptocode deleted the cryptocode:tsan/blacklist-mdb branch Jan 26, 2019

@zhyatt zhyatt moved this from RC2 to CP 3/RC 1 (2018-02-01) in V18 Feb 18, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.