]> Cypherpunks repositories - gostls13.git/commit
cmd/cgo, runtime, runtime/cgo: TSAN support for malloc
authorIan Lance Taylor <iant@golang.org>
Thu, 19 May 2016 23:27:23 +0000 (16:27 -0700)
committerIan Lance Taylor <iant@golang.org>
Wed, 25 May 2016 23:22:24 +0000 (23:22 +0000)
commita5d1a72a40b59db0d2f3f5d3fbb2ed60aafb7fdf
treeb6edf8790228d3dfe3381b2d1764b0c1e61b863d
parent10c8b2374f413ef6225555893cad5d2a530f77d5
cmd/cgo, runtime, runtime/cgo: TSAN support for malloc

Acquire and release the TSAN synchronization point when calling malloc,
just as we do when calling any other C function. If we don't do this,
TSAN will report false positive errors about races calling malloc and
free.

We used to have a special code path for malloc and free, going through
the runtime functions cmalloc and cfree. The special code path for cfree
was no longer used even before this CL. This CL stops using the special
code path for malloc, because there is no place along that path where we
could conditionally insert the TSAN synchronization. This CL removes
the support for the special code path for both functions.

Instead, cgo now automatically generates the malloc function as though
it were referenced as C.malloc.  We need to automatically generate it
even if C.malloc is not called, even if malloc and size_t are not
declared, to support cgo-provided functions like C.CString.

Change-Id: I829854ec0787a80f33fa0a8a0dc2ee1d617830e2
Reviewed-on: https://go-review.googlesource.com/23260
Reviewed-by: Dmitry Vyukov <dvyukov@google.com>
Run-TryBot: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Austin Clements <austin@google.com>
misc/cgo/testsanitizers/test.bash
misc/cgo/testsanitizers/tsan4.go [new file with mode: 0644]
src/cmd/cgo/out.go
src/runtime/cgo.go
src/runtime/cgo/callbacks.go
src/runtime/cgo/gcc_util.c
src/runtime/cgocall.go
src/runtime/proc.go