michaelh/zephyr
1
0
Fork 0
Code Releases Activity

shell: kernel_service: Use z_stack_space_get for isr stack stats

Browse source 
Use dedicated function for calculating stack usage of the
interrupt stack(s).

Signed-off-by: Krzysztof Chruscinski <krzysztof.chruscinski@nordicsemi.no>
This commit is contained in:
Krzysztof Chruscinski 2022-01-28 15:49:05 +01:00 • committed by Anas Nashif
commit caa3ee686a
1 changed files with 7 additions and 13 deletions
Download patch file Download diff file Expand all files Collapse all files

20
subsys/shell/modules/kernel_service.c
View file

@ -14,6 +14,7 @@
#include <device.h> #include <device.h>
#include <drivers/timer/system_timer.h> #include <drivers/timer/system_timer.h>
#include <kernel.h> #include <kernel.h>
#include <kernel_internal.h>
static int cmd_kernel_version(const struct shell *shell, static int cmd_kernel_version(const struct shell *shell,
size_t argc, char **argv) size_t argc, char **argv)
@ -187,9 +188,6 @@ extern K_KERNEL_STACK_ARRAY_DEFINE(z_interrupt_stacks, CONFIG_MP_NUM_CPUS,
static int cmd_kernel_stacks(const struct shell *shell, static int cmd_kernel_stacks(const struct shell *shell,
size_t argc, char **argv) size_t argc, char **argv)
{ {
uint8_t *buf;
size_t size, unused;
ARG_UNUSED(argc); ARG_UNUSED(argc);
ARG_UNUSED(argv); ARG_UNUSED(argv);
k_thread_foreach(shell_stack_dump, (void *)shell); k_thread_foreach(shell_stack_dump, (void *)shell);
@ -199,17 +197,13 @@ static int cmd_kernel_stacks(const struct shell *shell,
* stack buffers. * stack buffers.
*/ */
for (int i = 0; i < CONFIG_MP_NUM_CPUS; i++) { for (int i = 0; i < CONFIG_MP_NUM_CPUS; i++) {
buf = Z_KERNEL_STACK_BUFFER(z_interrupt_stacks[i]); size_t unused;
size = K_KERNEL_STACK_SIZEOF(z_interrupt_stacks[i]); const uint8_t *buf = Z_KERNEL_STACK_BUFFER(z_interrupt_stacks[i]);
size_t size = K_KERNEL_STACK_SIZEOF(z_interrupt_stacks[i]);
int err = z_stack_space_get(buf, size, &unused);
unused = 0; (void)err;
for (size_t i = 0; i < size; i++) { __ASSERT_NO_MSG(err == 0);
if (buf[i] == 0xAAU) {
unused++;
} else {
break;
}
}
shell_print(shell, shell_print(shell,
"%p IRQ %02d (real size %zu):\tunused %zu\tusage %zu / %zu (%zu %%)", "%p IRQ %02d (real size %zu):\tunused %zu\tusage %zu / %zu (%zu %%)",