runtime: reset scavenge address in scavengeAll
Currently scavengeAll (which is called by debug.FreeOSMemory) doesn't
reset the scavenge address before scavenging, meaning it could miss
large portions of the heap. Fix this by reseting the address before
scavenging, which will ensure it is able to walk over the entire heap.
Fixes #35858.
Change-Id: I4a7408050b8e134318ff94428f98cb96a1795aa9
Reviewed-on: https://go-review.googlesource.com/c/go/+/208960
Reviewed-by: Cherry Zhang <cherryyz@google.com>
Run-TryBot: Cherry Zhang <cherryyz@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
diff --git a/src/runtime/mheap.go b/src/runtime/mheap.go
index 3f3e367..114c97b 100644
--- a/src/runtime/mheap.go
+++ b/src/runtime/mheap.go
@@ -1424,6 +1424,8 @@
gp := getg()
gp.m.mallocing++
lock(&h.lock)
+ // Reset the scavenger address so we have access to the whole heap.
+ h.pages.resetScavengeAddr()
released := h.pages.scavenge(^uintptr(0), true)
unlock(&h.lock)
gp.m.mallocing--
