Skip to content

Commit 14ee77b

Browse files
committed
refactor: Add additional sim flow debug logs
Add logs for initializing a RunTest object (which leads to wildcard expansion) and for creating a build mode object. For large regression runs, this tells us what DVSim is doing with debug logs when it is spending a long time creating these objects (due to many duplicate seeds). Signed-off-by: Alex Jones <[email protected]>
1 parent 5070a78 commit 14ee77b

File tree

2 files changed

+14
-1
lines changed

2 files changed

+14
-1
lines changed

src/dvsim/job/deploy.py

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -548,6 +548,13 @@ def __init__(self, index, test, build_job, sim_cfg: "SimCfg") -> None:
548548
# Systemverilog accepts seeds with a maximum size of 32 bits.
549549
self.svseed = int(self.seed) & 0xFFFFFFFF
550550
self.simulated_time = JobTime()
551+
log.debug(
552+
"Initializing RunTest for %s test %s no. %d with seed %s",
553+
sim_cfg.name,
554+
getattr(test, "name", "[unknown]"),
555+
index,
556+
self.seed,
557+
)
551558
super().__init__(sim_cfg)
552559

553560
# Needs to be after the wildcard expansion to log anything meaningful

src/dvsim/sim/flow.py

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -467,7 +467,13 @@ def _create_deploy_objects(self) -> None:
467467

468468
self.builds = []
469469
build_map = {}
470-
for build_mode_obj in self.build_list:
470+
for i, build_mode_obj in enumerate(self.build_list, start=1):
471+
log.debug(
472+
"Creating build mode obj %d/%d: %s",
473+
i,
474+
len(self.build_list),
475+
build_mode_obj.name,
476+
)
471477
new_build = CompileSim(build_mode_obj, self)
472478

473479
# It is possible for tests to supply different build modes, but

0 commit comments

Comments
 (0)