On Sat, Feb 6, 2021 at 7:38 AM Gregory Nutt <[email protected]> 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,
>
>
>
