blob: 48fcd6cf7e2477fceba39086bb597bcfc36be60a [file] [log] [blame]
Tom Rini83d290c2018-05-06 17:58:06 -04001# SPDX-License-Identifier: GPL-2.0+
Simon Glass190064b2014-08-09 15:33:00 -06002# Copyright (c) 2014 Google, Inc
3#
Simon Glass190064b2014-08-09 15:33:00 -06004
5import errno
6import glob
7import os
8import shutil
Lothar Waßmann409fc022018-04-08 05:14:11 -06009import sys
Simon Glass190064b2014-08-09 15:33:00 -060010import threading
11
Simon Glassbf776672020-04-17 18:09:04 -060012from patman import command
13from patman import gitutil
Simon Glass190064b2014-08-09 15:33:00 -060014
Simon Glass88c8dcf2015-02-05 22:06:13 -070015RETURN_CODE_RETRY = -1
16
Thierry Redingf3d015c2014-08-19 10:22:39 +020017def Mkdir(dirname, parents = False):
Simon Glass190064b2014-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 Redingf3d015c2014-08-19 10:22:39 +020024 if parents:
25 os.makedirs(dirname)
26 else:
27 os.mkdir(dirname)
Simon Glass190064b2014-08-09 15:33:00 -060028 except OSError as err:
29 if err.errno == errno.EEXIST:
Lothar Waßmann409fc022018-04-08 05:14:11 -060030 if os.path.realpath('.') == os.path.realpath(dirname):
Simon Glassc05aa032019-10-31 07:42:53 -060031 print("Cannot create the current working directory '%s'!" % dirname)
Lothar Waßmann409fc022018-04-08 05:14:11 -060032 sys.exit(1)
Simon Glass190064b2014-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 Glasse9fbbf62020-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 Glassd829f122020-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 Glass190064b2014-08-09 15:33:00 -060047 """
48 def __init__(self):
49 self.board = None
50 self.commits = []
Simon Glasse9fbbf62020-03-18 09:42:41 -060051 self.keep_outputs = False
52 self.step = 1
Simon Glassd829f122020-03-18 09:42:42 -060053 self.work_in_output = False
Simon Glass190064b2014-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 Glasseb70a2c2020-04-09 15:08:51 -060093 def __init__(self, builder, thread_num, mrproper, per_board_out_dir):
Simon Glass190064b2014-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 Glasseb70a2c2020-04-09 15:08:51 -060098 self.mrproper = mrproper
Stephen Warrenf79f1e02016-04-11 10:48:44 -060099 self.per_board_out_dir = per_board_out_dir
Simon Glass190064b2014-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 Meierfd18a892014-08-20 22:10:29 +0200111 mrproper - can be called to clean source
Simon Glass190064b2014-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 Glassa9401b22016-11-16 14:09:25 -0700123 def RunCommit(self, commit_upto, brd, work_dir, do_config, config_only,
Simon Glassd829f122020-03-18 09:42:42 -0600124 force_build, force_build_failures, work_in_output):
Simon Glass190064b2014-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 Glassa9401b22016-11-16 14:09:25 -0700135 config_only: Only configure the source, do not build it
Simon Glass190064b2014-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 Glassd829f122020-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 Glass190064b2014-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 Glassd829f122020-03-18 09:42:42 -0600151 if work_in_output or self.builder.in_tree:
Simon Glass190064b2014-08-09 15:33:00 -0600152 out_dir = work_dir
153 else:
Stephen Warrenf79f1e02016-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 Glass190064b2014-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 Glassfb3954f2014-09-05 19:00:17 -0600165 if result.already_done:
Simon Glass190064b2014-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 Glasse74429b2018-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 Glass88c8dcf2015-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 Glassfb3954f2014-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 Glass190064b2014-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 Glassbb1501f2014-12-01 17:34:00 -0700210 env = self.toolchain.MakeEnvironment(self.builder.full_path)
Simon Glass190064b2014-08-09 15:33:00 -0600211 Mkdir(out_dir)
212 args = []
213 cwd = work_dir
Simon Glass48c1b6a2014-08-28 09:43:42 -0600214 src_dir = os.path.realpath(work_dir)
Simon Glass190064b2014-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 Warrenf79f1e02016-04-11 10:48:44 -0600224 out_dir = os.path.realpath(out_dir)
225 args.append('O=%s' % out_dir)
Simon Glass190064b2014-08-09 15:33:00 -0600226 cwd = None
Simon Glass48c1b6a2014-08-28 09:43:42 -0600227 src_dir = os.getcwd()
Simon Glass190064b2014-08-09 15:33:00 -0600228 else:
Stephen Warrenf79f1e02016-04-11 10:48:44 -0600229 args.append('O=%s' % out_rel_dir)
Tom Rinif5e5ece2015-04-01 07:47:41 -0400230 if self.builder.verbose_build:
231 args.append('V=1')
232 else:
Simon Glassd2ce6582014-12-01 17:34:07 -0700233 args.append('-s')
Simon Glass190064b2014-08-09 15:33:00 -0600234 if self.builder.num_jobs is not None:
235 args.extend(['-j', str(self.builder.num_jobs)])
Daniel Schwierzeck2371d1b2018-01-26 16:31:05 +0100236 if self.builder.warnings_as_errors:
237 args.append('KCFLAGS=-Werror')
Simon Glass190064b2014-08-09 15:33:00 -0600238 config_args = ['%s_defconfig' % brd.target]
239 config_out = ''
240 args.extend(self.builder.toolchains.GetMakeArguments(brd))
Simon Glass00beb242019-01-07 16:44:20 -0700241 args.extend(self.toolchain.MakeArgs())
Simon Glass190064b2014-08-09 15:33:00 -0600242
243 # If we need to reconfigure, do that now
244 if do_config:
Stephen Warrenf79f1e02016-04-11 10:48:44 -0600245 config_out = ''
Simon Glasseb70a2c2020-04-09 15:08:51 -0600246 if self.mrproper:
Stephen Warrenf79f1e02016-04-11 10:48:44 -0600247 result = self.Make(commit, brd, 'mrproper', cwd,
248 'mrproper', *args, env=env)
249 config_out += result.combined
Simon Glass190064b2014-08-09 15:33:00 -0600250 result = self.Make(commit, brd, 'config', cwd,
251 *(args + config_args), env=env)
Simon Glass40f11fc2015-02-05 22:06:12 -0700252 config_out += result.combined
Simon Glass190064b2014-08-09 15:33:00 -0600253 do_config = False # No need to configure next time
254 if result.return_code == 0:
Simon Glassa9401b22016-11-16 14:09:25 -0700255 if config_only:
Simon Glassb50113f2016-11-13 14:25:51 -0700256 args.append('cfg')
Simon Glass190064b2014-08-09 15:33:00 -0600257 result = self.Make(commit, brd, 'build', cwd, *args,
258 env=env)
Simon Glass48c1b6a2014-08-28 09:43:42 -0600259 result.stderr = result.stderr.replace(src_dir + '/', '')
Simon Glass40f11fc2015-02-05 22:06:12 -0700260 if self.builder.verbose_build:
261 result.stdout = config_out + result.stdout
Simon Glass190064b2014-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 Glassd829f122020-03-18 09:42:42 -0600273 def _WriteResult(self, result, keep_outputs, work_in_output):
Simon Glass190064b2014-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 Glassd829f122020-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 Glass190064b2014-08-09 15:33:00 -0600282 """
283 # Fatal error
284 if result.return_code < 0:
285 return
286
Simon Glass88c8dcf2015-02-05 22:06:13 -0700287 # If we think this might have been aborted with Ctrl-C, record the
288 # failure but not that we are 'done' with this board. A retry may fix
289 # it.
290 maybe_aborted = result.stderr and 'No child processes' in result.stderr
Simon Glass190064b2014-08-09 15:33:00 -0600291
292 if result.already_done:
293 return
294
295 # Write the output and stderr
296 output_dir = self.builder._GetOutputDir(result.commit_upto)
297 Mkdir(output_dir)
298 build_dir = self.builder.GetBuildDir(result.commit_upto,
299 result.brd.target)
300 Mkdir(build_dir)
301
302 outfile = os.path.join(build_dir, 'log')
303 with open(outfile, 'w') as fd:
304 if result.stdout:
Simon Glassc05aa032019-10-31 07:42:53 -0600305 fd.write(result.stdout)
Simon Glass190064b2014-08-09 15:33:00 -0600306
307 errfile = self.builder.GetErrFile(result.commit_upto,
308 result.brd.target)
309 if result.stderr:
310 with open(errfile, 'w') as fd:
Simon Glassc05aa032019-10-31 07:42:53 -0600311 fd.write(result.stderr)
Simon Glass190064b2014-08-09 15:33:00 -0600312 elif os.path.exists(errfile):
313 os.remove(errfile)
314
315 if result.toolchain:
316 # Write the build result and toolchain information.
317 done_file = self.builder.GetDoneFile(result.commit_upto,
318 result.brd.target)
319 with open(done_file, 'w') as fd:
Simon Glass88c8dcf2015-02-05 22:06:13 -0700320 if maybe_aborted:
321 # Special code to indicate we need to retry
322 fd.write('%s' % RETURN_CODE_RETRY)
323 else:
324 fd.write('%s' % result.return_code)
Simon Glass190064b2014-08-09 15:33:00 -0600325 with open(os.path.join(build_dir, 'toolchain'), 'w') as fd:
Simon Glassc05aa032019-10-31 07:42:53 -0600326 print('gcc', result.toolchain.gcc, file=fd)
327 print('path', result.toolchain.path, file=fd)
328 print('cross', result.toolchain.cross, file=fd)
329 print('arch', result.toolchain.arch, file=fd)
Simon Glass190064b2014-08-09 15:33:00 -0600330 fd.write('%s' % result.return_code)
331
Simon Glass190064b2014-08-09 15:33:00 -0600332 # Write out the image and function size information and an objdump
Simon Glassbb1501f2014-12-01 17:34:00 -0700333 env = result.toolchain.MakeEnvironment(self.builder.full_path)
Simon Glass166a98a2020-04-17 17:51:33 -0600334 with open(os.path.join(build_dir, 'out-env'), 'w') as fd:
Simon Glasse5fc79e2019-01-07 16:44:23 -0700335 for var in sorted(env.keys()):
Simon Glassc05aa032019-10-31 07:42:53 -0600336 print('%s="%s"' % (var, env[var]), file=fd)
Simon Glass190064b2014-08-09 15:33:00 -0600337 lines = []
338 for fname in ['u-boot', 'spl/u-boot-spl']:
339 cmd = ['%snm' % self.toolchain.cross, '--size-sort', fname]
340 nm_result = command.RunPipe([cmd], capture=True,
341 capture_stderr=True, cwd=result.out_dir,
342 raise_on_error=False, env=env)
343 if nm_result.stdout:
344 nm = self.builder.GetFuncSizesFile(result.commit_upto,
345 result.brd.target, fname)
346 with open(nm, 'w') as fd:
Simon Glassc05aa032019-10-31 07:42:53 -0600347 print(nm_result.stdout, end=' ', file=fd)
Simon Glass190064b2014-08-09 15:33:00 -0600348
349 cmd = ['%sobjdump' % self.toolchain.cross, '-h', fname]
350 dump_result = command.RunPipe([cmd], capture=True,
351 capture_stderr=True, cwd=result.out_dir,
352 raise_on_error=False, env=env)
353 rodata_size = ''
354 if dump_result.stdout:
355 objdump = self.builder.GetObjdumpFile(result.commit_upto,
356 result.brd.target, fname)
357 with open(objdump, 'w') as fd:
Simon Glassc05aa032019-10-31 07:42:53 -0600358 print(dump_result.stdout, end=' ', file=fd)
Simon Glass190064b2014-08-09 15:33:00 -0600359 for line in dump_result.stdout.splitlines():
360 fields = line.split()
361 if len(fields) > 5 and fields[1] == '.rodata':
362 rodata_size = fields[2]
363
364 cmd = ['%ssize' % self.toolchain.cross, fname]
365 size_result = command.RunPipe([cmd], capture=True,
366 capture_stderr=True, cwd=result.out_dir,
367 raise_on_error=False, env=env)
368 if size_result.stdout:
369 lines.append(size_result.stdout.splitlines()[1] + ' ' +
370 rodata_size)
371
Alex Kiernan0ddc5102018-05-31 04:48:33 +0000372 # Extract the environment from U-Boot and dump it out
373 cmd = ['%sobjcopy' % self.toolchain.cross, '-O', 'binary',
374 '-j', '.rodata.default_environment',
375 'env/built-in.o', 'uboot.env']
376 command.RunPipe([cmd], capture=True,
377 capture_stderr=True, cwd=result.out_dir,
378 raise_on_error=False, env=env)
379 ubootenv = os.path.join(result.out_dir, 'uboot.env')
Simon Glass60b285f2020-04-17 17:51:34 -0600380 if not work_in_output:
381 self.CopyFiles(result.out_dir, build_dir, '', ['uboot.env'])
Alex Kiernan0ddc5102018-05-31 04:48:33 +0000382
Simon Glass190064b2014-08-09 15:33:00 -0600383 # Write out the image sizes file. This is similar to the output
384 # of binutil's 'size' utility, but it omits the header line and
385 # adds an additional hex value at the end of each line for the
386 # rodata size
387 if len(lines):
388 sizes = self.builder.GetSizesFile(result.commit_upto,
389 result.brd.target)
390 with open(sizes, 'w') as fd:
Simon Glassc05aa032019-10-31 07:42:53 -0600391 print('\n'.join(lines), file=fd)
Simon Glass190064b2014-08-09 15:33:00 -0600392
Simon Glass60b285f2020-04-17 17:51:34 -0600393 if not work_in_output:
394 # Write out the configuration files, with a special case for SPL
395 for dirname in ['', 'spl', 'tpl']:
396 self.CopyFiles(
397 result.out_dir, build_dir, dirname,
398 ['u-boot.cfg', 'spl/u-boot-spl.cfg', 'tpl/u-boot-tpl.cfg',
399 '.config', 'include/autoconf.mk',
400 'include/generated/autoconf.h'])
Simon Glass970f9322015-02-05 22:06:14 -0700401
Simon Glass60b285f2020-04-17 17:51:34 -0600402 # Now write the actual build output
403 if keep_outputs:
404 self.CopyFiles(
405 result.out_dir, build_dir, '',
406 ['u-boot*', '*.bin', '*.map', '*.img', 'MLO', 'SPL',
407 'include/autoconf.mk', 'spl/u-boot-spl*'])
Simon Glass190064b2014-08-09 15:33:00 -0600408
Simon Glass970f9322015-02-05 22:06:14 -0700409 def CopyFiles(self, out_dir, build_dir, dirname, patterns):
410 """Copy files from the build directory to the output.
411
412 Args:
413 out_dir: Path to output directory containing the files
414 build_dir: Place to copy the files
415 dirname: Source directory, '' for normal U-Boot, 'spl' for SPL
416 patterns: A list of filenames (strings) to copy, each relative
417 to the build directory
418 """
419 for pattern in patterns:
420 file_list = glob.glob(os.path.join(out_dir, dirname, pattern))
421 for fname in file_list:
422 target = os.path.basename(fname)
423 if dirname:
424 base, ext = os.path.splitext(target)
425 if ext:
426 target = '%s-%s%s' % (base, dirname, ext)
427 shutil.copy(fname, os.path.join(build_dir, target))
Simon Glass190064b2014-08-09 15:33:00 -0600428
429 def RunJob(self, job):
430 """Run a single job
431
432 A job consists of a building a list of commits for a particular board.
433
434 Args:
435 job: Job to build
436 """
437 brd = job.board
438 work_dir = self.builder.GetThreadDir(self.thread_num)
439 self.toolchain = None
440 if job.commits:
441 # Run 'make board_defconfig' on the first commit
442 do_config = True
443 commit_upto = 0
444 force_build = False
445 for commit_upto in range(0, len(job.commits), job.step):
446 result, request_config = self.RunCommit(commit_upto, brd,
Simon Glassa9401b22016-11-16 14:09:25 -0700447 work_dir, do_config, self.builder.config_only,
Simon Glass190064b2014-08-09 15:33:00 -0600448 force_build or self.builder.force_build,
Simon Glassd829f122020-03-18 09:42:42 -0600449 self.builder.force_build_failures,
450 work_in_output=job.work_in_output)
Simon Glass190064b2014-08-09 15:33:00 -0600451 failed = result.return_code or result.stderr
452 did_config = do_config
453 if failed and not do_config:
454 # If our incremental build failed, try building again
455 # with a reconfig.
456 if self.builder.force_config_on_failure:
457 result, request_config = self.RunCommit(commit_upto,
Simon Glassd829f122020-03-18 09:42:42 -0600458 brd, work_dir, True, False, True, False,
459 work_in_output=job.work_in_output)
Simon Glass190064b2014-08-09 15:33:00 -0600460 did_config = True
461 if not self.builder.force_reconfig:
462 do_config = request_config
463
464 # If we built that commit, then config is done. But if we got
465 # an warning, reconfig next time to force it to build the same
466 # files that created warnings this time. Otherwise an
467 # incremental build may not build the same file, and we will
468 # think that the warning has gone away.
469 # We could avoid this by using -Werror everywhere...
470 # For errors, the problem doesn't happen, since presumably
471 # the build stopped and didn't generate output, so will retry
472 # that file next time. So we could detect warnings and deal
473 # with them specially here. For now, we just reconfigure if
474 # anything goes work.
475 # Of course this is substantially slower if there are build
476 # errors/warnings (e.g. 2-3x slower even if only 10% of builds
477 # have problems).
478 if (failed and not result.already_done and not did_config and
479 self.builder.force_config_on_failure):
480 # If this build failed, try the next one with a
481 # reconfigure.
482 # Sometimes if the board_config.h file changes it can mess
483 # with dependencies, and we get:
484 # make: *** No rule to make target `include/autoconf.mk',
485 # needed by `depend'.
486 do_config = True
487 force_build = True
488 else:
489 force_build = False
490 if self.builder.force_config_on_failure:
491 if failed:
492 do_config = True
493 result.commit_upto = commit_upto
494 if result.return_code < 0:
495 raise ValueError('Interrupt')
496
497 # We have the build results, so output the result
Simon Glassd829f122020-03-18 09:42:42 -0600498 self._WriteResult(result, job.keep_outputs, job.work_in_output)
Simon Glass190064b2014-08-09 15:33:00 -0600499 self.builder.out_queue.put(result)
500 else:
501 # Just build the currently checked-out build
502 result, request_config = self.RunCommit(None, brd, work_dir, True,
Simon Glassa9401b22016-11-16 14:09:25 -0700503 self.builder.config_only, True,
Simon Glassd829f122020-03-18 09:42:42 -0600504 self.builder.force_build_failures,
505 work_in_output=job.work_in_output)
Simon Glass190064b2014-08-09 15:33:00 -0600506 result.commit_upto = 0
Simon Glassd829f122020-03-18 09:42:42 -0600507 self._WriteResult(result, job.keep_outputs, job.work_in_output)
Simon Glass190064b2014-08-09 15:33:00 -0600508 self.builder.out_queue.put(result)
509
510 def run(self):
511 """Our thread's run function
512
513 This thread picks a job from the queue, runs it, and then goes to the
514 next job.
515 """
Simon Glass190064b2014-08-09 15:33:00 -0600516 while True:
517 job = self.builder.queue.get()
Simon Glass2880e6b2016-09-18 16:48:38 -0600518 self.RunJob(job)
Simon Glass190064b2014-08-09 15:33:00 -0600519 self.builder.queue.task_done()