Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Trap DV plan: first version #1623

Merged
merged 2 commits into from
Nov 13, 2023

Conversation

ASintzoff
Copy link
Contributor

Trap DV plan

Copy link
Contributor

✔️ successful run, report available here.

4 similar comments
Copy link
Contributor

✔️ successful run, report available here.

Copy link
Contributor

✔️ successful run, report available here.

Copy link
Contributor

✔️ successful run, report available here.

Copy link
Contributor

✔️ successful run, report available here.

@JeanRochCoulon JeanRochCoulon merged commit 97b6969 into openhwgroup:master Nov 13, 2023
13 checks passed
@ASintzoff ASintzoff deleted the trap_dvplan branch November 14, 2023 08:58
* **Comments**

Covered by ISACOV tests, not yet in ISACOV DV plan
### Sub-feature: 001_mtval
Copy link
Contributor

Choose a reason for hiding this comment

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

mtval should be in item not sub-feature. Should be updated for all following features and for all CSRs like mtvec, mtval, mepc, ...

Copy link
Contributor

Choose a reason for hiding this comment

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

Interruptions feature section is missing.

* **Verification Goals**

Check that when any illegal instruction exception is raised, `mtval` CSR contains the faulting instruction.
* **Pass/Fail Criteria:** NDY (Not Defined Yet)
Copy link
Contributor

Choose a reason for hiding this comment

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

Pass/Fail Criteria/Test Type and Coverage Method should be filed. Same for all not defined items.

@jquevremont jquevremont removed their request for review December 6, 2023 14:45
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.

3 participants