[release-branch.go1.19] runtime: allow for 5 more threads in TestWindowsStackMemory*

Original version of TestWindowsStackMemory did not consider sysmon and
other threads running during the test. Allow for 5 extra threads in this
test - this should cover any new threads in the future.

For #58570
Fixes #61054

Change-Id: I215790f9b94ff40a32ddd7aa54af715d1dc391c6
Reviewed-on: https://go-review.googlesource.com/c/go/+/473415
Reviewed-by: Michael Pratt <mpratt@google.com>
Run-TryBot: Alex Brainman <alex.brainman@gmail.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Cherry Mui <cherryyz@google.com>
(cherry picked from commit f6cbc1da05da141a78ee33954b52d17642c95130)
Reviewed-on: https://go-review.googlesource.com/c/go/+/506975
Run-TryBot: Michael Knyszek <mknyszek@google.com>
Auto-Submit: Michael Knyszek <mknyszek@google.com>
(cherry picked from commit 08a58dd8b6337c9bac93face4c386a918a3dd97b)
Reviewed-on: https://go-review.googlesource.com/c/go/+/506976
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
This commit is contained in:
Alex Brainman 2023-03-04 14:35:35 +11:00 committed by Gopher Robot
parent 9a2e6c9cc2
commit 16b198519b
2 changed files with 8 additions and 2 deletions

View File

@ -66,5 +66,8 @@ func StackMemory() {
if err != nil {
panic(err)
}
print((mem2 - mem1) / threadCount)
// assumes that this process creates 1 thread for each
// thread locked goroutine plus extra 5 threads
// like sysmon and others
print((mem2 - mem1) / (threadCount + 5))
}

View File

@ -50,5 +50,8 @@ func StackMemory() {
if err != nil {
panic(err)
}
print((mem2 - mem1) / threadCount)
// assumes that this process creates 1 thread for each
// thread locked goroutine plus extra 5 threads
// like sysmon and others
print((mem2 - mem1) / (threadCount + 5))
}