Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

Improvements: Test Control #336

Closed
strichmo opened this issue Nov 5, 2020 · 5 comments
Closed

Improvements: Test Control #336

strichmo opened this issue Nov 5, 2020 · 5 comments
Labels
improvement Issues to highlight long-term improvements in core-v-verif WAIVED: CV32E40P Issue does not apply to CV32E40P and is WAIVED

Comments

@strichmo
Copy link
Contributor

strichmo commented Nov 5, 2020

Use this issue to gather all ideas to improve or highlight long-term issues with any of the following:

  • Makefiles
  • Directory structure
  • Simulation control
  • Test/regression definitions

Add comments for all ideas to this issue. Please try to refrain from deeper discussion of inidividual items. A separate issue should be opened to enable and record that discussion.

@strichmo strichmo added improvement Issues to highlight long-term improvements in core-v-verif WAIVED: CV32E40P Issue does not apply to CV32E40P and is WAIVED labels Nov 5, 2020
@strichmo strichmo changed the title improvements: Test Control Improvements: Test Control Nov 5, 2020
@strichmo
Copy link
Contributor Author

Test relocatable results directory (e.g. _results) across all common Makefiles and targets)

@strichmo
Copy link
Contributor Author

Add configuration (CFG=) as a path in all output artifacts (e.g. _results//)

@strichmo
Copy link
Contributor Author

Consider making <RUN_INDEX> permanent part of all tests. In other words, any test could be run for multiple iterations with different seeds without requiring the _<RUN_INDEX> in the test.yaml file. For default simulations, the RUN_INDEX is 0. For example running the standard hello-world would put outputs in _results/hello-world_0

@MikeOpenHWGroup
Copy link
Member

Here's another one: make the test-program generator (e.g. corev-dv, force-riscv) and toolchain components of the UVM environment. This would have several benefits:

  • simply the compile and run-flow into a single step.
  • make program generation and/or toolchain aware of the Core configuration inputs (boot_addr_i, 'mtvec_addr_i`, etc.) and thus controllable by a UVM testcase.
  • state-aware instruction generation.

@MikeOpenHWGroup
Copy link
Member

This Issue is really a discussion (which has gone stale) and has, for the most part, been implemented in the CORE-V-VERIF scriptware. I am going to close it.

Watch for a Discussion thread on this topic.

@openhwgroup openhwgroup locked and limited conversation to collaborators Sep 1, 2023
@MikeOpenHWGroup MikeOpenHWGroup converted this issue into discussion #2173 Sep 1, 2023

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
improvement Issues to highlight long-term improvements in core-v-verif WAIVED: CV32E40P Issue does not apply to CV32E40P and is WAIVED
Projects
None yet
Development

No branches or pull requests

2 participants