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] External supervisor mode interrupt #2056

Closed
1 task done
AyoubJalali opened this issue Apr 18, 2024 · 3 comments · Fixed by #2716
Closed
1 task done

[BUG] External supervisor mode interrupt #2056

AyoubJalali opened this issue Apr 18, 2024 · 3 comments · Fixed by #2716
Assignees
Labels
CV32A60AX Part: Application configuration (former "step1") CV32A65X Part: Embedded configuration 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,
This a CV32A65X RTL bug, as the core configuration does not support S-mode, so should not support S-mode interrupt, but the CV32A65X has an input irq_i[0] for Supervisor external interrupt, when irq_i[0] is assert the MIP.SEIP is asserted, that the normal behavior if we support the S-mode, but the CV32A65X does not.

@AyoubJalali AyoubJalali added Type:Bug For bugs in the RTL, Documentation, Verification environment or Tool and Build system CV32A60AX Part: Application configuration (former "step1") CV32A65X Part: Embedded configuration labels Apr 18, 2024
@JeanRochCoulon
Copy link
Contributor

Great !

@Gchauvon
Copy link
Contributor

I can't reproduce this issue. Asserting irq_i[0] only raise MIP.MEIP and MIP.SEIP stay at 0.
The RTL seems fine since PR #2017 which was merged few weeks before your issue.

Can we close this issue ?

@AyoubJalali
Copy link
Contributor Author

AyoubJalali commented Jan 15, 2025

You should assert irq_i[1] in UVM TB so that you can see that MIP.SEIP is asserted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CV32A60AX Part: Application configuration (former "step1") CV32A65X Part: Embedded configuration Type:Bug For bugs in the RTL, Documentation, Verification environment or Tool and Build system
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants