Commit 4f48507d authored by Dave Cheney's avatar Dave Cheney

runtime: reduce pthread stack size in TestCgoCallbackGC

Fixes #11959

This test runs 100 concurrent callbacks from C to Go consuming 100
operating system threads, which at 8mb a piece (the default on linux/arm)
would reserve over 800mb of address space. This would frequently
cause the test to fail on platforms with ~1gb of ram, such as the
raspberry pi.

This change reduces the thread stack allocation to 256kb, a number picked
at random, but at 1/32th the previous size, should allow the test to
pass successfully on all platforms.

Change-Id: I8b8bbab30ea7b2972b3269a6ff91e6fe5bc717af
Reviewed-on: https://go-review.googlesource.com/13731
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: default avatarMartin Capitanio <capnm9@gmail.com>
Reviewed-by: default avatarMinux Ma <minux@golang.org>
parent 0b5bcf53
......@@ -222,7 +222,10 @@ static void *thr(void *arg) {
static void foo() {
pthread_t th;
pthread_create(&th, 0, thr, 0);
pthread_attr_t attr;
pthread_attr_init(&attr);
pthread_attr_setstacksize(&attr, 256 << 10);
pthread_create(&th, &attr, thr, 0);
pthread_join(th, 0);
}
*/
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment