]> Cypherpunks.ru repositories - gostls13.git/commitdiff
test: really force heap profile update in heapsampling.go
authorAustin Clements <austin@google.com>
Thu, 12 Nov 2015 17:24:36 +0000 (12:24 -0500)
committerAustin Clements <austin@google.com>
Thu, 12 Nov 2015 18:32:13 +0000 (18:32 +0000)
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>
test/heapsampling.go

index ed9208c1f424fbfa7700410959c981c863f3be2b..c00b866680a2534900091979c2a7e3054d0b56a3 100644 (file)
@@ -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)),