diff options
| author | Michael Anthony Knyszek <mknyszek@google.com> | 2022-11-28 19:53:20 +0000 |
|---|---|---|
| committer | Gopher Robot <gobot@golang.org> | 2023-04-05 21:45:24 +0000 |
| commit | 8d68b388d4d1debec8d349adac58dd9f1cb03d25 (patch) | |
| tree | f6d4383189471f5c536bd5c68cfa7a839083d76d /src | |
| parent | 96e8e62821ddb862e947fc9a1609e8e5b713cf14 (diff) | |
| download | go-8d68b388d4d1debec8d349adac58dd9f1cb03d25.tar.xz | |
runtime: flush each idle P's page cache at the end of each GC cycle
Currently pages may linger in an idle P's page cache, hiding the memory
from the scavenger precisely when it's useful to return memory to the OS
and reduce the application's footprint.
Change-Id: I49fbcd806b6c66991d1ca87949f76a9f06708e70
Reviewed-on: https://go-review.googlesource.com/c/go/+/453622
Run-TryBot: Michael Knyszek <mknyszek@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Michael Pratt <mpratt@google.com>
Auto-Submit: Michael Knyszek <mknyszek@google.com>
Diffstat (limited to 'src')
| -rw-r--r-- | src/runtime/mgc.go | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/src/runtime/mgc.go b/src/runtime/mgc.go index f630577914..7c7d1449a2 100644 --- a/src/runtime/mgc.go +++ b/src/runtime/mgc.go @@ -1104,9 +1104,21 @@ func gcMarkTermination() { // mcache before allocating, but idle Ps may not. Since this // is necessary to sweep all spans, we need to ensure all // mcaches are flushed before we start the next GC cycle. + // + // While we're here, flush the page cache for idle Ps to avoid + // having pages get stuck on them. These pages are hidden from + // the scavenger, so in small idle heaps a significant amount + // of additional memory might be held onto. systemstack(func() { forEachP(func(pp *p) { pp.mcache.prepareForSweep() + if pp.status == _Pidle { + systemstack(func() { + lock(&mheap_.lock) + pp.pcache.flush(&mheap_.pages) + unlock(&mheap_.lock) + }) + } }) }) // Now that we've swept stale spans in mcaches, they don't |