On Sat, Feb 6, 2021 at 7:38 AM Gregory Nutt <spudan...@gmail.com> wrote:

>
> > I am using NuttX-v10.0.1.
> > This is fixed in 625eef20f00fbe9bdb4995ad9dfb24b71227e954, which came at
> a
> > later time.
>
> Hmm.. I guess I still don't understand the failure.  That change was:
>
>       #ifdef CONFIG_STACK_COLORATION
>     -  if (tcb->pid != 0)
>     -    {
>     -      memset(tcb->stack_alloc_ptr, 0xaa, stack_size);
>     -    }
>     +  memset(tcb->stack_alloc_ptr, 0xaa, stack_size);
>       #endif
>
> pid == 0 is the startup, IDLE thread.  That is not the thread that you
> are creating.  So there is more involved.
>
>
It isn't a right to place to check pid == 0 here, because sched set pid to
the real value after arm_usestack return.
PR has more info: https://github.com/apache/incubator-nuttx/pull/2077


> > I just tried master, which brought the expected results.
>
> Well that is really all that is important,
>
>
>

Reply via email to