From 75d779566b75fc1a09239bfbca40b6dba42000fd Mon Sep 17 00:00:00 2001 From: Russ Cox Date: Wed, 22 Jul 2015 15:31:54 -0400 Subject: runtime/cgo: make compatible with race detector Some routines run without and m or g and cannot invoke the race detector runtime. They must be opaque to the runtime. That used to be true because they were written in C. Now that they are written in Go, disable the race detector annotations for those functions explicitly. Add test. Fixes #10874. Change-Id: Ia8cc28d51e7051528f9f9594b75634e6bb66a785 Reviewed-on: https://go-review.googlesource.com/12534 Reviewed-by: Ian Lance Taylor --- src/runtime/cgo/callbacks.go | 1 + src/runtime/sigqueue.go | 1 + 2 files changed, 2 insertions(+) (limited to 'src/runtime') diff --git a/src/runtime/cgo/callbacks.go b/src/runtime/cgo/callbacks.go index 1682341596..08f230d47e 100644 --- a/src/runtime/cgo/callbacks.go +++ b/src/runtime/cgo/callbacks.go @@ -37,6 +37,7 @@ var _runtime_cgo_panic_internal byte //go:cgo_export_static _cgo_panic //go:cgo_export_dynamic _cgo_panic //go:nosplit +//go:norace func _cgo_panic(a unsafe.Pointer, n int32) { _runtime_cgocallback(unsafe.Pointer(&_runtime_cgo_panic_internal), a, uintptr(n)) } diff --git a/src/runtime/sigqueue.go b/src/runtime/sigqueue.go index e078bfaf0e..3f50a59c14 100644 --- a/src/runtime/sigqueue.go +++ b/src/runtime/sigqueue.go @@ -164,6 +164,7 @@ func signal_ignore(s uint32) { // This runs on a foreign stack, without an m or a g. No stack split. //go:nosplit +//go:norace func badsignal(sig uintptr) { cgocallback(unsafe.Pointer(funcPC(badsignalgo)), noescape(unsafe.Pointer(&sig)), unsafe.Sizeof(sig)) } -- cgit v1.3-5-g9baa