summaryrefslogtreecommitdiff
path: root/src/debug
diff options
context:
space:
mode:
authorDmitriy Vyukov <dvyukov@google.com>2014-11-20 09:51:02 -0500
committerDmitriy Vyukov <dvyukov@google.com>2014-11-20 09:51:02 -0500
commitfbd764146d71d8929edadc4abd51e89932742000 (patch)
tree9b443eec83e97515584926a9e46517d51a194268 /src/debug
parent6382443a1f02e8f3c181621bc9c895e4ec8410c5 (diff)
downloadgo-fbd764146d71d8929edadc4abd51e89932742000.tar.gz
runtime: fix atomic operations on non-heap addresses
Race detector runtime does not tolerate operations on addresses that was not previously declared with __tsan_map_shadow (namely, data, bss and heap). The corresponding address checks for atomic operations were removed in https://codereview.appspot.com/111310044 Restore these checks. It's tricker than just not calling into race runtime, because it is the race runtime that makes the atomic operations themselves (if we do not call into race runtime we skip the atomic operation itself as well). So instead we call __tsan_go_ignore_sync_start/end around the atomic operation. This forces race runtime to skip all other processing except than doing the atomic operation itself. Fixes issue 9136. LGTM=rsc R=rsc CC=golang-codereviews https://codereview.appspot.com/179030043 Committer: Russ Cox <rsc@golang.org>
Diffstat (limited to 'src/debug')
0 files changed, 0 insertions, 0 deletions