blob: 063bbc01457e6aeb3e840583095ae49516de9228 [file] [log] [blame]
Tom Rini10e47792018-05-06 17:58:06 -04001# SPDX-License-Identifier: GPL-2.0+
Simon Glass4a1e88b2014-08-09 15:33:00 -06002# Copyright (c) 2014 Google, Inc
3#
Simon Glass4a1e88b2014-08-09 15:33:00 -06004
5import errno
6import glob
7import os
8import shutil
Lothar Waßmannce6df922018-04-08 05:14:11 -06009import sys
Simon Glass4a1e88b2014-08-09 15:33:00 -060010import threading
11
12import command
13import gitutil
14
Simon Glassfd3eea12015-02-05 22:06:13 -070015RETURN_CODE_RETRY = -1
16
Thierry Reding336d5bd2014-08-19 10:22:39 +020017def Mkdir(dirname, parents = False):
Simon Glass4a1e88b2014-08-09 15:33:00 -060018 """Make a directory if it doesn't already exist.
19
20 Args:
21 dirname: Directory to create
22 """
23 try:
Thierry Reding336d5bd2014-08-19 10:22:39 +020024 if parents:
25 os.makedirs(dirname)
26 else:
27 os.mkdir(dirname)
Simon Glass4a1e88b2014-08-09 15:33:00 -060028 except OSError as err:
29 if err.errno == errno.EEXIST:
Lothar Waßmannce6df922018-04-08 05:14:11 -060030 if os.path.realpath('.') == os.path.realpath(dirname):
Simon Glassc78ed662019-10-31 07:42:53 -060031 print("Cannot create the current working directory '%s'!" % dirname)
Lothar Waßmannce6df922018-04-08 05:14:11 -060032 sys.exit(1)
Simon Glass4a1e88b2014-08-09 15:33:00 -060033 pass
34 else:
35 raise
36
37class BuilderJob:
38 """Holds information about a job to be performed by a thread
39
40 Members:
41 board: Board object to build
Simon Glassdf890152020-03-18 09:42:41 -060042 commits: List of Commit objects to build
43 keep_outputs: True to save build output files
44 step: 1 to process every commit, n to process every nth commit
Simon Glassb6eb8cf2020-03-18 09:42:42 -060045 work_in_output: Use the output directory as the work directory and
46 don't write to a separate output directory.
Simon Glass4a1e88b2014-08-09 15:33:00 -060047 """
48 def __init__(self):
49 self.board = None
50 self.commits = []
Simon Glassdf890152020-03-18 09:42:41 -060051 self.keep_outputs = False
52 self.step = 1
Simon Glassb6eb8cf2020-03-18 09:42:42 -060053 self.work_in_output = False
Simon Glass4a1e88b2014-08-09 15:33:00 -060054
55
56class ResultThread(threading.Thread):
57 """This thread processes results from builder threads.
58
59 It simply passes the results on to the builder. There is only one
60 result thread, and this helps to serialise the build output.
61 """
62 def __init__(self, builder):
63 """Set up a new result thread
64
65 Args:
66 builder: Builder which will be sent each result
67 """
68 threading.Thread.__init__(self)
69 self.builder = builder
70
71 def run(self):
72 """Called to start up the result thread.
73
74 We collect the next result job and pass it on to the build.
75 """
76 while True:
77 result = self.builder.out_queue.get()
78 self.builder.ProcessResult(result)
79 self.builder.out_queue.task_done()
80
81
82class BuilderThread(threading.Thread):
83 """This thread builds U-Boot for a particular board.
84
85 An input queue provides each new job. We run 'make' to build U-Boot
86 and then pass the results on to the output queue.
87
88 Members:
89 builder: The builder which contains information we might need
90 thread_num: Our thread number (0-n-1), used to decide on a
91 temporary directory
92 """
Simon Glass6029af12020-04-09 15:08:51 -060093 def __init__(self, builder, thread_num, mrproper, per_board_out_dir):
Simon Glass4a1e88b2014-08-09 15:33:00 -060094 """Set up a new builder thread"""
95 threading.Thread.__init__(self)
96 self.builder = builder
97 self.thread_num = thread_num
Simon Glass6029af12020-04-09 15:08:51 -060098 self.mrproper = mrproper
Stephen Warren97c96902016-04-11 10:48:44 -060099 self.per_board_out_dir = per_board_out_dir
Simon Glass4a1e88b2014-08-09 15:33:00 -0600100
101 def Make(self, commit, brd, stage, cwd, *args, **kwargs):
102 """Run 'make' on a particular commit and board.
103
104 The source code will already be checked out, so the 'commit'
105 argument is only for information.
106
107 Args:
108 commit: Commit object that is being built
109 brd: Board object that is being built
110 stage: Stage of the build. Valid stages are:
Roger Meiere0a0e552014-08-20 22:10:29 +0200111 mrproper - can be called to clean source
Simon Glass4a1e88b2014-08-09 15:33:00 -0600112 config - called to configure for a board
113 build - the main make invocation - it does the build
114 args: A list of arguments to pass to 'make'
115 kwargs: A list of keyword arguments to pass to command.RunPipe()
116
117 Returns:
118 CommandResult object
119 """
120 return self.builder.do_make(commit, brd, stage, cwd, *args,
121 **kwargs)
122
Simon Glassb55b57d2016-11-16 14:09:25 -0700123 def RunCommit(self, commit_upto, brd, work_dir, do_config, config_only,
Simon Glassb6eb8cf2020-03-18 09:42:42 -0600124 force_build, force_build_failures, work_in_output):
Simon Glass4a1e88b2014-08-09 15:33:00 -0600125 """Build a particular commit.
126
127 If the build is already done, and we are not forcing a build, we skip
128 the build and just return the previously-saved results.
129
130 Args:
131 commit_upto: Commit number to build (0...n-1)
132 brd: Board object to build
133 work_dir: Directory to which the source will be checked out
134 do_config: True to run a make <board>_defconfig on the source
Simon Glassb55b57d2016-11-16 14:09:25 -0700135 config_only: Only configure the source, do not build it
Simon Glass4a1e88b2014-08-09 15:33:00 -0600136 force_build: Force a build even if one was previously done
137 force_build_failures: Force a bulid if the previous result showed
138 failure
Simon Glassb6eb8cf2020-03-18 09:42:42 -0600139 work_in_output: Use the output directory as the work directory and
140 don't write to a separate output directory.
Simon Glass4a1e88b2014-08-09 15:33:00 -0600141
142 Returns:
143 tuple containing:
144 - CommandResult object containing the results of the build
145 - boolean indicating whether 'make config' is still needed
146 """
147 # Create a default result - it will be overwritte by the call to
148 # self.Make() below, in the event that we do a build.
149 result = command.CommandResult()
150 result.return_code = 0
Simon Glassb6eb8cf2020-03-18 09:42:42 -0600151 if work_in_output or self.builder.in_tree:
Simon Glass4a1e88b2014-08-09 15:33:00 -0600152 out_dir = work_dir
153 else:
Stephen Warren97c96902016-04-11 10:48:44 -0600154 if self.per_board_out_dir:
155 out_rel_dir = os.path.join('..', brd.target)
156 else:
157 out_rel_dir = 'build'
158 out_dir = os.path.join(work_dir, out_rel_dir)
Simon Glass4a1e88b2014-08-09 15:33:00 -0600159
160 # Check if the job was already completed last time
161 done_file = self.builder.GetDoneFile(commit_upto, brd.target)
162 result.already_done = os.path.exists(done_file)
163 will_build = (force_build or force_build_failures or
164 not result.already_done)
Simon Glass09f1fe52014-09-05 19:00:17 -0600165 if result.already_done:
Simon Glass4a1e88b2014-08-09 15:33:00 -0600166 # Get the return code from that build and use it
167 with open(done_file, 'r') as fd:
Simon Glass4345a6d2018-12-10 09:05:23 -0700168 try:
169 result.return_code = int(fd.readline())
170 except ValueError:
171 # The file may be empty due to running out of disk space.
172 # Try a rebuild
173 result.return_code = RETURN_CODE_RETRY
Simon Glassfd3eea12015-02-05 22:06:13 -0700174
175 # Check the signal that the build needs to be retried
176 if result.return_code == RETURN_CODE_RETRY:
177 will_build = True
178 elif will_build:
Simon Glass09f1fe52014-09-05 19:00:17 -0600179 err_file = self.builder.GetErrFile(commit_upto, brd.target)
180 if os.path.exists(err_file) and os.stat(err_file).st_size:
181 result.stderr = 'bad'
182 elif not force_build:
183 # The build passed, so no need to build it again
184 will_build = False
Simon Glass4a1e88b2014-08-09 15:33:00 -0600185
186 if will_build:
187 # We are going to have to build it. First, get a toolchain
188 if not self.toolchain:
189 try:
190 self.toolchain = self.builder.toolchains.Select(brd.arch)
191 except ValueError as err:
192 result.return_code = 10
193 result.stdout = ''
194 result.stderr = str(err)
195 # TODO(sjg@chromium.org): This gets swallowed, but needs
196 # to be reported.
197
198 if self.toolchain:
199 # Checkout the right commit
200 if self.builder.commits:
201 commit = self.builder.commits[commit_upto]
202 if self.builder.checkout:
203 git_dir = os.path.join(work_dir, '.git')
204 gitutil.Checkout(commit.hash, git_dir, work_dir,
205 force=True)
206 else:
207 commit = 'current'
208
209 # Set up the environment and command line
Simon Glassd48a46c2014-12-01 17:34:00 -0700210 env = self.toolchain.MakeEnvironment(self.builder.full_path)
Simon Glass4a1e88b2014-08-09 15:33:00 -0600211 Mkdir(out_dir)
212 args = []
213 cwd = work_dir
Simon Glasse3096252014-08-28 09:43:42 -0600214 src_dir = os.path.realpath(work_dir)
Simon Glass4a1e88b2014-08-09 15:33:00 -0600215 if not self.builder.in_tree:
216 if commit_upto is None:
217 # In this case we are building in the original source
218 # directory (i.e. the current directory where buildman
219 # is invoked. The output directory is set to this
220 # thread's selected work directory.
221 #
222 # Symlinks can confuse U-Boot's Makefile since
223 # we may use '..' in our path, so remove them.
Stephen Warren97c96902016-04-11 10:48:44 -0600224 out_dir = os.path.realpath(out_dir)
225 args.append('O=%s' % out_dir)
Simon Glass4a1e88b2014-08-09 15:33:00 -0600226 cwd = None
Simon Glasse3096252014-08-28 09:43:42 -0600227 src_dir = os.getcwd()
Simon Glass4a1e88b2014-08-09 15:33:00 -0600228 else:
Stephen Warren97c96902016-04-11 10:48:44 -0600229 args.append('O=%s' % out_rel_dir)
Tom Rinif06dd812015-04-01 07:47:41 -0400230 if self.builder.verbose_build:
231 args.append('V=1')
232 else:
Simon Glass655b6102014-12-01 17:34:07 -0700233 args.append('-s')
Simon Glass4a1e88b2014-08-09 15:33:00 -0600234 if self.builder.num_jobs is not None:
235 args.extend(['-j', str(self.builder.num_jobs)])
Daniel Schwierzeck20e2ea92018-01-26 16:31:05 +0100236 if self.builder.warnings_as_errors:
237 args.append('KCFLAGS=-Werror')
Simon Glass4a1e88b2014-08-09 15:33:00 -0600238 config_args = ['%s_defconfig' % brd.target]
239 config_out = ''
240 args.extend(self.builder.toolchains.GetMakeArguments(brd))
Simon Glassf77ca5b2019-01-07 16:44:20 -0700241 args.extend(self.toolchain.MakeArgs())
Simon Glass4a1e88b2014-08-09 15:33:00 -0600242
243 # If we need to reconfigure, do that now
244 if do_config:
Stephen Warren97c96902016-04-11 10:48:44 -0600245 config_out = ''
Simon Glass6029af12020-04-09 15:08:51 -0600246 if self.mrproper:
Stephen Warren97c96902016-04-11 10:48:44 -0600247 result = self.Make(commit, brd, 'mrproper', cwd,
248 'mrproper', *args, env=env)
249 config_out += result.combined
Simon Glass4a1e88b2014-08-09 15:33:00 -0600250 result = self.Make(commit, brd, 'config', cwd,
251 *(args + config_args), env=env)
Simon Glass413f91a2015-02-05 22:06:12 -0700252 config_out += result.combined
Simon Glass4a1e88b2014-08-09 15:33:00 -0600253 do_config = False # No need to configure next time
254 if result.return_code == 0:
Simon Glassb55b57d2016-11-16 14:09:25 -0700255 if config_only:
Simon Glass739e8512016-11-13 14:25:51 -0700256 args.append('cfg')
Simon Glass4a1e88b2014-08-09 15:33:00 -0600257 result = self.Make(commit, brd, 'build', cwd, *args,
258 env=env)
Simon Glasse3096252014-08-28 09:43:42 -0600259 result.stderr = result.stderr.replace(src_dir + '/', '')
Simon Glass413f91a2015-02-05 22:06:12 -0700260 if self.builder.verbose_build:
261 result.stdout = config_out + result.stdout
Simon Glass4a1e88b2014-08-09 15:33:00 -0600262 else:
263 result.return_code = 1
264 result.stderr = 'No tool chain for %s\n' % brd.arch
265 result.already_done = False
266
267 result.toolchain = self.toolchain
268 result.brd = brd
269 result.commit_upto = commit_upto
270 result.out_dir = out_dir
271 return result, do_config
272
Simon Glassb6eb8cf2020-03-18 09:42:42 -0600273 def _WriteResult(self, result, keep_outputs, work_in_output):
Simon Glass4a1e88b2014-08-09 15:33:00 -0600274 """Write a built result to the output directory.
275
276 Args:
277 result: CommandResult object containing result to write
278 keep_outputs: True to store the output binaries, False
279 to delete them
Simon Glassb6eb8cf2020-03-18 09:42:42 -0600280 work_in_output: Use the output directory as the work directory and
281 don't write to a separate output directory.
Simon Glass4a1e88b2014-08-09 15:33:00 -0600282 """
Simon Glassb6eb8cf2020-03-18 09:42:42 -0600283 if work_in_output:
284 return
Simon Glass4a1e88b2014-08-09 15:33:00 -0600285 # Fatal error
286 if result.return_code < 0:
287 return
288
Simon Glassfd3eea12015-02-05 22:06:13 -0700289 # If we think this might have been aborted with Ctrl-C, record the
290 # failure but not that we are 'done' with this board. A retry may fix
291 # it.
292 maybe_aborted = result.stderr and 'No child processes' in result.stderr
Simon Glass4a1e88b2014-08-09 15:33:00 -0600293
294 if result.already_done:
295 return
296
297 # Write the output and stderr
298 output_dir = self.builder._GetOutputDir(result.commit_upto)
299 Mkdir(output_dir)
300 build_dir = self.builder.GetBuildDir(result.commit_upto,
301 result.brd.target)
302 Mkdir(build_dir)
303
304 outfile = os.path.join(build_dir, 'log')
305 with open(outfile, 'w') as fd:
306 if result.stdout:
Simon Glassc78ed662019-10-31 07:42:53 -0600307 fd.write(result.stdout)
Simon Glass4a1e88b2014-08-09 15:33:00 -0600308
309 errfile = self.builder.GetErrFile(result.commit_upto,
310 result.brd.target)
311 if result.stderr:
312 with open(errfile, 'w') as fd:
Simon Glassc78ed662019-10-31 07:42:53 -0600313 fd.write(result.stderr)
Simon Glass4a1e88b2014-08-09 15:33:00 -0600314 elif os.path.exists(errfile):
315 os.remove(errfile)
316
317 if result.toolchain:
318 # Write the build result and toolchain information.
319 done_file = self.builder.GetDoneFile(result.commit_upto,
320 result.brd.target)
321 with open(done_file, 'w') as fd:
Simon Glassfd3eea12015-02-05 22:06:13 -0700322 if maybe_aborted:
323 # Special code to indicate we need to retry
324 fd.write('%s' % RETURN_CODE_RETRY)
325 else:
326 fd.write('%s' % result.return_code)
Simon Glass4a1e88b2014-08-09 15:33:00 -0600327 with open(os.path.join(build_dir, 'toolchain'), 'w') as fd:
Simon Glassc78ed662019-10-31 07:42:53 -0600328 print('gcc', result.toolchain.gcc, file=fd)
329 print('path', result.toolchain.path, file=fd)
330 print('cross', result.toolchain.cross, file=fd)
331 print('arch', result.toolchain.arch, file=fd)
Simon Glass4a1e88b2014-08-09 15:33:00 -0600332 fd.write('%s' % result.return_code)
333
Simon Glass4a1e88b2014-08-09 15:33:00 -0600334 # Write out the image and function size information and an objdump
Simon Glassd48a46c2014-12-01 17:34:00 -0700335 env = result.toolchain.MakeEnvironment(self.builder.full_path)
Simon Glassff48a212020-04-17 17:51:33 -0600336 with open(os.path.join(build_dir, 'out-env'), 'w') as fd:
Simon Glass9e90d312019-01-07 16:44:23 -0700337 for var in sorted(env.keys()):
Simon Glassc78ed662019-10-31 07:42:53 -0600338 print('%s="%s"' % (var, env[var]), file=fd)
Simon Glass4a1e88b2014-08-09 15:33:00 -0600339 lines = []
340 for fname in ['u-boot', 'spl/u-boot-spl']:
341 cmd = ['%snm' % self.toolchain.cross, '--size-sort', fname]
342 nm_result = command.RunPipe([cmd], capture=True,
343 capture_stderr=True, cwd=result.out_dir,
344 raise_on_error=False, env=env)
345 if nm_result.stdout:
346 nm = self.builder.GetFuncSizesFile(result.commit_upto,
347 result.brd.target, fname)
348 with open(nm, 'w') as fd:
Simon Glassc78ed662019-10-31 07:42:53 -0600349 print(nm_result.stdout, end=' ', file=fd)
Simon Glass4a1e88b2014-08-09 15:33:00 -0600350
351 cmd = ['%sobjdump' % self.toolchain.cross, '-h', fname]
352 dump_result = command.RunPipe([cmd], capture=True,
353 capture_stderr=True, cwd=result.out_dir,
354 raise_on_error=False, env=env)
355 rodata_size = ''
356 if dump_result.stdout:
357 objdump = self.builder.GetObjdumpFile(result.commit_upto,
358 result.brd.target, fname)
359 with open(objdump, 'w') as fd:
Simon Glassc78ed662019-10-31 07:42:53 -0600360 print(dump_result.stdout, end=' ', file=fd)
Simon Glass4a1e88b2014-08-09 15:33:00 -0600361 for line in dump_result.stdout.splitlines():
362 fields = line.split()
363 if len(fields) > 5 and fields[1] == '.rodata':
364 rodata_size = fields[2]
365
366 cmd = ['%ssize' % self.toolchain.cross, fname]
367 size_result = command.RunPipe([cmd], capture=True,
368 capture_stderr=True, cwd=result.out_dir,
369 raise_on_error=False, env=env)
370 if size_result.stdout:
371 lines.append(size_result.stdout.splitlines()[1] + ' ' +
372 rodata_size)
373
Alex Kiernanf07ed232018-05-31 04:48:33 +0000374 # Extract the environment from U-Boot and dump it out
375 cmd = ['%sobjcopy' % self.toolchain.cross, '-O', 'binary',
376 '-j', '.rodata.default_environment',
377 'env/built-in.o', 'uboot.env']
378 command.RunPipe([cmd], capture=True,
379 capture_stderr=True, cwd=result.out_dir,
380 raise_on_error=False, env=env)
381 ubootenv = os.path.join(result.out_dir, 'uboot.env')
382 self.CopyFiles(result.out_dir, build_dir, '', ['uboot.env'])
383
Simon Glass4a1e88b2014-08-09 15:33:00 -0600384 # Write out the image sizes file. This is similar to the output
385 # of binutil's 'size' utility, but it omits the header line and
386 # adds an additional hex value at the end of each line for the
387 # rodata size
388 if len(lines):
389 sizes = self.builder.GetSizesFile(result.commit_upto,
390 result.brd.target)
391 with open(sizes, 'w') as fd:
Simon Glassc78ed662019-10-31 07:42:53 -0600392 print('\n'.join(lines), file=fd)
Simon Glass4a1e88b2014-08-09 15:33:00 -0600393
Simon Glass4b14c532015-02-05 22:06:14 -0700394 # Write out the configuration files, with a special case for SPL
395 for dirname in ['', 'spl', 'tpl']:
396 self.CopyFiles(result.out_dir, build_dir, dirname, ['u-boot.cfg',
397 'spl/u-boot-spl.cfg', 'tpl/u-boot-tpl.cfg', '.config',
398 'include/autoconf.mk', 'include/generated/autoconf.h'])
399
Simon Glass4a1e88b2014-08-09 15:33:00 -0600400 # Now write the actual build output
401 if keep_outputs:
Tom Rini4a42de42015-03-20 10:50:38 -0400402 self.CopyFiles(result.out_dir, build_dir, '', ['u-boot*', '*.bin',
Tom Rini6793aa82015-04-27 11:34:38 -0400403 '*.map', '*.img', 'MLO', 'SPL', 'include/autoconf.mk',
Tom Rini4a42de42015-03-20 10:50:38 -0400404 'spl/u-boot-spl*'])
Simon Glass4b14c532015-02-05 22:06:14 -0700405
406 def CopyFiles(self, out_dir, build_dir, dirname, patterns):
407 """Copy files from the build directory to the output.
Simon Glass4a1e88b2014-08-09 15:33:00 -0600408
Simon Glass4b14c532015-02-05 22:06:14 -0700409 Args:
410 out_dir: Path to output directory containing the files
411 build_dir: Place to copy the files
412 dirname: Source directory, '' for normal U-Boot, 'spl' for SPL
413 patterns: A list of filenames (strings) to copy, each relative
414 to the build directory
415 """
416 for pattern in patterns:
417 file_list = glob.glob(os.path.join(out_dir, dirname, pattern))
418 for fname in file_list:
419 target = os.path.basename(fname)
420 if dirname:
421 base, ext = os.path.splitext(target)
422 if ext:
423 target = '%s-%s%s' % (base, dirname, ext)
424 shutil.copy(fname, os.path.join(build_dir, target))
Simon Glass4a1e88b2014-08-09 15:33:00 -0600425
426 def RunJob(self, job):
427 """Run a single job
428
429 A job consists of a building a list of commits for a particular board.
430
431 Args:
432 job: Job to build
433 """
434 brd = job.board
435 work_dir = self.builder.GetThreadDir(self.thread_num)
436 self.toolchain = None
437 if job.commits:
438 # Run 'make board_defconfig' on the first commit
439 do_config = True
440 commit_upto = 0
441 force_build = False
442 for commit_upto in range(0, len(job.commits), job.step):
443 result, request_config = self.RunCommit(commit_upto, brd,
Simon Glassb55b57d2016-11-16 14:09:25 -0700444 work_dir, do_config, self.builder.config_only,
Simon Glass4a1e88b2014-08-09 15:33:00 -0600445 force_build or self.builder.force_build,
Simon Glassb6eb8cf2020-03-18 09:42:42 -0600446 self.builder.force_build_failures,
447 work_in_output=job.work_in_output)
Simon Glass4a1e88b2014-08-09 15:33:00 -0600448 failed = result.return_code or result.stderr
449 did_config = do_config
450 if failed and not do_config:
451 # If our incremental build failed, try building again
452 # with a reconfig.
453 if self.builder.force_config_on_failure:
454 result, request_config = self.RunCommit(commit_upto,
Simon Glassb6eb8cf2020-03-18 09:42:42 -0600455 brd, work_dir, True, False, True, False,
456 work_in_output=job.work_in_output)
Simon Glass4a1e88b2014-08-09 15:33:00 -0600457 did_config = True
458 if not self.builder.force_reconfig:
459 do_config = request_config
460
461 # If we built that commit, then config is done. But if we got
462 # an warning, reconfig next time to force it to build the same
463 # files that created warnings this time. Otherwise an
464 # incremental build may not build the same file, and we will
465 # think that the warning has gone away.
466 # We could avoid this by using -Werror everywhere...
467 # For errors, the problem doesn't happen, since presumably
468 # the build stopped and didn't generate output, so will retry
469 # that file next time. So we could detect warnings and deal
470 # with them specially here. For now, we just reconfigure if
471 # anything goes work.
472 # Of course this is substantially slower if there are build
473 # errors/warnings (e.g. 2-3x slower even if only 10% of builds
474 # have problems).
475 if (failed and not result.already_done and not did_config and
476 self.builder.force_config_on_failure):
477 # If this build failed, try the next one with a
478 # reconfigure.
479 # Sometimes if the board_config.h file changes it can mess
480 # with dependencies, and we get:
481 # make: *** No rule to make target `include/autoconf.mk',
482 # needed by `depend'.
483 do_config = True
484 force_build = True
485 else:
486 force_build = False
487 if self.builder.force_config_on_failure:
488 if failed:
489 do_config = True
490 result.commit_upto = commit_upto
491 if result.return_code < 0:
492 raise ValueError('Interrupt')
493
494 # We have the build results, so output the result
Simon Glassb6eb8cf2020-03-18 09:42:42 -0600495 self._WriteResult(result, job.keep_outputs, job.work_in_output)
Simon Glass4a1e88b2014-08-09 15:33:00 -0600496 self.builder.out_queue.put(result)
497 else:
498 # Just build the currently checked-out build
499 result, request_config = self.RunCommit(None, brd, work_dir, True,
Simon Glassb55b57d2016-11-16 14:09:25 -0700500 self.builder.config_only, True,
Simon Glassb6eb8cf2020-03-18 09:42:42 -0600501 self.builder.force_build_failures,
502 work_in_output=job.work_in_output)
Simon Glass4a1e88b2014-08-09 15:33:00 -0600503 result.commit_upto = 0
Simon Glassb6eb8cf2020-03-18 09:42:42 -0600504 self._WriteResult(result, job.keep_outputs, job.work_in_output)
Simon Glass4a1e88b2014-08-09 15:33:00 -0600505 self.builder.out_queue.put(result)
506
507 def run(self):
508 """Our thread's run function
509
510 This thread picks a job from the queue, runs it, and then goes to the
511 next job.
512 """
Simon Glass4a1e88b2014-08-09 15:33:00 -0600513 while True:
514 job = self.builder.queue.get()
Simon Glass52a9ce92016-09-18 16:48:38 -0600515 self.RunJob(job)
Simon Glass4a1e88b2014-08-09 15:33:00 -0600516 self.builder.queue.task_done()