From 468bc941886ae83b8d16af5deaaf12ea5cabb05b Mon Sep 17 00:00:00 2001 From: Matthew Dempsky Date: Mon, 20 Nov 2023 06:27:20 +0000 Subject: 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 Auto-Submit: Matthew Dempsky LUCI-TryBot-Result: Go LUCI --- src/runtime/runtime2.go | 8 -------- 1 file changed, 8 deletions(-) (limited to 'src/runtime/runtime2.go') diff --git a/src/runtime/runtime2.go b/src/runtime/runtime2.go index 6bdd66766d..e64c3c5695 100644 --- a/src/runtime/runtime2.go +++ b/src/runtime/runtime2.go @@ -593,8 +593,6 @@ type m struct { lockedInt uint32 // tracking for internal lockOSThread nextwaitm muintptr // next m waiting for lock - mLockProfile mLockProfile // fields relating to runtime.lock contention - // wait* are used to carry arguments from gopark into park_m, because // there's no stack to put them on. That is their sole purpose. waitunlockf func(*g, unsafe.Pointer) bool @@ -902,12 +900,6 @@ type schedt struct { // stwTotalTimeOther covers the others. stwTotalTimeGC timeHistogram stwTotalTimeOther timeHistogram - - // totalRuntimeLockWaitTime (plus the value of lockWaitTime on each M in - // allm) is the sum of time goroutines have spent in _Grunnable and with an - // M, but waiting for locks within the runtime. This field stores the value - // for Ms that have exited. - totalRuntimeLockWaitTime atomic.Int64 } // Values for the flags field of a sigTabT. -- cgit v1.3