cva6 icon indicating copy to clipboard operation
cva6 copied to clipboard

[Pmp] Verif plan TXT

Open CoralieAllioux opened this issue 1 year ago • 4 comments

This PR aims at resolving task https://github.com/openhwgroup/cva6/issues/1447

Here's a first version of the Verification Plan of PMP, in txt format. It is split in 2 files:

  • verif/docs/VerifPlans/PMP/pmp_verif_plan.txt
  • verif/docs/VerifPlans/PMP/pmp_verif_plan_features.txt

VPTool format is undergoing on our side. We'd like to share already the content to get first feedbacks.

CoralieAllioux avatar Aug 20 '24 09:08 CoralieAllioux

:heavy_check_mark: successful run, report available here.

github-actions[bot] avatar Aug 20 '24 09:08 github-actions[bot]

👋 Hi there!

This pull request seems inactive. Need more help or have updates? Feel free to let us know. If there are no updates within the next few days, we'll go ahead and close this PR. 😊

github-actions[bot] avatar Sep 20 '24 02:09 github-actions[bot]

Hi @JeanRochCoulon, can you give a look?

jquevremont avatar Sep 20 '24 07:09 jquevremont

👋 Hi there!

This pull request seems inactive. Need more help or have updates? Feel free to let us know. If there are no updates within the next few days, we'll go ahead and close this PR. 😊

github-actions[bot] avatar Oct 21 '24 02:10 github-actions[bot]

:heavy_check_mark: successful run, report available here.

github-actions[bot] avatar Nov 08 '24 09:11 github-actions[bot]

Added to this pull request are some PMP tests along with the Verif Plan integrated in VP_TOOL. Weird that CI is failing as no RTL has been updated and the PMP tests are not integrated into the regression scripts

OlivierBetschi avatar Nov 08 '24 13:11 OlivierBetschi

:heavy_check_mark: successful run, report available here.

github-actions[bot] avatar Nov 25 '24 07:11 github-actions[bot]

The PMP updated has been merged ( https://github.com/openhwgroup/cva6/pull/2528). @ASintzoff @JeanRochCoulon does the VP_TOOL update and provided tests look ok to you ? I have no idea why the ECA check is failing as I have signed the agreement. It seems to come from the latest merge of master.

OlivierBetschi avatar Dec 04 '24 09:12 OlivierBetschi

This PR modifies 50k lines: too much. Maybe a rebase issue. This PR should execute tests on the new RTL with extracted PMP. Let's see the results !

JeanRochCoulon avatar Dec 04 '24 09:12 JeanRochCoulon

:heavy_check_mark: successful run, report available here.

github-actions[bot] avatar Dec 04 '24 10:12 github-actions[bot]

I found. This PR is related to master_when_no_mmu, while it should be master.

JeanRochCoulon avatar Dec 04 '24 11:12 JeanRochCoulon

I opened a new PR against master : https://github.com/openhwgroup/cva6/pull/2648 I think we can close this one

OlivierBetschi avatar Dec 04 '24 11:12 OlivierBetschi