diff options
| author | David Crawshaw <crawshaw@golang.org> | 2016-10-30 21:19:59 -0400 |
|---|---|---|
| committer | David Crawshaw <crawshaw@golang.org> | 2016-10-31 04:49:52 +0000 |
| commit | f4c7a12c2c4b0aef99b4957a778736a4da9ae4ec (patch) | |
| tree | c5e6ba761e54e38dc8904a3d75f683a4d7817ab8 /src/runtime/symtab.go | |
| parent | 9da7058466c8c9f32e1481f28d57732832ee3b30 (diff) | |
| download | go-f4c7a12c2c4b0aef99b4957a778736a4da9ae4ec.tar.xz | |
runtime: make module typemaps visible to the GC
The map[typeOff]*_type object is created at run time and stored in
the moduledata. The moduledata object is marked by the linker as
SNOPTRDATA, so the reference is ignored by the GC. Running
misc/cgo/testplugin/test.bash with GOGC=1 will eventually collect
the typemap and crash.
This bug probably comes up in -linkshared binaries in Go 1.7.
I don't know why we haven't seen a report about this yet.
Fixes #17680
Change-Id: I0e9b5c006010e8edd51d9471651620ba665248d3
Reviewed-on: https://go-review.googlesource.com/32430
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Michael Hudson-Doyle <michael.hudson@canonical.com>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Diffstat (limited to 'src/runtime/symtab.go')
| -rw-r--r-- | src/runtime/symtab.go | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/src/runtime/symtab.go b/src/runtime/symtab.go index 98b5f900e6..d69f610ebb 100644 --- a/src/runtime/symtab.go +++ b/src/runtime/symtab.go @@ -222,6 +222,15 @@ type modulehash struct { runtimehash *string } +// pinnedTypemaps are the map[typeOff]*_type from the moduledata objects. +// +// These typemap objects are allocated at run time on the heap, but the +// only direct reference to them is in the moduledata, created by the +// linker and marked SNOPTRDATA so it is ignored by the GC. +// +// To make sure the map isn't collected, we keep a second reference here. +var pinnedTypemaps []map[typeOff]*_type + var firstmoduledata moduledata // linker symbol var lastmoduledatap *moduledata // linker symbol |