diff options
| author | Austin Clements <austin@google.com> | 2019-09-27 14:13:22 -0400 |
|---|---|---|
| committer | Austin Clements <austin@google.com> | 2019-10-25 23:25:32 +0000 |
| commit | 8c5861576a983684faac98c612c9c7e569974ffa (patch) | |
| tree | acaca58d2264cf3079b8ff561809dd1a31a2858c /src/runtime/sizeof_test.go | |
| parent | 1b79afe460c329c1db75456c3278600a4b451b41 (diff) | |
| download | go-8c5861576a983684faac98c612c9c7e569974ffa.tar.gz go-8c5861576a983684faac98c612c9c7e569974ffa.zip | |
runtime: remove g.gcscanvalid
Currently, gcscanvalid is used to resolve a race between attempts to
scan a stack. Now that there's a clear owner of the stack scan
operation, there's no longer any danger of racing or attempting to
scan a stack more than once, so this CL eliminates gcscanvalid.
I double-checked my reasoning by first adding a throw if gcscanvalid
was set in scanstack and verifying that all.bash still passed.
For #10958, #24543.
Fixes #24363.
Change-Id: I76794a5fcda325ed7cfc2b545e2a839b8b3bc713
Reviewed-on: https://go-review.googlesource.com/c/go/+/201139
Run-TryBot: Austin Clements <austin@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Cherry Zhang <cherryyz@google.com>
Diffstat (limited to 'src/runtime/sizeof_test.go')
| -rw-r--r-- | src/runtime/sizeof_test.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/runtime/sizeof_test.go b/src/runtime/sizeof_test.go index 852244d425..406a38aad9 100644 --- a/src/runtime/sizeof_test.go +++ b/src/runtime/sizeof_test.go @@ -21,7 +21,7 @@ func TestSizeof(t *testing.T) { _32bit uintptr // size on 32bit platforms _64bit uintptr // size on 64bit platforms }{ - {runtime.G{}, 216, 376}, // g, but exported for testing + {runtime.G{}, 212, 368}, // g, but exported for testing } for _, tt := range tests { |