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

Fix WFI resume on mstatus.MIE disabled #735

Merged
merged 2 commits into from
Oct 15, 2024

Conversation

piotro888
Copy link
Member

@piotro888 piotro888 commented Oct 8, 2024

Depends on #729 (see last commit for diff)

WFI should continue to next instruction if interrupt is pending and unmasked via mie, but global interrupt enable mstatus.MIE bit is disabled.
[Priv spec 3.3.3]

Previously it only triggers interrupt handler when mstatus.MIE is set, and waits otherwise.

@piotro888 piotro888 added the bug Something isn't working label Oct 8, 2024
@piotro888
Copy link
Member Author

rebased

@tilk tilk merged commit c3edca0 into kuznia-rdzeni:master Oct 15, 2024
13 checks passed
github-actions bot pushed a commit that referenced this pull request Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants