From 14ee77ba0db0198c0ec5c8209c2c9a4231ab0f71 Mon Sep 17 00:00:00 2001 From: Alex Jones Date: Thu, 12 Feb 2026 15:10:04 +0000 Subject: [PATCH] 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 --- src/dvsim/job/deploy.py | 7 +++++++ src/dvsim/sim/flow.py | 8 +++++++- 2 files changed, 14 insertions(+), 1 deletion(-) diff --git a/src/dvsim/job/deploy.py b/src/dvsim/job/deploy.py index f91cfbd7..0b72dddd 100644 --- a/src/dvsim/job/deploy.py +++ b/src/dvsim/job/deploy.py @@ -548,6 +548,13 @@ def __init__(self, index, test, build_job, sim_cfg: "SimCfg") -> None: # Systemverilog accepts seeds with a maximum size of 32 bits. self.svseed = int(self.seed) & 0xFFFFFFFF self.simulated_time = JobTime() + log.debug( + "Initializing RunTest for %s test %s no. %d with seed %s", + sim_cfg.name, + getattr(test, "name", "[unknown]"), + index, + self.seed, + ) super().__init__(sim_cfg) # Needs to be after the wildcard expansion to log anything meaningful diff --git a/src/dvsim/sim/flow.py b/src/dvsim/sim/flow.py index c5fd3a06..195bd9b4 100644 --- a/src/dvsim/sim/flow.py +++ b/src/dvsim/sim/flow.py @@ -467,7 +467,13 @@ def _create_deploy_objects(self) -> None: self.builds = [] build_map = {} - for build_mode_obj in self.build_list: + for i, build_mode_obj in enumerate(self.build_list, start=1): + log.debug( + "Creating build mode obj %d/%d: %s", + i, + len(self.build_list), + build_mode_obj.name, + ) new_build = CompileSim(build_mode_obj, self) # It is possible for tests to supply different build modes, but