]> Cypherpunks repositories - gostls13.git/commit
runtime: remove atomic store requirement on pageAlloc.chunks
authorMichael Anthony Knyszek <mknyszek@google.com>
Wed, 7 Sep 2022 20:03:25 +0000 (20:03 +0000)
committerGopher Robot <gobot@golang.org>
Thu, 8 Sep 2022 16:05:56 +0000 (16:05 +0000)
commit1c59199c919419293075ac9c93b27b033e2733f9
treeb5db2c2fd945220d147592d93e6c9951fbbd8d74
parentcd8aa4014947dda0eb822452d3c423b561c13eeb
runtime: remove atomic store requirement on pageAlloc.chunks

pageAlloc.chunks used to require an atomic store when growing the heap
because the scavenger would look at the list without locking the heap
lock. However, the scavenger doesn't do that anymore, and it looks like
nothing really does at all.

This change updates the comment and makes the store non-atomic.

Change-Id: Ib452d147861060f9f6e74e2d98ee111cf89ce8f5
Reviewed-on: https://go-review.googlesource.com/c/go/+/429219
Auto-Submit: Michael Knyszek <mknyszek@google.com>
Reviewed-by: Keith Randall <khr@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Michael Knyszek <mknyszek@google.com>
Reviewed-by: Keith Randall <khr@golang.org>
src/runtime/mpagealloc.go