Hello Adar Dembo,

I'd like you to do a code review. Please visit

    http://gerrit.cloudera.org:8080/10240

to review the following change.


Change subject: KUDU-2422. Fix TSAN hangs in Subprocess::Start
......................................................................

KUDU-2422. Fix TSAN hangs in Subprocess::Start

This pulls in an upstream patch from LLVM to fix an occasional hang when
starting processes in TSAN builds[1]

Tested with:

KUDU_ALLOW_SLOW_TESTS=0 ./build-support/dist_test.py loop -n 1000 \
  ./build/latest/bin/subprocess-test --gtest_filter=\*CurrentDir \
  --gtest_repeat=1000 --stress-cpu-threads=3

Without the patch, two of the tasks hung in Subprocess::Start and
failed. With the patch, all succeeded.

[1] https://github.com/google/sanitizers/issues/945

Change-Id: I9426b28c8a925b6d59ec4241aa12007a07f6ec70
---
M thirdparty/download-thirdparty.sh
A 
thirdparty/patches/llvm-tsan-disable-trace-switching-after-multithreaded-for.patch
2 files changed, 56 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/40/10240/1
--
To view, visit http://gerrit.cloudera.org:8080/10240
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I9426b28c8a925b6d59ec4241aa12007a07f6ec70
Gerrit-Change-Number: 10240
Gerrit-PatchSet: 1
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>

Reply via email to