[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on pull request #5261: xtensa: add svccall handler and use svc call in context switch

[GitBox <gi...@apache.org>]

xiaoxiang781216 edited a comment on pull request #5261:
URL: https://github.com/apache/incubator-nuttx/pull/5261#issuecomment-1017088834


   > @xiaoxiang781216 I will give this a good look during this week end if you can wait. Thanks.
   
   Sure, this is the most tough bug we hit on xtensa. The race happen here:
   https://github.com/apache/incubator-nuttx/pull/5261/files#diff-b1fb00346f50b11c84c36c75d51d0ae1551da08ed43959e4af932c7f6aac35d0L582-L586
   Line 582 restore PS(interrupt enable again) and the rest restore may be interrupted by the pending IRQ.
   The fix we want try is trigger a syscall call(like cortex-m or riscv) and reuse the interrupt handle process.
   The best trigger method is syscall intruction, but as @zhuyanlinzyl mention before, syscall block in the critical section, so we have to replace syscall with swint. You may have a better solution to fix this problem.


-- 
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