riscv-non-isa / riscv-sbi-doc

Documentation for the RISC-V Supervisor Binary Interface
https://jira.riscv.org/browse/RVG-49
Creative Commons Attribution 4.0 International
356 stars 91 forks source link

Add reset function #26

Closed aurel32 closed 4 years ago

aurel32 commented 5 years ago

The SBI specification provides a shutdown function, but no reset function. Reset support has been added recently to the QEMU "RISC-V VirtIO Board" machine. Would it be possible to add such a function to the SBI specifications? It's not clear to me what process has to be followed to update the specifications.

atishp04 commented 5 years ago

We are in the process of sending a patch to add the reset function. I will send the PR in couple of days.

On Sun, Oct 6, 2019 at 5:16 AM aurel32 notifications@github.com wrote:

The SBI specification provides a shutdown function, but no reset function. Reset support has been added recently to the QEMU "RISC-V VirtIO Board" machine. Would it be possible to add such a function to the SBI specifications? It's not clear to me what process has to be followed to update the specifications.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/riscv/riscv-sbi-doc/issues/26?email_source=notifications&email_token=AA6CNH64HUKQU64CQTRQVDTQNHJIVA5CNFSM4I53XWV2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HP4SZTQ, or mute the thread https://github.com/notifications/unsubscribe-auth/AA6CNHYVMFCTUGXYEBOGNFLQNHJIVANCNFSM4I53XWVQ .

-- Regards, Atish

atishp04 commented 5 years ago

@aurel32 : Reset function added https://github.com/riscv/riscv-sbi-doc/pull/27

aurel32 commented 5 years ago

@atishp04 Thanks a lot, this looks all good (I only checked the shutdown/reboot part).

shlevy commented 4 years ago

Looks like this got removed before that PR was merged?

avpatel commented 4 years ago

We had intentionally kept reset extension out of SBI v0.2.

Now that SBI v0.2 is tagged and ready to be ratified, I will send SBI reset extension patch again for review.

Regards, Anup

atishp04 commented 4 years ago

The reset extension is under discussion and there is a PR available #39 . Closing this issue.