aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAustin Clements <austin@google.com>2015-11-12 12:24:36 -0500
committerAustin Clements <austin@google.com>2015-11-12 18:32:13 +0000
commita9ca2137aa3ebcf9cf094b20cf30cf50ba577892 (patch)
tree5dc8d036d5503893941c9945134db1694822be45
parent80d0b98d809361c862670bd7293fbca22055581f (diff)
downloadgo-a9ca2137aa3ebcf9cf094b20cf30cf50ba577892.tar.xz
test: really force heap profile update in heapsampling.go
The heap profile is only guaranteed to be up-to-date after two GC cycles, so force two GCs instead of just one. Updates #13098. Change-Id: I4fb9287b698f4a3b90b8af9fc6a2efb3b082bfe5 Reviewed-on: https://go-review.googlesource.com/16848 Reviewed-by: Dmitry Vyukov <dvyukov@google.com>
-rw-r--r--test/heapsampling.go3
1 files changed, 3 insertions, 0 deletions
diff --git a/test/heapsampling.go b/test/heapsampling.go
index ed9208c1f4..c00b866680 100644
--- a/test/heapsampling.go
+++ b/test/heapsampling.go
@@ -92,6 +92,9 @@ func checkValue(fname string, ln int, name string, want, got int64) {
func getMemProfileRecords() []runtime.MemProfileRecord {
// Force the runtime to update the object and byte counts.
+ // This can take up to two GC cycles to get a complete
+ // snapshot of the current point in time.
+ runtime.GC()
runtime.GC()
// Find out how many records there are (MemProfile(nil, true)),