]> Cypherpunks repositories - gostls13.git/commitdiff
runtime: mark traceEnabled and traceShuttingDown as no:split
authorMauri de Souza Meneguzzo <mauri870@gmail.com>
Sun, 13 Aug 2023 14:04:53 +0000 (14:04 +0000)
committerMichael Knyszek <mknyszek@google.com>
Tue, 15 Aug 2023 00:22:07 +0000 (00:22 +0000)
This fixes a regression from CL 494181.
The traceEnabled function splits the stack and is being
called by reentersyscall that shouldn't call anything
that splits the stack. Same with traceShuttingDown.

Fixes #61975

Change-Id: I5eca0ba74cfa6acb0259e8400b03c2093cd59dd1
GitHub-Last-Rev: 9e55ae9d7cc700de9757d32a7905127a349d973a
GitHub-Pull-Request: golang/go#61981
Reviewed-on: https://go-review.googlesource.com/c/go/+/519055
Reviewed-by: Michael Knyszek <mknyszek@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Dmitri Shuralyov <dmitshur@google.com>

src/runtime/trace.go

index a4d50d77a0102d880dd9b0d0efc5df6917b05bbc..7d7987c90c4993414bdaeb1d7589612d54bdb79e 100644 (file)
@@ -257,11 +257,15 @@ func traceBufPtrOf(b *traceBuf) traceBufPtr {
 }
 
 // traceEnabled returns true if the trace is currently enabled.
+//
+//go:nosplit
 func traceEnabled() bool {
        return trace.enabled
 }
 
 // traceShuttingDown returns true if the trace is currently shutting down.
+//
+//go:nosplit
 func traceShuttingDown() bool {
        return trace.shutdown
 }