Skip to content

Fix for issue 325

Eclipse Webmaster requested to merge github/fork/davideschiavone/fix325 into master

Created by: davideschiavone

This should solve the issue #325 (closed)

I tried this on the core-v-verif , which needs some updates to be used with this version of the core.

For this reasons, I created a branch on the core-v-verif

https://github.com/openhwgroup/core-v-verif/tree/fix325_cv32e40p

This branch does not point to the right core, this should be done manually as I prefer that we do as following (in this order):

** @GTumbush tries this fix by:

  • using the fix325_cv32e40p branch of core-v-verif by doing git checkout fix325_cv32e40p

  • changing the core by using the core of this pull request. You can change the Makefile if you prefer, or adding a remote to that core, or simply replacing the core folder with a new fork of the core pointing to the right branch:

git clone git@github.com:davideschiavone/riscv.git
cd riscv
git checkout fix325 

Probably you need to rename to folder to cv32e40p

This is the output I got:

uvmt_cv32_tb.end_of_test: * Test Summary *
PPPPPPP    AAAAAA    SSSSSS    SSSSSS   EEEEEEEE  DDDDDDD     
PP    PP  AA    AA  SS    SS  SS    SS  EE        DD    DD    
PP    PP  AA    AA  SS        SS        EE        DD    DD    
PPPPPPP   AAAAAAAA   SSSSSS    SSSSSS   EEEEE     DD    DD    
PP        AA    AA        SS        SS  EE        DD    DD    
PP        AA    AA  SS    SS  SS    SS  EE        DD    DD    
PP        AA    AA   SSSSSS    SSSSSS   EEEEEEEE  DDDDDDD     
----------------------------------------------------------
             SIMULATION PASSED with WARNINGS              
----------------------------------------------------------

=N:[WriteMetrics] Writing coverage metrics...
=T:Simulation terminated by $finish at time 334081500 (/tools/Metrics/dsim/20200316.6.0/uvm-1.2/src/base/uvm_root.svh:517);
Run directory: /data/davide/GitHubRepos/openhwgroup/core-v-verif/master/cv32/sim/uvmt_cv32/dsim_results/riscv_ebreak_test_0
  • If you confirm so and if I didn't create any more regressions, we merge this PR, then I update the core-v-verif pointing to the right core and create another related PR

Do you agree @MikeOpenHWGroup , @GTumbush ?

Merge request reports

Loading