ptka commented on a change in pull request #5401:
URL: https://github.com/apache/incubator-nuttx/pull/5401#discussion_r807420665
##########
File path: arch/sim/src/sim/up_initialstate.c
##########
@@ -84,10 +97,10 @@ void up_initial_state(struct tcb_s *tcb)
* the return address.
*/
- tcb->xcp.regs[JB_SP] = (xcpt_reg_t)tcb->stack_base_ptr +
- tcb->adj_stack_size -
- sizeof(xcpt_reg_t);
+ tcb->xcp.regs[JB_SP] = (xcpt_reg_t)
+ STACK_ALIGN_DOWN((long)tcb->stack_base_ptr + tcb->adj_stack_size - 1);
Review comment:
I tested https://github.com/apache/incubator-nuttx/pull/5507 in
combination with this test on MacBook Air M1 2020 running Monterey 12.2.1.
Executed test: sim:nsh with additional ostest configured. Result: complete
ostest passed w/o error.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
