[go/release-branch.go1.25] [release-branch.go1.25] runtime: initialise debug settings much earlier in startup process

1 view
Skip to first unread message

Carlos Amedee (Gerrit)

unread,
12:39 PM (10 hours ago) 12:39 PM
to Steve Muir, goph...@pubsubhelper.golang.org, golang-...@googlegroups.com, Go LUCI, Cherry Mui, Michael Knyszek, golang-co...@googlegroups.com

Carlos Amedee submitted the change

Change information

Commit message:
[release-branch.go1.25] runtime: initialise debug settings much earlier in startup process

This is necessary specifically to set the value of `debug.decoratemappings`
sufficiently early in the startup sequence that all memory ranges allocated
can be named appropriately using the new Linux-specific naming API
introduced in #71546.

Example output (on ARM64):
https://gist.github.com/9muir/3667654b9c3f52e8be92756219371672

For: #75324
Fixes #75669

Change-Id: Ic0b16233f54a45adef1660c4d0df59af2f5af86a
Reviewed-on: https://go-review.googlesource.com/c/go/+/703476
Auto-Submit: Michael Knyszek <mkny...@google.com>
LUCI-TryBot-Result: Go LUCI <golang...@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Michael Knyszek <mkny...@google.com>
(cherry picked from commit 300d9d2714164e455abc7990d52de9de6b084df1)
Reviewed-on: https://go-review.googlesource.com/c/go/+/708359
Reviewed-by: Cherry Mui <cher...@google.com>
Files:
  • A src/runtime/decoratemappings_test.go
  • M src/runtime/export_test.go
  • M src/runtime/proc.go
  • M src/runtime/runtime1.go
  • M src/runtime/set_vma_name_linux.go
  • M src/runtime/set_vma_name_stub.go
Change size: M
Delta: 6 files changed, 109 insertions(+), 14 deletions(-)
Branch: refs/heads/release-branch.go1.25
Submit Requirements:
  • requirement satisfiedCode-Review: +2 by Cherry Mui
  • requirement satisfiedTryBots-Pass: LUCI-TryBot-Result+1 by Go LUCI
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: go
Gerrit-Branch: release-branch.go1.25
Gerrit-Change-Id: Ic0b16233f54a45adef1660c4d0df59af2f5af86a
Gerrit-Change-Number: 708359
Gerrit-PatchSet: 2
Gerrit-Owner: Carlos Amedee <car...@golang.org>
Gerrit-Reviewer: Carlos Amedee <car...@golang.org>
Gerrit-Reviewer: Cherry Mui <cher...@google.com>
Gerrit-Reviewer: Michael Knyszek <mkny...@google.com>
Gerrit-CC: Steve Muir <sjm...@google.com>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages