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

Posted to commits@nuttx.apache.org by "wengzhe (via GitHub)" <gi...@apache.org> on 2023/07/12 09:12:05 UTC

[GitHub] [nuttx] wengzhe opened a new pull request, #9794: arch/arm64: Enable FPU for qemu-armv8a:netnsh_smp and netnsh_smp_hv

wengzhe opened a new pull request, #9794:
URL: https://github.com/apache/nuttx/pull/9794

   ## Summary
   When reading https://github.com/apache/nuttx-apps/pull/1793, I found a previous problem https://github.com/apache/nuttx-apps/pull/1454#discussion_r1045718067 now still exist on current master (`inf` in `iperf` bandwidth).
   
   By step-by-step debugging, I found a value of `1000000.0` stored in register `d8` is overwritten as `0` after context switch, then `fdiv` results in `inf` because of divided by `0`. Finally I found that `ARCH_FPU` is not enabled while compiler is using FPU, and may lose values in floating-point registers.
   
   ## Impact
   `qemu-armv8a:netnsh_smp` and `qemu-armv8a:netnsh_smp_hv`
   
   ## Testing
   Manually
   


-- 
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] [nuttx] acassis commented on pull request #9794: arch/arm64: Enable ARCH_FPU for qemu-armv8a:netnsh_smp and netnsh_smp_hv

Posted by "acassis (via GitHub)" <gi...@apache.org>.

acassis commented on PR #9794:
URL: https://github.com/apache/nuttx/pull/9794#issuecomment-1632492968

   Nice finding @wengzhe !!!


-- 
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] [nuttx] acassis merged pull request #9794: arch/arm64: Enable ARCH_FPU for qemu-armv8a:netnsh_smp and netnsh_smp_hv

Posted by "acassis (via GitHub)" <gi...@apache.org>.

acassis merged PR #9794:
URL: https://github.com/apache/nuttx/pull/9794


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