Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 1 | # Copyright lowRISC contributors. |
| 2 | # Licensed under the Apache License, Version 2.0, see LICENSE for details. |
| 3 | # SPDX-License-Identifier: Apache-2.0 |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 4 | |
| 5 | import logging as log |
| 6 | import pprint |
Srikrishna Iyer | 7cf7cad | 2020-01-08 11:32:53 -0800 | [diff] [blame] | 7 | import random |
Srikrishna Iyer | 37b0c99 | 2021-03-23 16:27:26 -0700 | [diff] [blame] | 8 | import shlex |
Srikrishna Iyer | 956b3f8 | 2021-04-15 15:14:56 -0700 | [diff] [blame] | 9 | from pathlib import Path |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 10 | |
Srikrishna Iyer | 9d9d86f | 2021-03-02 00:15:51 -0800 | [diff] [blame] | 11 | from LauncherFactory import get_launcher |
Rupert Swarbrick | 6cc2011 | 2020-04-24 09:44:35 +0100 | [diff] [blame] | 12 | from sim_utils import get_cov_summary_table |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 13 | from tabulate import tabulate |
Srikrishna Iyer | caa0d89 | 2021-03-01 13:15:52 -0800 | [diff] [blame] | 14 | from utils import (VERBOSE, clean_odirs, find_and_substitute_wildcards, |
| 15 | rm_path, subst_wildcards) |
Rupert Swarbrick | a2892a5 | 2020-10-12 08:50:08 +0100 | [diff] [blame] | 16 | |
| 17 | |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 18 | class Deploy(): |
| 19 | """ |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 20 | Abstraction to create and maintain a runnable job (builds, runs, etc.). |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 21 | """ |
| 22 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 23 | # Indicate the target for each sub-class. |
| 24 | target = None |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 25 | |
Srikrishna Iyer | 6c531b0 | 2020-11-23 18:41:16 -0800 | [diff] [blame] | 26 | # List of variable names that are to be treated as "list of commands". |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 27 | # This tells '_construct_cmd' that these vars are lists that need to |
Srikrishna Iyer | 6c531b0 | 2020-11-23 18:41:16 -0800 | [diff] [blame] | 28 | # be joined with '&&' instead of a space. |
| 29 | cmds_list_vars = [] |
| 30 | |
Srikrishna Iyer | 1d2899f | 2021-02-11 18:23:15 -0800 | [diff] [blame] | 31 | # Represents the weight with which a job of this target is scheduled. These |
| 32 | # initial weights set for each of the targets below are roughly inversely |
| 33 | # proportional to their average runtimes. These are subject to change in |
| 34 | # future. Lower the runtime, the higher chance the it gets scheduled. It is |
| 35 | # useful to customize this only in case of targets that may coexist at a |
| 36 | # time. |
| 37 | # TODO: Allow these to be set in the HJson. |
| 38 | weight = 1 |
| 39 | |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 40 | def __str__(self): |
Rupert Swarbrick | 381770d | 2020-10-12 08:50:08 +0100 | [diff] [blame] | 41 | return (pprint.pformat(self.__dict__) |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 42 | if log.getLogger().isEnabledFor(VERBOSE) else self.full_name) |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 43 | |
Srikrishna Iyer | 7cf7cad | 2020-01-08 11:32:53 -0800 | [diff] [blame] | 44 | def __init__(self, sim_cfg): |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 45 | assert self.target is not None |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 46 | |
Srikrishna Iyer | 7cf7cad | 2020-01-08 11:32:53 -0800 | [diff] [blame] | 47 | # Cross ref the whole cfg object for ease. |
| 48 | self.sim_cfg = sim_cfg |
| 49 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 50 | # A list of jobs on which this job depends. |
Rupert Swarbrick | 381770d | 2020-10-12 08:50:08 +0100 | [diff] [blame] | 51 | self.dependencies = [] |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 52 | |
Cindy Chen | 924c606 | 2021-01-27 14:27:50 -0800 | [diff] [blame] | 53 | # Indicates whether running this job requires all dependencies to pass. |
| 54 | # If this flag is set to False, any passing dependency will trigger |
| 55 | # this current job to run |
| 56 | self.needs_all_dependencies_passing = True |
| 57 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 58 | # Declare attributes that need to be extracted from the HJSon cfg. |
| 59 | self._define_attrs() |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 60 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 61 | # Set class instance attributes. |
| 62 | self._set_attrs() |
| 63 | |
| 64 | # Check if all attributes that are needed are set. |
| 65 | self._check_attrs() |
| 66 | |
| 67 | # Do variable substitutions. |
| 68 | self._subst_vars() |
| 69 | |
| 70 | # List of vars required to be exported to sub-shell, as a dict. |
| 71 | self.exports = self._process_exports() |
| 72 | |
| 73 | # Construct the job's command. |
| 74 | self.cmd = self._construct_cmd() |
| 75 | |
Srikrishna Iyer | d562377 | 2021-06-18 22:23:07 -0700 | [diff] [blame] | 76 | # Launcher instance created later using create_launcher() method. |
| 77 | self.launcher = None |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 78 | |
| 79 | def _define_attrs(self): |
| 80 | """Defines the attributes this instance needs to have. |
| 81 | |
| 82 | These attributes are extracted from the Mode object / HJson config with |
| 83 | which this instance is created. There are two types of attributes - |
| 84 | one contributes to the generation of the command directly; the other |
| 85 | provides supplementary information pertaining to the job, such as |
| 86 | patterns that determine whether it passed or failed. These are |
| 87 | represented as dicts, whose values indicate in boolean whether the |
| 88 | extraction was successful. |
| 89 | """ |
| 90 | # These attributes are explicitly used to construct the job command. |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 91 | self.mandatory_cmd_attrs = {} |
| 92 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 93 | # These attributes may indirectly contribute to the construction of the |
| 94 | # command (through substitution vars) or other things such as pass / |
| 95 | # fail patterns. |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 96 | self.mandatory_misc_attrs = { |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 97 | "name": False, |
| 98 | "build_mode": False, |
| 99 | "flow_makefile": False, |
| 100 | "exports": False, |
| 101 | "dry_run": False |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 102 | } |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 103 | |
Srikrishna Iyer | 7cf7cad | 2020-01-08 11:32:53 -0800 | [diff] [blame] | 104 | # Function to parse a dict and extract the mandatory cmd and misc attrs. |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 105 | def _extract_attrs(self, ddict): |
| 106 | """Extracts the attributes from the supplied dict. |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 107 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 108 | 'ddict' is typically either the Mode object or the entire config |
| 109 | object's dict. It is used to retrieve the instance attributes defined |
| 110 | in 'mandatory_cmd_attrs' and 'mandatory_misc_attrs'. |
| 111 | """ |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 112 | ddict_keys = ddict.keys() |
| 113 | for key in self.mandatory_cmd_attrs.keys(): |
Rupert Swarbrick | 6cc2011 | 2020-04-24 09:44:35 +0100 | [diff] [blame] | 114 | if self.mandatory_cmd_attrs[key] is False: |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 115 | if key in ddict_keys: |
| 116 | setattr(self, key, ddict[key]) |
| 117 | self.mandatory_cmd_attrs[key] = True |
| 118 | |
| 119 | for key in self.mandatory_misc_attrs.keys(): |
Rupert Swarbrick | 6cc2011 | 2020-04-24 09:44:35 +0100 | [diff] [blame] | 120 | if self.mandatory_misc_attrs[key] is False: |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 121 | if key in ddict_keys: |
| 122 | setattr(self, key, ddict[key]) |
| 123 | self.mandatory_misc_attrs[key] = True |
| 124 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 125 | def _set_attrs(self): |
| 126 | """Sets additional attributes. |
| 127 | |
| 128 | Invokes '_extract_attrs()' to read in all the necessary instance |
| 129 | attributes. Based on those, some additional instance attributes may |
| 130 | be derived. Those are set by this method. |
| 131 | """ |
| 132 | self._extract_attrs(self.sim_cfg.__dict__) |
| 133 | |
Srikrishna Iyer | d562377 | 2021-06-18 22:23:07 -0700 | [diff] [blame] | 134 | # Enable GUI mode. |
| 135 | self.gui = self.sim_cfg.gui |
| 136 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 137 | # Output directory where the artifacts go (used by the launcher). |
| 138 | self.odir = getattr(self, self.target + "_dir") |
| 139 | |
| 140 | # Qualified name disambiguates the instance name with other instances |
| 141 | # of the same class (example: 'uart_smoke' reseeded multiple times |
| 142 | # needs to be disambiguated using the index -> '0.uart_smoke'. |
| 143 | self.qual_name = self.name |
| 144 | |
| 145 | # Full name disambiguates across multiple cfg being run (example: |
| 146 | # 'aes:default', 'uart:default' builds. |
| 147 | self.full_name = self.sim_cfg.name + ":" + self.qual_name |
| 148 | |
Srikrishna Iyer | 956b3f8 | 2021-04-15 15:14:56 -0700 | [diff] [blame] | 149 | # Job name is used to group the job by cfg and target. The scratch path |
| 150 | # directory name is assumed to be uniquified, in case there are more |
| 151 | # than one sim_cfgs with the same name. |
| 152 | self.job_name = "{}_{}".format( |
| 153 | Path(self.sim_cfg.scratch_path).name, self.target) |
Srikrishna Iyer | 9d9d86f | 2021-03-02 00:15:51 -0800 | [diff] [blame] | 154 | |
Srikrishna Iyer | 37b0c99 | 2021-03-23 16:27:26 -0700 | [diff] [blame] | 155 | # Input directories (other than self) this job depends on. |
| 156 | self.input_dirs = [] |
| 157 | |
| 158 | # Directories touched by this job. These directories are marked |
| 159 | # becuase they are used by dependent jobs as input. |
| 160 | self.output_dirs = [self.odir] |
| 161 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 162 | # Pass and fail patterns. |
| 163 | self.pass_patterns = [] |
| 164 | self.fail_patterns = [] |
| 165 | |
| 166 | def _check_attrs(self): |
| 167 | """Checks if all required class attributes are set. |
| 168 | |
| 169 | Invoked in __init__() after all attributes are extracted and set. |
| 170 | """ |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 171 | for attr in self.mandatory_cmd_attrs.keys(): |
| 172 | if self.mandatory_cmd_attrs[attr] is False: |
Srikrishna Iyer | 08ccc23 | 2021-03-31 00:01:54 -0700 | [diff] [blame] | 173 | raise AttributeError("Attribute {!r} not found for " |
| 174 | "{!r}.".format(attr, self.name)) |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 175 | |
| 176 | for attr in self.mandatory_misc_attrs.keys(): |
| 177 | if self.mandatory_misc_attrs[attr] is False: |
Srikrishna Iyer | 08ccc23 | 2021-03-31 00:01:54 -0700 | [diff] [blame] | 178 | raise AttributeError("Attribute {!r} not found for " |
| 179 | "{!r}.".format(attr, self.name)) |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 180 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 181 | def _subst_vars(self, ignored_subst_vars=[]): |
| 182 | """Recursively search and replace substitution variables. |
| 183 | |
| 184 | First pass: search within self dict. We ignore errors since some |
| 185 | substitions may be available in the second pass. Second pass: search |
| 186 | the entire sim_cfg object.""" |
| 187 | |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 188 | self.__dict__ = find_and_substitute_wildcards(self.__dict__, |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 189 | self.__dict__, |
| 190 | ignored_subst_vars, True) |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 191 | self.__dict__ = find_and_substitute_wildcards(self.__dict__, |
| 192 | self.sim_cfg.__dict__, |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 193 | ignored_subst_vars, |
| 194 | False) |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 195 | |
Srikrishna Iyer | 6333b7c | 2020-10-17 21:14:25 -0700 | [diff] [blame] | 196 | def _process_exports(self): |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 197 | """Convert 'exports' as a list of dicts in the HJson to a dict. |
Srikrishna Iyer | 6333b7c | 2020-10-17 21:14:25 -0700 | [diff] [blame] | 198 | |
| 199 | Exports is a list of key-value pairs that are to be exported to the |
| 200 | subprocess' environment so that the tools can lookup those options. |
| 201 | DVSim limits how the data is presented in the HJson (the value of a |
| 202 | HJson member cannot be an object). This method converts a list of dicts |
| 203 | into a dict variable, which makes it easy to merge the list of exports |
| 204 | with the subprocess' env where the ASIC tool is invoked. |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 205 | """ |
Srikrishna Iyer | 6333b7c | 2020-10-17 21:14:25 -0700 | [diff] [blame] | 206 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 207 | return {k: str(v) for item in self.exports for k, v in item.items()} |
| 208 | |
| 209 | def _construct_cmd(self): |
| 210 | """Construct the command that will eventually be launched.""" |
| 211 | |
Srikrishna Iyer | 9bff8ea | 2021-04-02 15:14:04 -0700 | [diff] [blame] | 212 | cmd = "make -f {} {}".format(self.flow_makefile, self.target) |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 213 | if self.dry_run is True: |
Srikrishna Iyer | 9bff8ea | 2021-04-02 15:14:04 -0700 | [diff] [blame] | 214 | cmd += " -n" |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 215 | for attr in sorted(self.mandatory_cmd_attrs.keys()): |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 216 | value = getattr(self, attr) |
| 217 | if type(value) is list: |
Srikrishna Iyer | 6c531b0 | 2020-11-23 18:41:16 -0800 | [diff] [blame] | 218 | # Join attributes that are list of commands with '&&' to chain |
| 219 | # them together when executed as a Make target's recipe. |
| 220 | separator = " && " if attr in self.cmds_list_vars else " " |
Srikrishna Iyer | 08ccc23 | 2021-03-31 00:01:54 -0700 | [diff] [blame] | 221 | value = separator.join(item.strip() for item in value) |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 222 | if type(value) is bool: |
| 223 | value = int(value) |
| 224 | if type(value) is str: |
| 225 | value = value.strip() |
Srikrishna Iyer | 9bff8ea | 2021-04-02 15:14:04 -0700 | [diff] [blame] | 226 | cmd += " {}={}".format(attr, shlex.quote(value)) |
| 227 | return cmd |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 228 | |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 229 | def is_equivalent_job(self, item): |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 230 | """Checks if job that would be dispatched with 'item' is equivalent to |
| 231 | 'self'. |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 232 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 233 | Determines if 'item' and 'self' would behave exactly the same way when |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 234 | deployed. If so, then there is no point in keeping both. The caller can |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 235 | choose to discard 'item' and pick 'self' instead. To do so, we check |
| 236 | the final resolved 'cmd' & the exports. The 'name' field will be unique |
| 237 | to 'item' and 'self', so we take that out of the comparison. |
| 238 | """ |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 239 | if type(self) != type(item): |
| 240 | return False |
| 241 | |
| 242 | # Check if the cmd field is identical. |
| 243 | item_cmd = item.cmd.replace(item.name, self.name) |
| 244 | if self.cmd != item_cmd: |
| 245 | return False |
| 246 | |
| 247 | # Check if exports have identical set of keys. |
| 248 | if self.exports.keys() != item.exports.keys(): |
| 249 | return False |
| 250 | |
| 251 | # Check if exports have identical values. |
| 252 | for key, val in self.exports.items(): |
| 253 | item_val = item.exports[key] |
| 254 | if type(item_val) is str: |
| 255 | item_val = item_val.replace(item.name, self.name) |
| 256 | if val != item_val: |
| 257 | return False |
| 258 | |
| 259 | log.log(VERBOSE, "Deploy job \"%s\" is equivalent to \"%s\"", |
| 260 | item.name, self.name) |
| 261 | return True |
| 262 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 263 | def pre_launch(self): |
| 264 | """Callback to perform additional pre-launch activities. |
Srikrishna Iyer | 6333b7c | 2020-10-17 21:14:25 -0700 | [diff] [blame] | 265 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 266 | This is invoked by launcher::_pre_launch(). |
Srikrishna Iyer | 2d15119 | 2021-02-10 16:56:16 -0800 | [diff] [blame] | 267 | """ |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 268 | pass |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 269 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 270 | def post_finish(self, status): |
| 271 | """Callback to perform additional post-finish activities. |
Srikrishna Iyer | 2d15119 | 2021-02-10 16:56:16 -0800 | [diff] [blame] | 272 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 273 | This is invoked by launcher::_post_finish(). |
Srikrishna Iyer | 2d15119 | 2021-02-10 16:56:16 -0800 | [diff] [blame] | 274 | """ |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 275 | pass |
Srikrishna Iyer | 2d15119 | 2021-02-10 16:56:16 -0800 | [diff] [blame] | 276 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 277 | def get_log_path(self): |
| 278 | """Returns the log file path.""" |
Srikrishna Iyer | 2d15119 | 2021-02-10 16:56:16 -0800 | [diff] [blame] | 279 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 280 | return "{}/{}.log".format(self.odir, self.target) |
Weicai Yang | fc2ff3b | 2020-03-19 18:05:14 -0700 | [diff] [blame] | 281 | |
Srikrishna Iyer | d562377 | 2021-06-18 22:23:07 -0700 | [diff] [blame] | 282 | def create_launcher(self): |
| 283 | """Creates the launcher instance. |
| 284 | |
| 285 | Note that the launcher instance for ALL jobs in the same job group must |
| 286 | be created before the Scheduler starts to dispatch one by one. |
| 287 | """ |
| 288 | # Retain the handle to self for lookup & callbacks. |
| 289 | self.launcher = get_launcher(self) |
| 290 | |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 291 | |
| 292 | class CompileSim(Deploy): |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 293 | """Abstraction for building the simulation executable.""" |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 294 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 295 | target = "build" |
Srikrishna Iyer | 6c531b0 | 2020-11-23 18:41:16 -0800 | [diff] [blame] | 296 | cmds_list_vars = ["pre_build_cmds", "post_build_cmds"] |
Srikrishna Iyer | 1d2899f | 2021-02-11 18:23:15 -0800 | [diff] [blame] | 297 | weight = 5 |
Srikrishna Iyer | 6c531b0 | 2020-11-23 18:41:16 -0800 | [diff] [blame] | 298 | |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 299 | def __init__(self, build_mode, sim_cfg): |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 300 | self.build_mode_obj = build_mode |
Srikrishna Iyer | 4c4b55d | 2022-03-29 23:08:47 -0700 | [diff] [blame] | 301 | self.seed = sim_cfg.build_seed |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 302 | super().__init__(sim_cfg) |
| 303 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 304 | def _define_attrs(self): |
| 305 | super()._define_attrs() |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 306 | self.mandatory_cmd_attrs.update({ |
Srikrishna Iyer | 8ce80d0 | 2020-02-05 10:53:19 -0800 | [diff] [blame] | 307 | # tool srcs |
Srikrishna Iyer | 981c36b | 2020-12-12 12:20:35 -0800 | [diff] [blame] | 308 | "proj_root": False, |
Srikrishna Iyer | 8ce80d0 | 2020-02-05 10:53:19 -0800 | [diff] [blame] | 309 | |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 310 | # Flist gen |
| 311 | "sv_flist_gen_cmd": False, |
| 312 | "sv_flist_gen_dir": False, |
| 313 | "sv_flist_gen_opts": False, |
| 314 | |
| 315 | # Build |
| 316 | "build_dir": False, |
Srikrishna Iyer | 6c531b0 | 2020-11-23 18:41:16 -0800 | [diff] [blame] | 317 | "pre_build_cmds": False, |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 318 | "build_cmd": False, |
Srikrishna Iyer | 6c531b0 | 2020-11-23 18:41:16 -0800 | [diff] [blame] | 319 | "build_opts": False, |
| 320 | "post_build_cmds": False, |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 321 | }) |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 322 | |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 323 | self.mandatory_misc_attrs.update({ |
Weicai Yang | 31029d2 | 2020-03-24 11:14:56 -0700 | [diff] [blame] | 324 | "cov_db_dir": False, |
| 325 | "build_pass_patterns": False, |
| 326 | "build_fail_patterns": False |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 327 | }) |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 328 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 329 | def _set_attrs(self): |
| 330 | super()._extract_attrs(self.build_mode_obj.__dict__) |
| 331 | super()._set_attrs() |
| 332 | |
Srikrishna Iyer | d562377 | 2021-06-18 22:23:07 -0700 | [diff] [blame] | 333 | # Dont run the compile job in GUI mode. |
| 334 | self.gui = False |
| 335 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 336 | # 'build_mode' is used as a substitution variable in the HJson. |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 337 | self.build_mode = self.name |
Srikrishna Iyer | 956b3f8 | 2021-04-15 15:14:56 -0700 | [diff] [blame] | 338 | self.job_name += f"_{self.build_mode}" |
Srikrishna Iyer | 08ccc23 | 2021-03-31 00:01:54 -0700 | [diff] [blame] | 339 | if self.sim_cfg.cov: |
| 340 | self.output_dirs += [self.cov_db_dir] |
Weicai Yang | 31029d2 | 2020-03-24 11:14:56 -0700 | [diff] [blame] | 341 | self.pass_patterns = self.build_pass_patterns |
| 342 | self.fail_patterns = self.build_fail_patterns |
Srikrishna Iyer | 4f0b090 | 2020-01-25 02:28:47 -0800 | [diff] [blame] | 343 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 344 | def pre_launch(self): |
Srikrishna Iyer | 2d15119 | 2021-02-10 16:56:16 -0800 | [diff] [blame] | 345 | # Delete old coverage database directories before building again. We |
Srikrishna Iyer | 08ccc23 | 2021-03-31 00:01:54 -0700 | [diff] [blame] | 346 | # need to do this because the build directory is not 'renewed'. |
Srikrishna Iyer | 2d15119 | 2021-02-10 16:56:16 -0800 | [diff] [blame] | 347 | rm_path(self.cov_db_dir) |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 348 | |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 349 | |
Michael Schaffner | 8ac6c4c | 2020-03-03 15:00:20 -0800 | [diff] [blame] | 350 | class CompileOneShot(Deploy): |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 351 | """Abstraction for building the design (used by non-DV flows).""" |
Michael Schaffner | 8ac6c4c | 2020-03-03 15:00:20 -0800 | [diff] [blame] | 352 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 353 | target = "build" |
Michael Schaffner | 8ac6c4c | 2020-03-03 15:00:20 -0800 | [diff] [blame] | 354 | |
| 355 | def __init__(self, build_mode, sim_cfg): |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 356 | self.build_mode_obj = build_mode |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 357 | super().__init__(sim_cfg) |
| 358 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 359 | def _define_attrs(self): |
| 360 | super()._define_attrs() |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 361 | self.mandatory_cmd_attrs.update({ |
Michael Schaffner | 8ac6c4c | 2020-03-03 15:00:20 -0800 | [diff] [blame] | 362 | # tool srcs |
Srikrishna Iyer | 981c36b | 2020-12-12 12:20:35 -0800 | [diff] [blame] | 363 | "proj_root": False, |
Michael Schaffner | 8ac6c4c | 2020-03-03 15:00:20 -0800 | [diff] [blame] | 364 | |
Michael Schaffner | 3d16099 | 2020-03-31 18:37:53 -0700 | [diff] [blame] | 365 | # Flist gen |
| 366 | "sv_flist_gen_cmd": False, |
| 367 | "sv_flist_gen_dir": False, |
| 368 | "sv_flist_gen_opts": False, |
| 369 | |
Michael Schaffner | 8ac6c4c | 2020-03-03 15:00:20 -0800 | [diff] [blame] | 370 | # Build |
| 371 | "build_dir": False, |
Srikrishna Iyer | 981c36b | 2020-12-12 12:20:35 -0800 | [diff] [blame] | 372 | "pre_build_cmds": False, |
Michael Schaffner | 8ac6c4c | 2020-03-03 15:00:20 -0800 | [diff] [blame] | 373 | "build_cmd": False, |
| 374 | "build_opts": False, |
Srikrishna Iyer | 981c36b | 2020-12-12 12:20:35 -0800 | [diff] [blame] | 375 | "post_build_cmds": False, |
Michael Schaffner | 3d16099 | 2020-03-31 18:37:53 -0700 | [diff] [blame] | 376 | "build_log": False, |
Michael Schaffner | 8ac6c4c | 2020-03-03 15:00:20 -0800 | [diff] [blame] | 377 | |
| 378 | # Report processing |
| 379 | "report_cmd": False, |
| 380 | "report_opts": False |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 381 | }) |
Michael Schaffner | 8ac6c4c | 2020-03-03 15:00:20 -0800 | [diff] [blame] | 382 | |
Srikrishna Iyer | 9bff8ea | 2021-04-02 15:14:04 -0700 | [diff] [blame] | 383 | self.mandatory_misc_attrs.update({"build_fail_patterns": False}) |
Srikrishna Iyer | 5b6f9d6 | 2021-04-02 02:12:49 -0700 | [diff] [blame] | 384 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 385 | def _set_attrs(self): |
| 386 | super()._extract_attrs(self.build_mode_obj.__dict__) |
| 387 | super()._set_attrs() |
| 388 | |
| 389 | # 'build_mode' is used as a substitution variable in the HJson. |
Michael Schaffner | 8ac6c4c | 2020-03-03 15:00:20 -0800 | [diff] [blame] | 390 | self.build_mode = self.name |
Srikrishna Iyer | 956b3f8 | 2021-04-15 15:14:56 -0700 | [diff] [blame] | 391 | self.job_name += f"_{self.build_mode}" |
Srikrishna Iyer | 5b6f9d6 | 2021-04-02 02:12:49 -0700 | [diff] [blame] | 392 | self.fail_patterns = self.build_fail_patterns |
Michael Schaffner | 8ac6c4c | 2020-03-03 15:00:20 -0800 | [diff] [blame] | 393 | |
| 394 | |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 395 | class RunTest(Deploy): |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 396 | """Abstraction for running tests. This is one per seed for each test.""" |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 397 | |
Srikrishna Iyer | 7cf7cad | 2020-01-08 11:32:53 -0800 | [diff] [blame] | 398 | # Initial seed values when running tests (if available). |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 399 | target = "run" |
Srikrishna Iyer | 7cf7cad | 2020-01-08 11:32:53 -0800 | [diff] [blame] | 400 | seeds = [] |
Srikrishna Iyer | 96e5410 | 2020-03-12 22:46:50 -0700 | [diff] [blame] | 401 | fixed_seed = None |
Srikrishna Iyer | 6c531b0 | 2020-11-23 18:41:16 -0800 | [diff] [blame] | 402 | cmds_list_vars = ["pre_run_cmds", "post_run_cmds"] |
| 403 | |
Rupert Swarbrick | 381770d | 2020-10-12 08:50:08 +0100 | [diff] [blame] | 404 | def __init__(self, index, test, build_job, sim_cfg): |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 405 | self.test_obj = test |
| 406 | self.index = index |
| 407 | self.seed = RunTest.get_seed() |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 408 | super().__init__(sim_cfg) |
| 409 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 410 | if build_job is not None: |
| 411 | self.dependencies.append(build_job) |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 412 | |
Srikrishna Iyer | 9d9d86f | 2021-03-02 00:15:51 -0800 | [diff] [blame] | 413 | # We did something wrong if build_mode is not the same as the build_job |
| 414 | # arg's name. |
| 415 | assert self.build_mode == build_job.name |
| 416 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 417 | def _define_attrs(self): |
| 418 | super()._define_attrs() |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 419 | self.mandatory_cmd_attrs.update({ |
Srikrishna Iyer | e9879ca | 2020-11-17 01:34:49 -0800 | [diff] [blame] | 420 | # tool srcs |
Srikrishna Iyer | 42d032f | 2020-03-04 23:55:44 -0800 | [diff] [blame] | 421 | "proj_root": False, |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 422 | "uvm_test": False, |
| 423 | "uvm_test_seq": False, |
Srikrishna Iyer | de5efa0 | 2020-11-20 23:17:11 -0800 | [diff] [blame] | 424 | "sw_images": False, |
Srikrishna Iyer | 7702be5 | 2020-03-07 01:00:28 -0800 | [diff] [blame] | 425 | "sw_build_device": False, |
| 426 | "sw_build_dir": False, |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 427 | "run_dir": False, |
Srikrishna Iyer | 6c531b0 | 2020-11-23 18:41:16 -0800 | [diff] [blame] | 428 | "pre_run_cmds": False, |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 429 | "run_cmd": False, |
Srikrishna Iyer | 6c531b0 | 2020-11-23 18:41:16 -0800 | [diff] [blame] | 430 | "run_opts": False, |
| 431 | "post_run_cmds": False, |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 432 | }) |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 433 | |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 434 | self.mandatory_misc_attrs.update({ |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 435 | "run_dir_name": False, |
Srikrishna Iyer | 1f5fae9 | 2021-02-18 21:57:25 -0800 | [diff] [blame] | 436 | "cov_db_dir": False, |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 437 | "cov_db_test_dir": False, |
Weicai Yang | 31029d2 | 2020-03-24 11:14:56 -0700 | [diff] [blame] | 438 | "run_pass_patterns": False, |
| 439 | "run_fail_patterns": False |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 440 | }) |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 441 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 442 | def _set_attrs(self): |
| 443 | super()._extract_attrs(self.test_obj.__dict__) |
| 444 | super()._set_attrs() |
Rupert Swarbrick | 381770d | 2020-10-12 08:50:08 +0100 | [diff] [blame] | 445 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 446 | # 'test' is used as a substitution variable in the HJson. |
Srikrishna Iyer | 09a81e9 | 2019-12-30 10:47:57 -0800 | [diff] [blame] | 447 | self.test = self.name |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 448 | self.build_mode = self.test_obj.build_mode.name |
| 449 | self.qual_name = self.run_dir_name + "." + str(self.seed) |
| 450 | self.full_name = self.sim_cfg.name + ":" + self.qual_name |
Srikrishna Iyer | 956b3f8 | 2021-04-15 15:14:56 -0700 | [diff] [blame] | 451 | self.job_name += f"_{self.build_mode}" |
Srikrishna Iyer | 08ccc23 | 2021-03-31 00:01:54 -0700 | [diff] [blame] | 452 | if self.sim_cfg.cov: |
Cindy Chen | d597758 | 2021-06-05 12:49:10 -0700 | [diff] [blame] | 453 | self.output_dirs += [self.cov_db_dir] |
Srikrishna Iyer | d37d10d | 2021-03-25 18:35:36 -0700 | [diff] [blame] | 454 | |
| 455 | # In GUI mode, the log file is not updated; hence, nothing to check. |
Srikrishna Iyer | d562377 | 2021-06-18 22:23:07 -0700 | [diff] [blame] | 456 | if not self.gui: |
Srikrishna Iyer | d37d10d | 2021-03-25 18:35:36 -0700 | [diff] [blame] | 457 | self.pass_patterns = self.run_pass_patterns |
| 458 | self.fail_patterns = self.run_fail_patterns |
Srikrishna Iyer | 4f0b090 | 2020-01-25 02:28:47 -0800 | [diff] [blame] | 459 | |
Srikrishna Iyer | d562377 | 2021-06-18 22:23:07 -0700 | [diff] [blame] | 460 | def pre_launch(self): |
| 461 | self.launcher.renew_odir = True |
| 462 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 463 | def post_finish(self, status): |
Rupert Swarbrick | a2892a5 | 2020-10-12 08:50:08 +0100 | [diff] [blame] | 464 | if status != 'P': |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 465 | # Delete the coverage data if available. |
Srikrishna Iyer | 2d15119 | 2021-02-10 16:56:16 -0800 | [diff] [blame] | 466 | rm_path(self.cov_db_test_dir) |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 467 | |
Srikrishna Iyer | 7cf7cad | 2020-01-08 11:32:53 -0800 | [diff] [blame] | 468 | @staticmethod |
| 469 | def get_seed(): |
Srikrishna Iyer | 96e5410 | 2020-03-12 22:46:50 -0700 | [diff] [blame] | 470 | # If --seeds option is passed, then those custom seeds are consumed |
| 471 | # first. If --fixed-seed <val> is also passed, the subsequent tests |
| 472 | # (once the custom seeds are consumed) will be run with the fixed seed. |
Philipp Wagner | 9a52178 | 2020-02-25 11:49:53 +0000 | [diff] [blame] | 473 | if not RunTest.seeds: |
Rupert Swarbrick | 6cc2011 | 2020-04-24 09:44:35 +0100 | [diff] [blame] | 474 | if RunTest.fixed_seed: |
| 475 | return RunTest.fixed_seed |
Srikrishna Iyer | a821bbc | 2020-01-31 00:28:02 -0800 | [diff] [blame] | 476 | for i in range(1000): |
Philipp Wagner | 75fd6c7 | 2020-02-25 11:47:41 +0000 | [diff] [blame] | 477 | seed = random.getrandbits(32) |
Srikrishna Iyer | a821bbc | 2020-01-31 00:28:02 -0800 | [diff] [blame] | 478 | RunTest.seeds.append(seed) |
Srikrishna Iyer | 7cf7cad | 2020-01-08 11:32:53 -0800 | [diff] [blame] | 479 | return RunTest.seeds.pop(0) |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 480 | |
| 481 | |
Weicai Yang | 080632d | 2020-10-16 17:52:14 -0700 | [diff] [blame] | 482 | class CovUnr(Deploy): |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 483 | """Abstraction for coverage UNR flow.""" |
Weicai Yang | 080632d | 2020-10-16 17:52:14 -0700 | [diff] [blame] | 484 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 485 | target = "cov_unr" |
Weicai Yang | 080632d | 2020-10-16 17:52:14 -0700 | [diff] [blame] | 486 | |
| 487 | def __init__(self, sim_cfg): |
Weicai Yang | 080632d | 2020-10-16 17:52:14 -0700 | [diff] [blame] | 488 | super().__init__(sim_cfg) |
| 489 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 490 | def _define_attrs(self): |
| 491 | super()._define_attrs() |
Weicai Yang | 080632d | 2020-10-16 17:52:14 -0700 | [diff] [blame] | 492 | self.mandatory_cmd_attrs.update({ |
| 493 | # tool srcs |
Srikrishna Iyer | 981c36b | 2020-12-12 12:20:35 -0800 | [diff] [blame] | 494 | "proj_root": False, |
Weicai Yang | 080632d | 2020-10-16 17:52:14 -0700 | [diff] [blame] | 495 | |
| 496 | # Need to generate filelist based on build mode |
| 497 | "sv_flist_gen_cmd": False, |
| 498 | "sv_flist_gen_dir": False, |
| 499 | "sv_flist_gen_opts": False, |
| 500 | "build_dir": False, |
| 501 | "cov_unr_build_cmd": False, |
| 502 | "cov_unr_build_opts": False, |
| 503 | "cov_unr_run_cmd": False, |
| 504 | "cov_unr_run_opts": False |
| 505 | }) |
| 506 | |
| 507 | self.mandatory_misc_attrs.update({ |
| 508 | "cov_unr_dir": False, |
Srikrishna Iyer | 8e21e51 | 2021-04-02 01:55:47 -0700 | [diff] [blame] | 509 | "cov_merge_db_dir": False, |
Weicai Yang | 080632d | 2020-10-16 17:52:14 -0700 | [diff] [blame] | 510 | "build_fail_patterns": False |
| 511 | }) |
| 512 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 513 | def _set_attrs(self): |
| 514 | super()._set_attrs() |
| 515 | self.qual_name = self.target |
| 516 | self.full_name = self.sim_cfg.name + ":" + self.qual_name |
Srikrishna Iyer | 8e21e51 | 2021-04-02 01:55:47 -0700 | [diff] [blame] | 517 | self.input_dirs += [self.cov_merge_db_dir] |
Weicai Yang | 080632d | 2020-10-16 17:52:14 -0700 | [diff] [blame] | 518 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 519 | # Reuse the build_fail_patterns set in the HJson. |
Weicai Yang | 080632d | 2020-10-16 17:52:14 -0700 | [diff] [blame] | 520 | self.fail_patterns = self.build_fail_patterns |
| 521 | |
Weicai Yang | 080632d | 2020-10-16 17:52:14 -0700 | [diff] [blame] | 522 | |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 523 | class CovMerge(Deploy): |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 524 | """Abstraction for merging coverage databases.""" |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 525 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 526 | target = "cov_merge" |
Srikrishna Iyer | 1d2899f | 2021-02-11 18:23:15 -0800 | [diff] [blame] | 527 | weight = 10 |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 528 | |
Rupert Swarbrick | f2bf370 | 2020-10-12 17:04:46 +0100 | [diff] [blame] | 529 | def __init__(self, run_items, sim_cfg): |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 530 | # Construct the cov_db_dirs right away from the run_items. This is a |
| 531 | # special variable used in the HJson. |
| 532 | self.cov_db_dirs = [] |
| 533 | for run in run_items: |
| 534 | if run.cov_db_dir not in self.cov_db_dirs: |
| 535 | self.cov_db_dirs.append(run.cov_db_dir) |
| 536 | |
Srikrishna Iyer | caa0d89 | 2021-03-01 13:15:52 -0800 | [diff] [blame] | 537 | # Early lookup the cov_merge_db_dir, which is a mandatory misc |
| 538 | # attribute anyway. We need it to compute additional cov db dirs. |
| 539 | self.cov_merge_db_dir = subst_wildcards("{cov_merge_db_dir}", |
| 540 | sim_cfg.__dict__) |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 541 | |
Srikrishna Iyer | caa0d89 | 2021-03-01 13:15:52 -0800 | [diff] [blame] | 542 | # Prune previous merged cov directories, keeping past 7 dbs. |
| 543 | prev_cov_db_dirs = clean_odirs(odir=self.cov_merge_db_dir, max_odirs=7) |
| 544 | |
| 545 | # If the --cov-merge-previous command line switch is passed, then |
| 546 | # merge coverage with the previous runs. |
| 547 | if sim_cfg.cov_merge_previous: |
| 548 | self.cov_db_dirs += [str(item) for item in prev_cov_db_dirs] |
| 549 | |
| 550 | super().__init__(sim_cfg) |
Rupert Swarbrick | f2bf370 | 2020-10-12 17:04:46 +0100 | [diff] [blame] | 551 | self.dependencies += run_items |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 552 | # Run coverage merge even if one test passes. |
Cindy Chen | 924c606 | 2021-01-27 14:27:50 -0800 | [diff] [blame] | 553 | self.needs_all_dependencies_passing = False |
Rupert Swarbrick | f2bf370 | 2020-10-12 17:04:46 +0100 | [diff] [blame] | 554 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 555 | # Append cov_db_dirs to the list of exports. |
Srikrishna Iyer | 37b0c99 | 2021-03-23 16:27:26 -0700 | [diff] [blame] | 556 | self.exports["cov_db_dirs"] = shlex.quote(" ".join(self.cov_db_dirs)) |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 557 | |
| 558 | def _define_attrs(self): |
| 559 | super()._define_attrs() |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 560 | self.mandatory_cmd_attrs.update({ |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 561 | "cov_merge_cmd": False, |
| 562 | "cov_merge_opts": False |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 563 | }) |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 564 | |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 565 | self.mandatory_misc_attrs.update({ |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 566 | "cov_merge_dir": False, |
| 567 | "cov_merge_db_dir": False |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 568 | }) |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 569 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 570 | def _set_attrs(self): |
| 571 | super()._set_attrs() |
| 572 | self.qual_name = self.target |
| 573 | self.full_name = self.sim_cfg.name + ":" + self.qual_name |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 574 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 575 | # For merging coverage db, the precise output dir is set in the HJson. |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 576 | self.odir = self.cov_merge_db_dir |
Srikrishna Iyer | 37b0c99 | 2021-03-23 16:27:26 -0700 | [diff] [blame] | 577 | self.input_dirs += self.cov_db_dirs |
Srikrishna Iyer | 08ccc23 | 2021-03-31 00:01:54 -0700 | [diff] [blame] | 578 | self.output_dirs = [self.odir] |
Srikrishna Iyer | 37b0c99 | 2021-03-23 16:27:26 -0700 | [diff] [blame] | 579 | |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 580 | |
| 581 | class CovReport(Deploy): |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 582 | """Abstraction for coverage report generation. """ |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 583 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 584 | target = "cov_report" |
Srikrishna Iyer | 1d2899f | 2021-02-11 18:23:15 -0800 | [diff] [blame] | 585 | weight = 10 |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 586 | |
Rupert Swarbrick | 381770d | 2020-10-12 08:50:08 +0100 | [diff] [blame] | 587 | def __init__(self, merge_job, sim_cfg): |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 588 | super().__init__(sim_cfg) |
Rupert Swarbrick | 381770d | 2020-10-12 08:50:08 +0100 | [diff] [blame] | 589 | self.dependencies.append(merge_job) |
| 590 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 591 | def _define_attrs(self): |
| 592 | super()._define_attrs() |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 593 | self.mandatory_cmd_attrs.update({ |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 594 | "cov_report_cmd": False, |
| 595 | "cov_report_opts": False |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 596 | }) |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 597 | |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 598 | self.mandatory_misc_attrs.update({ |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 599 | "cov_report_dir": False, |
| 600 | "cov_merge_db_dir": False, |
Srikrishna Iyer | 39ffebd | 2020-03-30 11:53:12 -0700 | [diff] [blame] | 601 | "cov_report_txt": False |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 602 | }) |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 603 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 604 | def _set_attrs(self): |
| 605 | super()._set_attrs() |
| 606 | self.qual_name = self.target |
| 607 | self.full_name = self.sim_cfg.name + ":" + self.qual_name |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 608 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 609 | # Keep track of coverage results, once the job is finished. |
| 610 | self.cov_total = "" |
| 611 | self.cov_results = "" |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 612 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 613 | def post_finish(self, status): |
| 614 | """Extract the coverage results summary for the dashboard. |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 615 | |
Srikrishna Iyer | 9b38141 | 2021-08-10 23:11:43 -0700 | [diff] [blame] | 616 | If the extraction fails, an appropriate exception is raised, which must |
| 617 | be caught by the caller to mark the job as a failure. |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 618 | """ |
| 619 | |
| 620 | if self.dry_run or status != 'P': |
| 621 | return |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 622 | |
Srikrishna Iyer | 9b38141 | 2021-08-10 23:11:43 -0700 | [diff] [blame] | 623 | results, self.cov_total = get_cov_summary_table( |
Rupert Swarbrick | a2892a5 | 2020-10-12 08:50:08 +0100 | [diff] [blame] | 624 | self.cov_report_txt, self.sim_cfg.tool) |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 625 | |
Rupert Swarbrick | a2892a5 | 2020-10-12 08:50:08 +0100 | [diff] [blame] | 626 | colalign = (("center", ) * len(results[0])) |
| 627 | self.cov_results = tabulate(results, |
| 628 | headers="firstrow", |
| 629 | tablefmt="pipe", |
| 630 | colalign=colalign) |
| 631 | |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 632 | |
| 633 | class CovAnalyze(Deploy): |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 634 | """Abstraction for running the coverage analysis tool.""" |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 635 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 636 | target = "cov_analyze" |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 637 | |
| 638 | def __init__(self, sim_cfg): |
Srikrishna Iyer | 8e21e51 | 2021-04-02 01:55:47 -0700 | [diff] [blame] | 639 | # Enforce GUI mode for coverage analysis. |
| 640 | sim_cfg.gui = True |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 641 | super().__init__(sim_cfg) |
| 642 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 643 | def _define_attrs(self): |
| 644 | super()._define_attrs() |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 645 | self.mandatory_cmd_attrs.update({ |
Srikrishna Iyer | 39ffebd | 2020-03-30 11:53:12 -0700 | [diff] [blame] | 646 | # tool srcs |
Srikrishna Iyer | 981c36b | 2020-12-12 12:20:35 -0800 | [diff] [blame] | 647 | "proj_root": False, |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 648 | "cov_analyze_cmd": False, |
| 649 | "cov_analyze_opts": False |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 650 | }) |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 651 | |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 652 | self.mandatory_misc_attrs.update({ |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 653 | "cov_analyze_dir": False, |
| 654 | "cov_merge_db_dir": False |
Srikrishna Iyer | e4662af | 2020-08-10 13:19:42 -0700 | [diff] [blame] | 655 | }) |
Srikrishna Iyer | 2a710a4 | 2020-02-10 10:39:15 -0800 | [diff] [blame] | 656 | |
Srikrishna Iyer | 6578374 | 2021-02-10 21:42:12 -0800 | [diff] [blame] | 657 | def _set_attrs(self): |
| 658 | super()._set_attrs() |
| 659 | self.qual_name = self.target |
| 660 | self.full_name = self.sim_cfg.name + ":" + self.qual_name |
Srikrishna Iyer | 8e21e51 | 2021-04-02 01:55:47 -0700 | [diff] [blame] | 661 | self.input_dirs += [self.cov_merge_db_dir] |