diff options
| author | Dmitriy Vyukov <dvyukov@google.com> | 2014-10-22 20:23:49 +0400 |
|---|---|---|
| committer | Dmitriy Vyukov <dvyukov@google.com> | 2014-10-22 20:23:49 +0400 |
| commit | 48b04a95e5f719cbef174124780dce00bb410c2c (patch) | |
| tree | a407a7fddb9024a171b3ce8b5a34e382f7c70daf /doc | |
| parent | 87b95ba6c803d8477c306cbf184e69b13d7a5f05 (diff) | |
| download | go-48b04a95e5f719cbef174124780dce00bb410c2c.tar.gz | |
sync: release Pool memory during second and later GCs
Pool memory was only being released during the first GC after the first Put.
Put assumes that p.local != nil means p is on the allPools list.
poolCleanup (called during each GC) removed each pool from allPools
but did not clear p.local, so each pool was cleared by exactly one GC
and then never cleared again.
This bug was introduced late in the Go 1.3 release cycle.
Fixes issue 8979.
LGTM=rsc
R=golang-codereviews, bradfitz, r, rsc
CC=golang-codereviews, khr
https://codereview.appspot.com/162980043
Diffstat (limited to 'doc')
0 files changed, 0 insertions, 0 deletions