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

[BUG] Division Operations made an error in calculating -1/1 #27

Open
1 task done
WEIhabi opened this issue Jun 20, 2023 · 0 comments
Open
1 task done

[BUG] Division Operations made an error in calculating -1/1 #27

WEIhabi opened this issue Jun 20, 2023 · 0 comments

Comments

@WEIhabi
Copy link

WEIhabi commented Jun 20, 2023

Is there an existing CVA6 bug for div?

  • I have searched the existing bug issues

Bug Description

Hi, when we performing correctness checks on the division ISA using a formal tool, we discovered that in the serdiv module, when encountering the case of -1/1, the answer is incorrectly computed as 0 instead of the expected correct value of -1.

We have also investigated this issue on the original master of CVA6 and found that someone has already proposed a solution and made improvements to address this problem. You can refer to:
openhwgroup#421

The following is a comparison between the error results obtained from checking the DUV using JasperGold FPV and the golden answer we wrote for SVA.

DUV error
SVA golden answer

@WEIhabi WEIhabi changed the title [BUG] <title>Division Operations made an error in calculating -1/1 [BUG] Division Operations made an error in calculating -1/1 Jun 20, 2023
maxbjurling added a commit to planvtech/cva6_pulp that referenced this issue Oct 11, 2023
…servation-erroneously-cleared-by-writeback

Wait for completion on write back before AMO
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant