Skip to content

Commit 70a1aae

Browse files
mknyszekgopherbot
authored andcommitted
[release-branch.go1.22] cmd/trace/v2: make the -pprof actually useful
In both the v1 and v2 cmd/trace, pprofMatchingGoroutines will generate no output at all if the filter name passed to it is the empty string. This is rather pointless because there are at least two places where we don't pass a name to filter. Modify pprofMatchingGoroutines to include *all* goroutines in the trace if the name to filter by is not specified. For #66782 Fixes #68542 Fixes #68546 Change-Id: I6b72298d676bc93892b075a7426e6e56bc6656c7 Reviewed-on: https://go-review.googlesource.com/c/go/+/578356 LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Carlos Amedee <carlos@golang.org> Auto-Submit: Michael Knyszek <mknyszek@google.com> (cherry picked from commit d1f2cd8) Reviewed-on: https://go-review.googlesource.com/c/go/+/600275 Reviewed-by: Michael Knyszek <mknyszek@google.com> Auto-Submit: Dmitri Shuralyov <dmitshur@google.com> Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
1 parent 2c88c1d commit 70a1aae

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/cmd/trace/v2/pprof.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ func pprofByRegion(compute computePprofFunc, t *parsedTrace) traceviewer.Profile
4848
func pprofMatchingGoroutines(name string, t *parsedTrace) (map[tracev2.GoID][]interval, error) {
4949
res := make(map[tracev2.GoID][]interval)
5050
for _, g := range t.summary.Goroutines {
51-
if g.Name != name {
51+
if name != "" && g.Name != name {
5252
continue
5353
}
5454
endTime := g.EndTime

0 commit comments

Comments
 (0)