From af7f417665fb1612eb9865c7ab4992bf095148e2 Mon Sep 17 00:00:00 2001 From: cuiweixie Date: Fri, 2 Sep 2022 10:53:13 +0800 Subject: [PATCH] runtime: convert local var stop at TestAfterStress to atomic type For #53821 Change-Id: I7e86dac34691f7752f68879ff379061f3435cd45 Reviewed-on: https://go-review.googlesource.com/c/go/+/427139 Reviewed-by: Michael Knyszek TryBot-Result: Gopher Robot Run-TryBot: xie cui <523516579@qq.com> Reviewed-by: Heschi Kreinick --- src/time/sleep_test.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/time/sleep_test.go b/src/time/sleep_test.go index 5a949b6f80e94..8aac3b68f674d 100644 --- a/src/time/sleep_test.go +++ b/src/time/sleep_test.go @@ -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, @@ -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)) {