Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Floating point multipication issue in RISCV cva6 #1416

Open
spidugu444 opened this issue Aug 11, 2022 · 4 comments
Open

Floating point multipication issue in RISCV cva6 #1416

spidugu444 opened this issue Aug 11, 2022 · 4 comments
Assignees
Labels
notCV32A65X It is not an CV32A65X issue PARAM:FPU Issue depends on the FPU parameter

Comments

@spidugu444
Copy link
Contributor

Hi All,

In CVA6 riscv-arch-test suite fmul tests are failing due to incorrect calculation as shown in below attached screenshot.

image

As shown in above in line 157 and 182 expected result is 0,as it is multiplied with zero but in verilator it is giving value as 80000000 where as in spike it is calculating correct result as 0.

@MikeOpenHWGroup
Copy link
Member

Thanks for this issue @spidugu444. I have assigned it to @JeanRochCoulon, the verification lead for the CVA6. As far as I know, the CVA6 team is not planning to support floating point operations in the initial release of CVA6, so this bug may not be a priority.

FYI, the floating point IP that is used by the CVA6 (and CV32E40P), known as fpnew, has recently been transferred from the pulp-platform team to the OpenHW Group and re-branded as cvfpu. The OpenHW Group is starting a complete verification of this IP now.

The first user of cvfpu will be the v2.0.0 release of CV32E40P. It is my expectation that the CVA6 team will integrate the fully verified cvfpu into CVA6, but I do not know the timeline for this. In the meantime, I will created an issue on the cvfpu repo to link to this issue.

@JeanRochCoulon
Copy link
Contributor

Thank for this feedback.
You are using Verilator, but do you have the bug when using VCS ? If no, which Verilator version do you use?

@spidugu444
Copy link
Contributor Author

@JeanRochCoulon fmul test is failed on VCS also, currently I am using verilator-4.110 version.

@MikeOpenHWGroup
Copy link
Member

I have opened CVFPU issue 63 for this issue. Please track debug of this problem in that issue.

@MikeOpenHWGroup MikeOpenHWGroup transferred this issue from openhwgroup/core-v-verif Sep 14, 2023
Abdulwadoodd pushed a commit to 10x-Engineers/cva6 that referenced this issue Sep 15, 2023
@JeanRochCoulon JeanRochCoulon added the PARAM:FPU Issue depends on the FPU parameter label Dec 6, 2023
@JeanRochCoulon JeanRochCoulon added the notCV32A65X It is not an CV32A65X issue label Apr 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
notCV32A65X It is not an CV32A65X issue PARAM:FPU Issue depends on the FPU parameter
Projects
None yet
Development

No branches or pull requests

3 participants