]> Cypherpunks repositories - gostls13.git/commitdiff
runtime: allow for 10 more threads in TestWindowsStackMemory*
authorCuong Manh Le <cuong.manhle.vn@gmail.com>
Mon, 23 Oct 2023 03:22:17 +0000 (10:22 +0700)
committerGopher Robot <gobot@golang.org>
Mon, 23 Oct 2023 17:31:31 +0000 (17:31 +0000)
CL 473415 allowed 5 more threads in TestWindowsStackMemory, to cover
sysmon and any new threads in future. However, during go1.22 dev cycle,
the test becomes flaky again, failing in windows-386 builder a couple of
times in CL 535975 and CL 536175 (and maybe others that haven't caught).

This CL increases the extra threads from 5 to 10, hopefully to make the
test stable again for windows-386. The theory is that Go process load a
bunch of DLLs, which may start their own threads. We could investigate
more deeply if the test still be flaky with 10 extra threads.

Fixes #58570

Change-Id: I255d0d31ed554859a5046fa76dfae1ba89a89aa3
Reviewed-on: https://go-review.googlesource.com/c/go/+/536058
Reviewed-by: Benny Siegert <bsiegert@gmail.com>
Reviewed-by: Quim Muntal <quimmuntal@gmail.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Auto-Submit: Cuong Manh Le <cuong.manhle.vn@gmail.com>
Reviewed-by: Michael Pratt <mpratt@google.com>
src/runtime/testdata/testprog/syscall_windows.go
src/runtime/testdata/testprogcgo/stack_windows.go

index 71bf384dc10a0dfb7ad6277efc7dc8525231f683..a9b8c090cf3b861f0764af79f37458643e6f89f3 100644 (file)
@@ -67,7 +67,7 @@ func StackMemory() {
                panic(err)
        }
        // assumes that this process creates 1 thread for each
-       // thread locked goroutine plus extra 5 threads
+       // thread locked goroutine plus extra 10 threads
        // like sysmon and others
-       print((mem2 - mem1) / (threadCount + 5))
+       print((mem2 - mem1) / (threadCount + 10))
 }
index 0be1126bee93df6245268343b3d0749cfb9675b6..d095093c85f6219a2848651b60cfb131ae463d27 100644 (file)
@@ -51,7 +51,7 @@ func StackMemory() {
                panic(err)
        }
        // assumes that this process creates 1 thread for each
-       // thread locked goroutine plus extra 5 threads
+       // thread locked goroutine plus extra 10 threads
        // like sysmon and others
-       print((mem2 - mem1) / (threadCount + 5))
+       print((mem2 - mem1) / (threadCount + 10))
 }