Skip to content

Update diagrams and minor fix to text#17

Merged
pperesse merged 1 commit intoriscv:mainfrom
geoffthorpe-mips:diagrams
Feb 16, 2026
Merged

Update diagrams and minor fix to text#17
pperesse merged 1 commit intoriscv:mainfrom
geoffthorpe-mips:diagrams

Conversation

@geoffthorpe-mips
Copy link

  • the diagrams now show transactions from all privilege levels
  • improved visual consistency
  • correction that Smlwidlist depends on Smlwid, not Smwid. (Though if Smwid is not enabled, then mlwidlist can't be lockable.)

@pperesse
Copy link
Collaborator

Similar issues are there in the RISC-V ISA World-ID Extensions section at the beginning, where we provide an overview and list of all the extensions.

The Smlwidlist extension depends on the Smwid extension

and

The only extension that is dependent on Smwid is Smlwidlist, the other Worlds extensions have no dependence on Smwid. If the Smwid extension is not enabled, the mwid CSR does not exist. If Smwid is enabled, the platform determines whether mwid is locked or unlocked on-reset.

Also, since you have a PR open, can we mention in this section the pinstraps associated with each extension? This is a feedback that was made to me directly which I think will provide clarity.

For instance:
With the Smwid extension, the mwid CSR is introduced to provide a lockable control of the WID value assigned to the hart and a platform-defined value pmwidlist may limit the set of WIDs that can be used with mwid.

The Smlwid extension introduces the mlwid CSR to control the WID of lower-privilege modes and a platform-defined value pmlwidlist may limit the set of WIDs that can be used with mlwid.

- the diagrams now show transactions from all privilege levels
- improved visual consistency
- correction that Smlwidlist depends on both Smlwid and Smwid.

Signed-off-by: Geoffrey Thorpe <gthorpe@mips.com>
@geoffthorpe-mips
Copy link
Author

I've (force-)pushed an update. Smlwidlist is declared as dependent on both Smwid and Smlwid, and I've added the requested clarifications on the platform values.

Copy link
Collaborator

@pperesse pperesse left a comment

Choose a reason for hiding this comment

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

Thanks @geoffthorpe-mips . Looks good to me

@pperesse pperesse merged commit 14cfec9 into riscv:main Feb 16, 2026
3 checks passed
@geoffthorpe-mips geoffthorpe-mips deleted the diagrams branch February 17, 2026 21:11
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