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

[rv_core_ibex] V1 Signoff #21023

Closed
msfschaffner opened this issue Jan 25, 2024 · 2 comments
Closed

[rv_core_ibex] V1 Signoff #21023

msfschaffner opened this issue Jan 25, 2024 · 2 comments
Assignees
Labels
Component:DV DV issue: testbench, test case, etc. IP:rv_core_ibex Type:Signoff

Comments

@msfschaffner
Copy link
Contributor

Description

Ensure V1 signoff criteria are fulfilled after focus area changes have landed.

@msfschaffner msfschaffner added this to the Earlgrey-PROD.M2 milestone Jan 25, 2024
@msfschaffner msfschaffner added the Component:DV DV issue: testbench, test case, etc. label Jan 25, 2024
@msfschaffner msfschaffner removed their assignment Feb 22, 2024
@hcallahan-lowrisc hcallahan-lowrisc self-assigned this Feb 27, 2024
@andreaskurth
Copy link
Contributor

@GregAC @hcallahan-lowrisc: Looking through V1 checklist items, I think only TESTPLAN_COMPLETED needs a closer look, right? Then, as described in the D2 signoff analysis, there were only 3 changes with actual RTL impact: two security fixes (lowRISC/ibex@8ec0c6f1 and lowRISC/ibex@35bbdb7b) and the removal of the S&P layer from scrambling (lowRISC/ibex@56413ecf). AFAIK from a functionality perspective, all these changes are well covered by existing tests, so the testplan doesn't need to be extended.

If you agree, I suggest closing this as completed.

@GregAC
Copy link
Contributor

GregAC commented Mar 28, 2024

@andreaskurth I agree, so V1 criteria remain fulfilled.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component:DV DV issue: testbench, test case, etc. IP:rv_core_ibex Type:Signoff
Projects
None yet
Development

No branches or pull requests

5 participants