languages/go
languages/go
1
0
Fork 0
mirror of https://github.com/golang/go synced 2024-07-01 07:56:09 +00:00
Code Issues Projects Releases Wiki Activity

test: ignore MemProfileRecords with no live objects in finprofiled.go

Browse Source 
This test erroneously assumes that there will always be at least one
live object accounted for in a MemProfileRecord. This is not true; all
memory allocated from a particular location could be dead.

Fixes #64153.

Change-Id: Iadb783ea9b247823439ddc74b62a4c8b2ce8e33e
Reviewed-on: https://go-review.googlesource.com/c/go/+/542736
Reviewed-by: Cherry Mui <cherryyz@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
This commit is contained in:
Michael Anthony Knyszek 2023-11-15 21:05:02 +00:00 committed by Michael Knyszek
parent 22278e3835
commit 0011342590
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
test/finprofiled.go
View File

@ -57,6 +57,11 @@ func main() {
for _, p := range prof {
bytes := p.AllocBytes - p.FreeBytes
nobj := p.AllocObjects - p.FreeObjects
if nobj == 0 {
// There may be a record that has had all of its objects
// freed. That's fine. Avoid a divide-by-zero and skip.
continue
}
size := bytes / nobj
if size == tinyBlockSize {
totalBytes += bytes