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.

Sign up for GitHub

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

Jump to bottom

CV32E40Pv2 Verification update Week 19 PR #2425

Merged
merged 25 commits into from
May 10, 2024
Merged

CV32E40Pv2 Verification update Week 19 PR #2425

merged 25 commits into from
May 10, 2024

Conversation

dd-BeeNee
Copy link

This Pull Request contains the work from this week (ww19). Contributors are @pascalgouedo, @dd-baoshan, @dd-BeeNee

The main updates are:

  1. update license header (as pointed out in last PR)
  1. Update code coverage waiver for F config following more detailed analysis
  1. correction from regression debug:
  1. Update test to improve function coverage
  1. revert v1 DvPlan back to their original v1 state (of v1 commit HASH 0cc43af), then consolidate all delta of v2 changes by Xavier into v2 DvPlan. From this onward, the v1 DvPlan are back to their clean state, and we will touch just the v2 DvPlan moving forward.

Pascal Gouedo and others added 25 commits May 3, 2024 16:43
Corrected ex_apu_valid_memorised generation for cp_f_multicycle_clk_w…
12ddce5 
…indow coverpoint.

Signed-off-by: Pascal Gouedo <pascal.gouedo@dolphin.fr>
Added 20 cycles bin for cp_f_multicycle_clk_window coverpoint when ap…
2e63927 
…u_rdata is memorized just for 1 cycle.

Signed-off-by: Pascal Gouedo <pascal.gouedo@dolphin.fr>
Merge pull request #246 from XavierAubert/cv32e40p/dev_dd_pgo
296c894 
Corrections for cp_f_multicycle_clk_window coverpoint
Moved wfi_mem_stress tests at the beginning of the execution list as …
17b999d 
…only few occurences are ultra-long.

Will let other tests execute in parallel.
Should reduce overall non-re time.

Signed-off-by: Pascal Gouedo <pascal.gouedo@dolphin.fr>
Merge pull request #247 from XavierAubert/cv32e40p/dev_dd_pgo
de94f79 
Moved wfi_mem_stress tests at the beginning of the execution list as …
Reverted PR #246 as it created a lot of errors in non-reg.
ecbd51d 
Need more analysis to correct it.

Signed-off-by: Pascal Gouedo <pascal.gouedo@dolphin.fr>
Merge pull request #248 from XavierAubert/cv32e40p/dev_dd_pgo
246fc95 
Reverted PR #246 as it created a lot of errors in non-reg.
@dd-BeeNee
Update license header
4fa94a0 
Signed-off-by: Bee Nee Lim <beenee.lim@dolphin.fr>
@dd-BeeNee
Merge pull request #249 from XavierAubert/cv32e40p/dev_bnl_ww18
9f60f65 
Update license header
@dd-BeeNee
Update code coverage waiver for F config after review
22d0a80 
Signed-off-by: Bee Nee Lim <beenee.lim@dolphin.fr>
@dd-BeeNee
Merge pull request #250 from XavierAubert/cv32e40p/dev_bnl_ww18a
89e6bc1 
Update code coverage waiver for F config after review
@dd-BeeNee
Update code coverage waiver for CFG_P_F0 after review.
a98537b 
Signed-off-by: Bee Nee Lim <beenee.lim@dolphin.fr>
@dd-BeeNee
Merge pull request #251 from XavierAubert/cv32e40p/dev_bnl_ww18b
664f754 
Update code coverage waiver for CFG_P_F0 after review.
@dd-baoshan
Add workaround for failing test on nested interrupt event
2e24a8e 
Signed-off-by: dd-baoshan <baoshan.mak@dolphin.fr>
@dd-baoshan
Merge pull request #252 from XavierAubert/cv32e40p/bsm-update_tb_file…
3a97fbc 
…s_WW18

Add workaround for failing test on nested interrupt event
@dd-BeeNee
Revert "Clean-up/Updates done on dvplan which some of them related to…
3e64498 
… functional coverage mapping"

This reverts commit 594cb7d.
@dd-BeeNee
Revert "updated external-debugger Vplan"
499cd28 
This reverts commit a332ec0.
@dd-BeeNee
Revert "Interrupt verif plan update for v2"
acf6cbd 
This reverts commit b3c633c.
@dd-baoshan
Update macro for fpu and zfinx functional coverage
57d5c77 
Signed-off-by: dd-baoshan <baoshan.mak@dolphin.fr>
@dd-baoshan
Update tests to improve functional coverage holes (uvme_cv32e40p_fp/z…
9eb8029 
…finx_instr_covg

Signed-off-by: dd-baoshan <baoshan.mak@dolphin.fr>
@dd-baoshan
Merge pull request #253 from XavierAubert/cv32e40p/bsm-update_tb_file…
2a81216 
…s_WW18

Cv32e40p/bsm update tb files ww18
@dd-BeeNee
to revert v1 DvPlan back to their original v1 state (of commit HASH 0…
817abf9 
…cc43af)

Signed-off-by: Bee Nee Lim <beenee.lim@dolphin.fr>
@dd-BeeNee
to consolidate all changes by Xavier into v2 DvPlan
239b720 
Signed-off-by: Bee Nee Lim <beenee.lim@dolphin.fr>
@dd-BeeNee
to revert DvPlan back to their original v1 state (remove redundant fi…
538deae 
…les)

Signed-off-by: Bee Nee Lim <beenee.lim@dolphin.fr>
@dd-BeeNee
Merge pull request #254 from XavierAubert/cv32e40p/dev_bnl_ww18c
3bb659a 
revert v1 DvPlan back to their original v1 state (of commit HASH 0cc43af), then consolidate all v2 changes by Xavier into v2 DvPlan file
@dd-BeeNee dd-BeeNee requested a review from MikeOpenHWGroup May 10, 2024 14:23
MikeOpenHWGroup
MikeOpenHWGroup approved these changes May 10, 2024
View reviewed changes
Copy link
Member

@MikeOpenHWGroup MikeOpenHWGroup left a comment

Choose a reason for hiding this comment

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

LGTM and it passes a local CI regression. Thanks for taking care of the copyright declaration on debug_hwloop_test/trigger_code.S.

@MikeOpenHWGroup MikeOpenHWGroup merged commit 88586e7 into openhwgroup:cv32e40p/dev May 10, 2024
1 check passed
@XavierAubert XavierAubert mentioned this pull request May 17, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MikeOpenHWGroup MikeOpenHWGroup MikeOpenHWGroup approved these changes

Assignees
No one assigned
Labels
cv32e40p
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dd-BeeNee @MikeOpenHWGroup @dd-baoshan