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 2022/01/27 06:49:21 UTC
[GitHub] [incubator-nuttx] anchao opened a new pull request #5355: cortex-m/doirq: do not update the CURRENT_REGS on nested interrupt handling
anchao opened a new pull request #5355:
URL: https://github.com/apache/incubator-nuttx/pull/5355
## Summary
cortex-m/doirq: do not update the CURRENT_REGS on nested interrupt handling
current implementation incorrectly update CURRENT_REGS to interrupt context if
trigger nested interrupt, (e.g, hard fault occurs during interrupt handling)
this would ambiguous for programs using CURRENT_REGS, this patch will prohibit
the update of CURRENT_REGS on on nested interrupt handling
Signed-off-by: chao.an <an...@xiaomi.com>
## Impact
Nested interrupt handling
## Testing
trigger hard fault on interrupt context and check the CURRENT_REGS
--
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
[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5355: cortex-m/doirq: do not update the CURRENT_REGS on nested interrupt handling
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on pull request #5355:
URL: https://github.com/apache/incubator-nuttx/pull/5355#issuecomment-1024066480
Should we fix other arch too?
--
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
[GitHub] [incubator-nuttx] anchao commented on pull request #5355: cortex-m/doirq: do not update the CURRENT_REGS on nested interrupt handling
Posted by GitBox <gi...@apache.org>.
anchao commented on pull request #5355:
URL: https://github.com/apache/incubator-nuttx/pull/5355#issuecomment-1024132582
> Should we fix other arch too?
let me do more testing on other platforms, this PR just for arm
--
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
[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5355: cortex-m/doirq: do not update the CURRENT_REGS on nested interrupt handling
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 merged pull request #5355:
URL: https://github.com/apache/incubator-nuttx/pull/5355
--
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