[GitHub] [incubator-nuttx] hotislandn opened a new pull request #2918: arch:rv32:up_sigdeliver missing fpu contexts.

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

hotislandn opened a new pull request #2918:
URL: https://github.com/apache/incubator-nuttx/pull/2918


   Signed-off-by: hotislandn <ho...@hotmail.com>
   
   ## Summary
   This is a bug fix that is reported on bl602evb with FPU enabled.
   When up_sigdeliver() is called, the up_copystate() will only copy the contents of int regs, since "FS" flag is "clean". Which leads to an invalid value in the stack that will be pop as 'fcsr' later in context restore. From now on, when the CPU encounters the float instruction, it will raise an "invalid instruction" fault.
   
   ## Impact
   rv32 targets which utils the signal deliver logic.
   
   ## Testing
   Tested and verified on bl602evb, it runs normally with this patch.
   


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

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

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #2918: arch:rv32:up_sigdeliver missing fpu contexts.

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

xiaoxiang781216 merged pull request #2918:
URL: https://github.com/apache/incubator-nuttx/pull/2918


   


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

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