]> Cypherpunks repositories - gostls13.git/commit
runtime: remove futile wakeups from trace
authorDmitry Vyukov <dvyukov@google.com>
Wed, 11 Mar 2015 15:29:12 +0000 (18:29 +0300)
committerDmitry Vyukov <dvyukov@google.com>
Tue, 17 Mar 2015 14:14:55 +0000 (14:14 +0000)
commit4396ea96c43e1df585c648762b6993e84cb6a2e5
tree4cc0f995c0198be8060dd347b9523d430cc6d8ba
parent1b49a86ecece3978ceba60c372327b9cbcc68501
runtime: remove futile wakeups from trace

Channels and sync.Mutex'es allow another goroutine to acquire resource
ahead of an unblocked goroutine. This is good for performance, but
leads to futile wakeups (the unblocked goroutine needs to block again).
Futile wakeups caused user confusion during the very first evaluation
of tracing functionality on a real server (a goroutine as if acquires a mutex
in a loop, while there is no loop in user code).

This change detects futile wakeups on channels and emits a special event
to denote the fact. Later parser finds entire wakeup sequences
(unblock->start->block) and removes them.

sync.Mutex will be supported in a separate change.

Change-Id: Iaaaee9d5c0921afc62b449a97447445030ac19d3
Reviewed-on: https://go-review.googlesource.com/7380
Reviewed-by: Keith Randall <khr@golang.org>
src/internal/trace/parser.go
src/runtime/chan.go
src/runtime/pprof/trace_test.go
src/runtime/select.go
src/runtime/trace.go