diff options
| author | Michael Anthony Knyszek <mknyszek@google.com> | 2025-12-11 19:23:19 +0000 |
|---|---|---|
| committer | Michael Knyszek <mknyszek@google.com> | 2025-12-12 12:02:59 -0800 |
| commit | 6455afbc6f93e7ddc5ffde38b563c4e1a6821d22 (patch) | |
| tree | 82b325a80d515e84f47ec1ec5db22e68989b8b4c /src/runtime/trace | |
| parent | 8f45611e78095c05075b4cc49e44aabde064e134 (diff) | |
| download | go-6455afbc6f93e7ddc5ffde38b563c4e1a6821d22.tar.xz | |
runtime: dropg after emitting trace event in preemptPark
Because we dropg before emitting a trace event in preemptPark, we end up
failing to emit a status for the goroutine if this happens to be the
first event for it in the generation. We only really see this with
multiple subscribers in TestSubscribers.
This is for two reasons:
1. If we are missing a status event for a non-initial generation then
the trace parser won't validate that (an oversight, but we can only
enforce that for new traces because of this bug), and
2. If we're starting the tracer fresh, then we have a STW which
effectively guarantees that the first event for a goroutine cannot
come from preemptPark.
Therefore, we cannot observe this situation unless the first generation
manifests the bug, but prior to having flight recording and/or multiple
subscribers being able to "cut" the trace data at any point, this was
impossible.
The fix is simple: dropg only after emitting the trace event. This is
also safe, because the tracer doesn't care. The tracer will also start
taking a stack trace of the goroutine in this circumstance, but that is
also safe, since we are able to generally unwind the stack of
asynchronously preempted goroutines, and here we're at the very, very
end of asynchronous preemption where all the state to do so is already
set up.
Fixes #75665.
Change-Id: I7ee1142697d0a53b62d4c5647aa53775d2f6976a
Reviewed-on: https://go-review.googlesource.com/c/go/+/729400
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Cherry Mui <cherryyz@google.com>
Diffstat (limited to 'src/runtime/trace')
| -rw-r--r-- | src/runtime/trace/subscribe_test.go | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/runtime/trace/subscribe_test.go b/src/runtime/trace/subscribe_test.go index 869ab0e75b..2822094a02 100644 --- a/src/runtime/trace/subscribe_test.go +++ b/src/runtime/trace/subscribe_test.go @@ -17,9 +17,10 @@ import ( func TestSubscribers(t *testing.T) { validate := func(t *testing.T, source string, tr []byte) { + t.Log("validating", source) defer func() { if t.Failed() { - testtrace.Dump(t, "trace", tr, *dumpTraces) + testtrace.Dump(t, "TestSubscribers."+source, tr, *dumpTraces) } }() |