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

riscv64: use SBI SRST extension instead of legacy SBI for system shutdown #47

Merged
merged 1 commit into from
Oct 24, 2024

Conversation

luojia65
Copy link
Contributor

@luojia65 luojia65 commented Oct 24, 2024

Legacy SBI have been deprecated years ago (on Jun 9, 2021). Modern RISC-V SBI firmware would use SBI SRST extension to shutdown an S-mode environment; we use sbi-rt crate for this purpose.

Unused feature gate opensbi is removed, as RISC-V SBI support does not usually satisfy which implementation we'd use, unless e.g. fixing implementation-specific bugs.

QEMU RISC-V test result (shutdown procedure works as expected):
图片

r? @gmacd

…down

Legacy SBI have been deprecated years ago (on Jun 9, 2021). Modern
RISC-V SBI firmware would use SBI SRST extension to shutdown an S-mode
environment; we use `sbi-rt` crate for this purpose. Unused feature gate
`opensbi` is removed, as RISC-V SBI support does not usually satisfy
which implementation we'd use, unless e.g. fixing implementation-specific
bugs.

Signed-off-by: Zhouqi Jiang <[email protected]>
@orangecms
Copy link
Contributor

Awesome, thank you! :)

@gmacd gmacd merged commit 4f56ee9 into r9os:main Oct 24, 2024
4 checks passed
@gmacd
Copy link
Contributor

gmacd commented Oct 24, 2024

Thanks for the PR! (I hope you don't mind me merging)

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

Successfully merging this pull request may close these issues.

3 participants