Commit 25398f36 authored by Benedikt Schmidt's avatar Benedikt Schmidt Committed by Chris Friedt
Browse files

shell: modules: do not use k_thread_foreach with shell callbacks



Always use k_thread_foreach_unlocked with callbacks which print
something out to the shell, as they might call arch_irq_unlock.
Fixes #66660.

Signed-off-by: default avatarBenedikt Schmidt <benedikt.schmidt@embedded-solutions.at>
(cherry picked from commit 4c731f27)
parent d3c2a245
Loading
Loading
Loading
Loading
+9 −8
Original line number Diff line number Diff line
@@ -141,11 +141,12 @@ static int cmd_kernel_threads(const struct shell *shell,
	shell_print(shell, "Scheduler: %u since last call", sys_clock_elapsed());
	shell_print(shell, "Threads:");

#ifdef CONFIG_SMP
	/*
	 * Use the unlocked version as the callback itself might call
	 * arch_irq_unlock.
	 */
	k_thread_foreach_unlocked(shell_tdata_dump, (void *)shell);
#else
	k_thread_foreach(shell_tdata_dump, (void *)shell);
#endif

	return 0;
}

@@ -190,11 +191,11 @@ static int cmd_kernel_stacks(const struct shell *shell,
	ARG_UNUSED(argc);
	ARG_UNUSED(argv);

#ifdef CONFIG_SMP
	/*
	 * Use the unlocked version as the callback itself might call
	 * arch_irq_unlock.
	 */
	k_thread_foreach_unlocked(shell_stack_dump, (void *)shell);
#else
	k_thread_foreach(shell_stack_dump, (void *)shell);
#endif

	/* Placeholder logic for interrupt stack until we have better
	 * kernel support, including dumping arch-specific exception-related