feat: use RustSBI 0.4.0 proc macros to simplify VCpu SBI implementation

[luojia65]

This pull request simplifies SBI implementation in RISCVVCpu by using RustSBI 0.4.0 Forward structure, it forwards all SBI calls on console, pmu, fence, reset and info (base extension) without missing any SBI features like extension detection and all functions in the extension.

For example, the original code only forwards remote_fence_i and remote_sfence_vma in fence (RFNC) extension, but by reusing RustSBI Forward feature we forward all 7 functions in the RFNC extension.

By using RustSBI 0.4.0 Forward, we save a lot of code on RISC-V hypervisor development.

Ref: https://github.com/KuangjuX/hypercraft/pull/3/files

r? @hky1999

[hky1999]

Thanks for your work, I'll review it later

[hky1999]

except for naked_asm modification, all LGTM

[luojia65]

except for naked_asm modification, all LGTM

Thanks for review! For naked_asm, I have two solutions to this problem.

The first approach is using the rustversion crate, as is mentioned in this pull request: rcore-os/buddy_system_allocator#3. The second approach is just reverting it to asm! as we only build this project under ArceOS toolchain version.

What's your opinion on this? @hky1999

[hky1999]

except for naked_asm modification, all LGTM

Thanks for review! For naked_asm, I have two solutions to this problem.

The first approach is using the rustversion crate, as is mentioned in this pull request: rcore-os/buddy_system_allocator#3. The second approach is just reverting it to asm! as we only build this project under ArceOS toolchain version.

What's your opinion on this? @hky1999

Sorry for delay, a bit busy these days... (so much email send from github I missed this

The first approach is using the rustversion crate, as is mentioned in this pull request: rcore-os/buddy_system_allocator#3.

It's a good solution but I'm worried that this will further increase the complexity of the code

The second approach is just reverting it to asm! as we only build this project under ArceOS toolchain version.

Let's just go back to this solution, after that I think this PR is ready to merge

[hky1999]

also, pay attention to the ci error @luojia65

(#![feature(asm_const)] is needed in the “outdated“ of rust toolchain

[luojia65]

@hky1999: Thanks! Will modify soon.

[hky1999]

LGTM
thanks for your work