blob: d20508f089975237ee54a403d93daa5ea8a07f83 [file] [log] [blame]
Simon Glassc05694f2013-04-03 11:07:16 +00001# Copyright (c) 2013 The Chromium OS Authors.
2#
Wolfgang Denkd79de1d2013-07-08 09:37:19 +02003# SPDX-License-Identifier: GPL-2.0+
Simon Glassc05694f2013-04-03 11:07:16 +00004#
5
Simon Glass3ec03d42014-08-09 15:32:58 -06006(Please read 'How to change from MAKEALL' if you are used to that tool)
7
Simon Glassc05694f2013-04-03 11:07:16 +00008What is this?
9=============
10
11This tool handles building U-Boot to check that you have not broken it
12with your patch series. It can build each individual commit and report
13which boards fail on which commits, and which errors come up. It aims
14to make full use of multi-processor machines.
15
16A key feature of buildman is its output summary, which allows warnings,
17errors or image size increases in a particular commit or board to be
18quickly identified and the offending commit pinpointed. This can be a big
19help for anyone working with >10 patches at a time.
20
21
22Caveats
23=======
24
25Buildman is still in its infancy. It is already a very useful tool, but
26expect to find problems and send patches.
27
28Buildman can be stopped and restarted, in which case it will continue
29where it left off. This should happen cleanly and without side-effects.
30If not, it is a bug, for which a patch would be welcome.
31
32Buildman gets so tied up in its work that it can ignore the outside world.
33You may need to press Ctrl-C several times to quit it. Also it will print
34out various exceptions when stopped.
35
36
37Theory of Operation
38===================
39
40(please read this section in full twice or you will be perpetually confused)
41
42Buildman is a builder. It is not make, although it runs make. It does not
43produce any useful output on the terminal while building, except for
Simon Glass78e418e2014-08-09 15:33:03 -060044progress information (except with -v, see below). All the output (errors,
45warnings and binaries if you are ask for them) is stored in output
46directories, which you can look at while the build is progressing, or when
47it is finished.
Simon Glassc05694f2013-04-03 11:07:16 +000048
49Buildman produces a concise summary of which boards succeeded and failed.
50It shows which commit introduced which board failure using a simple
51red/green colour coding. Full error information can be requested, in which
52case it is de-duped and displayed against the commit that introduced the
53error. An example workflow is below.
54
55Buildman stores image size information and can report changes in image size
56from commit to commit. An example of this is below.
57
58Buildman starts multiple threads, and each thread builds for one board at
59a time. A thread starts at the first commit, configures the source for your
60board and builds it. Then it checks out the next commit and does an
61incremental build. Eventually the thread reaches the last commit and stops.
62If errors or warnings are found along the way, the thread will reconfigure
63after every commit, and your build will be very slow. This is because a
64file that produces just a warning would not normally be rebuilt in an
65incremental build.
66
67Buildman works in an entirely separate place from your U-Boot repository.
68It creates a separate working directory for each thread, and puts the
69output files in the working directory, organised by commit name and board
70name, in a two-level hierarchy.
71
72Buildman is invoked in your U-Boot directory, the one with the .git
73directory. It clones this repository into a copy for each thread, and the
74threads do not affect the state of your git repository. Any checkouts done
75by the thread affect only the working directory for that thread.
76
Simon Glassfa1cfee2014-08-09 15:32:57 -060077Buildman automatically selects the correct tool chain for each board. You
78must supply suitable tool chains, but buildman takes care of selecting the
Simon Glassc05694f2013-04-03 11:07:16 +000079right one.
80
Simon Glass78e418e2014-08-09 15:33:03 -060081Buildman generally builds a branch (with the -b flag), and in this case
82builds the upstream commit as well, for comparison. It cannot build
83individual commits at present, unless (maybe) you point it at an empty
84branch. Put all your commits in a branch, set the branch's upstream to a
85valid value, and all will be well. Otherwise buildman will perform random
86actions. Use -n to check what the random actions might be.
87
88If you just want to build the current source tree, leave off the -b flag.
89This will display results and errors as they happen. You can still look
90at them later using -s. Note that buildman will assume that the source
91has changed, and will build all specified boards in this case.
Simon Glassc05694f2013-04-03 11:07:16 +000092
93Buildman is optimised for building many commits at once, for many boards.
94On multi-core machines, Buildman is fast because it uses most of the
95available CPU power. When it gets to the end, or if you are building just
96a few commits or boards, it will be pretty slow. As a tip, if you don't
97plan to use your machine for anything else, you can use -T to increase the
98number of threads beyond the default.
99
Stephen Warren08447632013-10-10 10:00:20 -0600100Buildman lets you build all boards, or a subset. Specify the subset by passing
101command-line arguments that list the desired board name, architecture name,
102SOC name, or anything else in the boards.cfg file. Multiple arguments are
103allowed. Each argument will be interpreted as a regular expression, so
104behaviour is a superset of exact or substring matching. Examples are:
105
106* 'tegra20' All boards with a Tegra20 SoC
107* 'tegra' All boards with any Tegra Soc (Tegra20, Tegra30, Tegra114...)
108* '^tegra[23]0$' All boards with either Tegra20 or Tegra30 SoC
109* 'powerpc' All PowerPC boards
Simon Glassc05694f2013-04-03 11:07:16 +0000110
Simon Glassaa40f9a2014-08-09 15:33:08 -0600111While the default is to OR the terms together, you can also make use of
112the '&' operator to limit the selection:
113
114* 'freescale & arm sandbox' All Freescale boards with ARM architecture,
115 plus sandbox
116
117It is convenient to use the -n option to see whaat will be built based on
118the subset given.
119
Simon Glassc05694f2013-04-03 11:07:16 +0000120Buildman does not store intermediate object files. It optionally copies
121the binary output into a directory when a build is successful. Size
122information is always recorded. It needs a fair bit of disk space to work,
123typically 250MB per thread.
124
125
126Setting up
127==========
128
1291. Get the U-Boot source. You probably already have it, but if not these
130steps should get you started with a repo and some commits for testing.
131
132$ cd /path/to/u-boot
133$ git clone git://git.denx.de/u-boot.git .
134$ git checkout -b my-branch origin/master
135$ # Add some commits to the branch, reading for testing
136
1372. Create ~/.buildman to tell buildman where to find tool chains. As an
138example:
139
140# Buildman settings file
141
142[toolchain]
143root: /
144rest: /toolchains/*
145eldk: /opt/eldk-4.2
Simon Glass9a016392014-08-09 15:33:07 -0600146arm: /opt/linaro/gcc-linaro-arm-linux-gnueabihf-4.8-2013.08_linux
147aarch64: /opt/linaro/gcc-linaro-aarch64-none-elf-4.8-2013.10_linux
Simon Glassc05694f2013-04-03 11:07:16 +0000148
149[toolchain-alias]
150x86: i386
151blackfin: bfin
152sh: sh4
153nds32: nds32le
154openrisc: or32
155
156
157This selects the available toolchain paths. Add the base directory for
158each of your toolchains here. Buildman will search inside these directories
159and also in any '/usr' and '/usr/bin' subdirectories.
160
161Make sure the tags (here root: rest: and eldk:) are unique.
162
163The toolchain-alias section indicates that the i386 toolchain should be used
164to build x86 commits.
165
166
1672. Check the available toolchains
168
169Run this check to make sure that you have a toolchain for every architecture.
170
171$ ./tools/buildman/buildman --list-tool-chains
172Scanning for tool chains
173 - scanning path '/'
174 - looking in '/.'
175 - looking in '/bin'
176 - looking in '/usr/bin'
177 - found '/usr/bin/gcc'
178Tool chain test: OK
179 - found '/usr/bin/c89-gcc'
180Tool chain test: OK
181 - found '/usr/bin/c99-gcc'
182Tool chain test: OK
183 - found '/usr/bin/x86_64-linux-gnu-gcc'
184Tool chain test: OK
185 - scanning path '/toolchains/powerpc-linux'
186 - looking in '/toolchains/powerpc-linux/.'
187 - looking in '/toolchains/powerpc-linux/bin'
188 - found '/toolchains/powerpc-linux/bin/powerpc-linux-gcc'
189Tool chain test: OK
190 - looking in '/toolchains/powerpc-linux/usr/bin'
191 - scanning path '/toolchains/nds32le-linux-glibc-v1f'
192 - looking in '/toolchains/nds32le-linux-glibc-v1f/.'
193 - looking in '/toolchains/nds32le-linux-glibc-v1f/bin'
194 - found '/toolchains/nds32le-linux-glibc-v1f/bin/nds32le-linux-gcc'
195Tool chain test: OK
196 - looking in '/toolchains/nds32le-linux-glibc-v1f/usr/bin'
197 - scanning path '/toolchains/nios2'
198 - looking in '/toolchains/nios2/.'
199 - looking in '/toolchains/nios2/bin'
200 - found '/toolchains/nios2/bin/nios2-linux-gcc'
201Tool chain test: OK
202 - found '/toolchains/nios2/bin/nios2-linux-uclibc-gcc'
203Tool chain test: OK
204 - looking in '/toolchains/nios2/usr/bin'
205 - found '/toolchains/nios2/usr/bin/nios2-linux-gcc'
206Tool chain test: OK
207 - found '/toolchains/nios2/usr/bin/nios2-linux-uclibc-gcc'
208Tool chain test: OK
209 - scanning path '/toolchains/microblaze-unknown-linux-gnu'
210 - looking in '/toolchains/microblaze-unknown-linux-gnu/.'
211 - looking in '/toolchains/microblaze-unknown-linux-gnu/bin'
212 - found '/toolchains/microblaze-unknown-linux-gnu/bin/microblaze-unknown-linux-gnu-gcc'
213Tool chain test: OK
214 - found '/toolchains/microblaze-unknown-linux-gnu/bin/mb-linux-gcc'
215Tool chain test: OK
216 - looking in '/toolchains/microblaze-unknown-linux-gnu/usr/bin'
217 - scanning path '/toolchains/mips-linux'
218 - looking in '/toolchains/mips-linux/.'
219 - looking in '/toolchains/mips-linux/bin'
220 - found '/toolchains/mips-linux/bin/mips-linux-gcc'
221Tool chain test: OK
222 - looking in '/toolchains/mips-linux/usr/bin'
223 - scanning path '/toolchains/old'
224 - looking in '/toolchains/old/.'
225 - looking in '/toolchains/old/bin'
226 - looking in '/toolchains/old/usr/bin'
227 - scanning path '/toolchains/i386-linux'
228 - looking in '/toolchains/i386-linux/.'
229 - looking in '/toolchains/i386-linux/bin'
230 - found '/toolchains/i386-linux/bin/i386-linux-gcc'
231Tool chain test: OK
232 - looking in '/toolchains/i386-linux/usr/bin'
233 - scanning path '/toolchains/bfin-uclinux'
234 - looking in '/toolchains/bfin-uclinux/.'
235 - looking in '/toolchains/bfin-uclinux/bin'
236 - found '/toolchains/bfin-uclinux/bin/bfin-uclinux-gcc'
237Tool chain test: OK
238 - looking in '/toolchains/bfin-uclinux/usr/bin'
239 - scanning path '/toolchains/sparc-elf'
240 - looking in '/toolchains/sparc-elf/.'
241 - looking in '/toolchains/sparc-elf/bin'
242 - found '/toolchains/sparc-elf/bin/sparc-elf-gcc'
243Tool chain test: OK
244 - looking in '/toolchains/sparc-elf/usr/bin'
245 - scanning path '/toolchains/arm-2010q1'
246 - looking in '/toolchains/arm-2010q1/.'
247 - looking in '/toolchains/arm-2010q1/bin'
248 - found '/toolchains/arm-2010q1/bin/arm-none-linux-gnueabi-gcc'
249Tool chain test: OK
250 - looking in '/toolchains/arm-2010q1/usr/bin'
251 - scanning path '/toolchains/from'
252 - looking in '/toolchains/from/.'
253 - looking in '/toolchains/from/bin'
254 - looking in '/toolchains/from/usr/bin'
255 - scanning path '/toolchains/sh4-gentoo-linux-gnu'
256 - looking in '/toolchains/sh4-gentoo-linux-gnu/.'
257 - looking in '/toolchains/sh4-gentoo-linux-gnu/bin'
258 - found '/toolchains/sh4-gentoo-linux-gnu/bin/sh4-gentoo-linux-gnu-gcc'
259Tool chain test: OK
260 - looking in '/toolchains/sh4-gentoo-linux-gnu/usr/bin'
261 - scanning path '/toolchains/avr32-linux'
262 - looking in '/toolchains/avr32-linux/.'
263 - looking in '/toolchains/avr32-linux/bin'
264 - found '/toolchains/avr32-linux/bin/avr32-gcc'
265Tool chain test: OK
266 - looking in '/toolchains/avr32-linux/usr/bin'
267 - scanning path '/toolchains/m68k-linux'
268 - looking in '/toolchains/m68k-linux/.'
269 - looking in '/toolchains/m68k-linux/bin'
270 - found '/toolchains/m68k-linux/bin/m68k-linux-gcc'
271Tool chain test: OK
272 - looking in '/toolchains/m68k-linux/usr/bin'
273List of available toolchains (17):
274arm : /toolchains/arm-2010q1/bin/arm-none-linux-gnueabi-gcc
275avr32 : /toolchains/avr32-linux/bin/avr32-gcc
276bfin : /toolchains/bfin-uclinux/bin/bfin-uclinux-gcc
277c89 : /usr/bin/c89-gcc
278c99 : /usr/bin/c99-gcc
279i386 : /toolchains/i386-linux/bin/i386-linux-gcc
280m68k : /toolchains/m68k-linux/bin/m68k-linux-gcc
281mb : /toolchains/microblaze-unknown-linux-gnu/bin/mb-linux-gcc
282microblaze: /toolchains/microblaze-unknown-linux-gnu/bin/microblaze-unknown-linux-gnu-gcc
283mips : /toolchains/mips-linux/bin/mips-linux-gcc
284nds32le : /toolchains/nds32le-linux-glibc-v1f/bin/nds32le-linux-gcc
285nios2 : /toolchains/nios2/bin/nios2-linux-gcc
286powerpc : /toolchains/powerpc-linux/bin/powerpc-linux-gcc
287sandbox : /usr/bin/gcc
288sh4 : /toolchains/sh4-gentoo-linux-gnu/bin/sh4-gentoo-linux-gnu-gcc
289sparc : /toolchains/sparc-elf/bin/sparc-elf-gcc
290x86_64 : /usr/bin/x86_64-linux-gnu-gcc
291
292
293You can see that everything is covered, even some strange ones that won't
294be used (c88 and c99). This is a feature.
295
296
297How to run it
298=============
299
300First do a dry run using the -n flag: (replace <branch> with a real, local
301branch with a valid upstream)
302
303$ ./tools/buildman/buildman -b <branch> -n
304
305If it can't detect the upstream branch, try checking out the branch, and
306doing something like 'git branch --set-upstream <branch> upstream/master'
307or something similar.
308
Simon Glassfa1cfee2014-08-09 15:32:57 -0600309As an example:
Simon Glassc05694f2013-04-03 11:07:16 +0000310
311Dry run, so not doing much. But I would do this:
312
313Building 18 commits for 1059 boards (4 threads, 1 job per thread)
314Build directory: ../lcd9b
315 5bb3505 Merge branch 'master' of git://git.denx.de/u-boot-arm
316 c18f1b4 tegra: Use const for pinmux_config_pingroup/table()
317 2f043ae tegra: Add display support to funcmux
318 e349900 tegra: fdt: Add pwm binding and node
319 424a5f0 tegra: fdt: Add LCD definitions for Tegra
320 0636ccf tegra: Add support for PWM
321 a994fe7 tegra: Add SOC support for display/lcd
322 fcd7350 tegra: Add LCD driver
323 4d46e9d tegra: Add LCD support to Nvidia boards
324 991bd48 arm: Add control over cachability of memory regions
325 54e8019 lcd: Add CONFIG_LCD_ALIGNMENT to select frame buffer alignment
326 d92aff7 lcd: Add support for flushing LCD fb from dcache after update
327 dbd0677 tegra: Align LCD frame buffer to section boundary
328 0cff9b8 tegra: Support control of cache settings for LCD
329 9c56900 tegra: fdt: Add LCD definitions for Seaboard
330 5cc29db lcd: Add CONFIG_CONSOLE_SCROLL_LINES option to speed console
331 cac5a23 tegra: Enable display/lcd support on Seaboard
332 49ff541 wip
333
334Total boards to build for each commit: 1059
335
336This shows that it will build all 1059 boards, using 4 threads (because
337we have a 4-core CPU). Each thread will run with -j1, meaning that each
338make job will use a single CPU. The list of commits to be built helps you
339confirm that things look about right. Notice that buildman has chosen a
340'base' directory for you, immediately above your source tree.
341
342Buildman works entirely inside the base directory, here ../lcd9b,
343creating a working directory for each thread, and creating output
344directories for each commit and board.
345
346
347Suggested Workflow
348==================
349
350To run the build for real, take off the -n:
351
352$ ./tools/buildman/buildman -b <branch>
353
354Buildman will set up some working directories, and get started. After a
355minute or so it will settle down to a steady pace, with a display like this:
356
357Building 18 commits for 1059 boards (4 threads, 1 job per thread)
358 528 36 124 /19062 1:13:30 : SIMPC8313_SP
359
360This means that it is building 19062 board/commit combinations. So far it
Simon Glassfa1cfee2014-08-09 15:32:57 -0600361has managed to successfully build 528. Another 36 have built with warnings,
Simon Glassc05694f2013-04-03 11:07:16 +0000362and 124 more didn't build at all. Buildman expects to complete the process
363in an hour and 15 minutes. Use this time to buy a faster computer.
364
365
366To find out how the build went, ask for a summary with -s. You can do this
367either before the build completes (presumably in another terminal) or or
368afterwards. Let's work through an example of how this is used:
369
370$ ./tools/buildman/buildman -b lcd9b -s
371...
37201: Merge branch 'master' of git://git.denx.de/u-boot-arm
373 powerpc: + galaxy5200_LOWBOOT
37402: tegra: Use const for pinmux_config_pingroup/table()
37503: tegra: Add display support to funcmux
37604: tegra: fdt: Add pwm binding and node
37705: tegra: fdt: Add LCD definitions for Tegra
37806: tegra: Add support for PWM
37907: tegra: Add SOC support for display/lcd
38008: tegra: Add LCD driver
38109: tegra: Add LCD support to Nvidia boards
38210: arm: Add control over cachability of memory regions
38311: lcd: Add CONFIG_LCD_ALIGNMENT to select frame buffer alignment
38412: lcd: Add support for flushing LCD fb from dcache after update
385 arm: + lubbock
38613: tegra: Align LCD frame buffer to section boundary
38714: tegra: Support control of cache settings for LCD
38815: tegra: fdt: Add LCD definitions for Seaboard
38916: lcd: Add CONFIG_CONSOLE_SCROLL_LINES option to speed console
39017: tegra: Enable display/lcd support on Seaboard
39118: wip
392
393This shows which commits have succeeded and which have failed. In this case
394the build is still in progress so many boards are not built yet (use -u to
395see which ones). But still we can see a few failures. The galaxy5200_LOWBOOT
396never builds correctly. This could be a problem with our toolchain, or it
397could be a bug in the upstream. The good news is that we probably don't need
398to blame our commits. The bad news is it isn't tested on that board.
399
400Commit 12 broke lubbock. That's what the '+ lubbock' means. The failure
401is never fixed by a later commit, or you would see lubbock again, in green,
402without the +.
403
404To see the actual error:
405
406$ ./tools/buildman/buildman -b <branch> -se lubbock
407...
40812: lcd: Add support for flushing LCD fb from dcache after update
409 arm: + lubbock
410+common/libcommon.o: In function `lcd_sync':
411+/u-boot/lcd9b/.bm-work/00/common/lcd.c:120: undefined reference to `flush_dcache_range'
412+arm-none-linux-gnueabi-ld: BFD (Sourcery G++ Lite 2010q1-202) 2.19.51.20090709 assertion fail /scratch/julian/2010q1-release-linux-lite/obj/binutils-src-2010q1-202-arm-none-linux-gnueabi-i686-pc-linux-gnu/bfd/elf32-arm.c:12572
413+make: *** [/u-boot/lcd9b/.bm-work/00/build/u-boot] Error 139
41413: tegra: Align LCD frame buffer to section boundary
41514: tegra: Support control of cache settings for LCD
41615: tegra: fdt: Add LCD definitions for Seaboard
41716: lcd: Add CONFIG_CONSOLE_SCROLL_LINES option to speed console
418-/u-boot/lcd9b/.bm-work/00/common/lcd.c:120: undefined reference to `flush_dcache_range'
419+/u-boot/lcd9b/.bm-work/00/common/lcd.c:125: undefined reference to `flush_dcache_range'
42017: tegra: Enable display/lcd support on Seaboard
42118: wip
422
423So the problem is in lcd.c, due to missing cache operations. This information
424should be enough to work out what that commit is doing to break these
425boards. (In this case pxa did not have cache operations defined).
426
427If you see error lines marked with - that means that the errors were fixed
428by that commit. Sometimes commits can be in the wrong order, so that a
429breakage is introduced for a few commits and fixed by later commits. This
430shows up clearly with buildman. You can then reorder the commits and try
431again.
432
433At commit 16, the error moves - you can see that the old error at line 120
434is fixed, but there is a new one at line 126. This is probably only because
Simon Glassfa1cfee2014-08-09 15:32:57 -0600435we added some code and moved the broken line father down the file.
Simon Glassc05694f2013-04-03 11:07:16 +0000436
437If many boards have the same error, then -e will display the error only
438once. This makes the output as concise as possible.
439
440The full build output in this case is available in:
441
442../lcd9b/12_of_18_gd92aff7_lcd--Add-support-for/lubbock/
443
444 done: Indicates the build was done, and holds the return code from make.
445 This is 0 for a good build, typically 2 for a failure.
446
447 err: Output from stderr, if any. Errors and warnings appear here.
448
449 log: Output from stdout. Normally there isn't any since buildman runs
450 in silent mode for now.
451
452 toolchain: Shows information about the toolchain used for the build.
453
454 sizes: Shows image size information.
455
456It is possible to get the build output there also. Use the -k option for
457this. In that case you will also see some output files, like:
458
459 System.map toolchain u-boot u-boot.bin u-boot.map autoconf.mk
460 (also SPL versions u-boot-spl and u-boot-spl.bin if available)
461
462
463Checking Image Sizes
464====================
465
466A key requirement for U-Boot is that you keep code/data size to a minimum.
467Where a new feature increases this noticeably it should normally be put
468behind a CONFIG flag so that boards can leave it off and keep the image
469size more or less the same with each new release.
470
471To check the impact of your commits on image size, use -S. For example:
472
473$ ./tools/buildman/buildman -b us-x86 -sS
474Summary of 10 commits for 1066 boards (4 threads, 1 job per thread)
47501: MAKEALL: add support for per architecture toolchains
47602: x86: Add function to get top of usable ram
477 x86: (for 1/3 boards) text -272.0 rodata +41.0
47803: x86: Add basic cache operations
47904: x86: Permit bootstage and timer data to be used prior to relocation
480 x86: (for 1/3 boards) data +16.0
48105: x86: Add an __end symbol to signal the end of the U-Boot binary
482 x86: (for 1/3 boards) text +76.0
48306: x86: Rearrange the output input to remove BSS
484 x86: (for 1/3 boards) bss -2140.0
48507: x86: Support relocation of FDT on start-up
486 x86: + coreboot-x86
48708: x86: Add error checking to x86 relocation code
48809: x86: Adjust link device tree include file
48910: x86: Enable CONFIG_OF_CONTROL on coreboot
490
491
492You can see that image size only changed on x86, which is good because this
493series is not supposed to change any other board. From commit 7 onwards the
494build fails so we don't get code size numbers. The numbers are fractional
495because they are an average of all boards for that architecture. The
496intention is to allow you to quickly find image size problems introduced by
497your commits.
498
499Note that the 'text' region and 'rodata' are split out. You should add the
500two together to get the total read-only size (reported as the first column
501in the output from binutil's 'size' utility).
502
503A useful option is --step which lets you skip some commits. For example
504--step 2 will show the image sizes for only every 2nd commit (so it will
505compare the image sizes of the 1st, 3rd, 5th... commits). You can also use
506--step 0 which will compare only the first and last commits. This is useful
507for an overview of how your entire series affects code size.
508
509You can also use -d to see a detailed size breakdown for each board. This
510list is sorted in order from largest growth to largest reduction.
511
512It is possible to go a little further with the -B option (--bloat). This
Simon Glassfa1cfee2014-08-09 15:32:57 -0600513shows where U-Boot has bloated, breaking the size change down to the function
Simon Glassc05694f2013-04-03 11:07:16 +0000514level. Example output is below:
515
516$ ./tools/buildman/buildman -b us-mem4 -sSdB
517...
51819: Roll crc32 into hash infrastructure
519 arm: (for 10/10 boards) all -143.4 bss +1.2 data -4.8 rodata -48.2 text -91.6
520 paz00 : all +23 bss -4 rodata -29 text +56
521 u-boot: add: 1/0, grow: 3/-2 bytes: 168/-104 (64)
522 function old new delta
523 hash_command 80 160 +80
524 crc32_wd_buf - 56 +56
525 ext4fs_read_file 540 568 +28
526 insert_var_value_sub 688 692 +4
527 run_list_real 1996 1992 -4
528 do_mem_crc 168 68 -100
529 trimslice : all -9 bss +16 rodata -29 text +4
530 u-boot: add: 1/0, grow: 1/-3 bytes: 136/-124 (12)
531 function old new delta
532 hash_command 80 160 +80
533 crc32_wd_buf - 56 +56
534 ext4fs_iterate_dir 672 668 -4
535 ext4fs_read_file 568 548 -20
536 do_mem_crc 168 68 -100
537 whistler : all -9 bss +16 rodata -29 text +4
538 u-boot: add: 1/0, grow: 1/-3 bytes: 136/-124 (12)
539 function old new delta
540 hash_command 80 160 +80
541 crc32_wd_buf - 56 +56
542 ext4fs_iterate_dir 672 668 -4
543 ext4fs_read_file 568 548 -20
544 do_mem_crc 168 68 -100
545 seaboard : all -9 bss -28 rodata -29 text +48
546 u-boot: add: 1/0, grow: 3/-2 bytes: 160/-104 (56)
547 function old new delta
548 hash_command 80 160 +80
549 crc32_wd_buf - 56 +56
550 ext4fs_read_file 548 568 +20
551 run_list_real 1996 2000 +4
552 do_nandboot 760 756 -4
553 do_mem_crc 168 68 -100
554 colibri_t20_iris: all -9 rodata -29 text +20
555 u-boot: add: 1/0, grow: 2/-3 bytes: 140/-112 (28)
556 function old new delta
557 hash_command 80 160 +80
558 crc32_wd_buf - 56 +56
559 read_abs_bbt 204 208 +4
560 do_nandboot 760 756 -4
561 ext4fs_read_file 576 568 -8
562 do_mem_crc 168 68 -100
563 ventana : all -37 bss -12 rodata -29 text +4
564 u-boot: add: 1/0, grow: 1/-3 bytes: 136/-124 (12)
565 function old new delta
566 hash_command 80 160 +80
567 crc32_wd_buf - 56 +56
568 ext4fs_iterate_dir 672 668 -4
569 ext4fs_read_file 568 548 -20
570 do_mem_crc 168 68 -100
571 harmony : all -37 bss -16 rodata -29 text +8
572 u-boot: add: 1/0, grow: 2/-3 bytes: 140/-124 (16)
573 function old new delta
574 hash_command 80 160 +80
575 crc32_wd_buf - 56 +56
576 nand_write_oob_syndrome 428 432 +4
577 ext4fs_iterate_dir 672 668 -4
578 ext4fs_read_file 568 548 -20
579 do_mem_crc 168 68 -100
580 medcom-wide : all -417 bss +28 data -16 rodata -93 text -336
581 u-boot: add: 1/-1, grow: 1/-2 bytes: 88/-376 (-288)
582 function old new delta
583 crc32_wd_buf - 56 +56
584 do_fat_read_at 2872 2904 +32
585 hash_algo 16 - -16
586 do_mem_crc 168 68 -100
587 hash_command 420 160 -260
588 tec : all -449 bss -4 data -16 rodata -93 text -336
589 u-boot: add: 1/-1, grow: 1/-2 bytes: 88/-376 (-288)
590 function old new delta
591 crc32_wd_buf - 56 +56
592 do_fat_read_at 2872 2904 +32
593 hash_algo 16 - -16
594 do_mem_crc 168 68 -100
595 hash_command 420 160 -260
596 plutux : all -481 bss +16 data -16 rodata -93 text -388
597 u-boot: add: 1/-1, grow: 1/-3 bytes: 68/-408 (-340)
598 function old new delta
599 crc32_wd_buf - 56 +56
600 do_load_serial_bin 1688 1700 +12
601 hash_algo 16 - -16
602 do_fat_read_at 2904 2872 -32
603 do_mem_crc 168 68 -100
604 hash_command 420 160 -260
605 powerpc: (for 5/5 boards) all +37.4 data -3.2 rodata -41.8 text +82.4
606 MPC8610HPCD : all +55 rodata -29 text +84
607 u-boot: add: 1/0, grow: 0/-1 bytes: 176/-96 (80)
608 function old new delta
609 hash_command - 176 +176
610 do_mem_crc 184 88 -96
611 MPC8641HPCN : all +55 rodata -29 text +84
612 u-boot: add: 1/0, grow: 0/-1 bytes: 176/-96 (80)
613 function old new delta
614 hash_command - 176 +176
615 do_mem_crc 184 88 -96
616 MPC8641HPCN_36BIT: all +55 rodata -29 text +84
617 u-boot: add: 1/0, grow: 0/-1 bytes: 176/-96 (80)
618 function old new delta
619 hash_command - 176 +176
620 do_mem_crc 184 88 -96
621 sbc8641d : all +55 rodata -29 text +84
622 u-boot: add: 1/0, grow: 0/-1 bytes: 176/-96 (80)
623 function old new delta
624 hash_command - 176 +176
625 do_mem_crc 184 88 -96
626 xpedite517x : all -33 data -16 rodata -93 text +76
627 u-boot: add: 1/-1, grow: 0/-1 bytes: 176/-112 (64)
628 function old new delta
629 hash_command - 176 +176
630 hash_algo 16 - -16
631 do_mem_crc 184 88 -96
632...
633
634
635This shows that commit 19 has increased text size for arm (although only one
636board was built) and by 96 bytes for powerpc. This increase was offset in both
637cases by reductions in rodata and data/bss.
638
639Shown below the summary lines is the sizes for each board. Below each board
640is the sizes for each function. This information starts with:
641
642 add - number of functions added / removed
643 grow - number of functions which grew / shrunk
644 bytes - number of bytes of code added to / removed from all functions,
645 plus the total byte change in brackets
646
647The change seems to be that hash_command() has increased by more than the
648do_mem_crc() function has decreased. The function sizes typically add up to
649roughly the text area size, but note that every read-only section except
650rodata is included in 'text', so the function total does not exactly
651correspond.
652
653It is common when refactoring code for the rodata to decrease as the text size
654increases, and vice versa.
655
656
Simon Glasscc246fb2013-09-23 17:35:17 -0600657Providing 'make' flags
658======================
659
660U-Boot's build system supports a few flags (such as BUILD_TAG) which affect
661the build product. These flags can be specified in the buildman settings
662file. They can also be useful when building U-Boot against other open source
663software.
664
665[make-flags]
666at91-boards=ENABLE_AT91_TEST=1
667snapper9260=${at91-boards} BUILD_TAG=442
668snapper9g45=${at91-boards} BUILD_TAG=443
669
670This will use 'make ENABLE_AT91_TEST=1 BUILD_TAG=442' for snapper9260
Andreas Bießmannb5c56e32013-11-05 10:37:09 +0100671and 'make ENABLE_AT91_TEST=1 BUILD_TAG=443' for snapper9g45. A special
Simon Glasscc246fb2013-09-23 17:35:17 -0600672variable ${target} is available to access the target name (snapper9260 and
673snapper9g20 in this case). Variables are resolved recursively.
674
675It is expected that any variables added are dealt with in U-Boot's
676config.mk file and documented in the README.
677
678
Simon Glass78e418e2014-08-09 15:33:03 -0600679Quick Sanity Check
680==================
681
682If you have made changes and want to do a quick sanity check of the
683currently-checked-out source, run buildman without the -b flag. This will
684build the selected boards and display build status and errors as it runs
685(i.e. -v amd -e are enabled automatically).
686
687
Simon Glassc05694f2013-04-03 11:07:16 +0000688Other options
689=============
690
691Buildman has various other command line options. Try --help to see them.
692
Simon Glassc2f91072014-08-28 09:43:39 -0600693When doing builds, Buildman's return code will reflect the overall result:
694
695 0 (success) No errors or warnings found
696 128 Errors found
697 129 Warnings found
698
Simon Glassc05694f2013-04-03 11:07:16 +0000699
Simon Glass3ec03d42014-08-09 15:32:58 -0600700How to change from MAKEALL
701==========================
702
703Buildman includes most of the features of MAKEALL and is generally faster
704and easier to use. In particular it builds entire branches: if a particular
705commit introduces an error in a particular board, buildman can easily show
706you this, even if a later commit fixes that error.
707
708The reasons to deprecate MAKEALL are:
709- We don't want to maintain two build systems
710- Buildman is typically faster
711- Buildman has a lot more features
712
713But still, many people will be sad to lose MAKEALL. If you are used to
714MAKEALL, here are a few pointers.
715
716First you need to set up your tool chains - see the 'Setting up' section
717for details. Once you have your required toolchain(s) detected then you are
718ready to go.
719
Simon Glass78e418e2014-08-09 15:33:03 -0600720To build the current source tree, run buildman without a -b flag:
721
722 ./tools/buildman/buildman <list of things to build>
723
724This will build the current source tree for the given boards and display
725the results and errors.
726
727However buildman usually works on entire branches, and for that you must
728specify a board flag:
Simon Glass3ec03d42014-08-09 15:32:58 -0600729
730 ./tools/buildman/buildman -b <branch_name> <list of things to build>
731
732followed by (afterwards, or perhaps concurrently in another terminal):
733
734 ./tools/buildman/buildman -b <branch_name> -s <list of things to build>
735
736to see the results of the build. Rather than showing you all the output,
737buildman just shows a summary, with red indicating that a commit introduced
738an error and green indicating that a commit fixed an error. Use the -e
739flag to see the full errors.
740
Simon Glass78e418e2014-08-09 15:33:03 -0600741If you really want to see build results as they happen, use -v when doing a
742build (and -e if you want to see errors as well).
743
Simon Glass3ec03d42014-08-09 15:32:58 -0600744You don't need to stick around on that branch while buildman is running. It
745checks out its own copy of the source code, so you can change branches,
746add commits, etc. without affecting the build in progress.
747
748The <list of things to build> can include board names, architectures or the
749like. There are no flags to disambiguate since ambiguities are rare. Using
750the examples from MAKEALL:
751
752Examples:
753 - build all Power Architecture boards:
754 MAKEALL -a powerpc
755 MAKEALL --arch powerpc
756 MAKEALL powerpc
757 ** buildman -b <branch> powerpc
758 - build all PowerPC boards manufactured by vendor "esd":
759 MAKEALL -a powerpc -v esd
760 ** buildman -b <branch> esd
761 - build all PowerPC boards manufactured either by "keymile" or "siemens":
762 MAKEALL -a powerpc -v keymile -v siemens
763 ** buildman -b <branch> keymile siemens
764 - build all Freescale boards with MPC83xx CPUs, plus all 4xx boards:
765 MAKEALL -c mpc83xx -v freescale 4xx
766 ** buildman -b <branch> mpc83xx freescale 4xx
767
768Buildman automatically tries to use all the CPUs in your machine. If you
769are building a lot of boards it will use one thread for every CPU core
770it detects in your machine. This is like MAKEALL's BUILD_NBUILDS option.
771You can use the -T flag to change the number of threads. If you are only
772building a few boards, buildman will automatically run make with the -j
773flag to increase the number of concurrent make tasks. It isn't normally
774that helpful to fiddle with this option, but if you use the BUILD_NCPUS
775option in MAKEALL then -j is the equivalent in buildman.
776
777Buildman puts its output in ../<branch_name> by default but you can change
778this with the -o option. Buildman normally does out-of-tree builds: use -i
779to disable that if you really want to. But be careful that once you have
780used -i you pollute buildman's copies of the source tree, and you will need
781to remove the build directory (normally ../<branch_name>) to run buildman
782in normal mode (without -i).
783
784Buildman doesn't keep the output result normally, but use the -k option to
785do this.
786
787Please read 'Theory of Operation' a few times as it will make a lot of
788things clearer.
789
790Some options you might like are:
791
792 -B shows which functions are growing/shrinking in which commit - great
793 for finding code bloat.
794 -S shows image sizes for each commit (just an overall summary)
795 -u shows boards that you haven't built yet
796 --step 0 will build just the upstream commit and the last commit of your
797 branch. This is often a quick sanity check that your branch doesn't
798 break anything. But note this does not check bisectability!
799
800
Simon Glassc05694f2013-04-03 11:07:16 +0000801TODO
802====
803
804This has mostly be written in my spare time as a response to my difficulties
805in testing large series of patches. Apart from tidying up there is quite a
806bit of scope for improvement. Things like better error diffs, easier access
807to log files, error display while building. Also it would be nice it buildman
808could 'hunt' for problems, perhaps by building a few boards for each arch,
809or checking commits for changed files and building only boards which use
810those files.
811
812
813Credits
814=======
815
816Thanks to Grant Grundler <grundler@chromium.org> for his ideas for improving
817the build speed by building all commits for a board instead of the other
818way around.
819
820
Simon Glassc05694f2013-04-03 11:07:16 +0000821Simon Glass
822sjg@chromium.org
823Halloween 2012
824Updated 12-12-12
825Updated 23-02-13