Skip to content

Add unratified Smpmpind Smcfiss Smucfiss#2213

Draft
ved-rivos wants to merge 1 commit intoriscv-software-src:masterfrom
ved-rivos:smcfiss
Draft

Add unratified Smpmpind Smcfiss Smucfiss#2213
ved-rivos wants to merge 1 commit intoriscv-software-src:masterfrom
ved-rivos:smcfiss

Conversation

@ved-rivos
Copy link
Copy Markdown
Contributor

@ved-rivos ved-rivos commented Jan 18, 2026

Spec: https://github.com/ved-rivos/riscv-isa-manual/blob/smpmpss/src/smcfiss.adoc
PoC: https://github.com/ved-rivos/opensbi/tree/smcfiss

$ cd opensbi
$ make PLATFORM=generic
$ spike --isa=rv64gc_zimop_zcmop_zaamo_zicfiss_zicfilp_smcsrind_smpmpind_smcfiss_smucfiss \
              --priv=mu \
              build/platform/generic/firmware/fw_payload.elf

Needs Zicfiss/Zicfilp enabled toolchain to build opensbi.

$ git clone git@github.com:sifive/riscv-gnu-toolchain.git -b cfi-dev
$ riscv-gnu-toolchain/configure --prefix=<path-to-where-to-build> \
       --with-arch=rv64gc_zicfilp_zicfiss --enable-linux --disable-gdb  \
       --with-extra-multilib-test="rv64gc_zicfilp_zicfiss-lp64d:-static"
$ make -j$(nproc)

encoding.h was hand edited as CSR numbers and new field positions are not yet assigned.

RFC only; not requesting to be merged.

Copy link
Copy Markdown
Collaborator

@aswaterman aswaterman left a comment

Choose a reason for hiding this comment

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

LGTM overall. I recommend changing the PR to Draft status until you think it should be considered for merging (which, of course, presupposes that encoding.h has been auto-generated).

@ved-rivos ved-rivos marked this pull request as draft January 23, 2026 23:07
@ved-rivos ved-rivos marked this pull request as ready for review January 23, 2026 23:07
@ved-rivos ved-rivos marked this pull request as draft January 23, 2026 23:07
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