lowRISC / lowRISC/opentitan

[dvsim] The launchers handle max_parallel very strangely

Open
#24,669 0 comments 0 reactions 1 assignee Claimed by @rswarbrick View on GitHub
Component:DV Tool:dvsim
Dominant language
SystemVerilog
Stars
3.6k
Forks
1.1k
Avg merge
2d 22h
Merged PRs (30d)
141

Description

### Description

For some reason, this is handled in a static variable in each class, which gets set up at the start of the `dvsim` main function. That seems very strange!

The `max_parallel` argument should probably be passed through:
- `FlowCfg.deploy_objects` (called in three places in `dvsim.py`)
- `Scheduler.__init__`
- `Deploy.create_launcher`
- `LauncherFactory.get_launcher`

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.