xref: /openbmc/u-boot/tools/buildman/builder.py (revision 189a496825aefe7c8e9be80c0f2a4cf5923d4f55)
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