]> Cypherpunks repositories - gostls13.git/commit
Revert "runtime: improve runtime-internal mutex profile tests"
authorRhys Hiltner <rhys.hiltner@gmail.com>
Wed, 29 May 2024 16:39:34 +0000 (16:39 +0000)
committerGopher Robot <gobot@golang.org>
Thu, 30 May 2024 17:50:53 +0000 (17:50 +0000)
commite6b8b2f3e236e8646962c68e172c304e11e9a8f0
tree59464ede0bc23f54a17ec0816acfa0e1784e0c85
parent1be701a344d1f1819dc08d78259684de1da6f923
Revert "runtime: improve runtime-internal mutex profile tests"

This reverts commit f9ba2cff2286d378eca28c841bea8488c69fc30e (CL 586237)

Reason for revert: This is part of a patch series that changed the
handling of contended lock2/unlock2 calls, reducing the maximum
throughput of contended runtime.mutex values, and causing a performance
regression on applications where that is (or became) the bottleneck.

This test verifies that the semantics of the mutex profile for
runtime.mutex values matches that of sync.Mutex values. Without the rest
of the patch series, this test would correctly identify that Go 1.22's
semantics are incorrect (issue #66999).

Updates #66999
Updates #67585

Change-Id: Id06ae01d7bc91c94054c80d273e6530cb2d59d10
Reviewed-on: https://go-review.googlesource.com/c/go/+/589096
Reviewed-by: Michael Pratt <mpratt@google.com>
Reviewed-by: Than McIntosh <thanm@google.com>
Auto-Submit: Rhys Hiltner <rhys.hiltner@gmail.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
src/runtime/export_test.go
src/runtime/metrics_test.go