From b1d44a81b450a23511dcbd31b783fe2cf4a935f9 Mon Sep 17 00:00:00 2001 From: Gregory Nutt Date: Thu, 25 Aug 2016 13:22:06 -0600 Subject: [PATCH] apps/system/system: A much larger stack is needed to run reliably on the simulation. --- examples/README.txt | 5 ++++- system/system/Kconfig | 8 +++++++- 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/examples/README.txt b/examples/README.txt index d30acde0d..47c4b3896 100644 --- a/examples/README.txt +++ b/examples/README.txt @@ -1890,7 +1890,10 @@ examples/smp examples/system ^^^^^^^^^^^^^^^ - This is a simple test of the system() command. + This is a simple test of the system() command. The test simply executes this + system command: + + ret = system("ls -Rl /"); examples/tcpecho ^^^^^^^^^^^^^^^^ diff --git a/system/system/Kconfig b/system/system/Kconfig index 846053b96..2463c39e7 100644 --- a/system/system/Kconfig +++ b/system/system/Kconfig @@ -15,10 +15,16 @@ if SYSTEM_SYSTEM config SYSTEM_SYSTEM_STACKSIZE int "system stack size" - default 1536 + default 2048 if !ARCH_SIM + default 4096 if ARCH_SIM ---help--- The size of stack allocated for the shell. + NOTE: I needed to set the stack size quite large to get this example + working on the simulated target (perhaps because of the 64-bit + stack?). I assume that a smaller stack would be okay on real + hardware, but I have not yet verified that. + config SYSTEM_SYSTEM_PRIORITY int "system priority" default 100