[dvsim] Fix for missing coverage
- The while loop in `Deploy.deploy()` method that sequences the deployed
processes had a bug that caused it to exit when there was more work to
be done. I reordered it so that it works correctly now. `cov_report`
should now get dispatched after `cov_merge`.
- This fixes #1792.
- Some minor fixes to the way things are displayed in stdout.
Signed-off-by: Srikrishna Iyer <sriyer@google.com>
diff --git a/util/dvsim/LintCfg.py b/util/dvsim/LintCfg.py
index 670e5e3..4adfd1e 100644
--- a/util/dvsim/LintCfg.py
+++ b/util/dvsim/LintCfg.py
@@ -41,7 +41,6 @@
# Set the title for lint results.
self.results_title = self.name.upper() + " Lint Results"
-
@staticmethod
def create_instance(flow_cfg_file, proj_root, args):
'''Create a new instance of this class as with given parameters.
@@ -125,7 +124,6 @@
results_str += "### " + self.timestamp_long + "\n"
results_str += "### Lint Tool: " + self.tool.upper() + "\n\n"
-
header = [
"Build Mode", "Tool Warnings", "Tool Errors", "Lint Warnings",
"Lint Errors"
@@ -213,8 +211,8 @@
# Write results to the scratch area
self.results_file = self.scratch_path + "/results_" + self.timestamp + ".md"
- log.info("Detailed results are available at %s", self.results_file)
with open(self.results_file, 'w') as f:
f.write(self.results_md)
+ log.info("[results page]: [%s] [%s]", self.name, results_file)
return self.results_md
