Skip to content

Commit

Permalink
runtime: convert local var stop at TestAfterStress to atomic type
Browse files Browse the repository at this point in the history
For #53821

Change-Id: I7e86dac34691f7752f68879ff379061f3435cd45
Reviewed-on: https://go-review.googlesource.com/c/go/+/427139
Reviewed-by: Michael Knyszek <mknyszek@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: xie cui <523516579@qq.com>
Reviewed-by: Heschi Kreinick <heschi@google.com>
  • Loading branch information
cuiweixie authored and mvdan committed Sep 5, 2022
1 parent 4ad55cd commit af7f417
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/time/sleep_test.go
Expand Up @@ -65,9 +65,9 @@ func TestAfterFunc(t *testing.T) {
}

func TestAfterStress(t *testing.T) {
stop := uint32(0)
var stop atomic.Bool
go func() {
for atomic.LoadUint32(&stop) == 0 {
for !stop.Load() {
runtime.GC()
// Yield so that the OS can wake up the timer thread,
// so that it can generate channel sends for the main goroutine,
Expand All @@ -80,7 +80,7 @@ func TestAfterStress(t *testing.T) {
<-ticker.C
}
ticker.Stop()
atomic.StoreUint32(&stop, 1)
stop.Store(true)
}

func benchmark(b *testing.B, bench func(n int)) {
Expand Down

0 comments on commit af7f417

Please sign in to comment.