Todd Lipcon has posted comments on this change. Change subject: locks: allow use of TSAN on many-core machines ......................................................................
Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6905/1//COMMIT_MSG Commit Message: PS1, Line 11: can make the : results a little harder to read, this changes percpu_rwlock in TSAN builds : to just use a simple lock. > Is there any danger that TSAN won't notice a race if we're using only one l I think if the rwlock itself were buggy, that's possible, but we've been using it long enough that I have good confidence in it. -- To view, visit http://gerrit.cloudera.org:8080/6905 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I08b41a5bbdbef9046b07d2908b2b824bb9dd2b76 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
