diff options
| author | Matthew Dempsky <mdempsky@google.com> | 2023-11-20 06:27:20 +0000 |
|---|---|---|
| committer | Gopher Robot <gobot@golang.org> | 2023-11-20 13:20:29 +0000 |
| commit | 468bc941886ae83b8d16af5deaaf12ea5cabb05b (patch) | |
| tree | 1d008384170d873009c247dcef90da14ac1afb95 /src/runtime/proc.go | |
| parent | 237715cf7c4c2aabf6d3c04473488c1b2477f8ea (diff) | |
| download | go-468bc941886ae83b8d16af5deaaf12ea5cabb05b.tar.xz | |
Revert "runtime: profile contended lock calls"
This reverts commit go.dev/cl/528657.
Reason for revert: broke a lot of builders.
Change-Id: I70c33062020e997c4df67b3eaa2e886cf0da961e
Reviewed-on: https://go-review.googlesource.com/c/go/+/543660
Reviewed-by: Than McIntosh <thanm@google.com>
Auto-Submit: Matthew Dempsky <mdempsky@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Diffstat (limited to 'src/runtime/proc.go')
| -rw-r--r-- | src/runtime/proc.go | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/runtime/proc.go b/src/runtime/proc.go index 66d7da887e..ea011e3192 100644 --- a/src/runtime/proc.go +++ b/src/runtime/proc.go @@ -919,7 +919,7 @@ func mcommoninit(mp *m, id int64) { // when it is just in a register or thread-local storage. mp.alllink = allm - // NumCgoCall() and others iterate over allm w/o schedlock, + // NumCgoCall() iterates over allm w/o schedlock, // so we need to publish it safely. atomicstorep(unsafe.Pointer(&allm), unsafe.Pointer(mp)) unlock(&sched.lock) @@ -1852,7 +1852,6 @@ found: unlock(&sched.lock) atomic.Xadd64(&ncgocall, int64(mp.ncgocall)) - sched.totalRuntimeLockWaitTime.Add(mp.mLockProfile.waitTime.Load()) // Release the P. handoffp(releasep()) @@ -5271,7 +5270,6 @@ func _ExternalCode() { _ExternalCode() } func _LostExternalCode() { _LostExternalCode() } func _GC() { _GC() } func _LostSIGPROFDuringAtomic64() { _LostSIGPROFDuringAtomic64() } -func _LostContendedLock() { _LostContendedLock() } func _VDSO() { _VDSO() } // Called if we receive a SIGPROF signal. |