You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2021/08/25 02:42:12 UTC

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #4400: arm/armv7: correct the stack usage of exception interrupt

xiaoxiang781216 commented on pull request #4400:
URL: https://github.com/apache/incubator-nuttx/pull/4400#issuecomment-905133292


   After the discussion, it's wrong to initialize the interrupt stack in the abort handler. Because the irq handler may generate the abort and the initialization in abort handler will destroy the irq stack content. So, the possible solution is:
   
   1. Keep as before which mean abort handler will use the same stack before aborting regardless CONFIG_ARCH_INTERRUPTSTACK
   2. Add abort stack if CONFIG_ARCH_INTERRUPTSTACK > 0 like irq/fiq
   
   Since the system normally will reboot in abort handler, item 1 is a reasonable action.


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org