From aeb4fbabc0aaa5293dd34821c450855e9ff82230 Mon Sep 17 00:00:00 2001 From: Michael Pratt Date: Tue, 28 Sep 2021 14:35:03 -0400 Subject: [PATCH] runtime: drop nowritebarrier from gcParkAssist Nothing in this function is at odds with having write barriers. It originally inherited the annotation from gcAssistAlloc http://golang.org/cl/30700, which subsequently dropped the annotation in http://golang.org/cl/32431 as it was unnecessary. Change-Id: Ie464e6b4ed957f57e922ec043728ff4e15bf35ad Reviewed-on: https://go-review.googlesource.com/c/go/+/352811 Trust: Michael Pratt Run-TryBot: Michael Pratt TryBot-Result: Go Bot Reviewed-by: Michael Knyszek --- src/runtime/mgcmark.go | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/runtime/mgcmark.go b/src/runtime/mgcmark.go index 64f1c79c36..eb5221443e 100644 --- a/src/runtime/mgcmark.go +++ b/src/runtime/mgcmark.go @@ -587,8 +587,6 @@ func gcWakeAllAssists() { // // gcParkAssist reports whether the assist is now satisfied. If it // returns false, the caller must retry the assist. -// -//go:nowritebarrier func gcParkAssist() bool { lock(&work.assistQueue.lock) // If the GC cycle finished while we were getting the lock,