diff mbox series

sandbox: improve description of CONFIG_SANDBOX_CRASH_RESET

Message ID 20240212083730.33956-1-heinrich.schuchardt@canonical.com
State Accepted
Commit 7b2d32a7d6a782063bed80fa2161305eba627caa
Delegated to: Tom Rini
Headers show
Series sandbox: improve description of CONFIG_SANDBOX_CRASH_RESET | expand

Commit Message

Heinrich Schuchardt Feb. 12, 2024, 8:37 a.m. UTC 
Mentions that command line option --signal is needed to make use of this
configuration option.

Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
---
 arch/sandbox/Kconfig | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

Comments

Tom Rini April 12, 2024, 6:52 p.m. UTC | #1 
On Mon, Feb 12, 2024 at 09:37:30AM +0100, Heinrich Schuchardt wrote:

> Mentions that command line option --signal is needed to make use of this
> configuration option.
> 
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>

Applied to u-boot/master, thanks!
diff mbox series

Patch

diff --git a/arch/sandbox/Kconfig b/arch/sandbox/Kconfig
index 0ce77de2fcb..1c8353d6156 100644
--- a/arch/sandbox/Kconfig
+++ b/arch/sandbox/Kconfig
@@ -58,10 +58,15 @@  config SANDBOX_CRASH_RESET
 	bool "Reset on crash"
 	help
 	  If an illegal instruction or an illegal memory access occurs, the
-	  sandbox by default writes a crash dump and exits. If you set this
-	  flag, the sandbox is reset instead. This may be useful when running
-	  test suites like the UEFI self certification test which continue
-	  with the next test after a crash.
+	  sandbox exits with an error by default.
+
+	  If the u-boot binary is invoked with --signals (or -S), U-Boot will
+	  handle the signal writing a crash dump before exiting.
+
+	  If you additionally set the CONFIG_SANDBOX_CRASH_RESET flag, the
+	  sandbox is reset after writing the crash dump. This may be useful
+	  when running test suites like the UEFI self certification test which
+	  continue with the next test after a crash.
 
 config SANDBOX_BITS_PER_LONG
 	int