qbit/go
1
0
Fork 0
mirror of https://github.com/golang/go synced 2024-11-12 06:20:22 -07:00
Code Releases Activity

runtime/pprof/internal/protopprof: skip TestTranslateCPUProfileWithSamples if < 2 mappings

Browse Source 
A Go binary may only have 1 executable memory region if it has been
linked using internal linking. This change means that the test will
be skipped if this is the case, rather than fail.

Fixes #17852.

Change-Id: I59459a0f90ae8963aeb9908e5cb9fb64d7d0e0f4
Reviewed-on: https://go-review.googlesource.com/32920
Run-TryBot: Michael Munday <munday@ca.ibm.com>
Run-TryBot: Michael Matloob <matloob@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Michael Matloob <matloob@golang.org>
This commit is contained in:
Michael Munday 2016-11-08 17:11:18 -05:00
parent 60a9bf9f95
commit 9c2037fbcf
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/runtime/pprof/internal/protopprof/protopprof_test.go
View File

@ -162,7 +162,9 @@ func TestTranslateCPUProfileWithSamples(t *testing.T) {
t.Fatalf("Cannot parse /proc/self/maps") t.Fatalf("Cannot parse /proc/self/maps")
} }
if len(mprof.Mapping) < 2 { if len(mprof.Mapping) < 2 {
t.Fatalf("Less than two mappings") // It is possible for a binary to only have 1 executable
// region of memory.
t.Skipf("need 2 or more mappings, got %v", len(mprof.Mapping))
} }
address1 := mprof.Mapping[0].Start address1 := mprof.Mapping[0].Start
address2 := mprof.Mapping[1].Start address2 := mprof.Mapping[1].Start