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.

Sign up for GitHub

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

Jump to bottom

Decoding LWU as a RV32 instruction #1342

Closed
1 task done
AyoubJalali opened this issue Aug 23, 2023 · 3 comments
Closed
1 task done

Decoding LWU as a RV32 instruction #1342

AyoubJalali opened this issue Aug 23, 2023 · 3 comments
Assignees
@AyoubJalali
Labels
Type:Bug For bugs in the RTL, Documentation, Verification environment or Tool and Build system

Comments

@AyoubJalali
Copy link
Contributor

Is there an existing CVA6 bug for this?

  • I have searched the existing bug issues

Bug Description

Hello,
I try inject some RV64 instruction with RV32, just to be sure that CVA6 raise an exception on RV64 instruction because the STEP1 configuration is a RV32, so I notice that CVA6 is decoding LWU as RV32 instruction.
image
to reproduce the BUG you just need to inject a LWU (in binary format) for example try to add in your test.S -> ### .4byte 0x6f9a6f83
@AyoubJalali AyoubJalali added the Type:Bug For bugs in the RTL, Documentation, Verification environment or Tool and Build system label Aug 23, 2023
@JeanRochCoulon
Copy link
Contributor

Hello @AyoubJalali
The merged PR #1344 fix the issue, can you confirm it and close the issue ?

@AyoubJalali
Copy link
Contributor Author

Great You Merge the Fix
Yes I confirm the PR #1344 fix the issue

@AyoubJalali
Copy link
Contributor Author

The bug is fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AyoubJalali AyoubJalali

Labels
Type:Bug For bugs in the RTL, Documentation, Verification environment or Tool and Build system
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@JeanRochCoulon @ASintzoff @AyoubJalali