Skip to content

Clarify sip can contain machine interrupts even though diagram shows 0s#3165

Open
David Harris (davidharrishmc) wants to merge 1 commit into
riscv:mainfrom
davidharrishmc:patch-13
Open

Clarify sip can contain machine interrupts even though diagram shows 0s#3165
David Harris (davidharrishmc) wants to merge 1 commit into
riscv:mainfrom
davidharrishmc:patch-13

Conversation

@davidharrishmc

Copy link
Copy Markdown
Contributor

Clarify the explanation regarding MEI, MTI, and MSI in the context of registers sip and sie.

Clarify the explanation regarding MEI, MTI, and MSI in the context of registers `sip` and `sie`.

Signed-off-by: David Harris <74973295+davidharrishmc@users.noreply.github.com>
@github-actions

github-actions Bot commented Jun 1, 2026

Copy link
Copy Markdown

Normative Rule Changes Detected

This PR modifies normatively tagged text. Please review the changes below to ensure they are intentional.

View Detected Changes

Normative Tag Change Report

riscv-spec Specification

================================================================================
Tag Changes Report
================================================================================

Reference file: ref/riscv-spec-norm-tags.json
Current file: build/riscv-spec-norm-tags.json
Modified 3 tags:
  * "norm:Zawrs_wrs-sto_stall_duration":
      Reference: "insn:wrs.sto[] (WRS-with-short-timeout) is
provided that works like insn:wrs.nto[] but bounds the st..."
      Current:   "insn:wrs.sto[] (WRS-with-short-timeout) is
provided that works like insn:wrs.nto[] but bounds the st..."
  * "norm:pm_uxl_clear":
      Reference: "Setting UXL/SXL/MXL to 1 will clear the corresponding pointer masking configuration bits."
      Current:   "Setting UXL/SXL to 1 will clear the corresponding pointer masking configuration bits."
  * "norm:sip_sie_bits_sz":
      Reference: "The standard portions (bits 15:0) of registers sip and sie are formatted as shown in Figures  and  r..."
      Current:   "The standard portions (bits 15:0) of registers sip and sie are formatted as shown in Figures  and  r..."

================================================================================
Summary: 3 total changes
  Added:    0
  Deleted:  0
  Modified: 3
================================================================================

What happens next:

  • This comment is informational only and does not block merging
  • When this PR is merged, a GitHub issue will be automatically created with the NormRules label for CSC tracking
  • If these changes are unintentional, please update the PR before merging

How to update reference files (if needed):

make update-ref
git add ref/*.json
git commit -m "Update normative tag reference files"

Note: New tags (additions) are automatically added to the reference files when PRs are merged to main. Only modifications and deletions require manual review.

This comment was automatically generated by the normative tag check workflow.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is true, but I don't think it's how it should be exposed in the supervisor spec. They're effectively custom interrupts at this level, rather than well-defined standard ones. I'll think about it and revisit this later.

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