mirror of
https://github.com/lowRISC/ibex.git
synced 2025-06-29 01:33:09 -04:00
Update code from upstream repository https://github.com/lowRISC/opentitan to revision 5cae0cf1fac783e0d0df8c8597bf65322a696a56 * Allow different assertion "backends" in prim_assert.sv (Rupert Swarbrick) * [prim_prince/doc] Update documentation (Michael Schaffner) * [prim_prince] Add option to instantiate a registers half-way (Michael Schaffner) * [prim_cipher_pkg] Reuse sbox4_8bit to build wider sbox layers (Michael Schaffner) * [dv/prim] add PRESENT testbench (Udi Jonnalagadda) * [uvmdvgen] Scoreboard update. (Srikrishna Iyer) * [flash_ctrl dv] Fix V1 tests (Srikrishna Iyer) * [prim_cipher_pkg] Replicate common subfunctions for other widths (Michael Schaffner) * [prim/present] fix PRESENT decryption bugs (Udi Jonnalagadda) * [prim/present] fix some PRESENT encryption bugs (Udi Jonnalagadda) * [dv] Add get_mem DPI function to Verilator simutil (Stefan Wallentowitz) * [lint/entropy_src] Add the entropy source to the lint regression (Michael Schaffner) * [style-lint] Fix some common style lint warnings (Michael Schaffner) * first set of security checks added to D2 checklist (Scott Johnson) * [fpv/tooling] add FPV class extension in dvsim (Cindy Chen) * [dvsim/lint] Minor fixes for printout issues and result parser status (Michael Schaffner) * [syn] Print detailed messages to .md if publication is disabled (Michael Schaffner) * [prim_util] Do not use $clog2() in Xcelium (Philipp Wagner) * [prim] Update ResetValue parameter in prim_flop_2sync (Timothy Chen) * Modified some command-line arguments for DSim (Aimee Sutton) * [prim_util] Make prim_util a package (Philipp Wagner) * [dv] Move mem checking to scb (Weicai Yang) * [lint] Make PINCONNECTEMPTY Verilator waiver common (Philipp Wagner) * [prim] - Fix generic flash enum reference (Timothy Chen) * [prim_ram_*adv] Mark cfg port as unused (Philipp Wagner) * [prim_fifo_sync] Use vbits() for simpler code (Philipp Wagner) * [prim_flash] Add reset to held_part (Eunchan Kim) * [lint] Add more lint waivers (Philipp Wagner) * [dv] Add random backdoor for csr_hw_reset (Weicai Yang) * [dv] Add set_freq_khz in clk_rst_if (Weicai Yang) * [prim] Close GAPI file handle in primgen (Philipp Wagner) * [fpv/prim_packer] fix CI failure due to index out of bound (Cindy Chen) * [prim_arbiter_*] Propagate parameter changes (Michael Schaffner) * [prim_arbiter_tree] Fix incorrect arbitration behavior (Michael Schaffner) * [prim_arbiter_ppc] Add more FPV fairness checks (Michael Schaffner) * [prim_ram*] Add an assertion that checks wmask consistency (Michael Schaffner) * [memutil] Increase max memory width to 256bit (Tom Roberts) * [flash] - Add flash info page support (Timothy Chen) Signed-off-by: Rupert Swarbrick <rswarbrick@lowrisc.org>
205 lines
7.3 KiB
Python
205 lines
7.3 KiB
Python
# Copyright lowRISC contributors.
|
|
# Licensed under the Apache License, Version 2.0, see LICENSE for details.
|
|
# SPDX-License-Identifier: Apache-2.0
|
|
r"""
|
|
Class describing lint configuration object
|
|
"""
|
|
|
|
import hjson
|
|
import logging as log
|
|
from pathlib import Path
|
|
|
|
from tabulate import tabulate
|
|
|
|
from OneShotCfg import OneShotCfg
|
|
from utils import print_msg_list, subst_wildcards
|
|
|
|
class LintCfg(OneShotCfg):
|
|
"""Derivative class for linting purposes.
|
|
"""
|
|
def __init__(self, flow_cfg_file, proj_root, args):
|
|
# This is a lint-specific attribute
|
|
self.is_style_lint = ""
|
|
super().__init__(flow_cfg_file, proj_root, args)
|
|
|
|
def __post_init__(self):
|
|
super().__post_init__()
|
|
|
|
# Convert to boolean
|
|
if self.is_style_lint == "True":
|
|
self.is_style_lint = True
|
|
else:
|
|
self.is_style_lint = False
|
|
|
|
# Set the title for lint results.
|
|
if self.is_style_lint:
|
|
self.results_title = self.name.upper() + " Style Lint Results"
|
|
else:
|
|
self.results_title = self.name.upper() + " Lint Results"
|
|
|
|
def gen_results_summary(self):
|
|
'''
|
|
Gathers the aggregated results from all sub configs
|
|
'''
|
|
|
|
# Generate results table for runs.
|
|
log.info("Create summary of lint results")
|
|
|
|
results_str = "## " + self.results_title + " (Summary)\n\n"
|
|
results_str += "### " + self.timestamp_long + "\n\n"
|
|
|
|
header = [
|
|
"Name", "Tool Warnings", "Tool Errors", "Lint Warnings",
|
|
"Lint Errors"
|
|
]
|
|
colalign = ("center", ) * len(header)
|
|
table = [header]
|
|
|
|
for cfg in self.cfgs:
|
|
|
|
results_page = cfg.results_server_dir + '/results.html'
|
|
results_page_url = results_page.replace(
|
|
cfg.results_server_prefix, cfg.results_server_url_prefix)
|
|
name_with_link = "[" + cfg.name.upper(
|
|
) + "](" + results_page_url + ")"
|
|
table.append([
|
|
name_with_link,
|
|
str(len(cfg.result_summary["warnings"])) + " W",
|
|
str(len(cfg.result_summary["errors"])) + " E",
|
|
str(len(cfg.result_summary["lint_warnings"])) + " W",
|
|
str(len(cfg.result_summary["lint_errors"])) + " E"
|
|
])
|
|
|
|
if len(table) > 1:
|
|
self.results_summary_md = results_str + tabulate(
|
|
table, headers="firstrow", tablefmt="pipe",
|
|
colalign=colalign) + "\n"
|
|
else:
|
|
self.results_summary_md = results_str + "\nNo results to display.\n"
|
|
|
|
print(self.results_summary_md)
|
|
|
|
# Return only the tables
|
|
return self.results_summary_md
|
|
|
|
def _gen_results(self):
|
|
# '''
|
|
# The function is called after the regression has completed. It looks
|
|
# for a regr_results.hjson file with aggregated results from the lint run.
|
|
# The hjson needs to have the following (potentially empty) fields
|
|
#
|
|
# {
|
|
# tool: ""
|
|
# errors: []
|
|
# warnings: []
|
|
# lint_errors: []
|
|
# lint_warning: []
|
|
# lint_infos: []
|
|
# }
|
|
#
|
|
# where each entry is a string representing a lint message. This allows
|
|
# to reuse the same LintCfg class with different tools since just the
|
|
# parsing script that transforms the tool output into the hjson above
|
|
# needs to be adapted.
|
|
#
|
|
# note that if this is a master config, the results will
|
|
# be generated using the _gen_results_summary function
|
|
# '''
|
|
|
|
# Generate results table for runs.
|
|
results_str = "## " + self.results_title + "\n\n"
|
|
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"
|
|
]
|
|
colalign = ("center", ) * len(header)
|
|
table = [header]
|
|
|
|
# aggregated counts
|
|
self.result_summary["warnings"] = []
|
|
self.result_summary["errors"] = []
|
|
self.result_summary["lint_warnings"] = []
|
|
self.result_summary["lint_errors"] = []
|
|
|
|
fail_msgs = ""
|
|
for mode in self.build_modes:
|
|
|
|
result_data = Path(
|
|
subst_wildcards(self.build_dir, {"build_mode": mode.name}) +
|
|
'/results.hjson')
|
|
log.info("looking for result data file at %s", result_data)
|
|
|
|
try:
|
|
with open(result_data, "r") as results_file:
|
|
self.result = hjson.load(results_file, use_decimal=True)
|
|
except IOError as err:
|
|
log.warning("%s", err)
|
|
self.result = {
|
|
"tool": "",
|
|
"errors": ["IOError: %s" % err],
|
|
"warnings": [],
|
|
"lint_errors": [],
|
|
"lint_warnings": [],
|
|
"lint_infos": []
|
|
}
|
|
if self.result:
|
|
table.append([
|
|
mode.name,
|
|
str(len(self.result["warnings"])) + " W ",
|
|
str(len(self.result["errors"])) + " E",
|
|
# We currently do not publish these infos at
|
|
# the moment len(self.result["lint_infos"]),
|
|
str(len(self.result["lint_warnings"])) + " W",
|
|
str(len(self.result["lint_errors"])) + " E"
|
|
])
|
|
else:
|
|
self.result = {
|
|
"tool": "",
|
|
"errors": [],
|
|
"warnings": [],
|
|
"lint_errors": [],
|
|
"lint_warnings": [],
|
|
"lint_infos": []
|
|
}
|
|
|
|
self.result_summary["warnings"] += self.result["warnings"]
|
|
self.result_summary["errors"] += self.result["errors"]
|
|
self.result_summary["lint_warnings"] += self.result[
|
|
"lint_warnings"]
|
|
self.result_summary["lint_errors"] += self.result["lint_errors"]
|
|
|
|
# Append detailed messages if they exist
|
|
hdr_key_pairs = [("Tool Warnings", "warnings"),
|
|
("Tool Errors", "errors"),
|
|
("Lint Warnings", "lint_warnings"),
|
|
("Lint Errors", "lint_errors")]
|
|
|
|
has_msg = False
|
|
for _, key in hdr_key_pairs:
|
|
if key in self.result:
|
|
has_msg = True
|
|
break
|
|
|
|
if has_msg:
|
|
fail_msgs += "\n### Errors and Warnings for Build Mode `'" + mode.name + "'`\n"
|
|
for hdr, key in hdr_key_pairs:
|
|
msgs = self.result.get(key)
|
|
fail_msgs += print_msg_list("#### " + hdr, msgs, self.max_msg_count)
|
|
|
|
if len(table) > 1:
|
|
self.results_md = results_str + tabulate(
|
|
table, headers="firstrow", tablefmt="pipe",
|
|
colalign=colalign) + "\n" + fail_msgs
|
|
else:
|
|
self.results_md = results_str + "\nNo results to display.\n"
|
|
|
|
# Write results to the scratch area
|
|
self.results_file = self.scratch_path + "/results_" + self.timestamp + ".md"
|
|
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
|