michaelh/zephyr
1
0
Fork 0
Code Releases Activity

shell: kernel: fix interrupt stack size calculation on SMP

Browse source 
When calculating the size of unused interrupt stacks on SMP,
the "unused" variable is not cleared between CPUs. So this
value keeps incrementing and does not reflect actual unused
size for CPUs other than the first one. So clear the "unused"
variable for each CPU.

Signed-off-by: Daniel Leung <daniel.leung@intel.com>
This commit is contained in:
Daniel Leung 2020-11-12 15:51:08 -08:00 • committed by Anas Nashif
commit c7a0cdd1a5
1 changed files with 2 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

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

@ -180,7 +180,7 @@ static int cmd_kernel_stacks(const struct shell *shell,
size_t argc, char **argv) size_t argc, char **argv)
{ {
uint8_t *buf; uint8_t *buf;
size_t size, unused = 0; size_t size, unused;
ARG_UNUSED(argc); ARG_UNUSED(argc);
ARG_UNUSED(argv); ARG_UNUSED(argv);
@ -194,6 +194,7 @@ static int cmd_kernel_stacks(const struct shell *shell,
buf = Z_KERNEL_STACK_BUFFER(z_interrupt_stacks[i]); buf = Z_KERNEL_STACK_BUFFER(z_interrupt_stacks[i]);
size = K_KERNEL_STACK_SIZEOF(z_interrupt_stacks[i]); size = K_KERNEL_STACK_SIZEOF(z_interrupt_stacks[i]);
unused = 0;
for (size_t i = 0; i < size; i++) { for (size_t i = 0; i < size; i++) {
if (buf[i] == 0xAAU) { if (buf[i] == 0xAAU) {
unused++; unused++;