From afcd8b452114d3ba3ed72d6d143942da7d8a30f4 Mon Sep 17 00:00:00 2001 From: zhaohengbo <1755923166@qq.com> Date: Sat, 15 Dec 2018 11:20:24 +0800 Subject: [PATCH] fix some bug when stack grows upward --- components/finsh/cmd.c | 2 +- src/scheduler.c | 7 +++++++ 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/components/finsh/cmd.c b/components/finsh/cmd.c index 0c06303b1..6bb8bce4d 100644 --- a/components/finsh/cmd.c +++ b/components/finsh/cmd.c @@ -116,7 +116,7 @@ static long _list_thread(struct rt_list_node *list) else if (stat == RT_THREAD_CLOSE) rt_kprintf(" close "); #if defined(ARCH_CPU_STACK_GROWS_UPWARD) - ptr = (rt_uint8_t *)thread->stack_addr + thread->stack_size; + ptr = (rt_uint8_t *)thread->stack_addr + thread->stack_size - 1; while (*ptr == '#')ptr --; rt_kprintf(" 0x%08x 0x%08x %02d%% 0x%08x %03d\n", diff --git a/src/scheduler.c b/src/scheduler.c index 14f7510e6..697df63ee 100644 --- a/src/scheduler.c +++ b/src/scheduler.c @@ -82,10 +82,17 @@ static void _rt_scheduler_stack_check(struct rt_thread *thread) { RT_ASSERT(thread != RT_NULL); +#if defined(ARCH_CPU_STACK_GROWS_UPWARD) + if (*((rt_uint8_t *)((rt_ubase_t)thread->stack_addr + thread->stack_size - 1)) != '#' || + (rt_ubase_t)thread->sp <= (rt_ubase_t)thread->stack_addr || + (rt_ubase_t)thread->sp > + (rt_ubase_t)thread->stack_addr + (rt_ubase_t)thread->stack_size) +#else if (*((rt_uint8_t *)thread->stack_addr) != '#' || (rt_ubase_t)thread->sp <= (rt_ubase_t)thread->stack_addr || (rt_ubase_t)thread->sp > (rt_ubase_t)thread->stack_addr + (rt_ubase_t)thread->stack_size) +#endif { rt_ubase_t level; -- GitLab