1
0
mirror of https://github.com/golang/go synced 2024-11-17 09:04:44 -07:00

runtime/pprof: TestLabelSystemstack parallelLabelHog.func1 must be labeled

The closure in parallelLabelHog should be labeled in a addition to
parallelLabelHog itself. Generally samples on that goroutine land on
labelHog, but there is a small portion of the closure outside of
labelHog.

Fixes #50740.

Change-Id: I363b6d8eec2e6920c215686e2039fce6d5b29a98
Reviewed-on: https://go-review.googlesource.com/c/go/+/380055
Reviewed-by: Bryan Mills <bcmills@google.com>
Trust: Michael Pratt <mpratt@google.com>
This commit is contained in:
Michael Pratt 2022-01-21 11:07:52 -05:00
parent d15481b8c7
commit 9eba5ff521

View File

@ -1448,7 +1448,7 @@ func TestLabelSystemstack(t *testing.T) {
for _, loc := range s.Location {
for _, l := range loc.Line {
switch l.Function.Name {
case "runtime/pprof.labelHog", "runtime/pprof.parallelLabelHog":
case "runtime/pprof.labelHog", "runtime/pprof.parallelLabelHog", "runtime/pprof.parallelLabelHog.func1":
mustBeLabeled = true
case "runtime/pprof.Do":
// Do sets the labels, so samples may