code review 179030043: runtime: fix atomic operations on non-heap addresses

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.

[dvyukov, 2014-11-20 11:24:24]

Hello rsc@golang.org (cc: golang-codereviews@googlegroups.com),

I'd like you to review this change to
https://dvyukov%40google.com@code.google.com/p/go/

[rsc, 2014-11-20 14:50:40]

LGTM

Thanks very much for the quick fix.

[rsc, 2014-11-20 14:51:10]

*** Submitted as https://code.google.com/p/go/source/detail?r=e4ab8f908aac ***

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>

[gobot, 2014-11-20 15:09:42]

This CL appears to have broken the plan9-386-cnielsen builder.
See http://build.golang.org/log/609ad1b791adc550c010bc17d479b92fad2510e5