1fc3fe1c2SSimon Glass# Copyright (c) 2013 The Chromium OS Authors. 2fc3fe1c2SSimon Glass# 3fc3fe1c2SSimon Glass# Bloat-o-meter code used here Copyright 2004 Matt Mackall <mpm@selenic.com> 4fc3fe1c2SSimon Glass# 51a459660SWolfgang Denk# SPDX-License-Identifier: GPL-2.0+ 6fc3fe1c2SSimon Glass# 7fc3fe1c2SSimon Glass 8fc3fe1c2SSimon Glassimport collections 9fc3fe1c2SSimon Glassimport errno 10fc3fe1c2SSimon Glassfrom datetime import datetime, timedelta 11fc3fe1c2SSimon Glassimport glob 12fc3fe1c2SSimon Glassimport os 13fc3fe1c2SSimon Glassimport re 14fc3fe1c2SSimon Glassimport Queue 15fc3fe1c2SSimon Glassimport shutil 16fc3fe1c2SSimon Glassimport string 17fc3fe1c2SSimon Glassimport sys 18fc3fe1c2SSimon Glassimport threading 19fc3fe1c2SSimon Glassimport time 20fc3fe1c2SSimon Glass 21fc3fe1c2SSimon Glassimport command 22fc3fe1c2SSimon Glassimport gitutil 23fc3fe1c2SSimon Glassimport terminal 24fc3fe1c2SSimon Glassimport toolchain 25fc3fe1c2SSimon Glass 26fc3fe1c2SSimon Glass 27fc3fe1c2SSimon Glass""" 28fc3fe1c2SSimon GlassTheory of Operation 29fc3fe1c2SSimon Glass 30fc3fe1c2SSimon GlassPlease see README for user documentation, and you should be familiar with 31fc3fe1c2SSimon Glassthat before trying to make sense of this. 32fc3fe1c2SSimon Glass 33fc3fe1c2SSimon GlassBuildman works by keeping the machine as busy as possible, building different 34fc3fe1c2SSimon Glasscommits for different boards on multiple CPUs at once. 35fc3fe1c2SSimon Glass 36fc3fe1c2SSimon GlassThe source repo (self.git_dir) contains all the commits to be built. Each 37fc3fe1c2SSimon Glassthread works on a single board at a time. It checks out the first commit, 38fc3fe1c2SSimon Glassconfigures it for that board, then builds it. Then it checks out the next 39fc3fe1c2SSimon Glasscommit and builds it (typically without re-configuring). When it runs out 40fc3fe1c2SSimon Glassof commits, it gets another job from the builder and starts again with that 41fc3fe1c2SSimon Glassboard. 42fc3fe1c2SSimon Glass 43fc3fe1c2SSimon GlassClearly the builder threads could work either way - they could check out a 44fc3fe1c2SSimon Glasscommit and then built it for all boards. Using separate directories for each 45fc3fe1c2SSimon Glasscommit/board pair they could leave their build product around afterwards 46fc3fe1c2SSimon Glassalso. 47fc3fe1c2SSimon Glass 48fc3fe1c2SSimon GlassThe intent behind building a single board for multiple commits, is to make 49fc3fe1c2SSimon Glassuse of incremental builds. Since each commit is built incrementally from 50fc3fe1c2SSimon Glassthe previous one, builds are faster. Reconfiguring for a different board 51fc3fe1c2SSimon Glassremoves all intermediate object files. 52fc3fe1c2SSimon Glass 53fc3fe1c2SSimon GlassMany threads can be working at once, but each has its own working directory. 54fc3fe1c2SSimon GlassWhen a thread finishes a build, it puts the output files into a result 55fc3fe1c2SSimon Glassdirectory. 56fc3fe1c2SSimon Glass 57fc3fe1c2SSimon GlassThe base directory used by buildman is normally '../<branch>', i.e. 58fc3fe1c2SSimon Glassa directory higher than the source repository and named after the branch 59fc3fe1c2SSimon Glassbeing built. 60fc3fe1c2SSimon Glass 61fc3fe1c2SSimon GlassWithin the base directory, we have one subdirectory for each commit. Within 62fc3fe1c2SSimon Glassthat is one subdirectory for each board. Within that is the build output for 63fc3fe1c2SSimon Glassthat commit/board combination. 64fc3fe1c2SSimon Glass 65fc3fe1c2SSimon GlassBuildman also create working directories for each thread, in a .bm-work/ 66fc3fe1c2SSimon Glasssubdirectory in the base dir. 67fc3fe1c2SSimon Glass 68fc3fe1c2SSimon GlassAs an example, say we are building branch 'us-net' for boards 'sandbox' and 69fc3fe1c2SSimon Glass'seaboard', and say that us-net has two commits. We will have directories 70fc3fe1c2SSimon Glasslike this: 71fc3fe1c2SSimon Glass 72fc3fe1c2SSimon Glassus-net/ base directory 73fc3fe1c2SSimon Glass 01_of_02_g4ed4ebc_net--Add-tftp-speed-/ 74fc3fe1c2SSimon Glass sandbox/ 75fc3fe1c2SSimon Glass u-boot.bin 76fc3fe1c2SSimon Glass seaboard/ 77fc3fe1c2SSimon Glass u-boot.bin 78fc3fe1c2SSimon Glass 02_of_02_g4ed4ebc_net--Check-tftp-comp/ 79fc3fe1c2SSimon Glass sandbox/ 80fc3fe1c2SSimon Glass u-boot.bin 81fc3fe1c2SSimon Glass seaboard/ 82fc3fe1c2SSimon Glass u-boot.bin 83fc3fe1c2SSimon Glass .bm-work/ 84fc3fe1c2SSimon Glass 00/ working directory for thread 0 (contains source checkout) 85fc3fe1c2SSimon Glass build/ build output 86fc3fe1c2SSimon Glass 01/ working directory for thread 1 87fc3fe1c2SSimon Glass build/ build output 88fc3fe1c2SSimon Glass ... 89fc3fe1c2SSimon Glassu-boot/ source directory 90fc3fe1c2SSimon Glass .git/ repository 91fc3fe1c2SSimon Glass""" 92fc3fe1c2SSimon Glass 93fc3fe1c2SSimon Glass# Possible build outcomes 94fc3fe1c2SSimon GlassOUTCOME_OK, OUTCOME_WARNING, OUTCOME_ERROR, OUTCOME_UNKNOWN = range(4) 95fc3fe1c2SSimon Glass 96fc3fe1c2SSimon Glass# Translate a commit subject into a valid filename 97fc3fe1c2SSimon Glasstrans_valid_chars = string.maketrans("/: ", "---") 98fc3fe1c2SSimon Glass 99fc3fe1c2SSimon Glass 100fc3fe1c2SSimon Glassdef Mkdir(dirname): 101fc3fe1c2SSimon Glass """Make a directory if it doesn't already exist. 102fc3fe1c2SSimon Glass 103fc3fe1c2SSimon Glass Args: 104fc3fe1c2SSimon Glass dirname: Directory to create 105fc3fe1c2SSimon Glass """ 106fc3fe1c2SSimon Glass try: 107fc3fe1c2SSimon Glass os.mkdir(dirname) 108fc3fe1c2SSimon Glass except OSError as err: 109fc3fe1c2SSimon Glass if err.errno == errno.EEXIST: 110fc3fe1c2SSimon Glass pass 111fc3fe1c2SSimon Glass else: 112fc3fe1c2SSimon Glass raise 113fc3fe1c2SSimon Glass 114fc3fe1c2SSimon Glassclass BuilderJob: 115fc3fe1c2SSimon Glass """Holds information about a job to be performed by a thread 116fc3fe1c2SSimon Glass 117fc3fe1c2SSimon Glass Members: 118fc3fe1c2SSimon Glass board: Board object to build 119fc3fe1c2SSimon Glass commits: List of commit options to build. 120fc3fe1c2SSimon Glass """ 121fc3fe1c2SSimon Glass def __init__(self): 122fc3fe1c2SSimon Glass self.board = None 123fc3fe1c2SSimon Glass self.commits = [] 124fc3fe1c2SSimon Glass 125fc3fe1c2SSimon Glass 126fc3fe1c2SSimon Glassclass ResultThread(threading.Thread): 127fc3fe1c2SSimon Glass """This thread processes results from builder threads. 128fc3fe1c2SSimon Glass 129fc3fe1c2SSimon Glass It simply passes the results on to the builder. There is only one 130fc3fe1c2SSimon Glass result thread, and this helps to serialise the build output. 131fc3fe1c2SSimon Glass """ 132fc3fe1c2SSimon Glass def __init__(self, builder): 133fc3fe1c2SSimon Glass """Set up a new result thread 134fc3fe1c2SSimon Glass 135fc3fe1c2SSimon Glass Args: 136fc3fe1c2SSimon Glass builder: Builder which will be sent each result 137fc3fe1c2SSimon Glass """ 138fc3fe1c2SSimon Glass threading.Thread.__init__(self) 139fc3fe1c2SSimon Glass self.builder = builder 140fc3fe1c2SSimon Glass 141fc3fe1c2SSimon Glass def run(self): 142fc3fe1c2SSimon Glass """Called to start up the result thread. 143fc3fe1c2SSimon Glass 144fc3fe1c2SSimon Glass We collect the next result job and pass it on to the build. 145fc3fe1c2SSimon Glass """ 146fc3fe1c2SSimon Glass while True: 147fc3fe1c2SSimon Glass result = self.builder.out_queue.get() 148fc3fe1c2SSimon Glass self.builder.ProcessResult(result) 149fc3fe1c2SSimon Glass self.builder.out_queue.task_done() 150fc3fe1c2SSimon Glass 151fc3fe1c2SSimon Glass 152fc3fe1c2SSimon Glassclass BuilderThread(threading.Thread): 153fc3fe1c2SSimon Glass """This thread builds U-Boot for a particular board. 154fc3fe1c2SSimon Glass 155fc3fe1c2SSimon Glass An input queue provides each new job. We run 'make' to build U-Boot 156fc3fe1c2SSimon Glass and then pass the results on to the output queue. 157fc3fe1c2SSimon Glass 158fc3fe1c2SSimon Glass Members: 159fc3fe1c2SSimon Glass builder: The builder which contains information we might need 160fc3fe1c2SSimon Glass thread_num: Our thread number (0-n-1), used to decide on a 161fc3fe1c2SSimon Glass temporary directory 162fc3fe1c2SSimon Glass """ 163fc3fe1c2SSimon Glass def __init__(self, builder, thread_num): 164fc3fe1c2SSimon Glass """Set up a new builder thread""" 165fc3fe1c2SSimon Glass threading.Thread.__init__(self) 166fc3fe1c2SSimon Glass self.builder = builder 167fc3fe1c2SSimon Glass self.thread_num = thread_num 168fc3fe1c2SSimon Glass 169fc3fe1c2SSimon Glass def Make(self, commit, brd, stage, cwd, *args, **kwargs): 170fc3fe1c2SSimon Glass """Run 'make' on a particular commit and board. 171fc3fe1c2SSimon Glass 172fc3fe1c2SSimon Glass The source code will already be checked out, so the 'commit' 173fc3fe1c2SSimon Glass argument is only for information. 174fc3fe1c2SSimon Glass 175fc3fe1c2SSimon Glass Args: 176fc3fe1c2SSimon Glass commit: Commit object that is being built 177fc3fe1c2SSimon Glass brd: Board object that is being built 178fc3fe1c2SSimon Glass stage: Stage of the build. Valid stages are: 179fc3fe1c2SSimon Glass distclean - can be called to clean source 180fc3fe1c2SSimon Glass config - called to configure for a board 181fc3fe1c2SSimon Glass build - the main make invocation - it does the build 182fc3fe1c2SSimon Glass args: A list of arguments to pass to 'make' 183fc3fe1c2SSimon Glass kwargs: A list of keyword arguments to pass to command.RunPipe() 184fc3fe1c2SSimon Glass 185fc3fe1c2SSimon Glass Returns: 186fc3fe1c2SSimon Glass CommandResult object 187fc3fe1c2SSimon Glass """ 188fc3fe1c2SSimon Glass return self.builder.do_make(commit, brd, stage, cwd, *args, 189fc3fe1c2SSimon Glass **kwargs) 190fc3fe1c2SSimon Glass 1914266dc28SSimon Glass def RunCommit(self, commit_upto, brd, work_dir, do_config, force_build, 1924266dc28SSimon Glass force_build_failures): 193fc3fe1c2SSimon Glass """Build a particular commit. 194fc3fe1c2SSimon Glass 195fc3fe1c2SSimon Glass If the build is already done, and we are not forcing a build, we skip 196fc3fe1c2SSimon Glass the build and just return the previously-saved results. 197fc3fe1c2SSimon Glass 198fc3fe1c2SSimon Glass Args: 199fc3fe1c2SSimon Glass commit_upto: Commit number to build (0...n-1) 200fc3fe1c2SSimon Glass brd: Board object to build 201fc3fe1c2SSimon Glass work_dir: Directory to which the source will be checked out 202fc3fe1c2SSimon Glass do_config: True to run a make <board>_config on the source 203fc3fe1c2SSimon Glass force_build: Force a build even if one was previously done 2044266dc28SSimon Glass force_build_failures: Force a bulid if the previous result showed 2054266dc28SSimon Glass failure 206fc3fe1c2SSimon Glass 207fc3fe1c2SSimon Glass Returns: 208fc3fe1c2SSimon Glass tuple containing: 209fc3fe1c2SSimon Glass - CommandResult object containing the results of the build 210fc3fe1c2SSimon Glass - boolean indicating whether 'make config' is still needed 211fc3fe1c2SSimon Glass """ 212fc3fe1c2SSimon Glass # Create a default result - it will be overwritte by the call to 213fc3fe1c2SSimon Glass # self.Make() below, in the event that we do a build. 214fc3fe1c2SSimon Glass result = command.CommandResult() 215fc3fe1c2SSimon Glass result.return_code = 0 216*189a4968SSimon Glass if self.builder.in_tree: 217*189a4968SSimon Glass out_dir = work_dir 218*189a4968SSimon Glass else: 219fc3fe1c2SSimon Glass out_dir = os.path.join(work_dir, 'build') 220fc3fe1c2SSimon Glass 221fc3fe1c2SSimon Glass # Check if the job was already completed last time 222fc3fe1c2SSimon Glass done_file = self.builder.GetDoneFile(commit_upto, brd.target) 223fc3fe1c2SSimon Glass result.already_done = os.path.exists(done_file) 2244266dc28SSimon Glass will_build = (force_build or force_build_failures or 2254266dc28SSimon Glass not result.already_done) 2264266dc28SSimon Glass if result.already_done and will_build: 227fc3fe1c2SSimon Glass # Get the return code from that build and use it 228fc3fe1c2SSimon Glass with open(done_file, 'r') as fd: 229fc3fe1c2SSimon Glass result.return_code = int(fd.readline()) 230fc3fe1c2SSimon Glass err_file = self.builder.GetErrFile(commit_upto, brd.target) 231fc3fe1c2SSimon Glass if os.path.exists(err_file) and os.stat(err_file).st_size: 232fc3fe1c2SSimon Glass result.stderr = 'bad' 2334266dc28SSimon Glass elif not force_build: 2344266dc28SSimon Glass # The build passed, so no need to build it again 2354266dc28SSimon Glass will_build = False 2364266dc28SSimon Glass 2374266dc28SSimon Glass if will_build: 238fc3fe1c2SSimon Glass # We are going to have to build it. First, get a toolchain 239fc3fe1c2SSimon Glass if not self.toolchain: 240fc3fe1c2SSimon Glass try: 241fc3fe1c2SSimon Glass self.toolchain = self.builder.toolchains.Select(brd.arch) 242fc3fe1c2SSimon Glass except ValueError as err: 243fc3fe1c2SSimon Glass result.return_code = 10 244fc3fe1c2SSimon Glass result.stdout = '' 245fc3fe1c2SSimon Glass result.stderr = str(err) 246fc3fe1c2SSimon Glass # TODO(sjg@chromium.org): This gets swallowed, but needs 247fc3fe1c2SSimon Glass # to be reported. 248fc3fe1c2SSimon Glass 249fc3fe1c2SSimon Glass if self.toolchain: 250fc3fe1c2SSimon Glass # Checkout the right commit 251fc3fe1c2SSimon Glass if commit_upto is not None: 252fc3fe1c2SSimon Glass commit = self.builder.commits[commit_upto] 253fc3fe1c2SSimon Glass if self.builder.checkout: 254fc3fe1c2SSimon Glass git_dir = os.path.join(work_dir, '.git') 255fc3fe1c2SSimon Glass gitutil.Checkout(commit.hash, git_dir, work_dir, 256fc3fe1c2SSimon Glass force=True) 257fc3fe1c2SSimon Glass else: 258fc3fe1c2SSimon Glass commit = self.builder.commit # Ick, fix this for BuildCommits() 259fc3fe1c2SSimon Glass 260fc3fe1c2SSimon Glass # Set up the environment and command line 261fc3fe1c2SSimon Glass env = self.toolchain.MakeEnvironment() 262fc3fe1c2SSimon Glass Mkdir(out_dir) 263*189a4968SSimon Glass args = [] 264*189a4968SSimon Glass if not self.builder.in_tree: 265*189a4968SSimon Glass args.append('O=build') 266*189a4968SSimon Glass args.append('-s') 267fc3fe1c2SSimon Glass if self.builder.num_jobs is not None: 268fc3fe1c2SSimon Glass args.extend(['-j', str(self.builder.num_jobs)]) 269fc3fe1c2SSimon Glass config_args = ['%s_config' % brd.target] 270fc3fe1c2SSimon Glass config_out = '' 2714281ad8eSSimon Glass args.extend(self.builder.toolchains.GetMakeArguments(brd)) 272fc3fe1c2SSimon Glass 273fc3fe1c2SSimon Glass # If we need to reconfigure, do that now 274fc3fe1c2SSimon Glass if do_config: 275fc3fe1c2SSimon Glass result = self.Make(commit, brd, 'distclean', work_dir, 276fc3fe1c2SSimon Glass 'distclean', *args, env=env) 277fc3fe1c2SSimon Glass result = self.Make(commit, brd, 'config', work_dir, 278fc3fe1c2SSimon Glass *(args + config_args), env=env) 279fc3fe1c2SSimon Glass config_out = result.combined 280fc3fe1c2SSimon Glass do_config = False # No need to configure next time 281fc3fe1c2SSimon Glass if result.return_code == 0: 282fc3fe1c2SSimon Glass result = self.Make(commit, brd, 'build', work_dir, *args, 283fc3fe1c2SSimon Glass env=env) 284fc3fe1c2SSimon Glass result.stdout = config_out + result.stdout 285fc3fe1c2SSimon Glass else: 286fc3fe1c2SSimon Glass result.return_code = 1 287fc3fe1c2SSimon Glass result.stderr = 'No tool chain for %s\n' % brd.arch 288fc3fe1c2SSimon Glass result.already_done = False 289fc3fe1c2SSimon Glass 290fc3fe1c2SSimon Glass result.toolchain = self.toolchain 291fc3fe1c2SSimon Glass result.brd = brd 292fc3fe1c2SSimon Glass result.commit_upto = commit_upto 293fc3fe1c2SSimon Glass result.out_dir = out_dir 294fc3fe1c2SSimon Glass return result, do_config 295fc3fe1c2SSimon Glass 296fc3fe1c2SSimon Glass def _WriteResult(self, result, keep_outputs): 297fc3fe1c2SSimon Glass """Write a built result to the output directory. 298fc3fe1c2SSimon Glass 299fc3fe1c2SSimon Glass Args: 300fc3fe1c2SSimon Glass result: CommandResult object containing result to write 301fc3fe1c2SSimon Glass keep_outputs: True to store the output binaries, False 302fc3fe1c2SSimon Glass to delete them 303fc3fe1c2SSimon Glass """ 304fc3fe1c2SSimon Glass # Fatal error 305fc3fe1c2SSimon Glass if result.return_code < 0: 306fc3fe1c2SSimon Glass return 307fc3fe1c2SSimon Glass 308fc3fe1c2SSimon Glass # Aborted? 309fc3fe1c2SSimon Glass if result.stderr and 'No child processes' in result.stderr: 310fc3fe1c2SSimon Glass return 311fc3fe1c2SSimon Glass 312fc3fe1c2SSimon Glass if result.already_done: 313fc3fe1c2SSimon Glass return 314fc3fe1c2SSimon Glass 315fc3fe1c2SSimon Glass # Write the output and stderr 316fc3fe1c2SSimon Glass output_dir = self.builder._GetOutputDir(result.commit_upto) 317fc3fe1c2SSimon Glass Mkdir(output_dir) 318fc3fe1c2SSimon Glass build_dir = self.builder.GetBuildDir(result.commit_upto, 319fc3fe1c2SSimon Glass result.brd.target) 320fc3fe1c2SSimon Glass Mkdir(build_dir) 321fc3fe1c2SSimon Glass 322fc3fe1c2SSimon Glass outfile = os.path.join(build_dir, 'log') 323fc3fe1c2SSimon Glass with open(outfile, 'w') as fd: 324fc3fe1c2SSimon Glass if result.stdout: 325fc3fe1c2SSimon Glass fd.write(result.stdout) 326fc3fe1c2SSimon Glass 327fc3fe1c2SSimon Glass errfile = self.builder.GetErrFile(result.commit_upto, 328fc3fe1c2SSimon Glass result.brd.target) 329fc3fe1c2SSimon Glass if result.stderr: 330fc3fe1c2SSimon Glass with open(errfile, 'w') as fd: 331fc3fe1c2SSimon Glass fd.write(result.stderr) 332fc3fe1c2SSimon Glass elif os.path.exists(errfile): 333fc3fe1c2SSimon Glass os.remove(errfile) 334fc3fe1c2SSimon Glass 335fc3fe1c2SSimon Glass if result.toolchain: 336fc3fe1c2SSimon Glass # Write the build result and toolchain information. 337fc3fe1c2SSimon Glass done_file = self.builder.GetDoneFile(result.commit_upto, 338fc3fe1c2SSimon Glass result.brd.target) 339fc3fe1c2SSimon Glass with open(done_file, 'w') as fd: 340fc3fe1c2SSimon Glass fd.write('%s' % result.return_code) 341fc3fe1c2SSimon Glass with open(os.path.join(build_dir, 'toolchain'), 'w') as fd: 342fc3fe1c2SSimon Glass print >>fd, 'gcc', result.toolchain.gcc 343fc3fe1c2SSimon Glass print >>fd, 'path', result.toolchain.path 344fc3fe1c2SSimon Glass print >>fd, 'cross', result.toolchain.cross 345fc3fe1c2SSimon Glass print >>fd, 'arch', result.toolchain.arch 346fc3fe1c2SSimon Glass fd.write('%s' % result.return_code) 347fc3fe1c2SSimon Glass 348fc3fe1c2SSimon Glass with open(os.path.join(build_dir, 'toolchain'), 'w') as fd: 349fc3fe1c2SSimon Glass print >>fd, 'gcc', result.toolchain.gcc 350fc3fe1c2SSimon Glass print >>fd, 'path', result.toolchain.path 351fc3fe1c2SSimon Glass 352fc3fe1c2SSimon Glass # Write out the image and function size information and an objdump 353fc3fe1c2SSimon Glass env = result.toolchain.MakeEnvironment() 354fc3fe1c2SSimon Glass lines = [] 355fc3fe1c2SSimon Glass for fname in ['u-boot', 'spl/u-boot-spl']: 356fc3fe1c2SSimon Glass cmd = ['%snm' % self.toolchain.cross, '--size-sort', fname] 357fc3fe1c2SSimon Glass nm_result = command.RunPipe([cmd], capture=True, 358fc3fe1c2SSimon Glass capture_stderr=True, cwd=result.out_dir, 359fc3fe1c2SSimon Glass raise_on_error=False, env=env) 360fc3fe1c2SSimon Glass if nm_result.stdout: 361fc3fe1c2SSimon Glass nm = self.builder.GetFuncSizesFile(result.commit_upto, 362fc3fe1c2SSimon Glass result.brd.target, fname) 363fc3fe1c2SSimon Glass with open(nm, 'w') as fd: 364fc3fe1c2SSimon Glass print >>fd, nm_result.stdout, 365fc3fe1c2SSimon Glass 366fc3fe1c2SSimon Glass cmd = ['%sobjdump' % self.toolchain.cross, '-h', fname] 367fc3fe1c2SSimon Glass dump_result = command.RunPipe([cmd], capture=True, 368fc3fe1c2SSimon Glass capture_stderr=True, cwd=result.out_dir, 369fc3fe1c2SSimon Glass raise_on_error=False, env=env) 370fc3fe1c2SSimon Glass rodata_size = '' 371fc3fe1c2SSimon Glass if dump_result.stdout: 372fc3fe1c2SSimon Glass objdump = self.builder.GetObjdumpFile(result.commit_upto, 373fc3fe1c2SSimon Glass result.brd.target, fname) 374fc3fe1c2SSimon Glass with open(objdump, 'w') as fd: 375fc3fe1c2SSimon Glass print >>fd, dump_result.stdout, 376fc3fe1c2SSimon Glass for line in dump_result.stdout.splitlines(): 377fc3fe1c2SSimon Glass fields = line.split() 378fc3fe1c2SSimon Glass if len(fields) > 5 and fields[1] == '.rodata': 379fc3fe1c2SSimon Glass rodata_size = fields[2] 380fc3fe1c2SSimon Glass 381fc3fe1c2SSimon Glass cmd = ['%ssize' % self.toolchain.cross, fname] 382fc3fe1c2SSimon Glass size_result = command.RunPipe([cmd], capture=True, 383fc3fe1c2SSimon Glass capture_stderr=True, cwd=result.out_dir, 384fc3fe1c2SSimon Glass raise_on_error=False, env=env) 385fc3fe1c2SSimon Glass if size_result.stdout: 386fc3fe1c2SSimon Glass lines.append(size_result.stdout.splitlines()[1] + ' ' + 387fc3fe1c2SSimon Glass rodata_size) 388fc3fe1c2SSimon Glass 389fc3fe1c2SSimon Glass # Write out the image sizes file. This is similar to the output 390fc3fe1c2SSimon Glass # of binutil's 'size' utility, but it omits the header line and 391fc3fe1c2SSimon Glass # adds an additional hex value at the end of each line for the 392fc3fe1c2SSimon Glass # rodata size 393fc3fe1c2SSimon Glass if len(lines): 394fc3fe1c2SSimon Glass sizes = self.builder.GetSizesFile(result.commit_upto, 395fc3fe1c2SSimon Glass result.brd.target) 396fc3fe1c2SSimon Glass with open(sizes, 'w') as fd: 397fc3fe1c2SSimon Glass print >>fd, '\n'.join(lines) 398fc3fe1c2SSimon Glass 399fc3fe1c2SSimon Glass # Now write the actual build output 400fc3fe1c2SSimon Glass if keep_outputs: 401fc3fe1c2SSimon Glass patterns = ['u-boot', '*.bin', 'u-boot.dtb', '*.map', 402fc3fe1c2SSimon Glass 'include/autoconf.mk', 'spl/u-boot-spl', 403fc3fe1c2SSimon Glass 'spl/u-boot-spl.bin'] 404fc3fe1c2SSimon Glass for pattern in patterns: 405fc3fe1c2SSimon Glass file_list = glob.glob(os.path.join(result.out_dir, pattern)) 406fc3fe1c2SSimon Glass for fname in file_list: 407fc3fe1c2SSimon Glass shutil.copy(fname, build_dir) 408fc3fe1c2SSimon Glass 409fc3fe1c2SSimon Glass 410fc3fe1c2SSimon Glass def RunJob(self, job): 411fc3fe1c2SSimon Glass """Run a single job 412fc3fe1c2SSimon Glass 413fc3fe1c2SSimon Glass A job consists of a building a list of commits for a particular board. 414fc3fe1c2SSimon Glass 415fc3fe1c2SSimon Glass Args: 416fc3fe1c2SSimon Glass job: Job to build 417fc3fe1c2SSimon Glass """ 418fc3fe1c2SSimon Glass brd = job.board 419fc3fe1c2SSimon Glass work_dir = self.builder.GetThreadDir(self.thread_num) 420fc3fe1c2SSimon Glass self.toolchain = None 421fc3fe1c2SSimon Glass if job.commits: 422fc3fe1c2SSimon Glass # Run 'make board_config' on the first commit 423fc3fe1c2SSimon Glass do_config = True 424fc3fe1c2SSimon Glass commit_upto = 0 425fc3fe1c2SSimon Glass force_build = False 426fc3fe1c2SSimon Glass for commit_upto in range(0, len(job.commits), job.step): 427fc3fe1c2SSimon Glass result, request_config = self.RunCommit(commit_upto, brd, 428fc3fe1c2SSimon Glass work_dir, do_config, 4294266dc28SSimon Glass force_build or self.builder.force_build, 4304266dc28SSimon Glass self.builder.force_build_failures) 431fc3fe1c2SSimon Glass failed = result.return_code or result.stderr 43221fe8ec3SSimon Glass did_config = do_config 433fc3fe1c2SSimon Glass if failed and not do_config: 434fc3fe1c2SSimon Glass # If our incremental build failed, try building again 435fc3fe1c2SSimon Glass # with a reconfig. 436fc3fe1c2SSimon Glass if self.builder.force_config_on_failure: 437fc3fe1c2SSimon Glass result, request_config = self.RunCommit(commit_upto, 4384266dc28SSimon Glass brd, work_dir, True, True, False) 43921fe8ec3SSimon Glass did_config = True 44097e91526SSimon Glass if not self.builder.force_reconfig: 441fc3fe1c2SSimon Glass do_config = request_config 442fc3fe1c2SSimon Glass 443fc3fe1c2SSimon Glass # If we built that commit, then config is done. But if we got 444fc3fe1c2SSimon Glass # an warning, reconfig next time to force it to build the same 445fc3fe1c2SSimon Glass # files that created warnings this time. Otherwise an 446fc3fe1c2SSimon Glass # incremental build may not build the same file, and we will 447fc3fe1c2SSimon Glass # think that the warning has gone away. 448fc3fe1c2SSimon Glass # We could avoid this by using -Werror everywhere... 449fc3fe1c2SSimon Glass # For errors, the problem doesn't happen, since presumably 450fc3fe1c2SSimon Glass # the build stopped and didn't generate output, so will retry 451fc3fe1c2SSimon Glass # that file next time. So we could detect warnings and deal 452fc3fe1c2SSimon Glass # with them specially here. For now, we just reconfigure if 453fc3fe1c2SSimon Glass # anything goes work. 454fc3fe1c2SSimon Glass # Of course this is substantially slower if there are build 455fc3fe1c2SSimon Glass # errors/warnings (e.g. 2-3x slower even if only 10% of builds 456fc3fe1c2SSimon Glass # have problems). 45721fe8ec3SSimon Glass if (failed and not result.already_done and not did_config and 458fc3fe1c2SSimon Glass self.builder.force_config_on_failure): 459fc3fe1c2SSimon Glass # If this build failed, try the next one with a 460fc3fe1c2SSimon Glass # reconfigure. 461fc3fe1c2SSimon Glass # Sometimes if the board_config.h file changes it can mess 462fc3fe1c2SSimon Glass # with dependencies, and we get: 463fc3fe1c2SSimon Glass # make: *** No rule to make target `include/autoconf.mk', 464fc3fe1c2SSimon Glass # needed by `depend'. 465fc3fe1c2SSimon Glass do_config = True 466fc3fe1c2SSimon Glass force_build = True 467fc3fe1c2SSimon Glass else: 468fc3fe1c2SSimon Glass force_build = False 469fc3fe1c2SSimon Glass if self.builder.force_config_on_failure: 470fc3fe1c2SSimon Glass if failed: 471fc3fe1c2SSimon Glass do_config = True 472fc3fe1c2SSimon Glass result.commit_upto = commit_upto 473fc3fe1c2SSimon Glass if result.return_code < 0: 474fc3fe1c2SSimon Glass raise ValueError('Interrupt') 475fc3fe1c2SSimon Glass 476fc3fe1c2SSimon Glass # We have the build results, so output the result 477fc3fe1c2SSimon Glass self._WriteResult(result, job.keep_outputs) 478fc3fe1c2SSimon Glass self.builder.out_queue.put(result) 479fc3fe1c2SSimon Glass else: 480fc3fe1c2SSimon Glass # Just build the currently checked-out build 481fc3fe1c2SSimon Glass result = self.RunCommit(None, True) 482fc3fe1c2SSimon Glass result.commit_upto = self.builder.upto 483fc3fe1c2SSimon Glass self.builder.out_queue.put(result) 484fc3fe1c2SSimon Glass 485fc3fe1c2SSimon Glass def run(self): 486fc3fe1c2SSimon Glass """Our thread's run function 487fc3fe1c2SSimon Glass 488fc3fe1c2SSimon Glass This thread picks a job from the queue, runs it, and then goes to the 489fc3fe1c2SSimon Glass next job. 490fc3fe1c2SSimon Glass """ 491fc3fe1c2SSimon Glass alive = True 492fc3fe1c2SSimon Glass while True: 493fc3fe1c2SSimon Glass job = self.builder.queue.get() 494fc3fe1c2SSimon Glass try: 495fc3fe1c2SSimon Glass if self.builder.active and alive: 496fc3fe1c2SSimon Glass self.RunJob(job) 497fc3fe1c2SSimon Glass except Exception as err: 498fc3fe1c2SSimon Glass alive = False 499fc3fe1c2SSimon Glass print err 500fc3fe1c2SSimon Glass self.builder.queue.task_done() 501fc3fe1c2SSimon Glass 502fc3fe1c2SSimon Glass 503fc3fe1c2SSimon Glassclass Builder: 504fc3fe1c2SSimon Glass """Class for building U-Boot for a particular commit. 505fc3fe1c2SSimon Glass 506fc3fe1c2SSimon Glass Public members: (many should ->private) 507fc3fe1c2SSimon Glass active: True if the builder is active and has not been stopped 508fc3fe1c2SSimon Glass already_done: Number of builds already completed 509fc3fe1c2SSimon Glass base_dir: Base directory to use for builder 510fc3fe1c2SSimon Glass checkout: True to check out source, False to skip that step. 511fc3fe1c2SSimon Glass This is used for testing. 512fc3fe1c2SSimon Glass col: terminal.Color() object 513fc3fe1c2SSimon Glass count: Number of commits to build 514fc3fe1c2SSimon Glass do_make: Method to call to invoke Make 515fc3fe1c2SSimon Glass fail: Number of builds that failed due to error 516fc3fe1c2SSimon Glass force_build: Force building even if a build already exists 517fc3fe1c2SSimon Glass force_config_on_failure: If a commit fails for a board, disable 518fc3fe1c2SSimon Glass incremental building for the next commit we build for that 519fc3fe1c2SSimon Glass board, so that we will see all warnings/errors again. 5204266dc28SSimon Glass force_build_failures: If a previously-built build (i.e. built on 5214266dc28SSimon Glass a previous run of buildman) is marked as failed, rebuild it. 522fc3fe1c2SSimon Glass git_dir: Git directory containing source repository 523fc3fe1c2SSimon Glass last_line_len: Length of the last line we printed (used for erasing 524fc3fe1c2SSimon Glass it with new progress information) 525fc3fe1c2SSimon Glass num_jobs: Number of jobs to run at once (passed to make as -j) 526fc3fe1c2SSimon Glass num_threads: Number of builder threads to run 527fc3fe1c2SSimon Glass out_queue: Queue of results to process 528fc3fe1c2SSimon Glass re_make_err: Compiled regular expression for ignore_lines 529fc3fe1c2SSimon Glass queue: Queue of jobs to run 530fc3fe1c2SSimon Glass threads: List of active threads 531fc3fe1c2SSimon Glass toolchains: Toolchains object to use for building 532fc3fe1c2SSimon Glass upto: Current commit number we are building (0.count-1) 533fc3fe1c2SSimon Glass warned: Number of builds that produced at least one warning 53497e91526SSimon Glass force_reconfig: Reconfigure U-Boot on each comiit. This disables 53597e91526SSimon Glass incremental building, where buildman reconfigures on the first 53697e91526SSimon Glass commit for a baord, and then just does an incremental build for 53797e91526SSimon Glass the following commits. In fact buildman will reconfigure and 53897e91526SSimon Glass retry for any failing commits, so generally the only effect of 53997e91526SSimon Glass this option is to slow things down. 540*189a4968SSimon Glass in_tree: Build U-Boot in-tree instead of specifying an output 541*189a4968SSimon Glass directory separate from the source code. This option is really 542*189a4968SSimon Glass only useful for testing in-tree builds. 543fc3fe1c2SSimon Glass 544fc3fe1c2SSimon Glass Private members: 545fc3fe1c2SSimon Glass _base_board_dict: Last-summarised Dict of boards 546fc3fe1c2SSimon Glass _base_err_lines: Last-summarised list of errors 547fc3fe1c2SSimon Glass _build_period_us: Time taken for a single build (float object). 548fc3fe1c2SSimon Glass _complete_delay: Expected delay until completion (timedelta) 549fc3fe1c2SSimon Glass _next_delay_update: Next time we plan to display a progress update 550fc3fe1c2SSimon Glass (datatime) 551fc3fe1c2SSimon Glass _show_unknown: Show unknown boards (those not built) in summary 552fc3fe1c2SSimon Glass _timestamps: List of timestamps for the completion of the last 553fc3fe1c2SSimon Glass last _timestamp_count builds. Each is a datetime object. 554fc3fe1c2SSimon Glass _timestamp_count: Number of timestamps to keep in our list. 555fc3fe1c2SSimon Glass _working_dir: Base working directory containing all threads 556fc3fe1c2SSimon Glass """ 557fc3fe1c2SSimon Glass class Outcome: 558fc3fe1c2SSimon Glass """Records a build outcome for a single make invocation 559fc3fe1c2SSimon Glass 560fc3fe1c2SSimon Glass Public Members: 561fc3fe1c2SSimon Glass rc: Outcome value (OUTCOME_...) 562fc3fe1c2SSimon Glass err_lines: List of error lines or [] if none 563fc3fe1c2SSimon Glass sizes: Dictionary of image size information, keyed by filename 564fc3fe1c2SSimon Glass - Each value is itself a dictionary containing 565fc3fe1c2SSimon Glass values for 'text', 'data' and 'bss', being the integer 566fc3fe1c2SSimon Glass size in bytes of each section. 567fc3fe1c2SSimon Glass func_sizes: Dictionary keyed by filename - e.g. 'u-boot'. Each 568fc3fe1c2SSimon Glass value is itself a dictionary: 569fc3fe1c2SSimon Glass key: function name 570fc3fe1c2SSimon Glass value: Size of function in bytes 571fc3fe1c2SSimon Glass """ 572fc3fe1c2SSimon Glass def __init__(self, rc, err_lines, sizes, func_sizes): 573fc3fe1c2SSimon Glass self.rc = rc 574fc3fe1c2SSimon Glass self.err_lines = err_lines 575fc3fe1c2SSimon Glass self.sizes = sizes 576fc3fe1c2SSimon Glass self.func_sizes = func_sizes 577fc3fe1c2SSimon Glass 578fc3fe1c2SSimon Glass def __init__(self, toolchains, base_dir, git_dir, num_threads, num_jobs, 579fc3fe1c2SSimon Glass checkout=True, show_unknown=True, step=1): 580fc3fe1c2SSimon Glass """Create a new Builder object 581fc3fe1c2SSimon Glass 582fc3fe1c2SSimon Glass Args: 583fc3fe1c2SSimon Glass toolchains: Toolchains object to use for building 584fc3fe1c2SSimon Glass base_dir: Base directory to use for builder 585fc3fe1c2SSimon Glass git_dir: Git directory containing source repository 586fc3fe1c2SSimon Glass num_threads: Number of builder threads to run 587fc3fe1c2SSimon Glass num_jobs: Number of jobs to run at once (passed to make as -j) 588fc3fe1c2SSimon Glass checkout: True to check out source, False to skip that step. 589fc3fe1c2SSimon Glass This is used for testing. 590fc3fe1c2SSimon Glass show_unknown: Show unknown boards (those not built) in summary 591fc3fe1c2SSimon Glass step: 1 to process every commit, n to process every nth commit 592fc3fe1c2SSimon Glass """ 593fc3fe1c2SSimon Glass self.toolchains = toolchains 594fc3fe1c2SSimon Glass self.base_dir = base_dir 595fc3fe1c2SSimon Glass self._working_dir = os.path.join(base_dir, '.bm-work') 596fc3fe1c2SSimon Glass self.threads = [] 597fc3fe1c2SSimon Glass self.active = True 598fc3fe1c2SSimon Glass self.do_make = self.Make 599fc3fe1c2SSimon Glass self.checkout = checkout 600fc3fe1c2SSimon Glass self.num_threads = num_threads 601fc3fe1c2SSimon Glass self.num_jobs = num_jobs 602fc3fe1c2SSimon Glass self.already_done = 0 603fc3fe1c2SSimon Glass self.force_build = False 604fc3fe1c2SSimon Glass self.git_dir = git_dir 605fc3fe1c2SSimon Glass self._show_unknown = show_unknown 606fc3fe1c2SSimon Glass self._timestamp_count = 10 607fc3fe1c2SSimon Glass self._build_period_us = None 608fc3fe1c2SSimon Glass self._complete_delay = None 609fc3fe1c2SSimon Glass self._next_delay_update = datetime.now() 610fc3fe1c2SSimon Glass self.force_config_on_failure = True 6114266dc28SSimon Glass self.force_build_failures = False 61297e91526SSimon Glass self.force_reconfig = False 613fc3fe1c2SSimon Glass self._step = step 614*189a4968SSimon Glass self.in_tree = False 615fc3fe1c2SSimon Glass 616fc3fe1c2SSimon Glass self.col = terminal.Color() 617fc3fe1c2SSimon Glass 618fc3fe1c2SSimon Glass self.queue = Queue.Queue() 619fc3fe1c2SSimon Glass self.out_queue = Queue.Queue() 620fc3fe1c2SSimon Glass for i in range(self.num_threads): 621fc3fe1c2SSimon Glass t = BuilderThread(self, i) 622fc3fe1c2SSimon Glass t.setDaemon(True) 623fc3fe1c2SSimon Glass t.start() 624fc3fe1c2SSimon Glass self.threads.append(t) 625fc3fe1c2SSimon Glass 626fc3fe1c2SSimon Glass self.last_line_len = 0 627fc3fe1c2SSimon Glass t = ResultThread(self) 628fc3fe1c2SSimon Glass t.setDaemon(True) 629fc3fe1c2SSimon Glass t.start() 630fc3fe1c2SSimon Glass self.threads.append(t) 631fc3fe1c2SSimon Glass 632fc3fe1c2SSimon Glass ignore_lines = ['(make.*Waiting for unfinished)', '(Segmentation fault)'] 633fc3fe1c2SSimon Glass self.re_make_err = re.compile('|'.join(ignore_lines)) 634fc3fe1c2SSimon Glass 635fc3fe1c2SSimon Glass def __del__(self): 636fc3fe1c2SSimon Glass """Get rid of all threads created by the builder""" 637fc3fe1c2SSimon Glass for t in self.threads: 638fc3fe1c2SSimon Glass del t 639fc3fe1c2SSimon Glass 640fc3fe1c2SSimon Glass def _AddTimestamp(self): 641fc3fe1c2SSimon Glass """Add a new timestamp to the list and record the build period. 642fc3fe1c2SSimon Glass 643fc3fe1c2SSimon Glass The build period is the length of time taken to perform a single 644fc3fe1c2SSimon Glass build (one board, one commit). 645fc3fe1c2SSimon Glass """ 646fc3fe1c2SSimon Glass now = datetime.now() 647fc3fe1c2SSimon Glass self._timestamps.append(now) 648fc3fe1c2SSimon Glass count = len(self._timestamps) 649fc3fe1c2SSimon Glass delta = self._timestamps[-1] - self._timestamps[0] 650fc3fe1c2SSimon Glass seconds = delta.total_seconds() 651fc3fe1c2SSimon Glass 652fc3fe1c2SSimon Glass # If we have enough data, estimate build period (time taken for a 653fc3fe1c2SSimon Glass # single build) and therefore completion time. 654fc3fe1c2SSimon Glass if count > 1 and self._next_delay_update < now: 655fc3fe1c2SSimon Glass self._next_delay_update = now + timedelta(seconds=2) 656fc3fe1c2SSimon Glass if seconds > 0: 657fc3fe1c2SSimon Glass self._build_period = float(seconds) / count 658fc3fe1c2SSimon Glass todo = self.count - self.upto 659fc3fe1c2SSimon Glass self._complete_delay = timedelta(microseconds= 660fc3fe1c2SSimon Glass self._build_period * todo * 1000000) 661fc3fe1c2SSimon Glass # Round it 662fc3fe1c2SSimon Glass self._complete_delay -= timedelta( 663fc3fe1c2SSimon Glass microseconds=self._complete_delay.microseconds) 664fc3fe1c2SSimon Glass 665fc3fe1c2SSimon Glass if seconds > 60: 666fc3fe1c2SSimon Glass self._timestamps.popleft() 667fc3fe1c2SSimon Glass count -= 1 668fc3fe1c2SSimon Glass 669fc3fe1c2SSimon Glass def ClearLine(self, length): 670fc3fe1c2SSimon Glass """Clear any characters on the current line 671fc3fe1c2SSimon Glass 672fc3fe1c2SSimon Glass Make way for a new line of length 'length', by outputting enough 673fc3fe1c2SSimon Glass spaces to clear out the old line. Then remember the new length for 674fc3fe1c2SSimon Glass next time. 675fc3fe1c2SSimon Glass 676fc3fe1c2SSimon Glass Args: 677fc3fe1c2SSimon Glass length: Length of new line, in characters 678fc3fe1c2SSimon Glass """ 679fc3fe1c2SSimon Glass if length < self.last_line_len: 680fc3fe1c2SSimon Glass print ' ' * (self.last_line_len - length), 681fc3fe1c2SSimon Glass print '\r', 682fc3fe1c2SSimon Glass self.last_line_len = length 683fc3fe1c2SSimon Glass sys.stdout.flush() 684fc3fe1c2SSimon Glass 685fc3fe1c2SSimon Glass def SelectCommit(self, commit, checkout=True): 686fc3fe1c2SSimon Glass """Checkout the selected commit for this build 687fc3fe1c2SSimon Glass """ 688fc3fe1c2SSimon Glass self.commit = commit 689fc3fe1c2SSimon Glass if checkout and self.checkout: 690fc3fe1c2SSimon Glass gitutil.Checkout(commit.hash) 691fc3fe1c2SSimon Glass 692fc3fe1c2SSimon Glass def Make(self, commit, brd, stage, cwd, *args, **kwargs): 693fc3fe1c2SSimon Glass """Run make 694fc3fe1c2SSimon Glass 695fc3fe1c2SSimon Glass Args: 696fc3fe1c2SSimon Glass commit: Commit object that is being built 697fc3fe1c2SSimon Glass brd: Board object that is being built 698fc3fe1c2SSimon Glass stage: Stage that we are at (distclean, config, build) 699fc3fe1c2SSimon Glass cwd: Directory where make should be run 700fc3fe1c2SSimon Glass args: Arguments to pass to make 701fc3fe1c2SSimon Glass kwargs: Arguments to pass to command.RunPipe() 702fc3fe1c2SSimon Glass """ 703fc3fe1c2SSimon Glass cmd = ['make'] + list(args) 704fc3fe1c2SSimon Glass result = command.RunPipe([cmd], capture=True, capture_stderr=True, 705fc3fe1c2SSimon Glass cwd=cwd, raise_on_error=False, **kwargs) 706fc3fe1c2SSimon Glass return result 707fc3fe1c2SSimon Glass 708fc3fe1c2SSimon Glass def ProcessResult(self, result): 709fc3fe1c2SSimon Glass """Process the result of a build, showing progress information 710fc3fe1c2SSimon Glass 711fc3fe1c2SSimon Glass Args: 712fc3fe1c2SSimon Glass result: A CommandResult object 713fc3fe1c2SSimon Glass """ 714fc3fe1c2SSimon Glass col = terminal.Color() 715fc3fe1c2SSimon Glass if result: 716fc3fe1c2SSimon Glass target = result.brd.target 717fc3fe1c2SSimon Glass 718fc3fe1c2SSimon Glass if result.return_code < 0: 719fc3fe1c2SSimon Glass self.active = False 720fc3fe1c2SSimon Glass command.StopAll() 721fc3fe1c2SSimon Glass return 722fc3fe1c2SSimon Glass 723fc3fe1c2SSimon Glass self.upto += 1 724fc3fe1c2SSimon Glass if result.return_code != 0: 725fc3fe1c2SSimon Glass self.fail += 1 726fc3fe1c2SSimon Glass elif result.stderr: 727fc3fe1c2SSimon Glass self.warned += 1 728fc3fe1c2SSimon Glass if result.already_done: 729fc3fe1c2SSimon Glass self.already_done += 1 730fc3fe1c2SSimon Glass else: 731fc3fe1c2SSimon Glass target = '(starting)' 732fc3fe1c2SSimon Glass 733fc3fe1c2SSimon Glass # Display separate counts for ok, warned and fail 734fc3fe1c2SSimon Glass ok = self.upto - self.warned - self.fail 735fc3fe1c2SSimon Glass line = '\r' + self.col.Color(self.col.GREEN, '%5d' % ok) 736fc3fe1c2SSimon Glass line += self.col.Color(self.col.YELLOW, '%5d' % self.warned) 737fc3fe1c2SSimon Glass line += self.col.Color(self.col.RED, '%5d' % self.fail) 738fc3fe1c2SSimon Glass 739fc3fe1c2SSimon Glass name = ' /%-5d ' % self.count 740fc3fe1c2SSimon Glass 741fc3fe1c2SSimon Glass # Add our current completion time estimate 742fc3fe1c2SSimon Glass self._AddTimestamp() 743fc3fe1c2SSimon Glass if self._complete_delay: 744fc3fe1c2SSimon Glass name += '%s : ' % self._complete_delay 745fc3fe1c2SSimon Glass # When building all boards for a commit, we can print a commit 746fc3fe1c2SSimon Glass # progress message. 747fc3fe1c2SSimon Glass if result and result.commit_upto is None: 748fc3fe1c2SSimon Glass name += 'commit %2d/%-3d' % (self.commit_upto + 1, 749fc3fe1c2SSimon Glass self.commit_count) 750fc3fe1c2SSimon Glass 751fc3fe1c2SSimon Glass name += target 752fc3fe1c2SSimon Glass print line + name, 753fc3fe1c2SSimon Glass length = 13 + len(name) 754fc3fe1c2SSimon Glass self.ClearLine(length) 755fc3fe1c2SSimon Glass 756fc3fe1c2SSimon Glass def _GetOutputDir(self, commit_upto): 757fc3fe1c2SSimon Glass """Get the name of the output directory for a commit number 758fc3fe1c2SSimon Glass 759fc3fe1c2SSimon Glass The output directory is typically .../<branch>/<commit>. 760fc3fe1c2SSimon Glass 761fc3fe1c2SSimon Glass Args: 762fc3fe1c2SSimon Glass commit_upto: Commit number to use (0..self.count-1) 763fc3fe1c2SSimon Glass """ 764fc3fe1c2SSimon Glass commit = self.commits[commit_upto] 765fc3fe1c2SSimon Glass subject = commit.subject.translate(trans_valid_chars) 766fc3fe1c2SSimon Glass commit_dir = ('%02d_of_%02d_g%s_%s' % (commit_upto + 1, 767fc3fe1c2SSimon Glass self.commit_count, commit.hash, subject[:20])) 768fc3fe1c2SSimon Glass output_dir = os.path.join(self.base_dir, commit_dir) 769fc3fe1c2SSimon Glass return output_dir 770fc3fe1c2SSimon Glass 771fc3fe1c2SSimon Glass def GetBuildDir(self, commit_upto, target): 772fc3fe1c2SSimon Glass """Get the name of the build directory for a commit number 773fc3fe1c2SSimon Glass 774fc3fe1c2SSimon Glass The build directory is typically .../<branch>/<commit>/<target>. 775fc3fe1c2SSimon Glass 776fc3fe1c2SSimon Glass Args: 777fc3fe1c2SSimon Glass commit_upto: Commit number to use (0..self.count-1) 778fc3fe1c2SSimon Glass target: Target name 779fc3fe1c2SSimon Glass """ 780fc3fe1c2SSimon Glass output_dir = self._GetOutputDir(commit_upto) 781fc3fe1c2SSimon Glass return os.path.join(output_dir, target) 782fc3fe1c2SSimon Glass 783fc3fe1c2SSimon Glass def GetDoneFile(self, commit_upto, target): 784fc3fe1c2SSimon Glass """Get the name of the done file for a commit number 785fc3fe1c2SSimon Glass 786fc3fe1c2SSimon Glass Args: 787fc3fe1c2SSimon Glass commit_upto: Commit number to use (0..self.count-1) 788fc3fe1c2SSimon Glass target: Target name 789fc3fe1c2SSimon Glass """ 790fc3fe1c2SSimon Glass return os.path.join(self.GetBuildDir(commit_upto, target), 'done') 791fc3fe1c2SSimon Glass 792fc3fe1c2SSimon Glass def GetSizesFile(self, commit_upto, target): 793fc3fe1c2SSimon Glass """Get the name of the sizes file for a commit number 794fc3fe1c2SSimon Glass 795fc3fe1c2SSimon Glass Args: 796fc3fe1c2SSimon Glass commit_upto: Commit number to use (0..self.count-1) 797fc3fe1c2SSimon Glass target: Target name 798fc3fe1c2SSimon Glass """ 799fc3fe1c2SSimon Glass return os.path.join(self.GetBuildDir(commit_upto, target), 'sizes') 800fc3fe1c2SSimon Glass 801fc3fe1c2SSimon Glass def GetFuncSizesFile(self, commit_upto, target, elf_fname): 802fc3fe1c2SSimon Glass """Get the name of the funcsizes file for a commit number and ELF file 803fc3fe1c2SSimon Glass 804fc3fe1c2SSimon Glass Args: 805fc3fe1c2SSimon Glass commit_upto: Commit number to use (0..self.count-1) 806fc3fe1c2SSimon Glass target: Target name 807fc3fe1c2SSimon Glass elf_fname: Filename of elf image 808fc3fe1c2SSimon Glass """ 809fc3fe1c2SSimon Glass return os.path.join(self.GetBuildDir(commit_upto, target), 810fc3fe1c2SSimon Glass '%s.sizes' % elf_fname.replace('/', '-')) 811fc3fe1c2SSimon Glass 812fc3fe1c2SSimon Glass def GetObjdumpFile(self, commit_upto, target, elf_fname): 813fc3fe1c2SSimon Glass """Get the name of the objdump file for a commit number and ELF file 814fc3fe1c2SSimon Glass 815fc3fe1c2SSimon Glass Args: 816fc3fe1c2SSimon Glass commit_upto: Commit number to use (0..self.count-1) 817fc3fe1c2SSimon Glass target: Target name 818fc3fe1c2SSimon Glass elf_fname: Filename of elf image 819fc3fe1c2SSimon Glass """ 820fc3fe1c2SSimon Glass return os.path.join(self.GetBuildDir(commit_upto, target), 821fc3fe1c2SSimon Glass '%s.objdump' % elf_fname.replace('/', '-')) 822fc3fe1c2SSimon Glass 823fc3fe1c2SSimon Glass def GetErrFile(self, commit_upto, target): 824fc3fe1c2SSimon Glass """Get the name of the err file for a commit number 825fc3fe1c2SSimon Glass 826fc3fe1c2SSimon Glass Args: 827fc3fe1c2SSimon Glass commit_upto: Commit number to use (0..self.count-1) 828fc3fe1c2SSimon Glass target: Target name 829fc3fe1c2SSimon Glass """ 830fc3fe1c2SSimon Glass output_dir = self.GetBuildDir(commit_upto, target) 831fc3fe1c2SSimon Glass return os.path.join(output_dir, 'err') 832fc3fe1c2SSimon Glass 833fc3fe1c2SSimon Glass def FilterErrors(self, lines): 834fc3fe1c2SSimon Glass """Filter out errors in which we have no interest 835fc3fe1c2SSimon Glass 836fc3fe1c2SSimon Glass We should probably use map(). 837fc3fe1c2SSimon Glass 838fc3fe1c2SSimon Glass Args: 839fc3fe1c2SSimon Glass lines: List of error lines, each a string 840fc3fe1c2SSimon Glass Returns: 841fc3fe1c2SSimon Glass New list with only interesting lines included 842fc3fe1c2SSimon Glass """ 843fc3fe1c2SSimon Glass out_lines = [] 844fc3fe1c2SSimon Glass for line in lines: 845fc3fe1c2SSimon Glass if not self.re_make_err.search(line): 846fc3fe1c2SSimon Glass out_lines.append(line) 847fc3fe1c2SSimon Glass return out_lines 848fc3fe1c2SSimon Glass 849fc3fe1c2SSimon Glass def ReadFuncSizes(self, fname, fd): 850fc3fe1c2SSimon Glass """Read function sizes from the output of 'nm' 851fc3fe1c2SSimon Glass 852fc3fe1c2SSimon Glass Args: 853fc3fe1c2SSimon Glass fd: File containing data to read 854fc3fe1c2SSimon Glass fname: Filename we are reading from (just for errors) 855fc3fe1c2SSimon Glass 856fc3fe1c2SSimon Glass Returns: 857fc3fe1c2SSimon Glass Dictionary containing size of each function in bytes, indexed by 858fc3fe1c2SSimon Glass function name. 859fc3fe1c2SSimon Glass """ 860fc3fe1c2SSimon Glass sym = {} 861fc3fe1c2SSimon Glass for line in fd.readlines(): 862fc3fe1c2SSimon Glass try: 863fc3fe1c2SSimon Glass size, type, name = line[:-1].split() 864fc3fe1c2SSimon Glass except: 865fc3fe1c2SSimon Glass print "Invalid line in file '%s': '%s'" % (fname, line[:-1]) 866fc3fe1c2SSimon Glass continue 867fc3fe1c2SSimon Glass if type in 'tTdDbB': 868fc3fe1c2SSimon Glass # function names begin with '.' on 64-bit powerpc 869fc3fe1c2SSimon Glass if '.' in name[1:]: 870fc3fe1c2SSimon Glass name = 'static.' + name.split('.')[0] 871fc3fe1c2SSimon Glass sym[name] = sym.get(name, 0) + int(size, 16) 872fc3fe1c2SSimon Glass return sym 873fc3fe1c2SSimon Glass 874fc3fe1c2SSimon Glass def GetBuildOutcome(self, commit_upto, target, read_func_sizes): 875fc3fe1c2SSimon Glass """Work out the outcome of a build. 876fc3fe1c2SSimon Glass 877fc3fe1c2SSimon Glass Args: 878fc3fe1c2SSimon Glass commit_upto: Commit number to check (0..n-1) 879fc3fe1c2SSimon Glass target: Target board to check 880fc3fe1c2SSimon Glass read_func_sizes: True to read function size information 881fc3fe1c2SSimon Glass 882fc3fe1c2SSimon Glass Returns: 883fc3fe1c2SSimon Glass Outcome object 884fc3fe1c2SSimon Glass """ 885fc3fe1c2SSimon Glass done_file = self.GetDoneFile(commit_upto, target) 886fc3fe1c2SSimon Glass sizes_file = self.GetSizesFile(commit_upto, target) 887fc3fe1c2SSimon Glass sizes = {} 888fc3fe1c2SSimon Glass func_sizes = {} 889fc3fe1c2SSimon Glass if os.path.exists(done_file): 890fc3fe1c2SSimon Glass with open(done_file, 'r') as fd: 891fc3fe1c2SSimon Glass return_code = int(fd.readline()) 892fc3fe1c2SSimon Glass err_lines = [] 893fc3fe1c2SSimon Glass err_file = self.GetErrFile(commit_upto, target) 894fc3fe1c2SSimon Glass if os.path.exists(err_file): 895fc3fe1c2SSimon Glass with open(err_file, 'r') as fd: 896fc3fe1c2SSimon Glass err_lines = self.FilterErrors(fd.readlines()) 897fc3fe1c2SSimon Glass 898fc3fe1c2SSimon Glass # Decide whether the build was ok, failed or created warnings 899fc3fe1c2SSimon Glass if return_code: 900fc3fe1c2SSimon Glass rc = OUTCOME_ERROR 901fc3fe1c2SSimon Glass elif len(err_lines): 902fc3fe1c2SSimon Glass rc = OUTCOME_WARNING 903fc3fe1c2SSimon Glass else: 904fc3fe1c2SSimon Glass rc = OUTCOME_OK 905fc3fe1c2SSimon Glass 906fc3fe1c2SSimon Glass # Convert size information to our simple format 907fc3fe1c2SSimon Glass if os.path.exists(sizes_file): 908fc3fe1c2SSimon Glass with open(sizes_file, 'r') as fd: 909fc3fe1c2SSimon Glass for line in fd.readlines(): 910fc3fe1c2SSimon Glass values = line.split() 911fc3fe1c2SSimon Glass rodata = 0 912fc3fe1c2SSimon Glass if len(values) > 6: 913fc3fe1c2SSimon Glass rodata = int(values[6], 16) 914fc3fe1c2SSimon Glass size_dict = { 915fc3fe1c2SSimon Glass 'all' : int(values[0]) + int(values[1]) + 916fc3fe1c2SSimon Glass int(values[2]), 917fc3fe1c2SSimon Glass 'text' : int(values[0]) - rodata, 918fc3fe1c2SSimon Glass 'data' : int(values[1]), 919fc3fe1c2SSimon Glass 'bss' : int(values[2]), 920fc3fe1c2SSimon Glass 'rodata' : rodata, 921fc3fe1c2SSimon Glass } 922fc3fe1c2SSimon Glass sizes[values[5]] = size_dict 923fc3fe1c2SSimon Glass 924fc3fe1c2SSimon Glass if read_func_sizes: 925fc3fe1c2SSimon Glass pattern = self.GetFuncSizesFile(commit_upto, target, '*') 926fc3fe1c2SSimon Glass for fname in glob.glob(pattern): 927fc3fe1c2SSimon Glass with open(fname, 'r') as fd: 928fc3fe1c2SSimon Glass dict_name = os.path.basename(fname).replace('.sizes', 929fc3fe1c2SSimon Glass '') 930fc3fe1c2SSimon Glass func_sizes[dict_name] = self.ReadFuncSizes(fname, fd) 931fc3fe1c2SSimon Glass 932fc3fe1c2SSimon Glass return Builder.Outcome(rc, err_lines, sizes, func_sizes) 933fc3fe1c2SSimon Glass 934fc3fe1c2SSimon Glass return Builder.Outcome(OUTCOME_UNKNOWN, [], {}, {}) 935fc3fe1c2SSimon Glass 936fc3fe1c2SSimon Glass def GetResultSummary(self, boards_selected, commit_upto, read_func_sizes): 937fc3fe1c2SSimon Glass """Calculate a summary of the results of building a commit. 938fc3fe1c2SSimon Glass 939fc3fe1c2SSimon Glass Args: 940fc3fe1c2SSimon Glass board_selected: Dict containing boards to summarise 941fc3fe1c2SSimon Glass commit_upto: Commit number to summarize (0..self.count-1) 942fc3fe1c2SSimon Glass read_func_sizes: True to read function size information 943fc3fe1c2SSimon Glass 944fc3fe1c2SSimon Glass Returns: 945fc3fe1c2SSimon Glass Tuple: 946fc3fe1c2SSimon Glass Dict containing boards which passed building this commit. 947fc3fe1c2SSimon Glass keyed by board.target 948fc3fe1c2SSimon Glass List containing a summary of error/warning lines 949fc3fe1c2SSimon Glass """ 950fc3fe1c2SSimon Glass board_dict = {} 951fc3fe1c2SSimon Glass err_lines_summary = [] 952fc3fe1c2SSimon Glass 953fc3fe1c2SSimon Glass for board in boards_selected.itervalues(): 954fc3fe1c2SSimon Glass outcome = self.GetBuildOutcome(commit_upto, board.target, 955fc3fe1c2SSimon Glass read_func_sizes) 956fc3fe1c2SSimon Glass board_dict[board.target] = outcome 957fc3fe1c2SSimon Glass for err in outcome.err_lines: 958fc3fe1c2SSimon Glass if err and not err.rstrip() in err_lines_summary: 959fc3fe1c2SSimon Glass err_lines_summary.append(err.rstrip()) 960fc3fe1c2SSimon Glass return board_dict, err_lines_summary 961fc3fe1c2SSimon Glass 962fc3fe1c2SSimon Glass def AddOutcome(self, board_dict, arch_list, changes, char, color): 963fc3fe1c2SSimon Glass """Add an output to our list of outcomes for each architecture 964fc3fe1c2SSimon Glass 965fc3fe1c2SSimon Glass This simple function adds failing boards (changes) to the 966fc3fe1c2SSimon Glass relevant architecture string, so we can print the results out 967fc3fe1c2SSimon Glass sorted by architecture. 968fc3fe1c2SSimon Glass 969fc3fe1c2SSimon Glass Args: 970fc3fe1c2SSimon Glass board_dict: Dict containing all boards 971fc3fe1c2SSimon Glass arch_list: Dict keyed by arch name. Value is a string containing 972fc3fe1c2SSimon Glass a list of board names which failed for that arch. 973fc3fe1c2SSimon Glass changes: List of boards to add to arch_list 974fc3fe1c2SSimon Glass color: terminal.Colour object 975fc3fe1c2SSimon Glass """ 976fc3fe1c2SSimon Glass done_arch = {} 977fc3fe1c2SSimon Glass for target in changes: 978fc3fe1c2SSimon Glass if target in board_dict: 979fc3fe1c2SSimon Glass arch = board_dict[target].arch 980fc3fe1c2SSimon Glass else: 981fc3fe1c2SSimon Glass arch = 'unknown' 982fc3fe1c2SSimon Glass str = self.col.Color(color, ' ' + target) 983fc3fe1c2SSimon Glass if not arch in done_arch: 984fc3fe1c2SSimon Glass str = self.col.Color(color, char) + ' ' + str 985fc3fe1c2SSimon Glass done_arch[arch] = True 986fc3fe1c2SSimon Glass if not arch in arch_list: 987fc3fe1c2SSimon Glass arch_list[arch] = str 988fc3fe1c2SSimon Glass else: 989fc3fe1c2SSimon Glass arch_list[arch] += str 990fc3fe1c2SSimon Glass 991fc3fe1c2SSimon Glass 992fc3fe1c2SSimon Glass def ColourNum(self, num): 993fc3fe1c2SSimon Glass color = self.col.RED if num > 0 else self.col.GREEN 994fc3fe1c2SSimon Glass if num == 0: 995fc3fe1c2SSimon Glass return '0' 996fc3fe1c2SSimon Glass return self.col.Color(color, str(num)) 997fc3fe1c2SSimon Glass 998fc3fe1c2SSimon Glass def ResetResultSummary(self, board_selected): 999fc3fe1c2SSimon Glass """Reset the results summary ready for use. 1000fc3fe1c2SSimon Glass 1001fc3fe1c2SSimon Glass Set up the base board list to be all those selected, and set the 1002fc3fe1c2SSimon Glass error lines to empty. 1003fc3fe1c2SSimon Glass 1004fc3fe1c2SSimon Glass Following this, calls to PrintResultSummary() will use this 1005fc3fe1c2SSimon Glass information to work out what has changed. 1006fc3fe1c2SSimon Glass 1007fc3fe1c2SSimon Glass Args: 1008fc3fe1c2SSimon Glass board_selected: Dict containing boards to summarise, keyed by 1009fc3fe1c2SSimon Glass board.target 1010fc3fe1c2SSimon Glass """ 1011fc3fe1c2SSimon Glass self._base_board_dict = {} 1012fc3fe1c2SSimon Glass for board in board_selected: 1013fc3fe1c2SSimon Glass self._base_board_dict[board] = Builder.Outcome(0, [], [], {}) 1014fc3fe1c2SSimon Glass self._base_err_lines = [] 1015fc3fe1c2SSimon Glass 1016fc3fe1c2SSimon Glass def PrintFuncSizeDetail(self, fname, old, new): 1017fc3fe1c2SSimon Glass grow, shrink, add, remove, up, down = 0, 0, 0, 0, 0, 0 1018fc3fe1c2SSimon Glass delta, common = [], {} 1019fc3fe1c2SSimon Glass 1020fc3fe1c2SSimon Glass for a in old: 1021fc3fe1c2SSimon Glass if a in new: 1022fc3fe1c2SSimon Glass common[a] = 1 1023fc3fe1c2SSimon Glass 1024fc3fe1c2SSimon Glass for name in old: 1025fc3fe1c2SSimon Glass if name not in common: 1026fc3fe1c2SSimon Glass remove += 1 1027fc3fe1c2SSimon Glass down += old[name] 1028fc3fe1c2SSimon Glass delta.append([-old[name], name]) 1029fc3fe1c2SSimon Glass 1030fc3fe1c2SSimon Glass for name in new: 1031fc3fe1c2SSimon Glass if name not in common: 1032fc3fe1c2SSimon Glass add += 1 1033fc3fe1c2SSimon Glass up += new[name] 1034fc3fe1c2SSimon Glass delta.append([new[name], name]) 1035fc3fe1c2SSimon Glass 1036fc3fe1c2SSimon Glass for name in common: 1037fc3fe1c2SSimon Glass diff = new.get(name, 0) - old.get(name, 0) 1038fc3fe1c2SSimon Glass if diff > 0: 1039fc3fe1c2SSimon Glass grow, up = grow + 1, up + diff 1040fc3fe1c2SSimon Glass elif diff < 0: 1041fc3fe1c2SSimon Glass shrink, down = shrink + 1, down - diff 1042fc3fe1c2SSimon Glass delta.append([diff, name]) 1043fc3fe1c2SSimon Glass 1044fc3fe1c2SSimon Glass delta.sort() 1045fc3fe1c2SSimon Glass delta.reverse() 1046fc3fe1c2SSimon Glass 1047fc3fe1c2SSimon Glass args = [add, -remove, grow, -shrink, up, -down, up - down] 1048fc3fe1c2SSimon Glass if max(args) == 0: 1049fc3fe1c2SSimon Glass return 1050fc3fe1c2SSimon Glass args = [self.ColourNum(x) for x in args] 1051fc3fe1c2SSimon Glass indent = ' ' * 15 1052fc3fe1c2SSimon Glass print ('%s%s: add: %s/%s, grow: %s/%s bytes: %s/%s (%s)' % 1053fc3fe1c2SSimon Glass tuple([indent, self.col.Color(self.col.YELLOW, fname)] + args)) 1054fc3fe1c2SSimon Glass print '%s %-38s %7s %7s %+7s' % (indent, 'function', 'old', 'new', 1055fc3fe1c2SSimon Glass 'delta') 1056fc3fe1c2SSimon Glass for diff, name in delta: 1057fc3fe1c2SSimon Glass if diff: 1058fc3fe1c2SSimon Glass color = self.col.RED if diff > 0 else self.col.GREEN 1059fc3fe1c2SSimon Glass msg = '%s %-38s %7s %7s %+7d' % (indent, name, 1060fc3fe1c2SSimon Glass old.get(name, '-'), new.get(name,'-'), diff) 1061fc3fe1c2SSimon Glass print self.col.Color(color, msg) 1062fc3fe1c2SSimon Glass 1063fc3fe1c2SSimon Glass 1064fc3fe1c2SSimon Glass def PrintSizeDetail(self, target_list, show_bloat): 1065fc3fe1c2SSimon Glass """Show details size information for each board 1066fc3fe1c2SSimon Glass 1067fc3fe1c2SSimon Glass Args: 1068fc3fe1c2SSimon Glass target_list: List of targets, each a dict containing: 1069fc3fe1c2SSimon Glass 'target': Target name 1070fc3fe1c2SSimon Glass 'total_diff': Total difference in bytes across all areas 1071fc3fe1c2SSimon Glass <part_name>: Difference for that part 1072fc3fe1c2SSimon Glass show_bloat: Show detail for each function 1073fc3fe1c2SSimon Glass """ 1074fc3fe1c2SSimon Glass targets_by_diff = sorted(target_list, reverse=True, 1075fc3fe1c2SSimon Glass key=lambda x: x['_total_diff']) 1076fc3fe1c2SSimon Glass for result in targets_by_diff: 1077fc3fe1c2SSimon Glass printed_target = False 1078fc3fe1c2SSimon Glass for name in sorted(result): 1079fc3fe1c2SSimon Glass diff = result[name] 1080fc3fe1c2SSimon Glass if name.startswith('_'): 1081fc3fe1c2SSimon Glass continue 1082fc3fe1c2SSimon Glass if diff != 0: 1083fc3fe1c2SSimon Glass color = self.col.RED if diff > 0 else self.col.GREEN 1084fc3fe1c2SSimon Glass msg = ' %s %+d' % (name, diff) 1085fc3fe1c2SSimon Glass if not printed_target: 1086fc3fe1c2SSimon Glass print '%10s %-15s:' % ('', result['_target']), 1087fc3fe1c2SSimon Glass printed_target = True 1088fc3fe1c2SSimon Glass print self.col.Color(color, msg), 1089fc3fe1c2SSimon Glass if printed_target: 1090fc3fe1c2SSimon Glass print 1091fc3fe1c2SSimon Glass if show_bloat: 1092fc3fe1c2SSimon Glass target = result['_target'] 1093fc3fe1c2SSimon Glass outcome = result['_outcome'] 1094fc3fe1c2SSimon Glass base_outcome = self._base_board_dict[target] 1095fc3fe1c2SSimon Glass for fname in outcome.func_sizes: 1096fc3fe1c2SSimon Glass self.PrintFuncSizeDetail(fname, 1097fc3fe1c2SSimon Glass base_outcome.func_sizes[fname], 1098fc3fe1c2SSimon Glass outcome.func_sizes[fname]) 1099fc3fe1c2SSimon Glass 1100fc3fe1c2SSimon Glass 1101fc3fe1c2SSimon Glass def PrintSizeSummary(self, board_selected, board_dict, show_detail, 1102fc3fe1c2SSimon Glass show_bloat): 1103fc3fe1c2SSimon Glass """Print a summary of image sizes broken down by section. 1104fc3fe1c2SSimon Glass 1105fc3fe1c2SSimon Glass The summary takes the form of one line per architecture. The 1106fc3fe1c2SSimon Glass line contains deltas for each of the sections (+ means the section 1107fc3fe1c2SSimon Glass got bigger, - means smaller). The nunmbers are the average number 1108fc3fe1c2SSimon Glass of bytes that a board in this section increased by. 1109fc3fe1c2SSimon Glass 1110fc3fe1c2SSimon Glass For example: 1111fc3fe1c2SSimon Glass powerpc: (622 boards) text -0.0 1112fc3fe1c2SSimon Glass arm: (285 boards) text -0.0 1113fc3fe1c2SSimon Glass nds32: (3 boards) text -8.0 1114fc3fe1c2SSimon Glass 1115fc3fe1c2SSimon Glass Args: 1116fc3fe1c2SSimon Glass board_selected: Dict containing boards to summarise, keyed by 1117fc3fe1c2SSimon Glass board.target 1118fc3fe1c2SSimon Glass board_dict: Dict containing boards for which we built this 1119fc3fe1c2SSimon Glass commit, keyed by board.target. The value is an Outcome object. 1120fc3fe1c2SSimon Glass show_detail: Show detail for each board 1121fc3fe1c2SSimon Glass show_bloat: Show detail for each function 1122fc3fe1c2SSimon Glass """ 1123fc3fe1c2SSimon Glass arch_list = {} 1124fc3fe1c2SSimon Glass arch_count = {} 1125fc3fe1c2SSimon Glass 1126fc3fe1c2SSimon Glass # Calculate changes in size for different image parts 1127fc3fe1c2SSimon Glass # The previous sizes are in Board.sizes, for each board 1128fc3fe1c2SSimon Glass for target in board_dict: 1129fc3fe1c2SSimon Glass if target not in board_selected: 1130fc3fe1c2SSimon Glass continue 1131fc3fe1c2SSimon Glass base_sizes = self._base_board_dict[target].sizes 1132fc3fe1c2SSimon Glass outcome = board_dict[target] 1133fc3fe1c2SSimon Glass sizes = outcome.sizes 1134fc3fe1c2SSimon Glass 1135fc3fe1c2SSimon Glass # Loop through the list of images, creating a dict of size 1136fc3fe1c2SSimon Glass # changes for each image/part. We end up with something like 1137fc3fe1c2SSimon Glass # {'target' : 'snapper9g45, 'data' : 5, 'u-boot-spl:text' : -4} 1138fc3fe1c2SSimon Glass # which means that U-Boot data increased by 5 bytes and SPL 1139fc3fe1c2SSimon Glass # text decreased by 4. 1140fc3fe1c2SSimon Glass err = {'_target' : target} 1141fc3fe1c2SSimon Glass for image in sizes: 1142fc3fe1c2SSimon Glass if image in base_sizes: 1143fc3fe1c2SSimon Glass base_image = base_sizes[image] 1144fc3fe1c2SSimon Glass # Loop through the text, data, bss parts 1145fc3fe1c2SSimon Glass for part in sorted(sizes[image]): 1146fc3fe1c2SSimon Glass diff = sizes[image][part] - base_image[part] 1147fc3fe1c2SSimon Glass col = None 1148fc3fe1c2SSimon Glass if diff: 1149fc3fe1c2SSimon Glass if image == 'u-boot': 1150fc3fe1c2SSimon Glass name = part 1151fc3fe1c2SSimon Glass else: 1152fc3fe1c2SSimon Glass name = image + ':' + part 1153fc3fe1c2SSimon Glass err[name] = diff 1154fc3fe1c2SSimon Glass arch = board_selected[target].arch 1155fc3fe1c2SSimon Glass if not arch in arch_count: 1156fc3fe1c2SSimon Glass arch_count[arch] = 1 1157fc3fe1c2SSimon Glass else: 1158fc3fe1c2SSimon Glass arch_count[arch] += 1 1159fc3fe1c2SSimon Glass if not sizes: 1160fc3fe1c2SSimon Glass pass # Only add to our list when we have some stats 1161fc3fe1c2SSimon Glass elif not arch in arch_list: 1162fc3fe1c2SSimon Glass arch_list[arch] = [err] 1163fc3fe1c2SSimon Glass else: 1164fc3fe1c2SSimon Glass arch_list[arch].append(err) 1165fc3fe1c2SSimon Glass 1166fc3fe1c2SSimon Glass # We now have a list of image size changes sorted by arch 1167fc3fe1c2SSimon Glass # Print out a summary of these 1168fc3fe1c2SSimon Glass for arch, target_list in arch_list.iteritems(): 1169fc3fe1c2SSimon Glass # Get total difference for each type 1170fc3fe1c2SSimon Glass totals = {} 1171fc3fe1c2SSimon Glass for result in target_list: 1172fc3fe1c2SSimon Glass total = 0 1173fc3fe1c2SSimon Glass for name, diff in result.iteritems(): 1174fc3fe1c2SSimon Glass if name.startswith('_'): 1175fc3fe1c2SSimon Glass continue 1176fc3fe1c2SSimon Glass total += diff 1177fc3fe1c2SSimon Glass if name in totals: 1178fc3fe1c2SSimon Glass totals[name] += diff 1179fc3fe1c2SSimon Glass else: 1180fc3fe1c2SSimon Glass totals[name] = diff 1181fc3fe1c2SSimon Glass result['_total_diff'] = total 1182fc3fe1c2SSimon Glass result['_outcome'] = board_dict[result['_target']] 1183fc3fe1c2SSimon Glass 1184fc3fe1c2SSimon Glass count = len(target_list) 1185fc3fe1c2SSimon Glass printed_arch = False 1186fc3fe1c2SSimon Glass for name in sorted(totals): 1187fc3fe1c2SSimon Glass diff = totals[name] 1188fc3fe1c2SSimon Glass if diff: 1189fc3fe1c2SSimon Glass # Display the average difference in this name for this 1190fc3fe1c2SSimon Glass # architecture 1191fc3fe1c2SSimon Glass avg_diff = float(diff) / count 1192fc3fe1c2SSimon Glass color = self.col.RED if avg_diff > 0 else self.col.GREEN 1193fc3fe1c2SSimon Glass msg = ' %s %+1.1f' % (name, avg_diff) 1194fc3fe1c2SSimon Glass if not printed_arch: 1195fc3fe1c2SSimon Glass print '%10s: (for %d/%d boards)' % (arch, count, 1196fc3fe1c2SSimon Glass arch_count[arch]), 1197fc3fe1c2SSimon Glass printed_arch = True 1198fc3fe1c2SSimon Glass print self.col.Color(color, msg), 1199fc3fe1c2SSimon Glass 1200fc3fe1c2SSimon Glass if printed_arch: 1201fc3fe1c2SSimon Glass print 1202fc3fe1c2SSimon Glass if show_detail: 1203fc3fe1c2SSimon Glass self.PrintSizeDetail(target_list, show_bloat) 1204fc3fe1c2SSimon Glass 1205fc3fe1c2SSimon Glass 1206fc3fe1c2SSimon Glass def PrintResultSummary(self, board_selected, board_dict, err_lines, 1207fc3fe1c2SSimon Glass show_sizes, show_detail, show_bloat): 1208fc3fe1c2SSimon Glass """Compare results with the base results and display delta. 1209fc3fe1c2SSimon Glass 1210fc3fe1c2SSimon Glass Only boards mentioned in board_selected will be considered. This 1211fc3fe1c2SSimon Glass function is intended to be called repeatedly with the results of 1212fc3fe1c2SSimon Glass each commit. It therefore shows a 'diff' between what it saw in 1213fc3fe1c2SSimon Glass the last call and what it sees now. 1214fc3fe1c2SSimon Glass 1215fc3fe1c2SSimon Glass Args: 1216fc3fe1c2SSimon Glass board_selected: Dict containing boards to summarise, keyed by 1217fc3fe1c2SSimon Glass board.target 1218fc3fe1c2SSimon Glass board_dict: Dict containing boards for which we built this 1219fc3fe1c2SSimon Glass commit, keyed by board.target. The value is an Outcome object. 1220fc3fe1c2SSimon Glass err_lines: A list of errors for this commit, or [] if there is 1221fc3fe1c2SSimon Glass none, or we don't want to print errors 1222fc3fe1c2SSimon Glass show_sizes: Show image size deltas 1223fc3fe1c2SSimon Glass show_detail: Show detail for each board 1224fc3fe1c2SSimon Glass show_bloat: Show detail for each function 1225fc3fe1c2SSimon Glass """ 1226fc3fe1c2SSimon Glass better = [] # List of boards fixed since last commit 1227fc3fe1c2SSimon Glass worse = [] # List of new broken boards since last commit 1228fc3fe1c2SSimon Glass new = [] # List of boards that didn't exist last time 1229fc3fe1c2SSimon Glass unknown = [] # List of boards that were not built 1230fc3fe1c2SSimon Glass 1231fc3fe1c2SSimon Glass for target in board_dict: 1232fc3fe1c2SSimon Glass if target not in board_selected: 1233fc3fe1c2SSimon Glass continue 1234fc3fe1c2SSimon Glass 1235fc3fe1c2SSimon Glass # If the board was built last time, add its outcome to a list 1236fc3fe1c2SSimon Glass if target in self._base_board_dict: 1237fc3fe1c2SSimon Glass base_outcome = self._base_board_dict[target].rc 1238fc3fe1c2SSimon Glass outcome = board_dict[target] 1239fc3fe1c2SSimon Glass if outcome.rc == OUTCOME_UNKNOWN: 1240fc3fe1c2SSimon Glass unknown.append(target) 1241fc3fe1c2SSimon Glass elif outcome.rc < base_outcome: 1242fc3fe1c2SSimon Glass better.append(target) 1243fc3fe1c2SSimon Glass elif outcome.rc > base_outcome: 1244fc3fe1c2SSimon Glass worse.append(target) 1245fc3fe1c2SSimon Glass else: 1246fc3fe1c2SSimon Glass new.append(target) 1247fc3fe1c2SSimon Glass 1248fc3fe1c2SSimon Glass # Get a list of errors that have appeared, and disappeared 1249fc3fe1c2SSimon Glass better_err = [] 1250fc3fe1c2SSimon Glass worse_err = [] 1251fc3fe1c2SSimon Glass for line in err_lines: 1252fc3fe1c2SSimon Glass if line not in self._base_err_lines: 1253fc3fe1c2SSimon Glass worse_err.append('+' + line) 1254fc3fe1c2SSimon Glass for line in self._base_err_lines: 1255fc3fe1c2SSimon Glass if line not in err_lines: 1256fc3fe1c2SSimon Glass better_err.append('-' + line) 1257fc3fe1c2SSimon Glass 1258fc3fe1c2SSimon Glass # Display results by arch 1259fc3fe1c2SSimon Glass if better or worse or unknown or new or worse_err or better_err: 1260fc3fe1c2SSimon Glass arch_list = {} 1261fc3fe1c2SSimon Glass self.AddOutcome(board_selected, arch_list, better, '', 1262fc3fe1c2SSimon Glass self.col.GREEN) 1263fc3fe1c2SSimon Glass self.AddOutcome(board_selected, arch_list, worse, '+', 1264fc3fe1c2SSimon Glass self.col.RED) 1265fc3fe1c2SSimon Glass self.AddOutcome(board_selected, arch_list, new, '*', self.col.BLUE) 1266fc3fe1c2SSimon Glass if self._show_unknown: 1267fc3fe1c2SSimon Glass self.AddOutcome(board_selected, arch_list, unknown, '?', 1268fc3fe1c2SSimon Glass self.col.MAGENTA) 1269fc3fe1c2SSimon Glass for arch, target_list in arch_list.iteritems(): 1270fc3fe1c2SSimon Glass print '%10s: %s' % (arch, target_list) 1271fc3fe1c2SSimon Glass if better_err: 1272fc3fe1c2SSimon Glass print self.col.Color(self.col.GREEN, '\n'.join(better_err)) 1273fc3fe1c2SSimon Glass if worse_err: 1274fc3fe1c2SSimon Glass print self.col.Color(self.col.RED, '\n'.join(worse_err)) 1275fc3fe1c2SSimon Glass 1276fc3fe1c2SSimon Glass if show_sizes: 1277fc3fe1c2SSimon Glass self.PrintSizeSummary(board_selected, board_dict, show_detail, 1278fc3fe1c2SSimon Glass show_bloat) 1279fc3fe1c2SSimon Glass 1280fc3fe1c2SSimon Glass # Save our updated information for the next call to this function 1281fc3fe1c2SSimon Glass self._base_board_dict = board_dict 1282fc3fe1c2SSimon Glass self._base_err_lines = err_lines 1283fc3fe1c2SSimon Glass 1284fc3fe1c2SSimon Glass # Get a list of boards that did not get built, if needed 1285fc3fe1c2SSimon Glass not_built = [] 1286fc3fe1c2SSimon Glass for board in board_selected: 1287fc3fe1c2SSimon Glass if not board in board_dict: 1288fc3fe1c2SSimon Glass not_built.append(board) 1289fc3fe1c2SSimon Glass if not_built: 1290fc3fe1c2SSimon Glass print "Boards not built (%d): %s" % (len(not_built), 1291fc3fe1c2SSimon Glass ', '.join(not_built)) 1292fc3fe1c2SSimon Glass 1293fc3fe1c2SSimon Glass 1294fc3fe1c2SSimon Glass def ShowSummary(self, commits, board_selected, show_errors, show_sizes, 1295fc3fe1c2SSimon Glass show_detail, show_bloat): 1296fc3fe1c2SSimon Glass """Show a build summary for U-Boot for a given board list. 1297fc3fe1c2SSimon Glass 1298fc3fe1c2SSimon Glass Reset the result summary, then repeatedly call GetResultSummary on 1299fc3fe1c2SSimon Glass each commit's results, then display the differences we see. 1300fc3fe1c2SSimon Glass 1301fc3fe1c2SSimon Glass Args: 1302fc3fe1c2SSimon Glass commit: Commit objects to summarise 1303fc3fe1c2SSimon Glass board_selected: Dict containing boards to summarise 1304fc3fe1c2SSimon Glass show_errors: Show errors that occured 1305fc3fe1c2SSimon Glass show_sizes: Show size deltas 1306fc3fe1c2SSimon Glass show_detail: Show detail for each board 1307fc3fe1c2SSimon Glass show_bloat: Show detail for each function 1308fc3fe1c2SSimon Glass """ 1309fc3fe1c2SSimon Glass self.commit_count = len(commits) 1310fc3fe1c2SSimon Glass self.commits = commits 1311fc3fe1c2SSimon Glass self.ResetResultSummary(board_selected) 1312fc3fe1c2SSimon Glass 1313fc3fe1c2SSimon Glass for commit_upto in range(0, self.commit_count, self._step): 1314fc3fe1c2SSimon Glass board_dict, err_lines = self.GetResultSummary(board_selected, 1315fc3fe1c2SSimon Glass commit_upto, read_func_sizes=show_bloat) 1316fc3fe1c2SSimon Glass msg = '%02d: %s' % (commit_upto + 1, commits[commit_upto].subject) 1317fc3fe1c2SSimon Glass print self.col.Color(self.col.BLUE, msg) 1318fc3fe1c2SSimon Glass self.PrintResultSummary(board_selected, board_dict, 1319fc3fe1c2SSimon Glass err_lines if show_errors else [], show_sizes, show_detail, 1320fc3fe1c2SSimon Glass show_bloat) 1321fc3fe1c2SSimon Glass 1322fc3fe1c2SSimon Glass 1323fc3fe1c2SSimon Glass def SetupBuild(self, board_selected, commits): 1324fc3fe1c2SSimon Glass """Set up ready to start a build. 1325fc3fe1c2SSimon Glass 1326fc3fe1c2SSimon Glass Args: 1327fc3fe1c2SSimon Glass board_selected: Selected boards to build 1328fc3fe1c2SSimon Glass commits: Selected commits to build 1329fc3fe1c2SSimon Glass """ 1330fc3fe1c2SSimon Glass # First work out how many commits we will build 1331fc3fe1c2SSimon Glass count = (len(commits) + self._step - 1) / self._step 1332fc3fe1c2SSimon Glass self.count = len(board_selected) * count 1333fc3fe1c2SSimon Glass self.upto = self.warned = self.fail = 0 1334fc3fe1c2SSimon Glass self._timestamps = collections.deque() 1335fc3fe1c2SSimon Glass 1336fc3fe1c2SSimon Glass def BuildBoardsForCommit(self, board_selected, keep_outputs): 1337fc3fe1c2SSimon Glass """Build all boards for a single commit""" 1338fc3fe1c2SSimon Glass self.SetupBuild(board_selected) 1339fc3fe1c2SSimon Glass self.count = len(board_selected) 1340fc3fe1c2SSimon Glass for brd in board_selected.itervalues(): 1341fc3fe1c2SSimon Glass job = BuilderJob() 1342fc3fe1c2SSimon Glass job.board = brd 1343fc3fe1c2SSimon Glass job.commits = None 1344fc3fe1c2SSimon Glass job.keep_outputs = keep_outputs 1345fc3fe1c2SSimon Glass self.queue.put(brd) 1346fc3fe1c2SSimon Glass 1347fc3fe1c2SSimon Glass self.queue.join() 1348fc3fe1c2SSimon Glass self.out_queue.join() 1349fc3fe1c2SSimon Glass print 1350fc3fe1c2SSimon Glass self.ClearLine(0) 1351fc3fe1c2SSimon Glass 1352fc3fe1c2SSimon Glass def BuildCommits(self, commits, board_selected, show_errors, keep_outputs): 1353fc3fe1c2SSimon Glass """Build all boards for all commits (non-incremental)""" 1354fc3fe1c2SSimon Glass self.commit_count = len(commits) 1355fc3fe1c2SSimon Glass 1356fc3fe1c2SSimon Glass self.ResetResultSummary(board_selected) 1357fc3fe1c2SSimon Glass for self.commit_upto in range(self.commit_count): 1358fc3fe1c2SSimon Glass self.SelectCommit(commits[self.commit_upto]) 1359fc3fe1c2SSimon Glass self.SelectOutputDir() 1360fc3fe1c2SSimon Glass Mkdir(self.output_dir) 1361fc3fe1c2SSimon Glass 1362fc3fe1c2SSimon Glass self.BuildBoardsForCommit(board_selected, keep_outputs) 1363fc3fe1c2SSimon Glass board_dict, err_lines = self.GetResultSummary() 1364fc3fe1c2SSimon Glass self.PrintResultSummary(board_selected, board_dict, 1365fc3fe1c2SSimon Glass err_lines if show_errors else []) 1366fc3fe1c2SSimon Glass 1367fc3fe1c2SSimon Glass if self.already_done: 1368fc3fe1c2SSimon Glass print '%d builds already done' % self.already_done 1369fc3fe1c2SSimon Glass 1370fc3fe1c2SSimon Glass def GetThreadDir(self, thread_num): 1371fc3fe1c2SSimon Glass """Get the directory path to the working dir for a thread. 1372fc3fe1c2SSimon Glass 1373fc3fe1c2SSimon Glass Args: 1374fc3fe1c2SSimon Glass thread_num: Number of thread to check. 1375fc3fe1c2SSimon Glass """ 1376fc3fe1c2SSimon Glass return os.path.join(self._working_dir, '%02d' % thread_num) 1377fc3fe1c2SSimon Glass 1378fc3fe1c2SSimon Glass def _PrepareThread(self, thread_num): 1379fc3fe1c2SSimon Glass """Prepare the working directory for a thread. 1380fc3fe1c2SSimon Glass 1381fc3fe1c2SSimon Glass This clones or fetches the repo into the thread's work directory. 1382fc3fe1c2SSimon Glass 1383fc3fe1c2SSimon Glass Args: 1384fc3fe1c2SSimon Glass thread_num: Thread number (0, 1, ...) 1385fc3fe1c2SSimon Glass """ 1386fc3fe1c2SSimon Glass thread_dir = self.GetThreadDir(thread_num) 1387fc3fe1c2SSimon Glass Mkdir(thread_dir) 1388fc3fe1c2SSimon Glass git_dir = os.path.join(thread_dir, '.git') 1389fc3fe1c2SSimon Glass 1390fc3fe1c2SSimon Glass # Clone the repo if it doesn't already exist 1391fc3fe1c2SSimon Glass # TODO(sjg@chromium): Perhaps some git hackery to symlink instead, so 1392fc3fe1c2SSimon Glass # we have a private index but uses the origin repo's contents? 1393fc3fe1c2SSimon Glass if self.git_dir: 1394fc3fe1c2SSimon Glass src_dir = os.path.abspath(self.git_dir) 1395fc3fe1c2SSimon Glass if os.path.exists(git_dir): 1396fc3fe1c2SSimon Glass gitutil.Fetch(git_dir, thread_dir) 1397fc3fe1c2SSimon Glass else: 1398fc3fe1c2SSimon Glass print 'Cloning repo for thread %d' % thread_num 1399fc3fe1c2SSimon Glass gitutil.Clone(src_dir, thread_dir) 1400fc3fe1c2SSimon Glass 1401fc3fe1c2SSimon Glass def _PrepareWorkingSpace(self, max_threads): 1402fc3fe1c2SSimon Glass """Prepare the working directory for use. 1403fc3fe1c2SSimon Glass 1404fc3fe1c2SSimon Glass Set up the git repo for each thread. 1405fc3fe1c2SSimon Glass 1406fc3fe1c2SSimon Glass Args: 1407fc3fe1c2SSimon Glass max_threads: Maximum number of threads we expect to need. 1408fc3fe1c2SSimon Glass """ 1409fc3fe1c2SSimon Glass Mkdir(self._working_dir) 1410fc3fe1c2SSimon Glass for thread in range(max_threads): 1411fc3fe1c2SSimon Glass self._PrepareThread(thread) 1412fc3fe1c2SSimon Glass 1413fc3fe1c2SSimon Glass def _PrepareOutputSpace(self): 1414fc3fe1c2SSimon Glass """Get the output directories ready to receive files. 1415fc3fe1c2SSimon Glass 1416fc3fe1c2SSimon Glass We delete any output directories which look like ones we need to 1417fc3fe1c2SSimon Glass create. Having left over directories is confusing when the user wants 1418fc3fe1c2SSimon Glass to check the output manually. 1419fc3fe1c2SSimon Glass """ 1420fc3fe1c2SSimon Glass dir_list = [] 1421fc3fe1c2SSimon Glass for commit_upto in range(self.commit_count): 1422fc3fe1c2SSimon Glass dir_list.append(self._GetOutputDir(commit_upto)) 1423fc3fe1c2SSimon Glass 1424fc3fe1c2SSimon Glass for dirname in glob.glob(os.path.join(self.base_dir, '*')): 1425fc3fe1c2SSimon Glass if dirname not in dir_list: 1426fc3fe1c2SSimon Glass shutil.rmtree(dirname) 1427fc3fe1c2SSimon Glass 1428fc3fe1c2SSimon Glass def BuildBoards(self, commits, board_selected, show_errors, keep_outputs): 1429fc3fe1c2SSimon Glass """Build all commits for a list of boards 1430fc3fe1c2SSimon Glass 1431fc3fe1c2SSimon Glass Args: 1432fc3fe1c2SSimon Glass commits: List of commits to be build, each a Commit object 1433fc3fe1c2SSimon Glass boards_selected: Dict of selected boards, key is target name, 1434fc3fe1c2SSimon Glass value is Board object 1435fc3fe1c2SSimon Glass show_errors: True to show summarised error/warning info 1436fc3fe1c2SSimon Glass keep_outputs: True to save build output files 1437fc3fe1c2SSimon Glass """ 1438fc3fe1c2SSimon Glass self.commit_count = len(commits) 1439fc3fe1c2SSimon Glass self.commits = commits 1440fc3fe1c2SSimon Glass 1441fc3fe1c2SSimon Glass self.ResetResultSummary(board_selected) 1442fc3fe1c2SSimon Glass Mkdir(self.base_dir) 1443fc3fe1c2SSimon Glass self._PrepareWorkingSpace(min(self.num_threads, len(board_selected))) 1444fc3fe1c2SSimon Glass self._PrepareOutputSpace() 1445fc3fe1c2SSimon Glass self.SetupBuild(board_selected, commits) 1446fc3fe1c2SSimon Glass self.ProcessResult(None) 1447fc3fe1c2SSimon Glass 1448fc3fe1c2SSimon Glass # Create jobs to build all commits for each board 1449fc3fe1c2SSimon Glass for brd in board_selected.itervalues(): 1450fc3fe1c2SSimon Glass job = BuilderJob() 1451fc3fe1c2SSimon Glass job.board = brd 1452fc3fe1c2SSimon Glass job.commits = commits 1453fc3fe1c2SSimon Glass job.keep_outputs = keep_outputs 1454fc3fe1c2SSimon Glass job.step = self._step 1455fc3fe1c2SSimon Glass self.queue.put(job) 1456fc3fe1c2SSimon Glass 1457fc3fe1c2SSimon Glass # Wait until all jobs are started 1458fc3fe1c2SSimon Glass self.queue.join() 1459fc3fe1c2SSimon Glass 1460fc3fe1c2SSimon Glass # Wait until we have processed all output 1461fc3fe1c2SSimon Glass self.out_queue.join() 1462fc3fe1c2SSimon Glass print 1463fc3fe1c2SSimon Glass self.ClearLine(0) 1464