Increase default stack size

[eloparco]

With the previous stack size, I can't even run the simple no_pthread example on MacOS.

./iwasm wasm-apps/no_pthread.wasm
[3/3] Linking C executable iwasm

#00 main
#01 __main_void
#02 _start

Exception: native stack overflow

[wenyongh]

LGTM

[yamt]

With the previous stack size, I can't even run the simple no_pthread example on MacOS.

./iwasm wasm-apps/no_pthread.wasm
[3/3] Linking C executable iwasm

#00 main
#01 __main_void
#02 _start

Exception: native stack overflow

it works for me on macOS 12.7.1 / x86-64.

spacetanuki% ./iwasm wasm-apps/no_pthread.wasm
New thread ID: 1, starting parameter: 52
Updated value: 60
Thread completed, new value: 60, thread id: 1
spacetanuki% 

[yamt]

i guess it's simpler to allow APP_THREAD_STACK_SIZE_DEFAULT=0, which means "leave it to the system default". how do you think?

[wenyongh]

i guess it's simpler to allow APP_THREAD_STACK_SIZE_DEFAULT=0, which means "leave it to the system default". how do you think?

It should be good if we only use 0 for pthread_create, but there are some other places using it and APP_THREAD_STACK_SIZE_MIN, and it may be used in non-posix platforms:

wasm-micro-runtime/core/shared/platform/common/posix/posix_thread.c

Lines 431 to 435 in 23d2e06

	max_stack_size =
	(size_t)(APP_THREAD_STACK_SIZE_MAX + page_size - 1) & ~(page_size - 1);
	if (max_stack_size < APP_THREAD_STACK_SIZE_DEFAULT)
	max_stack_size = APP_THREAD_STACK_SIZE_DEFAULT;

wasm-micro-runtime/core/shared/platform/zephyr/zephyr_thread.c

Lines 331 to 335 in 23d2e06

	#if BH_ENABLE_ZEPHYR_MPU_STACK == 0
	if (stack_size < APP_THREAD_STACK_SIZE_MIN)
	stack_size = APP_THREAD_STACK_SIZE_MIN;
	thread_data_size = offsetof(os_thread_data, stack) + stack_size;
	#else

Seems it's better to set a non-zero value for it. Another option may be let each platform set its value individually.