sergiotarxz/nuttx-apps
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

wamr: bump the default stack guard size

Browse Source 
using 0 by default seems a bit too aggressive to me.

1024 here is what would have been used before this kconfig knob
was introduced by https://github.com/apache/nuttx-apps/pull/1259.
It also matches some expectations in the recent wamrc.
40b430fd24/core/iwasm/compilation/aot_emit_aot_file.c (L2672-L2678)
This commit is contained in:
YAMAMOTO Takashi 2023-12-25 16:14:15 +09:00 committed by Alin Jerpelea
parent 9c48a8d4fc
commit 70108ae59e
1 changed files with 11 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
interpreters/wamr/Kconfig
View File

@ -119,10 +119,18 @@ config INTERPRETERS_WAMR_TAIL_CALL
config INTERPRETERS_WAMR_STACK_GUARD_SIZE
int "Custom stack guard size"
default 0
default 1024
---help---
Reserve some space in stack as guard to detect stack overflow,
use stack base by default but may not safe enough.
Reserve some space in stack as guard to detect stack overflow.
Please consider this extra stack consumption when deciding the stack
size of your NuttX task on which you run WAMR.
The default value here (1024) was chosen to match the default value on
the majority of other platforms, including Linux. It also matches
the expections in the WAMR AoT compiler.
On the other hand, it might be a bit larger than what's strictly
necessary, especially for typical NuttX targets.
config INTERPRETERS_WAMR_THREAD_MGR
bool "Enable thread manager"