core-v-verif icon indicating copy to clipboard operation
core-v-verif copied to clipboard

define methodology for initializing memory in TB and RM outside of defined program

Open strichmo opened this issue 4 years ago • 2 comments
trafficstars

Type

  • Methodology

The issue is that currently the testbench will initialize memory in the testbench RTL (i.e. the mm_ram) and the reference model in the ISS infrastructure to the the same values. There are some issues with this:

  • It can be considered bad test design to read values that are not explicitly initialized nor implicitly initialized via the elf (i.e. the test program).
  • Out-of-bounds data memory accesses should be caught as soon as possible. The current methodology may hide a rogue pointer or GPR pointer value well beyond the initial failure point in a test.

Steps to Reproduce

No specific test to reproduce the issue. The methodology needs to be defined and clarified.

The methodology will be implemented in the cv32e40x first then propagated to the other cores.

This issue will serve as a discussion and initial documentation of the solution. Once determined the verification strategy document will be updated.

strichmo avatar Jul 22 '21 12:07 strichmo

This is not directly related, but the init code needs to be aware of the remapping done in the mm_ram.

MikeOpenHWGroup avatar Jul 23 '21 12:07 MikeOpenHWGroup

Reassigning this to @MikeOpenHWGroup as I have not made progress on the issue.

strichmo avatar Dec 09 '21 23:12 strichmo