]> Cypherpunks repositories - gostls13.git/commit
runtime: remove logical stack sentinel for runtime lock stacks
authorNick Ripley <nick.ripley@datadoghq.com>
Tue, 2 Dec 2025 19:46:39 +0000 (19:46 +0000)
committerMichael Knyszek <mknyszek@google.com>
Fri, 23 Jan 2026 16:26:04 +0000 (08:26 -0800)
commit6b7206feb2df3116bd5c471813353ffec1ff25fe
tree3efb8613192aaa200e14f3c78c2a8384dd4e1ca3
parent23f7ba554d0e3ffbe18d04e79c34984e050869e1
runtime: remove logical stack sentinel for runtime lock stacks

When CL 533258 added frame pointer support for the block and mutex
profiles, it deferred skipping/inline expansion, similar to the
execution tracer. The first frame indicated whether this expansion was
needed by holding either the number of frames to skip (implying inline
expansion) or the logicalStackSentinel placeholder to indicate that no
extra handling is needed. However, CL 598515 switched to doing the
skipping/inline expansion at sample time. After this, the sentinel value
is unused. This CL removes the sentinel from runtime lock profiling,
correcting an oversight from that CL. This didn't cause any problems
since post-processing would just ignore the bogus sentinel value; this
is just a cleanup.

Change-Id: Idbec11f57ac7a57426cd8a064782c4fe6a6a6964
Reviewed-on: https://go-review.googlesource.com/c/go/+/726040
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Michael Pratt <mpratt@google.com>
Reviewed-by: Carlos Amedee <carlos@golang.org>
src/runtime/mprof.go