Skip to content

Conversation

BillXiang
Copy link

Summary of the PR

Enable CONFIG_RISCV_SBI_V01 in the riscv64 kernel—required for SBI-based
workflows such as [#352].

Requirements

Before submitting your PR, please make sure you addressed the following
requirements:

  • All commits in this PR have Signed-Off-By trailers (with
    git commit -s), and the commit message has max 60 characters for the
    summary and max 75 characters for each description line.
  • All added/changed functionality has a corresponding unit/integration
    test.
  • All added/changed public-facing functionality has entries in the "Upcoming
    Release" section of CHANGELOG.md (if no such section exists, please create one).
  • Any newly added unsafe code is properly documented.

@BillXiang
Copy link
Author

@RuoqingHe could you have a look at this one?

pushd linux
# Enable kvm module instead of inserting manually
sed -i "s|^CONFIG_KVM=.*|CONFIG_KVM=y|g" arch/riscv/configs/defconfig
echo "CONFIG_RISCV_SBI_V01=y" >> arch/riscv/configs/defconfig
Copy link
Member

@RuoqingHe RuoqingHe Sep 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is need for KVM to process SBI legacy interfaces, and put vcpu_sbi_ext_v01 into working, would you please document this line a bit, like add a comment above, and add some descriptions in the commit message 🙂

Enable legacy SBI calls and activate vcpu_sbi_ext_v01 for kvm.

Signed-off-by: BillXiang <[email protected]>
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.

2 participants