xref: /rk3399_rockchip-uboot/tools/buildman/builder.py (revision 190064b4daa30ce8efa7bcbee12ac05b780fb9c8)
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 Glassfrom datetime import datetime, timedelta
10fc3fe1c2SSimon Glassimport glob
11fc3fe1c2SSimon Glassimport os
12fc3fe1c2SSimon Glassimport re
13fc3fe1c2SSimon Glassimport Queue
14fc3fe1c2SSimon Glassimport shutil
15fc3fe1c2SSimon Glassimport string
16fc3fe1c2SSimon Glassimport sys
17fc3fe1c2SSimon Glassimport time
18fc3fe1c2SSimon Glass
19*190064b4SSimon Glassimport builderthread
20fc3fe1c2SSimon Glassimport command
21fc3fe1c2SSimon Glassimport gitutil
22fc3fe1c2SSimon Glassimport terminal
23fc3fe1c2SSimon Glassimport toolchain
24fc3fe1c2SSimon Glass
25fc3fe1c2SSimon Glass
26fc3fe1c2SSimon Glass"""
27fc3fe1c2SSimon GlassTheory of Operation
28fc3fe1c2SSimon Glass
29fc3fe1c2SSimon GlassPlease see README for user documentation, and you should be familiar with
30fc3fe1c2SSimon Glassthat before trying to make sense of this.
31fc3fe1c2SSimon Glass
32fc3fe1c2SSimon GlassBuildman works by keeping the machine as busy as possible, building different
33fc3fe1c2SSimon Glasscommits for different boards on multiple CPUs at once.
34fc3fe1c2SSimon Glass
35fc3fe1c2SSimon GlassThe source repo (self.git_dir) contains all the commits to be built. Each
36fc3fe1c2SSimon Glassthread works on a single board at a time. It checks out the first commit,
37fc3fe1c2SSimon Glassconfigures it for that board, then builds it. Then it checks out the next
38fc3fe1c2SSimon Glasscommit and builds it (typically without re-configuring). When it runs out
39fc3fe1c2SSimon Glassof commits, it gets another job from the builder and starts again with that
40fc3fe1c2SSimon Glassboard.
41fc3fe1c2SSimon Glass
42fc3fe1c2SSimon GlassClearly the builder threads could work either way - they could check out a
43fc3fe1c2SSimon Glasscommit and then built it for all boards. Using separate directories for each
44fc3fe1c2SSimon Glasscommit/board pair they could leave their build product around afterwards
45fc3fe1c2SSimon Glassalso.
46fc3fe1c2SSimon Glass
47fc3fe1c2SSimon GlassThe intent behind building a single board for multiple commits, is to make
48fc3fe1c2SSimon Glassuse of incremental builds. Since each commit is built incrementally from
49fc3fe1c2SSimon Glassthe previous one, builds are faster. Reconfiguring for a different board
50fc3fe1c2SSimon Glassremoves all intermediate object files.
51fc3fe1c2SSimon Glass
52fc3fe1c2SSimon GlassMany threads can be working at once, but each has its own working directory.
53fc3fe1c2SSimon GlassWhen a thread finishes a build, it puts the output files into a result
54fc3fe1c2SSimon Glassdirectory.
55fc3fe1c2SSimon Glass
56fc3fe1c2SSimon GlassThe base directory used by buildman is normally '../<branch>', i.e.
57fc3fe1c2SSimon Glassa directory higher than the source repository and named after the branch
58fc3fe1c2SSimon Glassbeing built.
59fc3fe1c2SSimon Glass
60fc3fe1c2SSimon GlassWithin the base directory, we have one subdirectory for each commit. Within
61fc3fe1c2SSimon Glassthat is one subdirectory for each board. Within that is the build output for
62fc3fe1c2SSimon Glassthat commit/board combination.
63fc3fe1c2SSimon Glass
64fc3fe1c2SSimon GlassBuildman also create working directories for each thread, in a .bm-work/
65fc3fe1c2SSimon Glasssubdirectory in the base dir.
66fc3fe1c2SSimon Glass
67fc3fe1c2SSimon GlassAs an example, say we are building branch 'us-net' for boards 'sandbox' and
68fc3fe1c2SSimon Glass'seaboard', and say that us-net has two commits. We will have directories
69fc3fe1c2SSimon Glasslike this:
70fc3fe1c2SSimon Glass
71fc3fe1c2SSimon Glassus-net/             base directory
72fc3fe1c2SSimon Glass    01_of_02_g4ed4ebc_net--Add-tftp-speed-/
73fc3fe1c2SSimon Glass        sandbox/
74fc3fe1c2SSimon Glass            u-boot.bin
75fc3fe1c2SSimon Glass        seaboard/
76fc3fe1c2SSimon Glass            u-boot.bin
77fc3fe1c2SSimon Glass    02_of_02_g4ed4ebc_net--Check-tftp-comp/
78fc3fe1c2SSimon Glass        sandbox/
79fc3fe1c2SSimon Glass            u-boot.bin
80fc3fe1c2SSimon Glass        seaboard/
81fc3fe1c2SSimon Glass            u-boot.bin
82fc3fe1c2SSimon Glass    .bm-work/
83fc3fe1c2SSimon Glass        00/         working directory for thread 0 (contains source checkout)
84fc3fe1c2SSimon Glass            build/  build output
85fc3fe1c2SSimon Glass        01/         working directory for thread 1
86fc3fe1c2SSimon Glass            build/  build output
87fc3fe1c2SSimon Glass        ...
88fc3fe1c2SSimon Glassu-boot/             source directory
89fc3fe1c2SSimon Glass    .git/           repository
90fc3fe1c2SSimon Glass"""
91fc3fe1c2SSimon Glass
92fc3fe1c2SSimon Glass# Possible build outcomes
93fc3fe1c2SSimon GlassOUTCOME_OK, OUTCOME_WARNING, OUTCOME_ERROR, OUTCOME_UNKNOWN = range(4)
94fc3fe1c2SSimon Glass
95fc3fe1c2SSimon Glass# Translate a commit subject into a valid filename
96fc3fe1c2SSimon Glasstrans_valid_chars = string.maketrans("/: ", "---")
97fc3fe1c2SSimon Glass
98fc3fe1c2SSimon Glass
99fc3fe1c2SSimon Glassclass Builder:
100fc3fe1c2SSimon Glass    """Class for building U-Boot for a particular commit.
101fc3fe1c2SSimon Glass
102fc3fe1c2SSimon Glass    Public members: (many should ->private)
103fc3fe1c2SSimon Glass        active: True if the builder is active and has not been stopped
104fc3fe1c2SSimon Glass        already_done: Number of builds already completed
105fc3fe1c2SSimon Glass        base_dir: Base directory to use for builder
106fc3fe1c2SSimon Glass        checkout: True to check out source, False to skip that step.
107fc3fe1c2SSimon Glass            This is used for testing.
108fc3fe1c2SSimon Glass        col: terminal.Color() object
109fc3fe1c2SSimon Glass        count: Number of commits to build
110fc3fe1c2SSimon Glass        do_make: Method to call to invoke Make
111fc3fe1c2SSimon Glass        fail: Number of builds that failed due to error
112fc3fe1c2SSimon Glass        force_build: Force building even if a build already exists
113fc3fe1c2SSimon Glass        force_config_on_failure: If a commit fails for a board, disable
114fc3fe1c2SSimon Glass            incremental building for the next commit we build for that
115fc3fe1c2SSimon Glass            board, so that we will see all warnings/errors again.
1164266dc28SSimon Glass        force_build_failures: If a previously-built build (i.e. built on
1174266dc28SSimon Glass            a previous run of buildman) is marked as failed, rebuild it.
118fc3fe1c2SSimon Glass        git_dir: Git directory containing source repository
119fc3fe1c2SSimon Glass        last_line_len: Length of the last line we printed (used for erasing
120fc3fe1c2SSimon Glass            it with new progress information)
121fc3fe1c2SSimon Glass        num_jobs: Number of jobs to run at once (passed to make as -j)
122fc3fe1c2SSimon Glass        num_threads: Number of builder threads to run
123fc3fe1c2SSimon Glass        out_queue: Queue of results to process
124fc3fe1c2SSimon Glass        re_make_err: Compiled regular expression for ignore_lines
125fc3fe1c2SSimon Glass        queue: Queue of jobs to run
126fc3fe1c2SSimon Glass        threads: List of active threads
127fc3fe1c2SSimon Glass        toolchains: Toolchains object to use for building
128fc3fe1c2SSimon Glass        upto: Current commit number we are building (0.count-1)
129fc3fe1c2SSimon Glass        warned: Number of builds that produced at least one warning
13097e91526SSimon Glass        force_reconfig: Reconfigure U-Boot on each comiit. This disables
13197e91526SSimon Glass            incremental building, where buildman reconfigures on the first
13297e91526SSimon Glass            commit for a baord, and then just does an incremental build for
13397e91526SSimon Glass            the following commits. In fact buildman will reconfigure and
13497e91526SSimon Glass            retry for any failing commits, so generally the only effect of
13597e91526SSimon Glass            this option is to slow things down.
136189a4968SSimon Glass        in_tree: Build U-Boot in-tree instead of specifying an output
137189a4968SSimon Glass            directory separate from the source code. This option is really
138189a4968SSimon Glass            only useful for testing in-tree builds.
139fc3fe1c2SSimon Glass
140fc3fe1c2SSimon Glass    Private members:
141fc3fe1c2SSimon Glass        _base_board_dict: Last-summarised Dict of boards
142fc3fe1c2SSimon Glass        _base_err_lines: Last-summarised list of errors
143fc3fe1c2SSimon Glass        _build_period_us: Time taken for a single build (float object).
144fc3fe1c2SSimon Glass        _complete_delay: Expected delay until completion (timedelta)
145fc3fe1c2SSimon Glass        _next_delay_update: Next time we plan to display a progress update
146fc3fe1c2SSimon Glass                (datatime)
147fc3fe1c2SSimon Glass        _show_unknown: Show unknown boards (those not built) in summary
148fc3fe1c2SSimon Glass        _timestamps: List of timestamps for the completion of the last
149fc3fe1c2SSimon Glass            last _timestamp_count builds. Each is a datetime object.
150fc3fe1c2SSimon Glass        _timestamp_count: Number of timestamps to keep in our list.
151fc3fe1c2SSimon Glass        _working_dir: Base working directory containing all threads
152fc3fe1c2SSimon Glass    """
153fc3fe1c2SSimon Glass    class Outcome:
154fc3fe1c2SSimon Glass        """Records a build outcome for a single make invocation
155fc3fe1c2SSimon Glass
156fc3fe1c2SSimon Glass        Public Members:
157fc3fe1c2SSimon Glass            rc: Outcome value (OUTCOME_...)
158fc3fe1c2SSimon Glass            err_lines: List of error lines or [] if none
159fc3fe1c2SSimon Glass            sizes: Dictionary of image size information, keyed by filename
160fc3fe1c2SSimon Glass                - Each value is itself a dictionary containing
161fc3fe1c2SSimon Glass                    values for 'text', 'data' and 'bss', being the integer
162fc3fe1c2SSimon Glass                    size in bytes of each section.
163fc3fe1c2SSimon Glass            func_sizes: Dictionary keyed by filename - e.g. 'u-boot'. Each
164fc3fe1c2SSimon Glass                    value is itself a dictionary:
165fc3fe1c2SSimon Glass                        key: function name
166fc3fe1c2SSimon Glass                        value: Size of function in bytes
167fc3fe1c2SSimon Glass        """
168fc3fe1c2SSimon Glass        def __init__(self, rc, err_lines, sizes, func_sizes):
169fc3fe1c2SSimon Glass            self.rc = rc
170fc3fe1c2SSimon Glass            self.err_lines = err_lines
171fc3fe1c2SSimon Glass            self.sizes = sizes
172fc3fe1c2SSimon Glass            self.func_sizes = func_sizes
173fc3fe1c2SSimon Glass
174fc3fe1c2SSimon Glass    def __init__(self, toolchains, base_dir, git_dir, num_threads, num_jobs,
17599796923SMasahiro Yamada                 gnu_make='make', checkout=True, show_unknown=True, step=1):
176fc3fe1c2SSimon Glass        """Create a new Builder object
177fc3fe1c2SSimon Glass
178fc3fe1c2SSimon Glass        Args:
179fc3fe1c2SSimon Glass            toolchains: Toolchains object to use for building
180fc3fe1c2SSimon Glass            base_dir: Base directory to use for builder
181fc3fe1c2SSimon Glass            git_dir: Git directory containing source repository
182fc3fe1c2SSimon Glass            num_threads: Number of builder threads to run
183fc3fe1c2SSimon Glass            num_jobs: Number of jobs to run at once (passed to make as -j)
18499796923SMasahiro Yamada            gnu_make: the command name of GNU Make.
185fc3fe1c2SSimon Glass            checkout: True to check out source, False to skip that step.
186fc3fe1c2SSimon Glass                This is used for testing.
187fc3fe1c2SSimon Glass            show_unknown: Show unknown boards (those not built) in summary
188fc3fe1c2SSimon Glass            step: 1 to process every commit, n to process every nth commit
189fc3fe1c2SSimon Glass        """
190fc3fe1c2SSimon Glass        self.toolchains = toolchains
191fc3fe1c2SSimon Glass        self.base_dir = base_dir
192fc3fe1c2SSimon Glass        self._working_dir = os.path.join(base_dir, '.bm-work')
193fc3fe1c2SSimon Glass        self.threads = []
194fc3fe1c2SSimon Glass        self.active = True
195fc3fe1c2SSimon Glass        self.do_make = self.Make
19699796923SMasahiro Yamada        self.gnu_make = gnu_make
197fc3fe1c2SSimon Glass        self.checkout = checkout
198fc3fe1c2SSimon Glass        self.num_threads = num_threads
199fc3fe1c2SSimon Glass        self.num_jobs = num_jobs
200fc3fe1c2SSimon Glass        self.already_done = 0
201fc3fe1c2SSimon Glass        self.force_build = False
202fc3fe1c2SSimon Glass        self.git_dir = git_dir
203fc3fe1c2SSimon Glass        self._show_unknown = show_unknown
204fc3fe1c2SSimon Glass        self._timestamp_count = 10
205fc3fe1c2SSimon Glass        self._build_period_us = None
206fc3fe1c2SSimon Glass        self._complete_delay = None
207fc3fe1c2SSimon Glass        self._next_delay_update = datetime.now()
208fc3fe1c2SSimon Glass        self.force_config_on_failure = True
2094266dc28SSimon Glass        self.force_build_failures = False
21097e91526SSimon Glass        self.force_reconfig = False
211fc3fe1c2SSimon Glass        self._step = step
212189a4968SSimon Glass        self.in_tree = False
213fc3fe1c2SSimon Glass
214fc3fe1c2SSimon Glass        self.col = terminal.Color()
215fc3fe1c2SSimon Glass
216fc3fe1c2SSimon Glass        self.queue = Queue.Queue()
217fc3fe1c2SSimon Glass        self.out_queue = Queue.Queue()
218fc3fe1c2SSimon Glass        for i in range(self.num_threads):
219*190064b4SSimon Glass            t = builderthread.BuilderThread(self, i)
220fc3fe1c2SSimon Glass            t.setDaemon(True)
221fc3fe1c2SSimon Glass            t.start()
222fc3fe1c2SSimon Glass            self.threads.append(t)
223fc3fe1c2SSimon Glass
224fc3fe1c2SSimon Glass        self.last_line_len = 0
225*190064b4SSimon Glass        t = builderthread.ResultThread(self)
226fc3fe1c2SSimon Glass        t.setDaemon(True)
227fc3fe1c2SSimon Glass        t.start()
228fc3fe1c2SSimon Glass        self.threads.append(t)
229fc3fe1c2SSimon Glass
230fc3fe1c2SSimon Glass        ignore_lines = ['(make.*Waiting for unfinished)', '(Segmentation fault)']
231fc3fe1c2SSimon Glass        self.re_make_err = re.compile('|'.join(ignore_lines))
232fc3fe1c2SSimon Glass
233fc3fe1c2SSimon Glass    def __del__(self):
234fc3fe1c2SSimon Glass        """Get rid of all threads created by the builder"""
235fc3fe1c2SSimon Glass        for t in self.threads:
236fc3fe1c2SSimon Glass            del t
237fc3fe1c2SSimon Glass
238fc3fe1c2SSimon Glass    def _AddTimestamp(self):
239fc3fe1c2SSimon Glass        """Add a new timestamp to the list and record the build period.
240fc3fe1c2SSimon Glass
241fc3fe1c2SSimon Glass        The build period is the length of time taken to perform a single
242fc3fe1c2SSimon Glass        build (one board, one commit).
243fc3fe1c2SSimon Glass        """
244fc3fe1c2SSimon Glass        now = datetime.now()
245fc3fe1c2SSimon Glass        self._timestamps.append(now)
246fc3fe1c2SSimon Glass        count = len(self._timestamps)
247fc3fe1c2SSimon Glass        delta = self._timestamps[-1] - self._timestamps[0]
248fc3fe1c2SSimon Glass        seconds = delta.total_seconds()
249fc3fe1c2SSimon Glass
250fc3fe1c2SSimon Glass        # If we have enough data, estimate build period (time taken for a
251fc3fe1c2SSimon Glass        # single build) and therefore completion time.
252fc3fe1c2SSimon Glass        if count > 1 and self._next_delay_update < now:
253fc3fe1c2SSimon Glass            self._next_delay_update = now + timedelta(seconds=2)
254fc3fe1c2SSimon Glass            if seconds > 0:
255fc3fe1c2SSimon Glass                self._build_period = float(seconds) / count
256fc3fe1c2SSimon Glass                todo = self.count - self.upto
257fc3fe1c2SSimon Glass                self._complete_delay = timedelta(microseconds=
258fc3fe1c2SSimon Glass                        self._build_period * todo * 1000000)
259fc3fe1c2SSimon Glass                # Round it
260fc3fe1c2SSimon Glass                self._complete_delay -= timedelta(
261fc3fe1c2SSimon Glass                        microseconds=self._complete_delay.microseconds)
262fc3fe1c2SSimon Glass
263fc3fe1c2SSimon Glass        if seconds > 60:
264fc3fe1c2SSimon Glass            self._timestamps.popleft()
265fc3fe1c2SSimon Glass            count -= 1
266fc3fe1c2SSimon Glass
267fc3fe1c2SSimon Glass    def ClearLine(self, length):
268fc3fe1c2SSimon Glass        """Clear any characters on the current line
269fc3fe1c2SSimon Glass
270fc3fe1c2SSimon Glass        Make way for a new line of length 'length', by outputting enough
271fc3fe1c2SSimon Glass        spaces to clear out the old line. Then remember the new length for
272fc3fe1c2SSimon Glass        next time.
273fc3fe1c2SSimon Glass
274fc3fe1c2SSimon Glass        Args:
275fc3fe1c2SSimon Glass            length: Length of new line, in characters
276fc3fe1c2SSimon Glass        """
277fc3fe1c2SSimon Glass        if length < self.last_line_len:
278fc3fe1c2SSimon Glass            print ' ' * (self.last_line_len - length),
279fc3fe1c2SSimon Glass            print '\r',
280fc3fe1c2SSimon Glass        self.last_line_len = length
281fc3fe1c2SSimon Glass        sys.stdout.flush()
282fc3fe1c2SSimon Glass
283fc3fe1c2SSimon Glass    def SelectCommit(self, commit, checkout=True):
284fc3fe1c2SSimon Glass        """Checkout the selected commit for this build
285fc3fe1c2SSimon Glass        """
286fc3fe1c2SSimon Glass        self.commit = commit
287fc3fe1c2SSimon Glass        if checkout and self.checkout:
288fc3fe1c2SSimon Glass            gitutil.Checkout(commit.hash)
289fc3fe1c2SSimon Glass
290fc3fe1c2SSimon Glass    def Make(self, commit, brd, stage, cwd, *args, **kwargs):
291fc3fe1c2SSimon Glass        """Run make
292fc3fe1c2SSimon Glass
293fc3fe1c2SSimon Glass        Args:
294fc3fe1c2SSimon Glass            commit: Commit object that is being built
295fc3fe1c2SSimon Glass            brd: Board object that is being built
296fc3fe1c2SSimon Glass            stage: Stage that we are at (distclean, config, build)
297fc3fe1c2SSimon Glass            cwd: Directory where make should be run
298fc3fe1c2SSimon Glass            args: Arguments to pass to make
299fc3fe1c2SSimon Glass            kwargs: Arguments to pass to command.RunPipe()
300fc3fe1c2SSimon Glass        """
30199796923SMasahiro Yamada        cmd = [self.gnu_make] + list(args)
302fc3fe1c2SSimon Glass        result = command.RunPipe([cmd], capture=True, capture_stderr=True,
303fc3fe1c2SSimon Glass                cwd=cwd, raise_on_error=False, **kwargs)
304fc3fe1c2SSimon Glass        return result
305fc3fe1c2SSimon Glass
306fc3fe1c2SSimon Glass    def ProcessResult(self, result):
307fc3fe1c2SSimon Glass        """Process the result of a build, showing progress information
308fc3fe1c2SSimon Glass
309fc3fe1c2SSimon Glass        Args:
310fc3fe1c2SSimon Glass            result: A CommandResult object
311fc3fe1c2SSimon Glass        """
312fc3fe1c2SSimon Glass        col = terminal.Color()
313fc3fe1c2SSimon Glass        if result:
314fc3fe1c2SSimon Glass            target = result.brd.target
315fc3fe1c2SSimon Glass
316fc3fe1c2SSimon Glass            if result.return_code < 0:
317fc3fe1c2SSimon Glass                self.active = False
318fc3fe1c2SSimon Glass                command.StopAll()
319fc3fe1c2SSimon Glass                return
320fc3fe1c2SSimon Glass
321fc3fe1c2SSimon Glass            self.upto += 1
322fc3fe1c2SSimon Glass            if result.return_code != 0:
323fc3fe1c2SSimon Glass                self.fail += 1
324fc3fe1c2SSimon Glass            elif result.stderr:
325fc3fe1c2SSimon Glass                self.warned += 1
326fc3fe1c2SSimon Glass            if result.already_done:
327fc3fe1c2SSimon Glass                self.already_done += 1
328fc3fe1c2SSimon Glass        else:
329fc3fe1c2SSimon Glass            target = '(starting)'
330fc3fe1c2SSimon Glass
331fc3fe1c2SSimon Glass        # Display separate counts for ok, warned and fail
332fc3fe1c2SSimon Glass        ok = self.upto - self.warned - self.fail
333fc3fe1c2SSimon Glass        line = '\r' + self.col.Color(self.col.GREEN, '%5d' % ok)
334fc3fe1c2SSimon Glass        line += self.col.Color(self.col.YELLOW, '%5d' % self.warned)
335fc3fe1c2SSimon Glass        line += self.col.Color(self.col.RED, '%5d' % self.fail)
336fc3fe1c2SSimon Glass
337fc3fe1c2SSimon Glass        name = ' /%-5d  ' % self.count
338fc3fe1c2SSimon Glass
339fc3fe1c2SSimon Glass        # Add our current completion time estimate
340fc3fe1c2SSimon Glass        self._AddTimestamp()
341fc3fe1c2SSimon Glass        if self._complete_delay:
342fc3fe1c2SSimon Glass            name += '%s  : ' % self._complete_delay
343fc3fe1c2SSimon Glass        # When building all boards for a commit, we can print a commit
344fc3fe1c2SSimon Glass        # progress message.
345fc3fe1c2SSimon Glass        if result and result.commit_upto is None:
346fc3fe1c2SSimon Glass            name += 'commit %2d/%-3d' % (self.commit_upto + 1,
347fc3fe1c2SSimon Glass                    self.commit_count)
348fc3fe1c2SSimon Glass
349fc3fe1c2SSimon Glass        name += target
350fc3fe1c2SSimon Glass        print line + name,
351fc3fe1c2SSimon Glass        length = 13 + len(name)
352fc3fe1c2SSimon Glass        self.ClearLine(length)
353fc3fe1c2SSimon Glass
354fc3fe1c2SSimon Glass    def _GetOutputDir(self, commit_upto):
355fc3fe1c2SSimon Glass        """Get the name of the output directory for a commit number
356fc3fe1c2SSimon Glass
357fc3fe1c2SSimon Glass        The output directory is typically .../<branch>/<commit>.
358fc3fe1c2SSimon Glass
359fc3fe1c2SSimon Glass        Args:
360fc3fe1c2SSimon Glass            commit_upto: Commit number to use (0..self.count-1)
361fc3fe1c2SSimon Glass        """
362fea5858eSSimon Glass        if self.commits:
363fc3fe1c2SSimon Glass            commit = self.commits[commit_upto]
364fc3fe1c2SSimon Glass            subject = commit.subject.translate(trans_valid_chars)
365fc3fe1c2SSimon Glass            commit_dir = ('%02d_of_%02d_g%s_%s' % (commit_upto + 1,
366fc3fe1c2SSimon Glass                    self.commit_count, commit.hash, subject[:20]))
367fea5858eSSimon Glass        else:
368fea5858eSSimon Glass            commit_dir = 'current'
369fc3fe1c2SSimon Glass        output_dir = os.path.join(self.base_dir, commit_dir)
370fc3fe1c2SSimon Glass        return output_dir
371fc3fe1c2SSimon Glass
372fc3fe1c2SSimon Glass    def GetBuildDir(self, commit_upto, target):
373fc3fe1c2SSimon Glass        """Get the name of the build directory for a commit number
374fc3fe1c2SSimon Glass
375fc3fe1c2SSimon Glass        The build directory is typically .../<branch>/<commit>/<target>.
376fc3fe1c2SSimon Glass
377fc3fe1c2SSimon Glass        Args:
378fc3fe1c2SSimon Glass            commit_upto: Commit number to use (0..self.count-1)
379fc3fe1c2SSimon Glass            target: Target name
380fc3fe1c2SSimon Glass        """
381fc3fe1c2SSimon Glass        output_dir = self._GetOutputDir(commit_upto)
382fc3fe1c2SSimon Glass        return os.path.join(output_dir, target)
383fc3fe1c2SSimon Glass
384fc3fe1c2SSimon Glass    def GetDoneFile(self, commit_upto, target):
385fc3fe1c2SSimon Glass        """Get the name of the done file for a commit number
386fc3fe1c2SSimon Glass
387fc3fe1c2SSimon Glass        Args:
388fc3fe1c2SSimon Glass            commit_upto: Commit number to use (0..self.count-1)
389fc3fe1c2SSimon Glass            target: Target name
390fc3fe1c2SSimon Glass        """
391fc3fe1c2SSimon Glass        return os.path.join(self.GetBuildDir(commit_upto, target), 'done')
392fc3fe1c2SSimon Glass
393fc3fe1c2SSimon Glass    def GetSizesFile(self, commit_upto, target):
394fc3fe1c2SSimon Glass        """Get the name of the sizes file for a commit number
395fc3fe1c2SSimon Glass
396fc3fe1c2SSimon Glass        Args:
397fc3fe1c2SSimon Glass            commit_upto: Commit number to use (0..self.count-1)
398fc3fe1c2SSimon Glass            target: Target name
399fc3fe1c2SSimon Glass        """
400fc3fe1c2SSimon Glass        return os.path.join(self.GetBuildDir(commit_upto, target), 'sizes')
401fc3fe1c2SSimon Glass
402fc3fe1c2SSimon Glass    def GetFuncSizesFile(self, commit_upto, target, elf_fname):
403fc3fe1c2SSimon Glass        """Get the name of the funcsizes file for a commit number and ELF file
404fc3fe1c2SSimon Glass
405fc3fe1c2SSimon Glass        Args:
406fc3fe1c2SSimon Glass            commit_upto: Commit number to use (0..self.count-1)
407fc3fe1c2SSimon Glass            target: Target name
408fc3fe1c2SSimon Glass            elf_fname: Filename of elf image
409fc3fe1c2SSimon Glass        """
410fc3fe1c2SSimon Glass        return os.path.join(self.GetBuildDir(commit_upto, target),
411fc3fe1c2SSimon Glass                            '%s.sizes' % elf_fname.replace('/', '-'))
412fc3fe1c2SSimon Glass
413fc3fe1c2SSimon Glass    def GetObjdumpFile(self, commit_upto, target, elf_fname):
414fc3fe1c2SSimon Glass        """Get the name of the objdump file for a commit number and ELF file
415fc3fe1c2SSimon Glass
416fc3fe1c2SSimon Glass        Args:
417fc3fe1c2SSimon Glass            commit_upto: Commit number to use (0..self.count-1)
418fc3fe1c2SSimon Glass            target: Target name
419fc3fe1c2SSimon Glass            elf_fname: Filename of elf image
420fc3fe1c2SSimon Glass        """
421fc3fe1c2SSimon Glass        return os.path.join(self.GetBuildDir(commit_upto, target),
422fc3fe1c2SSimon Glass                            '%s.objdump' % elf_fname.replace('/', '-'))
423fc3fe1c2SSimon Glass
424fc3fe1c2SSimon Glass    def GetErrFile(self, commit_upto, target):
425fc3fe1c2SSimon Glass        """Get the name of the err file for a commit number
426fc3fe1c2SSimon Glass
427fc3fe1c2SSimon Glass        Args:
428fc3fe1c2SSimon Glass            commit_upto: Commit number to use (0..self.count-1)
429fc3fe1c2SSimon Glass            target: Target name
430fc3fe1c2SSimon Glass        """
431fc3fe1c2SSimon Glass        output_dir = self.GetBuildDir(commit_upto, target)
432fc3fe1c2SSimon Glass        return os.path.join(output_dir, 'err')
433fc3fe1c2SSimon Glass
434fc3fe1c2SSimon Glass    def FilterErrors(self, lines):
435fc3fe1c2SSimon Glass        """Filter out errors in which we have no interest
436fc3fe1c2SSimon Glass
437fc3fe1c2SSimon Glass        We should probably use map().
438fc3fe1c2SSimon Glass
439fc3fe1c2SSimon Glass        Args:
440fc3fe1c2SSimon Glass            lines: List of error lines, each a string
441fc3fe1c2SSimon Glass        Returns:
442fc3fe1c2SSimon Glass            New list with only interesting lines included
443fc3fe1c2SSimon Glass        """
444fc3fe1c2SSimon Glass        out_lines = []
445fc3fe1c2SSimon Glass        for line in lines:
446fc3fe1c2SSimon Glass            if not self.re_make_err.search(line):
447fc3fe1c2SSimon Glass                out_lines.append(line)
448fc3fe1c2SSimon Glass        return out_lines
449fc3fe1c2SSimon Glass
450fc3fe1c2SSimon Glass    def ReadFuncSizes(self, fname, fd):
451fc3fe1c2SSimon Glass        """Read function sizes from the output of 'nm'
452fc3fe1c2SSimon Glass
453fc3fe1c2SSimon Glass        Args:
454fc3fe1c2SSimon Glass            fd: File containing data to read
455fc3fe1c2SSimon Glass            fname: Filename we are reading from (just for errors)
456fc3fe1c2SSimon Glass
457fc3fe1c2SSimon Glass        Returns:
458fc3fe1c2SSimon Glass            Dictionary containing size of each function in bytes, indexed by
459fc3fe1c2SSimon Glass            function name.
460fc3fe1c2SSimon Glass        """
461fc3fe1c2SSimon Glass        sym = {}
462fc3fe1c2SSimon Glass        for line in fd.readlines():
463fc3fe1c2SSimon Glass            try:
464fc3fe1c2SSimon Glass                size, type, name = line[:-1].split()
465fc3fe1c2SSimon Glass            except:
466fc3fe1c2SSimon Glass                print "Invalid line in file '%s': '%s'" % (fname, line[:-1])
467fc3fe1c2SSimon Glass                continue
468fc3fe1c2SSimon Glass            if type in 'tTdDbB':
469fc3fe1c2SSimon Glass                # function names begin with '.' on 64-bit powerpc
470fc3fe1c2SSimon Glass                if '.' in name[1:]:
471fc3fe1c2SSimon Glass                    name = 'static.' + name.split('.')[0]
472fc3fe1c2SSimon Glass                sym[name] = sym.get(name, 0) + int(size, 16)
473fc3fe1c2SSimon Glass        return sym
474fc3fe1c2SSimon Glass
475fc3fe1c2SSimon Glass    def GetBuildOutcome(self, commit_upto, target, read_func_sizes):
476fc3fe1c2SSimon Glass        """Work out the outcome of a build.
477fc3fe1c2SSimon Glass
478fc3fe1c2SSimon Glass        Args:
479fc3fe1c2SSimon Glass            commit_upto: Commit number to check (0..n-1)
480fc3fe1c2SSimon Glass            target: Target board to check
481fc3fe1c2SSimon Glass            read_func_sizes: True to read function size information
482fc3fe1c2SSimon Glass
483fc3fe1c2SSimon Glass        Returns:
484fc3fe1c2SSimon Glass            Outcome object
485fc3fe1c2SSimon Glass        """
486fc3fe1c2SSimon Glass        done_file = self.GetDoneFile(commit_upto, target)
487fc3fe1c2SSimon Glass        sizes_file = self.GetSizesFile(commit_upto, target)
488fc3fe1c2SSimon Glass        sizes = {}
489fc3fe1c2SSimon Glass        func_sizes = {}
490fc3fe1c2SSimon Glass        if os.path.exists(done_file):
491fc3fe1c2SSimon Glass            with open(done_file, 'r') as fd:
492fc3fe1c2SSimon Glass                return_code = int(fd.readline())
493fc3fe1c2SSimon Glass                err_lines = []
494fc3fe1c2SSimon Glass                err_file = self.GetErrFile(commit_upto, target)
495fc3fe1c2SSimon Glass                if os.path.exists(err_file):
496fc3fe1c2SSimon Glass                    with open(err_file, 'r') as fd:
497fc3fe1c2SSimon Glass                        err_lines = self.FilterErrors(fd.readlines())
498fc3fe1c2SSimon Glass
499fc3fe1c2SSimon Glass                # Decide whether the build was ok, failed or created warnings
500fc3fe1c2SSimon Glass                if return_code:
501fc3fe1c2SSimon Glass                    rc = OUTCOME_ERROR
502fc3fe1c2SSimon Glass                elif len(err_lines):
503fc3fe1c2SSimon Glass                    rc = OUTCOME_WARNING
504fc3fe1c2SSimon Glass                else:
505fc3fe1c2SSimon Glass                    rc = OUTCOME_OK
506fc3fe1c2SSimon Glass
507fc3fe1c2SSimon Glass                # Convert size information to our simple format
508fc3fe1c2SSimon Glass                if os.path.exists(sizes_file):
509fc3fe1c2SSimon Glass                    with open(sizes_file, 'r') as fd:
510fc3fe1c2SSimon Glass                        for line in fd.readlines():
511fc3fe1c2SSimon Glass                            values = line.split()
512fc3fe1c2SSimon Glass                            rodata = 0
513fc3fe1c2SSimon Glass                            if len(values) > 6:
514fc3fe1c2SSimon Glass                                rodata = int(values[6], 16)
515fc3fe1c2SSimon Glass                            size_dict = {
516fc3fe1c2SSimon Glass                                'all' : int(values[0]) + int(values[1]) +
517fc3fe1c2SSimon Glass                                        int(values[2]),
518fc3fe1c2SSimon Glass                                'text' : int(values[0]) - rodata,
519fc3fe1c2SSimon Glass                                'data' : int(values[1]),
520fc3fe1c2SSimon Glass                                'bss' : int(values[2]),
521fc3fe1c2SSimon Glass                                'rodata' : rodata,
522fc3fe1c2SSimon Glass                            }
523fc3fe1c2SSimon Glass                            sizes[values[5]] = size_dict
524fc3fe1c2SSimon Glass
525fc3fe1c2SSimon Glass            if read_func_sizes:
526fc3fe1c2SSimon Glass                pattern = self.GetFuncSizesFile(commit_upto, target, '*')
527fc3fe1c2SSimon Glass                for fname in glob.glob(pattern):
528fc3fe1c2SSimon Glass                    with open(fname, 'r') as fd:
529fc3fe1c2SSimon Glass                        dict_name = os.path.basename(fname).replace('.sizes',
530fc3fe1c2SSimon Glass                                                                    '')
531fc3fe1c2SSimon Glass                        func_sizes[dict_name] = self.ReadFuncSizes(fname, fd)
532fc3fe1c2SSimon Glass
533fc3fe1c2SSimon Glass            return Builder.Outcome(rc, err_lines, sizes, func_sizes)
534fc3fe1c2SSimon Glass
535fc3fe1c2SSimon Glass        return Builder.Outcome(OUTCOME_UNKNOWN, [], {}, {})
536fc3fe1c2SSimon Glass
537fc3fe1c2SSimon Glass    def GetResultSummary(self, boards_selected, commit_upto, read_func_sizes):
538fc3fe1c2SSimon Glass        """Calculate a summary of the results of building a commit.
539fc3fe1c2SSimon Glass
540fc3fe1c2SSimon Glass        Args:
541fc3fe1c2SSimon Glass            board_selected: Dict containing boards to summarise
542fc3fe1c2SSimon Glass            commit_upto: Commit number to summarize (0..self.count-1)
543fc3fe1c2SSimon Glass            read_func_sizes: True to read function size information
544fc3fe1c2SSimon Glass
545fc3fe1c2SSimon Glass        Returns:
546fc3fe1c2SSimon Glass            Tuple:
547fc3fe1c2SSimon Glass                Dict containing boards which passed building this commit.
548fc3fe1c2SSimon Glass                    keyed by board.target
549fc3fe1c2SSimon Glass                List containing a summary of error/warning lines
550fc3fe1c2SSimon Glass        """
551fc3fe1c2SSimon Glass        board_dict = {}
552fc3fe1c2SSimon Glass        err_lines_summary = []
553fc3fe1c2SSimon Glass
554fc3fe1c2SSimon Glass        for board in boards_selected.itervalues():
555fc3fe1c2SSimon Glass            outcome = self.GetBuildOutcome(commit_upto, board.target,
556fc3fe1c2SSimon Glass                                           read_func_sizes)
557fc3fe1c2SSimon Glass            board_dict[board.target] = outcome
558fc3fe1c2SSimon Glass            for err in outcome.err_lines:
559fc3fe1c2SSimon Glass                if err and not err.rstrip() in err_lines_summary:
560fc3fe1c2SSimon Glass                    err_lines_summary.append(err.rstrip())
561fc3fe1c2SSimon Glass        return board_dict, err_lines_summary
562fc3fe1c2SSimon Glass
563fc3fe1c2SSimon Glass    def AddOutcome(self, board_dict, arch_list, changes, char, color):
564fc3fe1c2SSimon Glass        """Add an output to our list of outcomes for each architecture
565fc3fe1c2SSimon Glass
566fc3fe1c2SSimon Glass        This simple function adds failing boards (changes) to the
567fc3fe1c2SSimon Glass        relevant architecture string, so we can print the results out
568fc3fe1c2SSimon Glass        sorted by architecture.
569fc3fe1c2SSimon Glass
570fc3fe1c2SSimon Glass        Args:
571fc3fe1c2SSimon Glass             board_dict: Dict containing all boards
572fc3fe1c2SSimon Glass             arch_list: Dict keyed by arch name. Value is a string containing
573fc3fe1c2SSimon Glass                    a list of board names which failed for that arch.
574fc3fe1c2SSimon Glass             changes: List of boards to add to arch_list
575fc3fe1c2SSimon Glass             color: terminal.Colour object
576fc3fe1c2SSimon Glass        """
577fc3fe1c2SSimon Glass        done_arch = {}
578fc3fe1c2SSimon Glass        for target in changes:
579fc3fe1c2SSimon Glass            if target in board_dict:
580fc3fe1c2SSimon Glass                arch = board_dict[target].arch
581fc3fe1c2SSimon Glass            else:
582fc3fe1c2SSimon Glass                arch = 'unknown'
583fc3fe1c2SSimon Glass            str = self.col.Color(color, ' ' + target)
584fc3fe1c2SSimon Glass            if not arch in done_arch:
585fc3fe1c2SSimon Glass                str = self.col.Color(color, char) + '  ' + str
586fc3fe1c2SSimon Glass                done_arch[arch] = True
587fc3fe1c2SSimon Glass            if not arch in arch_list:
588fc3fe1c2SSimon Glass                arch_list[arch] = str
589fc3fe1c2SSimon Glass            else:
590fc3fe1c2SSimon Glass                arch_list[arch] += str
591fc3fe1c2SSimon Glass
592fc3fe1c2SSimon Glass
593fc3fe1c2SSimon Glass    def ColourNum(self, num):
594fc3fe1c2SSimon Glass        color = self.col.RED if num > 0 else self.col.GREEN
595fc3fe1c2SSimon Glass        if num == 0:
596fc3fe1c2SSimon Glass            return '0'
597fc3fe1c2SSimon Glass        return self.col.Color(color, str(num))
598fc3fe1c2SSimon Glass
599fc3fe1c2SSimon Glass    def ResetResultSummary(self, board_selected):
600fc3fe1c2SSimon Glass        """Reset the results summary ready for use.
601fc3fe1c2SSimon Glass
602fc3fe1c2SSimon Glass        Set up the base board list to be all those selected, and set the
603fc3fe1c2SSimon Glass        error lines to empty.
604fc3fe1c2SSimon Glass
605fc3fe1c2SSimon Glass        Following this, calls to PrintResultSummary() will use this
606fc3fe1c2SSimon Glass        information to work out what has changed.
607fc3fe1c2SSimon Glass
608fc3fe1c2SSimon Glass        Args:
609fc3fe1c2SSimon Glass            board_selected: Dict containing boards to summarise, keyed by
610fc3fe1c2SSimon Glass                board.target
611fc3fe1c2SSimon Glass        """
612fc3fe1c2SSimon Glass        self._base_board_dict = {}
613fc3fe1c2SSimon Glass        for board in board_selected:
614fc3fe1c2SSimon Glass            self._base_board_dict[board] = Builder.Outcome(0, [], [], {})
615fc3fe1c2SSimon Glass        self._base_err_lines = []
616fc3fe1c2SSimon Glass
617fc3fe1c2SSimon Glass    def PrintFuncSizeDetail(self, fname, old, new):
618fc3fe1c2SSimon Glass        grow, shrink, add, remove, up, down = 0, 0, 0, 0, 0, 0
619fc3fe1c2SSimon Glass        delta, common = [], {}
620fc3fe1c2SSimon Glass
621fc3fe1c2SSimon Glass        for a in old:
622fc3fe1c2SSimon Glass            if a in new:
623fc3fe1c2SSimon Glass                common[a] = 1
624fc3fe1c2SSimon Glass
625fc3fe1c2SSimon Glass        for name in old:
626fc3fe1c2SSimon Glass            if name not in common:
627fc3fe1c2SSimon Glass                remove += 1
628fc3fe1c2SSimon Glass                down += old[name]
629fc3fe1c2SSimon Glass                delta.append([-old[name], name])
630fc3fe1c2SSimon Glass
631fc3fe1c2SSimon Glass        for name in new:
632fc3fe1c2SSimon Glass            if name not in common:
633fc3fe1c2SSimon Glass                add += 1
634fc3fe1c2SSimon Glass                up += new[name]
635fc3fe1c2SSimon Glass                delta.append([new[name], name])
636fc3fe1c2SSimon Glass
637fc3fe1c2SSimon Glass        for name in common:
638fc3fe1c2SSimon Glass                diff = new.get(name, 0) - old.get(name, 0)
639fc3fe1c2SSimon Glass                if diff > 0:
640fc3fe1c2SSimon Glass                    grow, up = grow + 1, up + diff
641fc3fe1c2SSimon Glass                elif diff < 0:
642fc3fe1c2SSimon Glass                    shrink, down = shrink + 1, down - diff
643fc3fe1c2SSimon Glass                delta.append([diff, name])
644fc3fe1c2SSimon Glass
645fc3fe1c2SSimon Glass        delta.sort()
646fc3fe1c2SSimon Glass        delta.reverse()
647fc3fe1c2SSimon Glass
648fc3fe1c2SSimon Glass        args = [add, -remove, grow, -shrink, up, -down, up - down]
649fc3fe1c2SSimon Glass        if max(args) == 0:
650fc3fe1c2SSimon Glass            return
651fc3fe1c2SSimon Glass        args = [self.ColourNum(x) for x in args]
652fc3fe1c2SSimon Glass        indent = ' ' * 15
653fc3fe1c2SSimon Glass        print ('%s%s: add: %s/%s, grow: %s/%s bytes: %s/%s (%s)' %
654fc3fe1c2SSimon Glass               tuple([indent, self.col.Color(self.col.YELLOW, fname)] + args))
655fc3fe1c2SSimon Glass        print '%s  %-38s %7s %7s %+7s' % (indent, 'function', 'old', 'new',
656fc3fe1c2SSimon Glass                                        'delta')
657fc3fe1c2SSimon Glass        for diff, name in delta:
658fc3fe1c2SSimon Glass            if diff:
659fc3fe1c2SSimon Glass                color = self.col.RED if diff > 0 else self.col.GREEN
660fc3fe1c2SSimon Glass                msg = '%s  %-38s %7s %7s %+7d' % (indent, name,
661fc3fe1c2SSimon Glass                        old.get(name, '-'), new.get(name,'-'), diff)
662fc3fe1c2SSimon Glass                print self.col.Color(color, msg)
663fc3fe1c2SSimon Glass
664fc3fe1c2SSimon Glass
665fc3fe1c2SSimon Glass    def PrintSizeDetail(self, target_list, show_bloat):
666fc3fe1c2SSimon Glass        """Show details size information for each board
667fc3fe1c2SSimon Glass
668fc3fe1c2SSimon Glass        Args:
669fc3fe1c2SSimon Glass            target_list: List of targets, each a dict containing:
670fc3fe1c2SSimon Glass                    'target': Target name
671fc3fe1c2SSimon Glass                    'total_diff': Total difference in bytes across all areas
672fc3fe1c2SSimon Glass                    <part_name>: Difference for that part
673fc3fe1c2SSimon Glass            show_bloat: Show detail for each function
674fc3fe1c2SSimon Glass        """
675fc3fe1c2SSimon Glass        targets_by_diff = sorted(target_list, reverse=True,
676fc3fe1c2SSimon Glass        key=lambda x: x['_total_diff'])
677fc3fe1c2SSimon Glass        for result in targets_by_diff:
678fc3fe1c2SSimon Glass            printed_target = False
679fc3fe1c2SSimon Glass            for name in sorted(result):
680fc3fe1c2SSimon Glass                diff = result[name]
681fc3fe1c2SSimon Glass                if name.startswith('_'):
682fc3fe1c2SSimon Glass                    continue
683fc3fe1c2SSimon Glass                if diff != 0:
684fc3fe1c2SSimon Glass                    color = self.col.RED if diff > 0 else self.col.GREEN
685fc3fe1c2SSimon Glass                msg = ' %s %+d' % (name, diff)
686fc3fe1c2SSimon Glass                if not printed_target:
687fc3fe1c2SSimon Glass                    print '%10s  %-15s:' % ('', result['_target']),
688fc3fe1c2SSimon Glass                    printed_target = True
689fc3fe1c2SSimon Glass                print self.col.Color(color, msg),
690fc3fe1c2SSimon Glass            if printed_target:
691fc3fe1c2SSimon Glass                print
692fc3fe1c2SSimon Glass                if show_bloat:
693fc3fe1c2SSimon Glass                    target = result['_target']
694fc3fe1c2SSimon Glass                    outcome = result['_outcome']
695fc3fe1c2SSimon Glass                    base_outcome = self._base_board_dict[target]
696fc3fe1c2SSimon Glass                    for fname in outcome.func_sizes:
697fc3fe1c2SSimon Glass                        self.PrintFuncSizeDetail(fname,
698fc3fe1c2SSimon Glass                                                 base_outcome.func_sizes[fname],
699fc3fe1c2SSimon Glass                                                 outcome.func_sizes[fname])
700fc3fe1c2SSimon Glass
701fc3fe1c2SSimon Glass
702fc3fe1c2SSimon Glass    def PrintSizeSummary(self, board_selected, board_dict, show_detail,
703fc3fe1c2SSimon Glass                         show_bloat):
704fc3fe1c2SSimon Glass        """Print a summary of image sizes broken down by section.
705fc3fe1c2SSimon Glass
706fc3fe1c2SSimon Glass        The summary takes the form of one line per architecture. The
707fc3fe1c2SSimon Glass        line contains deltas for each of the sections (+ means the section
708fc3fe1c2SSimon Glass        got bigger, - means smaller). The nunmbers are the average number
709fc3fe1c2SSimon Glass        of bytes that a board in this section increased by.
710fc3fe1c2SSimon Glass
711fc3fe1c2SSimon Glass        For example:
712fc3fe1c2SSimon Glass           powerpc: (622 boards)   text -0.0
713fc3fe1c2SSimon Glass          arm: (285 boards)   text -0.0
714fc3fe1c2SSimon Glass          nds32: (3 boards)   text -8.0
715fc3fe1c2SSimon Glass
716fc3fe1c2SSimon Glass        Args:
717fc3fe1c2SSimon Glass            board_selected: Dict containing boards to summarise, keyed by
718fc3fe1c2SSimon Glass                board.target
719fc3fe1c2SSimon Glass            board_dict: Dict containing boards for which we built this
720fc3fe1c2SSimon Glass                commit, keyed by board.target. The value is an Outcome object.
721fc3fe1c2SSimon Glass            show_detail: Show detail for each board
722fc3fe1c2SSimon Glass            show_bloat: Show detail for each function
723fc3fe1c2SSimon Glass        """
724fc3fe1c2SSimon Glass        arch_list = {}
725fc3fe1c2SSimon Glass        arch_count = {}
726fc3fe1c2SSimon Glass
727fc3fe1c2SSimon Glass        # Calculate changes in size for different image parts
728fc3fe1c2SSimon Glass        # The previous sizes are in Board.sizes, for each board
729fc3fe1c2SSimon Glass        for target in board_dict:
730fc3fe1c2SSimon Glass            if target not in board_selected:
731fc3fe1c2SSimon Glass                continue
732fc3fe1c2SSimon Glass            base_sizes = self._base_board_dict[target].sizes
733fc3fe1c2SSimon Glass            outcome = board_dict[target]
734fc3fe1c2SSimon Glass            sizes = outcome.sizes
735fc3fe1c2SSimon Glass
736fc3fe1c2SSimon Glass            # Loop through the list of images, creating a dict of size
737fc3fe1c2SSimon Glass            # changes for each image/part. We end up with something like
738fc3fe1c2SSimon Glass            # {'target' : 'snapper9g45, 'data' : 5, 'u-boot-spl:text' : -4}
739fc3fe1c2SSimon Glass            # which means that U-Boot data increased by 5 bytes and SPL
740fc3fe1c2SSimon Glass            # text decreased by 4.
741fc3fe1c2SSimon Glass            err = {'_target' : target}
742fc3fe1c2SSimon Glass            for image in sizes:
743fc3fe1c2SSimon Glass                if image in base_sizes:
744fc3fe1c2SSimon Glass                    base_image = base_sizes[image]
745fc3fe1c2SSimon Glass                    # Loop through the text, data, bss parts
746fc3fe1c2SSimon Glass                    for part in sorted(sizes[image]):
747fc3fe1c2SSimon Glass                        diff = sizes[image][part] - base_image[part]
748fc3fe1c2SSimon Glass                        col = None
749fc3fe1c2SSimon Glass                        if diff:
750fc3fe1c2SSimon Glass                            if image == 'u-boot':
751fc3fe1c2SSimon Glass                                name = part
752fc3fe1c2SSimon Glass                            else:
753fc3fe1c2SSimon Glass                                name = image + ':' + part
754fc3fe1c2SSimon Glass                            err[name] = diff
755fc3fe1c2SSimon Glass            arch = board_selected[target].arch
756fc3fe1c2SSimon Glass            if not arch in arch_count:
757fc3fe1c2SSimon Glass                arch_count[arch] = 1
758fc3fe1c2SSimon Glass            else:
759fc3fe1c2SSimon Glass                arch_count[arch] += 1
760fc3fe1c2SSimon Glass            if not sizes:
761fc3fe1c2SSimon Glass                pass    # Only add to our list when we have some stats
762fc3fe1c2SSimon Glass            elif not arch in arch_list:
763fc3fe1c2SSimon Glass                arch_list[arch] = [err]
764fc3fe1c2SSimon Glass            else:
765fc3fe1c2SSimon Glass                arch_list[arch].append(err)
766fc3fe1c2SSimon Glass
767fc3fe1c2SSimon Glass        # We now have a list of image size changes sorted by arch
768fc3fe1c2SSimon Glass        # Print out a summary of these
769fc3fe1c2SSimon Glass        for arch, target_list in arch_list.iteritems():
770fc3fe1c2SSimon Glass            # Get total difference for each type
771fc3fe1c2SSimon Glass            totals = {}
772fc3fe1c2SSimon Glass            for result in target_list:
773fc3fe1c2SSimon Glass                total = 0
774fc3fe1c2SSimon Glass                for name, diff in result.iteritems():
775fc3fe1c2SSimon Glass                    if name.startswith('_'):
776fc3fe1c2SSimon Glass                        continue
777fc3fe1c2SSimon Glass                    total += diff
778fc3fe1c2SSimon Glass                    if name in totals:
779fc3fe1c2SSimon Glass                        totals[name] += diff
780fc3fe1c2SSimon Glass                    else:
781fc3fe1c2SSimon Glass                        totals[name] = diff
782fc3fe1c2SSimon Glass                result['_total_diff'] = total
783fc3fe1c2SSimon Glass                result['_outcome'] = board_dict[result['_target']]
784fc3fe1c2SSimon Glass
785fc3fe1c2SSimon Glass            count = len(target_list)
786fc3fe1c2SSimon Glass            printed_arch = False
787fc3fe1c2SSimon Glass            for name in sorted(totals):
788fc3fe1c2SSimon Glass                diff = totals[name]
789fc3fe1c2SSimon Glass                if diff:
790fc3fe1c2SSimon Glass                    # Display the average difference in this name for this
791fc3fe1c2SSimon Glass                    # architecture
792fc3fe1c2SSimon Glass                    avg_diff = float(diff) / count
793fc3fe1c2SSimon Glass                    color = self.col.RED if avg_diff > 0 else self.col.GREEN
794fc3fe1c2SSimon Glass                    msg = ' %s %+1.1f' % (name, avg_diff)
795fc3fe1c2SSimon Glass                    if not printed_arch:
796fc3fe1c2SSimon Glass                        print '%10s: (for %d/%d boards)' % (arch, count,
797fc3fe1c2SSimon Glass                                arch_count[arch]),
798fc3fe1c2SSimon Glass                        printed_arch = True
799fc3fe1c2SSimon Glass                    print self.col.Color(color, msg),
800fc3fe1c2SSimon Glass
801fc3fe1c2SSimon Glass            if printed_arch:
802fc3fe1c2SSimon Glass                print
803fc3fe1c2SSimon Glass                if show_detail:
804fc3fe1c2SSimon Glass                    self.PrintSizeDetail(target_list, show_bloat)
805fc3fe1c2SSimon Glass
806fc3fe1c2SSimon Glass
807fc3fe1c2SSimon Glass    def PrintResultSummary(self, board_selected, board_dict, err_lines,
808fc3fe1c2SSimon Glass                           show_sizes, show_detail, show_bloat):
809fc3fe1c2SSimon Glass        """Compare results with the base results and display delta.
810fc3fe1c2SSimon Glass
811fc3fe1c2SSimon Glass        Only boards mentioned in board_selected will be considered. This
812fc3fe1c2SSimon Glass        function is intended to be called repeatedly with the results of
813fc3fe1c2SSimon Glass        each commit. It therefore shows a 'diff' between what it saw in
814fc3fe1c2SSimon Glass        the last call and what it sees now.
815fc3fe1c2SSimon Glass
816fc3fe1c2SSimon Glass        Args:
817fc3fe1c2SSimon Glass            board_selected: Dict containing boards to summarise, keyed by
818fc3fe1c2SSimon Glass                board.target
819fc3fe1c2SSimon Glass            board_dict: Dict containing boards for which we built this
820fc3fe1c2SSimon Glass                commit, keyed by board.target. The value is an Outcome object.
821fc3fe1c2SSimon Glass            err_lines: A list of errors for this commit, or [] if there is
822fc3fe1c2SSimon Glass                none, or we don't want to print errors
823fc3fe1c2SSimon Glass            show_sizes: Show image size deltas
824fc3fe1c2SSimon Glass            show_detail: Show detail for each board
825fc3fe1c2SSimon Glass            show_bloat: Show detail for each function
826fc3fe1c2SSimon Glass        """
827fc3fe1c2SSimon Glass        better = []     # List of boards fixed since last commit
828fc3fe1c2SSimon Glass        worse = []      # List of new broken boards since last commit
829fc3fe1c2SSimon Glass        new = []        # List of boards that didn't exist last time
830fc3fe1c2SSimon Glass        unknown = []    # List of boards that were not built
831fc3fe1c2SSimon Glass
832fc3fe1c2SSimon Glass        for target in board_dict:
833fc3fe1c2SSimon Glass            if target not in board_selected:
834fc3fe1c2SSimon Glass                continue
835fc3fe1c2SSimon Glass
836fc3fe1c2SSimon Glass            # If the board was built last time, add its outcome to a list
837fc3fe1c2SSimon Glass            if target in self._base_board_dict:
838fc3fe1c2SSimon Glass                base_outcome = self._base_board_dict[target].rc
839fc3fe1c2SSimon Glass                outcome = board_dict[target]
840fc3fe1c2SSimon Glass                if outcome.rc == OUTCOME_UNKNOWN:
841fc3fe1c2SSimon Glass                    unknown.append(target)
842fc3fe1c2SSimon Glass                elif outcome.rc < base_outcome:
843fc3fe1c2SSimon Glass                    better.append(target)
844fc3fe1c2SSimon Glass                elif outcome.rc > base_outcome:
845fc3fe1c2SSimon Glass                    worse.append(target)
846fc3fe1c2SSimon Glass            else:
847fc3fe1c2SSimon Glass                new.append(target)
848fc3fe1c2SSimon Glass
849fc3fe1c2SSimon Glass        # Get a list of errors that have appeared, and disappeared
850fc3fe1c2SSimon Glass        better_err = []
851fc3fe1c2SSimon Glass        worse_err = []
852fc3fe1c2SSimon Glass        for line in err_lines:
853fc3fe1c2SSimon Glass            if line not in self._base_err_lines:
854fc3fe1c2SSimon Glass                worse_err.append('+' + line)
855fc3fe1c2SSimon Glass        for line in self._base_err_lines:
856fc3fe1c2SSimon Glass            if line not in err_lines:
857fc3fe1c2SSimon Glass                better_err.append('-' + line)
858fc3fe1c2SSimon Glass
859fc3fe1c2SSimon Glass        # Display results by arch
860fc3fe1c2SSimon Glass        if better or worse or unknown or new or worse_err or better_err:
861fc3fe1c2SSimon Glass            arch_list = {}
862fc3fe1c2SSimon Glass            self.AddOutcome(board_selected, arch_list, better, '',
863fc3fe1c2SSimon Glass                    self.col.GREEN)
864fc3fe1c2SSimon Glass            self.AddOutcome(board_selected, arch_list, worse, '+',
865fc3fe1c2SSimon Glass                    self.col.RED)
866fc3fe1c2SSimon Glass            self.AddOutcome(board_selected, arch_list, new, '*', self.col.BLUE)
867fc3fe1c2SSimon Glass            if self._show_unknown:
868fc3fe1c2SSimon Glass                self.AddOutcome(board_selected, arch_list, unknown, '?',
869fc3fe1c2SSimon Glass                        self.col.MAGENTA)
870fc3fe1c2SSimon Glass            for arch, target_list in arch_list.iteritems():
871fc3fe1c2SSimon Glass                print '%10s: %s' % (arch, target_list)
872fc3fe1c2SSimon Glass            if better_err:
873fc3fe1c2SSimon Glass                print self.col.Color(self.col.GREEN, '\n'.join(better_err))
874fc3fe1c2SSimon Glass            if worse_err:
875fc3fe1c2SSimon Glass                print self.col.Color(self.col.RED, '\n'.join(worse_err))
876fc3fe1c2SSimon Glass
877fc3fe1c2SSimon Glass        if show_sizes:
878fc3fe1c2SSimon Glass            self.PrintSizeSummary(board_selected, board_dict, show_detail,
879fc3fe1c2SSimon Glass                                  show_bloat)
880fc3fe1c2SSimon Glass
881fc3fe1c2SSimon Glass        # Save our updated information for the next call to this function
882fc3fe1c2SSimon Glass        self._base_board_dict = board_dict
883fc3fe1c2SSimon Glass        self._base_err_lines = err_lines
884fc3fe1c2SSimon Glass
885fc3fe1c2SSimon Glass        # Get a list of boards that did not get built, if needed
886fc3fe1c2SSimon Glass        not_built = []
887fc3fe1c2SSimon Glass        for board in board_selected:
888fc3fe1c2SSimon Glass            if not board in board_dict:
889fc3fe1c2SSimon Glass                not_built.append(board)
890fc3fe1c2SSimon Glass        if not_built:
891fc3fe1c2SSimon Glass            print "Boards not built (%d): %s" % (len(not_built),
892fc3fe1c2SSimon Glass                    ', '.join(not_built))
893fc3fe1c2SSimon Glass
894fc3fe1c2SSimon Glass
895fc3fe1c2SSimon Glass    def ShowSummary(self, commits, board_selected, show_errors, show_sizes,
896fc3fe1c2SSimon Glass                    show_detail, show_bloat):
897fc3fe1c2SSimon Glass        """Show a build summary for U-Boot for a given board list.
898fc3fe1c2SSimon Glass
899fc3fe1c2SSimon Glass        Reset the result summary, then repeatedly call GetResultSummary on
900fc3fe1c2SSimon Glass        each commit's results, then display the differences we see.
901fc3fe1c2SSimon Glass
902fc3fe1c2SSimon Glass        Args:
903fc3fe1c2SSimon Glass            commit: Commit objects to summarise
904fc3fe1c2SSimon Glass            board_selected: Dict containing boards to summarise
905fc3fe1c2SSimon Glass            show_errors: Show errors that occured
906fc3fe1c2SSimon Glass            show_sizes: Show size deltas
907fc3fe1c2SSimon Glass            show_detail: Show detail for each board
908fc3fe1c2SSimon Glass            show_bloat: Show detail for each function
909fc3fe1c2SSimon Glass        """
910fea5858eSSimon Glass        self.commit_count = len(commits) if commits else 1
911fc3fe1c2SSimon Glass        self.commits = commits
912fc3fe1c2SSimon Glass        self.ResetResultSummary(board_selected)
913fc3fe1c2SSimon Glass
914fc3fe1c2SSimon Glass        for commit_upto in range(0, self.commit_count, self._step):
915fc3fe1c2SSimon Glass            board_dict, err_lines = self.GetResultSummary(board_selected,
916fc3fe1c2SSimon Glass                    commit_upto, read_func_sizes=show_bloat)
917fea5858eSSimon Glass            if commits:
918fea5858eSSimon Glass                msg = '%02d: %s' % (commit_upto + 1,
919fea5858eSSimon Glass                        commits[commit_upto].subject)
920fea5858eSSimon Glass            else:
921fea5858eSSimon Glass                msg = 'current'
922fc3fe1c2SSimon Glass            print self.col.Color(self.col.BLUE, msg)
923fc3fe1c2SSimon Glass            self.PrintResultSummary(board_selected, board_dict,
924fc3fe1c2SSimon Glass                    err_lines if show_errors else [], show_sizes, show_detail,
925fc3fe1c2SSimon Glass                    show_bloat)
926fc3fe1c2SSimon Glass
927fc3fe1c2SSimon Glass
928fc3fe1c2SSimon Glass    def SetupBuild(self, board_selected, commits):
929fc3fe1c2SSimon Glass        """Set up ready to start a build.
930fc3fe1c2SSimon Glass
931fc3fe1c2SSimon Glass        Args:
932fc3fe1c2SSimon Glass            board_selected: Selected boards to build
933fc3fe1c2SSimon Glass            commits: Selected commits to build
934fc3fe1c2SSimon Glass        """
935fc3fe1c2SSimon Glass        # First work out how many commits we will build
936fea5858eSSimon Glass        count = (self.commit_count + self._step - 1) / self._step
937fc3fe1c2SSimon Glass        self.count = len(board_selected) * count
938fc3fe1c2SSimon Glass        self.upto = self.warned = self.fail = 0
939fc3fe1c2SSimon Glass        self._timestamps = collections.deque()
940fc3fe1c2SSimon Glass
941fc3fe1c2SSimon Glass    def BuildBoardsForCommit(self, board_selected, keep_outputs):
942fc3fe1c2SSimon Glass        """Build all boards for a single commit"""
943fc3fe1c2SSimon Glass        self.SetupBuild(board_selected)
944fc3fe1c2SSimon Glass        self.count = len(board_selected)
945fc3fe1c2SSimon Glass        for brd in board_selected.itervalues():
946fc3fe1c2SSimon Glass            job = BuilderJob()
947fc3fe1c2SSimon Glass            job.board = brd
948fc3fe1c2SSimon Glass            job.commits = None
949fc3fe1c2SSimon Glass            job.keep_outputs = keep_outputs
950fc3fe1c2SSimon Glass            self.queue.put(brd)
951fc3fe1c2SSimon Glass
952fc3fe1c2SSimon Glass        self.queue.join()
953fc3fe1c2SSimon Glass        self.out_queue.join()
954fc3fe1c2SSimon Glass        print
955fc3fe1c2SSimon Glass        self.ClearLine(0)
956fc3fe1c2SSimon Glass
957fc3fe1c2SSimon Glass    def BuildCommits(self, commits, board_selected, show_errors, keep_outputs):
958fc3fe1c2SSimon Glass        """Build all boards for all commits (non-incremental)"""
959fc3fe1c2SSimon Glass        self.commit_count = len(commits)
960fc3fe1c2SSimon Glass
961fc3fe1c2SSimon Glass        self.ResetResultSummary(board_selected)
962fc3fe1c2SSimon Glass        for self.commit_upto in range(self.commit_count):
963fc3fe1c2SSimon Glass            self.SelectCommit(commits[self.commit_upto])
964fc3fe1c2SSimon Glass            self.SelectOutputDir()
965*190064b4SSimon Glass            builderthread.Mkdir(self.output_dir)
966fc3fe1c2SSimon Glass
967fc3fe1c2SSimon Glass            self.BuildBoardsForCommit(board_selected, keep_outputs)
968fc3fe1c2SSimon Glass            board_dict, err_lines = self.GetResultSummary()
969fc3fe1c2SSimon Glass            self.PrintResultSummary(board_selected, board_dict,
970fc3fe1c2SSimon Glass                err_lines if show_errors else [])
971fc3fe1c2SSimon Glass
972fc3fe1c2SSimon Glass        if self.already_done:
973fc3fe1c2SSimon Glass            print '%d builds already done' % self.already_done
974fc3fe1c2SSimon Glass
975fc3fe1c2SSimon Glass    def GetThreadDir(self, thread_num):
976fc3fe1c2SSimon Glass        """Get the directory path to the working dir for a thread.
977fc3fe1c2SSimon Glass
978fc3fe1c2SSimon Glass        Args:
979fc3fe1c2SSimon Glass            thread_num: Number of thread to check.
980fc3fe1c2SSimon Glass        """
981fc3fe1c2SSimon Glass        return os.path.join(self._working_dir, '%02d' % thread_num)
982fc3fe1c2SSimon Glass
983fea5858eSSimon Glass    def _PrepareThread(self, thread_num, setup_git):
984fc3fe1c2SSimon Glass        """Prepare the working directory for a thread.
985fc3fe1c2SSimon Glass
986fc3fe1c2SSimon Glass        This clones or fetches the repo into the thread's work directory.
987fc3fe1c2SSimon Glass
988fc3fe1c2SSimon Glass        Args:
989fc3fe1c2SSimon Glass            thread_num: Thread number (0, 1, ...)
990fea5858eSSimon Glass            setup_git: True to set up a git repo clone
991fc3fe1c2SSimon Glass        """
992fc3fe1c2SSimon Glass        thread_dir = self.GetThreadDir(thread_num)
993*190064b4SSimon Glass        builderthread.Mkdir(thread_dir)
994fc3fe1c2SSimon Glass        git_dir = os.path.join(thread_dir, '.git')
995fc3fe1c2SSimon Glass
996fc3fe1c2SSimon Glass        # Clone the repo if it doesn't already exist
997fc3fe1c2SSimon Glass        # TODO(sjg@chromium): Perhaps some git hackery to symlink instead, so
998fc3fe1c2SSimon Glass        # we have a private index but uses the origin repo's contents?
999fea5858eSSimon Glass        if setup_git and self.git_dir:
1000fc3fe1c2SSimon Glass            src_dir = os.path.abspath(self.git_dir)
1001fc3fe1c2SSimon Glass            if os.path.exists(git_dir):
1002fc3fe1c2SSimon Glass                gitutil.Fetch(git_dir, thread_dir)
1003fc3fe1c2SSimon Glass            else:
1004fc3fe1c2SSimon Glass                print 'Cloning repo for thread %d' % thread_num
1005fc3fe1c2SSimon Glass                gitutil.Clone(src_dir, thread_dir)
1006fc3fe1c2SSimon Glass
1007fea5858eSSimon Glass    def _PrepareWorkingSpace(self, max_threads, setup_git):
1008fc3fe1c2SSimon Glass        """Prepare the working directory for use.
1009fc3fe1c2SSimon Glass
1010fc3fe1c2SSimon Glass        Set up the git repo for each thread.
1011fc3fe1c2SSimon Glass
1012fc3fe1c2SSimon Glass        Args:
1013fc3fe1c2SSimon Glass            max_threads: Maximum number of threads we expect to need.
1014fea5858eSSimon Glass            setup_git: True to set up a git repo clone
1015fc3fe1c2SSimon Glass        """
1016*190064b4SSimon Glass        builderthread.Mkdir(self._working_dir)
1017fc3fe1c2SSimon Glass        for thread in range(max_threads):
1018fea5858eSSimon Glass            self._PrepareThread(thread, setup_git)
1019fc3fe1c2SSimon Glass
1020fc3fe1c2SSimon Glass    def _PrepareOutputSpace(self):
1021fc3fe1c2SSimon Glass        """Get the output directories ready to receive files.
1022fc3fe1c2SSimon Glass
1023fc3fe1c2SSimon Glass        We delete any output directories which look like ones we need to
1024fc3fe1c2SSimon Glass        create. Having left over directories is confusing when the user wants
1025fc3fe1c2SSimon Glass        to check the output manually.
1026fc3fe1c2SSimon Glass        """
1027fc3fe1c2SSimon Glass        dir_list = []
1028fc3fe1c2SSimon Glass        for commit_upto in range(self.commit_count):
1029fc3fe1c2SSimon Glass            dir_list.append(self._GetOutputDir(commit_upto))
1030fc3fe1c2SSimon Glass
1031fc3fe1c2SSimon Glass        for dirname in glob.glob(os.path.join(self.base_dir, '*')):
1032fc3fe1c2SSimon Glass            if dirname not in dir_list:
1033fc3fe1c2SSimon Glass                shutil.rmtree(dirname)
1034fc3fe1c2SSimon Glass
1035fc3fe1c2SSimon Glass    def BuildBoards(self, commits, board_selected, show_errors, keep_outputs):
1036fc3fe1c2SSimon Glass        """Build all commits for a list of boards
1037fc3fe1c2SSimon Glass
1038fc3fe1c2SSimon Glass        Args:
1039fc3fe1c2SSimon Glass            commits: List of commits to be build, each a Commit object
1040fc3fe1c2SSimon Glass            boards_selected: Dict of selected boards, key is target name,
1041fc3fe1c2SSimon Glass                    value is Board object
1042fc3fe1c2SSimon Glass            show_errors: True to show summarised error/warning info
1043fc3fe1c2SSimon Glass            keep_outputs: True to save build output files
1044fc3fe1c2SSimon Glass        """
1045fea5858eSSimon Glass        self.commit_count = len(commits) if commits else 1
1046fc3fe1c2SSimon Glass        self.commits = commits
1047fc3fe1c2SSimon Glass
1048fc3fe1c2SSimon Glass        self.ResetResultSummary(board_selected)
1049*190064b4SSimon Glass        builderthread.Mkdir(self.base_dir)
1050fea5858eSSimon Glass        self._PrepareWorkingSpace(min(self.num_threads, len(board_selected)),
1051fea5858eSSimon Glass                commits is not None)
1052fc3fe1c2SSimon Glass        self._PrepareOutputSpace()
1053fc3fe1c2SSimon Glass        self.SetupBuild(board_selected, commits)
1054fc3fe1c2SSimon Glass        self.ProcessResult(None)
1055fc3fe1c2SSimon Glass
1056fc3fe1c2SSimon Glass        # Create jobs to build all commits for each board
1057fc3fe1c2SSimon Glass        for brd in board_selected.itervalues():
1058*190064b4SSimon Glass            job = builderthread.BuilderJob()
1059fc3fe1c2SSimon Glass            job.board = brd
1060fc3fe1c2SSimon Glass            job.commits = commits
1061fc3fe1c2SSimon Glass            job.keep_outputs = keep_outputs
1062fc3fe1c2SSimon Glass            job.step = self._step
1063fc3fe1c2SSimon Glass            self.queue.put(job)
1064fc3fe1c2SSimon Glass
1065fc3fe1c2SSimon Glass        # Wait until all jobs are started
1066fc3fe1c2SSimon Glass        self.queue.join()
1067fc3fe1c2SSimon Glass
1068fc3fe1c2SSimon Glass        # Wait until we have processed all output
1069fc3fe1c2SSimon Glass        self.out_queue.join()
1070fc3fe1c2SSimon Glass        print
1071fc3fe1c2SSimon Glass        self.ClearLine(0)
1072