buildman - U-Boot multi-threaded builder and summary tool
This tool handles building U-Boot to check that you have not broken it with your patch series. It can build each individual commit and report which boards fail on which commits, and which errors come up. It also shows differences in image sizes due to particular commits. Buildman aims to make full use of multi-processor machines. Documentation and caveats are in tools/buildman/README. Signed-off-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
parent
3fefd5efa6
commit
fc3fe1c287
1
tools/buildman/.gitignore
vendored
Normal file
1
tools/buildman/.gitignore
vendored
Normal file
@ -0,0 +1 @@
|
||||
*.pyc
|
679
tools/buildman/README
Normal file
679
tools/buildman/README
Normal file
@ -0,0 +1,679 @@
|
||||
# Copyright (c) 2013 The Chromium OS Authors.
|
||||
#
|
||||
# See file CREDITS for list of people who contributed to this
|
||||
# project.
|
||||
#
|
||||
# This program is free software; you can redistribute it and/or
|
||||
# modify it under the terms of the GNU General Public License as
|
||||
# published by the Free Software Foundation; either version 2 of
|
||||
# the License, or (at your option) any later version.
|
||||
#
|
||||
# This program is distributed in the hope that it will be useful,
|
||||
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||
# GNU General Public License for more details.
|
||||
#
|
||||
# You should have received a copy of the GNU General Public License
|
||||
# along with this program; if not, write to the Free Software
|
||||
# Foundation, Inc., 59 Temple Place, Suite 330, Boston,
|
||||
# MA 02111-1307 USA
|
||||
#
|
||||
|
||||
What is this?
|
||||
=============
|
||||
|
||||
This tool handles building U-Boot to check that you have not broken it
|
||||
with your patch series. It can build each individual commit and report
|
||||
which boards fail on which commits, and which errors come up. It aims
|
||||
to make full use of multi-processor machines.
|
||||
|
||||
A key feature of buildman is its output summary, which allows warnings,
|
||||
errors or image size increases in a particular commit or board to be
|
||||
quickly identified and the offending commit pinpointed. This can be a big
|
||||
help for anyone working with >10 patches at a time.
|
||||
|
||||
|
||||
Caveats
|
||||
=======
|
||||
|
||||
Buildman is still in its infancy. It is already a very useful tool, but
|
||||
expect to find problems and send patches.
|
||||
|
||||
Buildman can be stopped and restarted, in which case it will continue
|
||||
where it left off. This should happen cleanly and without side-effects.
|
||||
If not, it is a bug, for which a patch would be welcome.
|
||||
|
||||
Buildman gets so tied up in its work that it can ignore the outside world.
|
||||
You may need to press Ctrl-C several times to quit it. Also it will print
|
||||
out various exceptions when stopped.
|
||||
|
||||
|
||||
Theory of Operation
|
||||
===================
|
||||
|
||||
(please read this section in full twice or you will be perpetually confused)
|
||||
|
||||
Buildman is a builder. It is not make, although it runs make. It does not
|
||||
produce any useful output on the terminal while building, except for
|
||||
progress information. All the output (errors, warnings and binaries if you
|
||||
are ask for them) is stored in output directories, which you can look at
|
||||
while the build is progressing, or when it is finished.
|
||||
|
||||
Buildman produces a concise summary of which boards succeeded and failed.
|
||||
It shows which commit introduced which board failure using a simple
|
||||
red/green colour coding. Full error information can be requested, in which
|
||||
case it is de-duped and displayed against the commit that introduced the
|
||||
error. An example workflow is below.
|
||||
|
||||
Buildman stores image size information and can report changes in image size
|
||||
from commit to commit. An example of this is below.
|
||||
|
||||
Buildman starts multiple threads, and each thread builds for one board at
|
||||
a time. A thread starts at the first commit, configures the source for your
|
||||
board and builds it. Then it checks out the next commit and does an
|
||||
incremental build. Eventually the thread reaches the last commit and stops.
|
||||
If errors or warnings are found along the way, the thread will reconfigure
|
||||
after every commit, and your build will be very slow. This is because a
|
||||
file that produces just a warning would not normally be rebuilt in an
|
||||
incremental build.
|
||||
|
||||
Buildman works in an entirely separate place from your U-Boot repository.
|
||||
It creates a separate working directory for each thread, and puts the
|
||||
output files in the working directory, organised by commit name and board
|
||||
name, in a two-level hierarchy.
|
||||
|
||||
Buildman is invoked in your U-Boot directory, the one with the .git
|
||||
directory. It clones this repository into a copy for each thread, and the
|
||||
threads do not affect the state of your git repository. Any checkouts done
|
||||
by the thread affect only the working directory for that thread.
|
||||
|
||||
Buildman automatically selects the correct toolchain for each board. You
|
||||
must supply suitable toolchains, but buildman takes care of selecting the
|
||||
right one.
|
||||
|
||||
Buildman always builds a branch, and always builds the upstream commit as
|
||||
well, for comparison. It cannot build individual commits at present, unless
|
||||
(maybe) you point it at an empty branch. Put all your commits in a branch,
|
||||
set the branch's upstream to a valid value, and all will be well. Otherwise
|
||||
buildman will perform random actions. Use -n to check what the random
|
||||
actions might be.
|
||||
|
||||
Buildman is optimised for building many commits at once, for many boards.
|
||||
On multi-core machines, Buildman is fast because it uses most of the
|
||||
available CPU power. When it gets to the end, or if you are building just
|
||||
a few commits or boards, it will be pretty slow. As a tip, if you don't
|
||||
plan to use your machine for anything else, you can use -T to increase the
|
||||
number of threads beyond the default.
|
||||
|
||||
Buildman lets you build all boards, or a subset. Specify the subset using
|
||||
the board name, architecture name, SOC name, or anything else in the
|
||||
boards.cfg file. So 'at91' will build all AT91 boards (arm), powerpc will
|
||||
build all PowerPC boards.
|
||||
|
||||
Buildman does not store intermediate object files. It optionally copies
|
||||
the binary output into a directory when a build is successful. Size
|
||||
information is always recorded. It needs a fair bit of disk space to work,
|
||||
typically 250MB per thread.
|
||||
|
||||
|
||||
Setting up
|
||||
==========
|
||||
|
||||
1. Get the U-Boot source. You probably already have it, but if not these
|
||||
steps should get you started with a repo and some commits for testing.
|
||||
|
||||
$ cd /path/to/u-boot
|
||||
$ git clone git://git.denx.de/u-boot.git .
|
||||
$ git checkout -b my-branch origin/master
|
||||
$ # Add some commits to the branch, reading for testing
|
||||
|
||||
2. Create ~/.buildman to tell buildman where to find tool chains. As an
|
||||
example:
|
||||
|
||||
# Buildman settings file
|
||||
|
||||
[toolchain]
|
||||
root: /
|
||||
rest: /toolchains/*
|
||||
eldk: /opt/eldk-4.2
|
||||
|
||||
[toolchain-alias]
|
||||
x86: i386
|
||||
blackfin: bfin
|
||||
sh: sh4
|
||||
nds32: nds32le
|
||||
openrisc: or32
|
||||
|
||||
|
||||
This selects the available toolchain paths. Add the base directory for
|
||||
each of your toolchains here. Buildman will search inside these directories
|
||||
and also in any '/usr' and '/usr/bin' subdirectories.
|
||||
|
||||
Make sure the tags (here root: rest: and eldk:) are unique.
|
||||
|
||||
The toolchain-alias section indicates that the i386 toolchain should be used
|
||||
to build x86 commits.
|
||||
|
||||
|
||||
2. Check the available toolchains
|
||||
|
||||
Run this check to make sure that you have a toolchain for every architecture.
|
||||
|
||||
$ ./tools/buildman/buildman --list-tool-chains
|
||||
Scanning for tool chains
|
||||
- scanning path '/'
|
||||
- looking in '/.'
|
||||
- looking in '/bin'
|
||||
- looking in '/usr/bin'
|
||||
- found '/usr/bin/gcc'
|
||||
Tool chain test: OK
|
||||
- found '/usr/bin/c89-gcc'
|
||||
Tool chain test: OK
|
||||
- found '/usr/bin/c99-gcc'
|
||||
Tool chain test: OK
|
||||
- found '/usr/bin/x86_64-linux-gnu-gcc'
|
||||
Tool chain test: OK
|
||||
- scanning path '/toolchains/powerpc-linux'
|
||||
- looking in '/toolchains/powerpc-linux/.'
|
||||
- looking in '/toolchains/powerpc-linux/bin'
|
||||
- found '/toolchains/powerpc-linux/bin/powerpc-linux-gcc'
|
||||
Tool chain test: OK
|
||||
- looking in '/toolchains/powerpc-linux/usr/bin'
|
||||
- scanning path '/toolchains/nds32le-linux-glibc-v1f'
|
||||
- looking in '/toolchains/nds32le-linux-glibc-v1f/.'
|
||||
- looking in '/toolchains/nds32le-linux-glibc-v1f/bin'
|
||||
- found '/toolchains/nds32le-linux-glibc-v1f/bin/nds32le-linux-gcc'
|
||||
Tool chain test: OK
|
||||
- looking in '/toolchains/nds32le-linux-glibc-v1f/usr/bin'
|
||||
- scanning path '/toolchains/nios2'
|
||||
- looking in '/toolchains/nios2/.'
|
||||
- looking in '/toolchains/nios2/bin'
|
||||
- found '/toolchains/nios2/bin/nios2-linux-gcc'
|
||||
Tool chain test: OK
|
||||
- found '/toolchains/nios2/bin/nios2-linux-uclibc-gcc'
|
||||
Tool chain test: OK
|
||||
- looking in '/toolchains/nios2/usr/bin'
|
||||
- found '/toolchains/nios2/usr/bin/nios2-linux-gcc'
|
||||
Tool chain test: OK
|
||||
- found '/toolchains/nios2/usr/bin/nios2-linux-uclibc-gcc'
|
||||
Tool chain test: OK
|
||||
- scanning path '/toolchains/microblaze-unknown-linux-gnu'
|
||||
- looking in '/toolchains/microblaze-unknown-linux-gnu/.'
|
||||
- looking in '/toolchains/microblaze-unknown-linux-gnu/bin'
|
||||
- found '/toolchains/microblaze-unknown-linux-gnu/bin/microblaze-unknown-linux-gnu-gcc'
|
||||
Tool chain test: OK
|
||||
- found '/toolchains/microblaze-unknown-linux-gnu/bin/mb-linux-gcc'
|
||||
Tool chain test: OK
|
||||
- looking in '/toolchains/microblaze-unknown-linux-gnu/usr/bin'
|
||||
- scanning path '/toolchains/mips-linux'
|
||||
- looking in '/toolchains/mips-linux/.'
|
||||
- looking in '/toolchains/mips-linux/bin'
|
||||
- found '/toolchains/mips-linux/bin/mips-linux-gcc'
|
||||
Tool chain test: OK
|
||||
- looking in '/toolchains/mips-linux/usr/bin'
|
||||
- scanning path '/toolchains/old'
|
||||
- looking in '/toolchains/old/.'
|
||||
- looking in '/toolchains/old/bin'
|
||||
- looking in '/toolchains/old/usr/bin'
|
||||
- scanning path '/toolchains/i386-linux'
|
||||
- looking in '/toolchains/i386-linux/.'
|
||||
- looking in '/toolchains/i386-linux/bin'
|
||||
- found '/toolchains/i386-linux/bin/i386-linux-gcc'
|
||||
Tool chain test: OK
|
||||
- looking in '/toolchains/i386-linux/usr/bin'
|
||||
- scanning path '/toolchains/bfin-uclinux'
|
||||
- looking in '/toolchains/bfin-uclinux/.'
|
||||
- looking in '/toolchains/bfin-uclinux/bin'
|
||||
- found '/toolchains/bfin-uclinux/bin/bfin-uclinux-gcc'
|
||||
Tool chain test: OK
|
||||
- looking in '/toolchains/bfin-uclinux/usr/bin'
|
||||
- scanning path '/toolchains/sparc-elf'
|
||||
- looking in '/toolchains/sparc-elf/.'
|
||||
- looking in '/toolchains/sparc-elf/bin'
|
||||
- found '/toolchains/sparc-elf/bin/sparc-elf-gcc'
|
||||
Tool chain test: OK
|
||||
- looking in '/toolchains/sparc-elf/usr/bin'
|
||||
- scanning path '/toolchains/arm-2010q1'
|
||||
- looking in '/toolchains/arm-2010q1/.'
|
||||
- looking in '/toolchains/arm-2010q1/bin'
|
||||
- found '/toolchains/arm-2010q1/bin/arm-none-linux-gnueabi-gcc'
|
||||
Tool chain test: OK
|
||||
- looking in '/toolchains/arm-2010q1/usr/bin'
|
||||
- scanning path '/toolchains/from'
|
||||
- looking in '/toolchains/from/.'
|
||||
- looking in '/toolchains/from/bin'
|
||||
- looking in '/toolchains/from/usr/bin'
|
||||
- scanning path '/toolchains/sh4-gentoo-linux-gnu'
|
||||
- looking in '/toolchains/sh4-gentoo-linux-gnu/.'
|
||||
- looking in '/toolchains/sh4-gentoo-linux-gnu/bin'
|
||||
- found '/toolchains/sh4-gentoo-linux-gnu/bin/sh4-gentoo-linux-gnu-gcc'
|
||||
Tool chain test: OK
|
||||
- looking in '/toolchains/sh4-gentoo-linux-gnu/usr/bin'
|
||||
- scanning path '/toolchains/avr32-linux'
|
||||
- looking in '/toolchains/avr32-linux/.'
|
||||
- looking in '/toolchains/avr32-linux/bin'
|
||||
- found '/toolchains/avr32-linux/bin/avr32-gcc'
|
||||
Tool chain test: OK
|
||||
- looking in '/toolchains/avr32-linux/usr/bin'
|
||||
- scanning path '/toolchains/m68k-linux'
|
||||
- looking in '/toolchains/m68k-linux/.'
|
||||
- looking in '/toolchains/m68k-linux/bin'
|
||||
- found '/toolchains/m68k-linux/bin/m68k-linux-gcc'
|
||||
Tool chain test: OK
|
||||
- looking in '/toolchains/m68k-linux/usr/bin'
|
||||
List of available toolchains (17):
|
||||
arm : /toolchains/arm-2010q1/bin/arm-none-linux-gnueabi-gcc
|
||||
avr32 : /toolchains/avr32-linux/bin/avr32-gcc
|
||||
bfin : /toolchains/bfin-uclinux/bin/bfin-uclinux-gcc
|
||||
c89 : /usr/bin/c89-gcc
|
||||
c99 : /usr/bin/c99-gcc
|
||||
i386 : /toolchains/i386-linux/bin/i386-linux-gcc
|
||||
m68k : /toolchains/m68k-linux/bin/m68k-linux-gcc
|
||||
mb : /toolchains/microblaze-unknown-linux-gnu/bin/mb-linux-gcc
|
||||
microblaze: /toolchains/microblaze-unknown-linux-gnu/bin/microblaze-unknown-linux-gnu-gcc
|
||||
mips : /toolchains/mips-linux/bin/mips-linux-gcc
|
||||
nds32le : /toolchains/nds32le-linux-glibc-v1f/bin/nds32le-linux-gcc
|
||||
nios2 : /toolchains/nios2/bin/nios2-linux-gcc
|
||||
powerpc : /toolchains/powerpc-linux/bin/powerpc-linux-gcc
|
||||
sandbox : /usr/bin/gcc
|
||||
sh4 : /toolchains/sh4-gentoo-linux-gnu/bin/sh4-gentoo-linux-gnu-gcc
|
||||
sparc : /toolchains/sparc-elf/bin/sparc-elf-gcc
|
||||
x86_64 : /usr/bin/x86_64-linux-gnu-gcc
|
||||
|
||||
|
||||
You can see that everything is covered, even some strange ones that won't
|
||||
be used (c88 and c99). This is a feature.
|
||||
|
||||
|
||||
How to run it
|
||||
=============
|
||||
|
||||
First do a dry run using the -n flag: (replace <branch> with a real, local
|
||||
branch with a valid upstream)
|
||||
|
||||
$ ./tools/buildman/buildman -b <branch> -n
|
||||
|
||||
If it can't detect the upstream branch, try checking out the branch, and
|
||||
doing something like 'git branch --set-upstream <branch> upstream/master'
|
||||
or something similar.
|
||||
|
||||
As an exmmple:
|
||||
|
||||
Dry run, so not doing much. But I would do this:
|
||||
|
||||
Building 18 commits for 1059 boards (4 threads, 1 job per thread)
|
||||
Build directory: ../lcd9b
|
||||
5bb3505 Merge branch 'master' of git://git.denx.de/u-boot-arm
|
||||
c18f1b4 tegra: Use const for pinmux_config_pingroup/table()
|
||||
2f043ae tegra: Add display support to funcmux
|
||||
e349900 tegra: fdt: Add pwm binding and node
|
||||
424a5f0 tegra: fdt: Add LCD definitions for Tegra
|
||||
0636ccf tegra: Add support for PWM
|
||||
a994fe7 tegra: Add SOC support for display/lcd
|
||||
fcd7350 tegra: Add LCD driver
|
||||
4d46e9d tegra: Add LCD support to Nvidia boards
|
||||
991bd48 arm: Add control over cachability of memory regions
|
||||
54e8019 lcd: Add CONFIG_LCD_ALIGNMENT to select frame buffer alignment
|
||||
d92aff7 lcd: Add support for flushing LCD fb from dcache after update
|
||||
dbd0677 tegra: Align LCD frame buffer to section boundary
|
||||
0cff9b8 tegra: Support control of cache settings for LCD
|
||||
9c56900 tegra: fdt: Add LCD definitions for Seaboard
|
||||
5cc29db lcd: Add CONFIG_CONSOLE_SCROLL_LINES option to speed console
|
||||
cac5a23 tegra: Enable display/lcd support on Seaboard
|
||||
49ff541 wip
|
||||
|
||||
Total boards to build for each commit: 1059
|
||||
|
||||
This shows that it will build all 1059 boards, using 4 threads (because
|
||||
we have a 4-core CPU). Each thread will run with -j1, meaning that each
|
||||
make job will use a single CPU. The list of commits to be built helps you
|
||||
confirm that things look about right. Notice that buildman has chosen a
|
||||
'base' directory for you, immediately above your source tree.
|
||||
|
||||
Buildman works entirely inside the base directory, here ../lcd9b,
|
||||
creating a working directory for each thread, and creating output
|
||||
directories for each commit and board.
|
||||
|
||||
|
||||
Suggested Workflow
|
||||
==================
|
||||
|
||||
To run the build for real, take off the -n:
|
||||
|
||||
$ ./tools/buildman/buildman -b <branch>
|
||||
|
||||
Buildman will set up some working directories, and get started. After a
|
||||
minute or so it will settle down to a steady pace, with a display like this:
|
||||
|
||||
Building 18 commits for 1059 boards (4 threads, 1 job per thread)
|
||||
528 36 124 /19062 1:13:30 : SIMPC8313_SP
|
||||
|
||||
This means that it is building 19062 board/commit combinations. So far it
|
||||
has managed to succesfully build 528. Another 36 have built with warnings,
|
||||
and 124 more didn't build at all. Buildman expects to complete the process
|
||||
in an hour and 15 minutes. Use this time to buy a faster computer.
|
||||
|
||||
|
||||
To find out how the build went, ask for a summary with -s. You can do this
|
||||
either before the build completes (presumably in another terminal) or or
|
||||
afterwards. Let's work through an example of how this is used:
|
||||
|
||||
$ ./tools/buildman/buildman -b lcd9b -s
|
||||
...
|
||||
01: Merge branch 'master' of git://git.denx.de/u-boot-arm
|
||||
powerpc: + galaxy5200_LOWBOOT
|
||||
02: tegra: Use const for pinmux_config_pingroup/table()
|
||||
03: tegra: Add display support to funcmux
|
||||
04: tegra: fdt: Add pwm binding and node
|
||||
05: tegra: fdt: Add LCD definitions for Tegra
|
||||
06: tegra: Add support for PWM
|
||||
07: tegra: Add SOC support for display/lcd
|
||||
08: tegra: Add LCD driver
|
||||
09: tegra: Add LCD support to Nvidia boards
|
||||
10: arm: Add control over cachability of memory regions
|
||||
11: lcd: Add CONFIG_LCD_ALIGNMENT to select frame buffer alignment
|
||||
12: lcd: Add support for flushing LCD fb from dcache after update
|
||||
arm: + lubbock
|
||||
13: tegra: Align LCD frame buffer to section boundary
|
||||
14: tegra: Support control of cache settings for LCD
|
||||
15: tegra: fdt: Add LCD definitions for Seaboard
|
||||
16: lcd: Add CONFIG_CONSOLE_SCROLL_LINES option to speed console
|
||||
17: tegra: Enable display/lcd support on Seaboard
|
||||
18: wip
|
||||
|
||||
This shows which commits have succeeded and which have failed. In this case
|
||||
the build is still in progress so many boards are not built yet (use -u to
|
||||
see which ones). But still we can see a few failures. The galaxy5200_LOWBOOT
|
||||
never builds correctly. This could be a problem with our toolchain, or it
|
||||
could be a bug in the upstream. The good news is that we probably don't need
|
||||
to blame our commits. The bad news is it isn't tested on that board.
|
||||
|
||||
Commit 12 broke lubbock. That's what the '+ lubbock' means. The failure
|
||||
is never fixed by a later commit, or you would see lubbock again, in green,
|
||||
without the +.
|
||||
|
||||
To see the actual error:
|
||||
|
||||
$ ./tools/buildman/buildman -b <branch> -se lubbock
|
||||
...
|
||||
12: lcd: Add support for flushing LCD fb from dcache after update
|
||||
arm: + lubbock
|
||||
+common/libcommon.o: In function `lcd_sync':
|
||||
+/u-boot/lcd9b/.bm-work/00/common/lcd.c:120: undefined reference to `flush_dcache_range'
|
||||
+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
|
||||
+make: *** [/u-boot/lcd9b/.bm-work/00/build/u-boot] Error 139
|
||||
13: tegra: Align LCD frame buffer to section boundary
|
||||
14: tegra: Support control of cache settings for LCD
|
||||
15: tegra: fdt: Add LCD definitions for Seaboard
|
||||
16: lcd: Add CONFIG_CONSOLE_SCROLL_LINES option to speed console
|
||||
-/u-boot/lcd9b/.bm-work/00/common/lcd.c:120: undefined reference to `flush_dcache_range'
|
||||
+/u-boot/lcd9b/.bm-work/00/common/lcd.c:125: undefined reference to `flush_dcache_range'
|
||||
17: tegra: Enable display/lcd support on Seaboard
|
||||
18: wip
|
||||
|
||||
So the problem is in lcd.c, due to missing cache operations. This information
|
||||
should be enough to work out what that commit is doing to break these
|
||||
boards. (In this case pxa did not have cache operations defined).
|
||||
|
||||
If you see error lines marked with - that means that the errors were fixed
|
||||
by that commit. Sometimes commits can be in the wrong order, so that a
|
||||
breakage is introduced for a few commits and fixed by later commits. This
|
||||
shows up clearly with buildman. You can then reorder the commits and try
|
||||
again.
|
||||
|
||||
At commit 16, the error moves - you can see that the old error at line 120
|
||||
is fixed, but there is a new one at line 126. This is probably only because
|
||||
we added some code and moved the broken line futher down the file.
|
||||
|
||||
If many boards have the same error, then -e will display the error only
|
||||
once. This makes the output as concise as possible.
|
||||
|
||||
The full build output in this case is available in:
|
||||
|
||||
../lcd9b/12_of_18_gd92aff7_lcd--Add-support-for/lubbock/
|
||||
|
||||
done: Indicates the build was done, and holds the return code from make.
|
||||
This is 0 for a good build, typically 2 for a failure.
|
||||
|
||||
err: Output from stderr, if any. Errors and warnings appear here.
|
||||
|
||||
log: Output from stdout. Normally there isn't any since buildman runs
|
||||
in silent mode for now.
|
||||
|
||||
toolchain: Shows information about the toolchain used for the build.
|
||||
|
||||
sizes: Shows image size information.
|
||||
|
||||
It is possible to get the build output there also. Use the -k option for
|
||||
this. In that case you will also see some output files, like:
|
||||
|
||||
System.map toolchain u-boot u-boot.bin u-boot.map autoconf.mk
|
||||
(also SPL versions u-boot-spl and u-boot-spl.bin if available)
|
||||
|
||||
|
||||
Checking Image Sizes
|
||||
====================
|
||||
|
||||
A key requirement for U-Boot is that you keep code/data size to a minimum.
|
||||
Where a new feature increases this noticeably it should normally be put
|
||||
behind a CONFIG flag so that boards can leave it off and keep the image
|
||||
size more or less the same with each new release.
|
||||
|
||||
To check the impact of your commits on image size, use -S. For example:
|
||||
|
||||
$ ./tools/buildman/buildman -b us-x86 -sS
|
||||
Summary of 10 commits for 1066 boards (4 threads, 1 job per thread)
|
||||
01: MAKEALL: add support for per architecture toolchains
|
||||
02: x86: Add function to get top of usable ram
|
||||
x86: (for 1/3 boards) text -272.0 rodata +41.0
|
||||
03: x86: Add basic cache operations
|
||||
04: x86: Permit bootstage and timer data to be used prior to relocation
|
||||
x86: (for 1/3 boards) data +16.0
|
||||
05: x86: Add an __end symbol to signal the end of the U-Boot binary
|
||||
x86: (for 1/3 boards) text +76.0
|
||||
06: x86: Rearrange the output input to remove BSS
|
||||
x86: (for 1/3 boards) bss -2140.0
|
||||
07: x86: Support relocation of FDT on start-up
|
||||
x86: + coreboot-x86
|
||||
08: x86: Add error checking to x86 relocation code
|
||||
09: x86: Adjust link device tree include file
|
||||
10: x86: Enable CONFIG_OF_CONTROL on coreboot
|
||||
|
||||
|
||||
You can see that image size only changed on x86, which is good because this
|
||||
series is not supposed to change any other board. From commit 7 onwards the
|
||||
build fails so we don't get code size numbers. The numbers are fractional
|
||||
because they are an average of all boards for that architecture. The
|
||||
intention is to allow you to quickly find image size problems introduced by
|
||||
your commits.
|
||||
|
||||
Note that the 'text' region and 'rodata' are split out. You should add the
|
||||
two together to get the total read-only size (reported as the first column
|
||||
in the output from binutil's 'size' utility).
|
||||
|
||||
A useful option is --step which lets you skip some commits. For example
|
||||
--step 2 will show the image sizes for only every 2nd commit (so it will
|
||||
compare the image sizes of the 1st, 3rd, 5th... commits). You can also use
|
||||
--step 0 which will compare only the first and last commits. This is useful
|
||||
for an overview of how your entire series affects code size.
|
||||
|
||||
You can also use -d to see a detailed size breakdown for each board. This
|
||||
list is sorted in order from largest growth to largest reduction.
|
||||
|
||||
It is possible to go a little further with the -B option (--bloat). This
|
||||
shows where U-Boot has bloted, breaking the size change down to the function
|
||||
level. Example output is below:
|
||||
|
||||
$ ./tools/buildman/buildman -b us-mem4 -sSdB
|
||||
...
|
||||
19: Roll crc32 into hash infrastructure
|
||||
arm: (for 10/10 boards) all -143.4 bss +1.2 data -4.8 rodata -48.2 text -91.6
|
||||
paz00 : all +23 bss -4 rodata -29 text +56
|
||||
u-boot: add: 1/0, grow: 3/-2 bytes: 168/-104 (64)
|
||||
function old new delta
|
||||
hash_command 80 160 +80
|
||||
crc32_wd_buf - 56 +56
|
||||
ext4fs_read_file 540 568 +28
|
||||
insert_var_value_sub 688 692 +4
|
||||
run_list_real 1996 1992 -4
|
||||
do_mem_crc 168 68 -100
|
||||
trimslice : all -9 bss +16 rodata -29 text +4
|
||||
u-boot: add: 1/0, grow: 1/-3 bytes: 136/-124 (12)
|
||||
function old new delta
|
||||
hash_command 80 160 +80
|
||||
crc32_wd_buf - 56 +56
|
||||
ext4fs_iterate_dir 672 668 -4
|
||||
ext4fs_read_file 568 548 -20
|
||||
do_mem_crc 168 68 -100
|
||||
whistler : all -9 bss +16 rodata -29 text +4
|
||||
u-boot: add: 1/0, grow: 1/-3 bytes: 136/-124 (12)
|
||||
function old new delta
|
||||
hash_command 80 160 +80
|
||||
crc32_wd_buf - 56 +56
|
||||
ext4fs_iterate_dir 672 668 -4
|
||||
ext4fs_read_file 568 548 -20
|
||||
do_mem_crc 168 68 -100
|
||||
seaboard : all -9 bss -28 rodata -29 text +48
|
||||
u-boot: add: 1/0, grow: 3/-2 bytes: 160/-104 (56)
|
||||
function old new delta
|
||||
hash_command 80 160 +80
|
||||
crc32_wd_buf - 56 +56
|
||||
ext4fs_read_file 548 568 +20
|
||||
run_list_real 1996 2000 +4
|
||||
do_nandboot 760 756 -4
|
||||
do_mem_crc 168 68 -100
|
||||
colibri_t20_iris: all -9 rodata -29 text +20
|
||||
u-boot: add: 1/0, grow: 2/-3 bytes: 140/-112 (28)
|
||||
function old new delta
|
||||
hash_command 80 160 +80
|
||||
crc32_wd_buf - 56 +56
|
||||
read_abs_bbt 204 208 +4
|
||||
do_nandboot 760 756 -4
|
||||
ext4fs_read_file 576 568 -8
|
||||
do_mem_crc 168 68 -100
|
||||
ventana : all -37 bss -12 rodata -29 text +4
|
||||
u-boot: add: 1/0, grow: 1/-3 bytes: 136/-124 (12)
|
||||
function old new delta
|
||||
hash_command 80 160 +80
|
||||
crc32_wd_buf - 56 +56
|
||||
ext4fs_iterate_dir 672 668 -4
|
||||
ext4fs_read_file 568 548 -20
|
||||
do_mem_crc 168 68 -100
|
||||
harmony : all -37 bss -16 rodata -29 text +8
|
||||
u-boot: add: 1/0, grow: 2/-3 bytes: 140/-124 (16)
|
||||
function old new delta
|
||||
hash_command 80 160 +80
|
||||
crc32_wd_buf - 56 +56
|
||||
nand_write_oob_syndrome 428 432 +4
|
||||
ext4fs_iterate_dir 672 668 -4
|
||||
ext4fs_read_file 568 548 -20
|
||||
do_mem_crc 168 68 -100
|
||||
medcom-wide : all -417 bss +28 data -16 rodata -93 text -336
|
||||
u-boot: add: 1/-1, grow: 1/-2 bytes: 88/-376 (-288)
|
||||
function old new delta
|
||||
crc32_wd_buf - 56 +56
|
||||
do_fat_read_at 2872 2904 +32
|
||||
hash_algo 16 - -16
|
||||
do_mem_crc 168 68 -100
|
||||
hash_command 420 160 -260
|
||||
tec : all -449 bss -4 data -16 rodata -93 text -336
|
||||
u-boot: add: 1/-1, grow: 1/-2 bytes: 88/-376 (-288)
|
||||
function old new delta
|
||||
crc32_wd_buf - 56 +56
|
||||
do_fat_read_at 2872 2904 +32
|
||||
hash_algo 16 - -16
|
||||
do_mem_crc 168 68 -100
|
||||
hash_command 420 160 -260
|
||||
plutux : all -481 bss +16 data -16 rodata -93 text -388
|
||||
u-boot: add: 1/-1, grow: 1/-3 bytes: 68/-408 (-340)
|
||||
function old new delta
|
||||
crc32_wd_buf - 56 +56
|
||||
do_load_serial_bin 1688 1700 +12
|
||||
hash_algo 16 - -16
|
||||
do_fat_read_at 2904 2872 -32
|
||||
do_mem_crc 168 68 -100
|
||||
hash_command 420 160 -260
|
||||
powerpc: (for 5/5 boards) all +37.4 data -3.2 rodata -41.8 text +82.4
|
||||
MPC8610HPCD : all +55 rodata -29 text +84
|
||||
u-boot: add: 1/0, grow: 0/-1 bytes: 176/-96 (80)
|
||||
function old new delta
|
||||
hash_command - 176 +176
|
||||
do_mem_crc 184 88 -96
|
||||
MPC8641HPCN : all +55 rodata -29 text +84
|
||||
u-boot: add: 1/0, grow: 0/-1 bytes: 176/-96 (80)
|
||||
function old new delta
|
||||
hash_command - 176 +176
|
||||
do_mem_crc 184 88 -96
|
||||
MPC8641HPCN_36BIT: all +55 rodata -29 text +84
|
||||
u-boot: add: 1/0, grow: 0/-1 bytes: 176/-96 (80)
|
||||
function old new delta
|
||||
hash_command - 176 +176
|
||||
do_mem_crc 184 88 -96
|
||||
sbc8641d : all +55 rodata -29 text +84
|
||||
u-boot: add: 1/0, grow: 0/-1 bytes: 176/-96 (80)
|
||||
function old new delta
|
||||
hash_command - 176 +176
|
||||
do_mem_crc 184 88 -96
|
||||
xpedite517x : all -33 data -16 rodata -93 text +76
|
||||
u-boot: add: 1/-1, grow: 0/-1 bytes: 176/-112 (64)
|
||||
function old new delta
|
||||
hash_command - 176 +176
|
||||
hash_algo 16 - -16
|
||||
do_mem_crc 184 88 -96
|
||||
...
|
||||
|
||||
|
||||
This shows that commit 19 has increased text size for arm (although only one
|
||||
board was built) and by 96 bytes for powerpc. This increase was offset in both
|
||||
cases by reductions in rodata and data/bss.
|
||||
|
||||
Shown below the summary lines is the sizes for each board. Below each board
|
||||
is the sizes for each function. This information starts with:
|
||||
|
||||
add - number of functions added / removed
|
||||
grow - number of functions which grew / shrunk
|
||||
bytes - number of bytes of code added to / removed from all functions,
|
||||
plus the total byte change in brackets
|
||||
|
||||
The change seems to be that hash_command() has increased by more than the
|
||||
do_mem_crc() function has decreased. The function sizes typically add up to
|
||||
roughly the text area size, but note that every read-only section except
|
||||
rodata is included in 'text', so the function total does not exactly
|
||||
correspond.
|
||||
|
||||
It is common when refactoring code for the rodata to decrease as the text size
|
||||
increases, and vice versa.
|
||||
|
||||
|
||||
Other options
|
||||
=============
|
||||
|
||||
Buildman has various other command line options. Try --help to see them.
|
||||
|
||||
|
||||
TODO
|
||||
====
|
||||
|
||||
This has mostly be written in my spare time as a response to my difficulties
|
||||
in testing large series of patches. Apart from tidying up there is quite a
|
||||
bit of scope for improvement. Things like better error diffs, easier access
|
||||
to log files, error display while building. Also it would be nice it buildman
|
||||
could 'hunt' for problems, perhaps by building a few boards for each arch,
|
||||
or checking commits for changed files and building only boards which use
|
||||
those files.
|
||||
|
||||
|
||||
Credits
|
||||
=======
|
||||
|
||||
Thanks to Grant Grundler <grundler@chromium.org> for his ideas for improving
|
||||
the build speed by building all commits for a board instead of the other
|
||||
way around.
|
||||
|
||||
|
||||
|
||||
Simon Glass
|
||||
sjg@chromium.org
|
||||
Halloween 2012
|
||||
Updated 12-12-12
|
||||
Updated 23-02-13
|
167
tools/buildman/board.py
Normal file
167
tools/buildman/board.py
Normal file
@ -0,0 +1,167 @@
|
||||
# Copyright (c) 2012 The Chromium OS Authors.
|
||||
#
|
||||
# See file CREDITS for list of people who contributed to this
|
||||
# project.
|
||||
#
|
||||
# This program is free software; you can redistribute it and/or
|
||||
# modify it under the terms of the GNU General Public License as
|
||||
# published by the Free Software Foundation; either version 2 of
|
||||
# the License, or (at your option) any later version.
|
||||
#
|
||||
# This program is distributed in the hope that it will be useful,
|
||||
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||
# GNU General Public License for more details.
|
||||
#
|
||||
# You should have received a copy of the GNU General Public License
|
||||
# along with this program; if not, write to the Free Software
|
||||
# Foundation, Inc., 59 Temple Place, Suite 330, Boston,
|
||||
# MA 02111-1307 USA
|
||||
#
|
||||
|
||||
class Board:
|
||||
"""A particular board that we can build"""
|
||||
def __init__(self, target, arch, cpu, board_name, vendor, soc, options):
|
||||
"""Create a new board type.
|
||||
|
||||
Args:
|
||||
target: Target name (use make <target>_config to configure)
|
||||
arch: Architecture name (e.g. arm)
|
||||
cpu: Cpu name (e.g. arm1136)
|
||||
board_name: Name of board (e.g. integrator)
|
||||
vendor: Name of vendor (e.g. armltd)
|
||||
soc: Name of SOC, or '' if none (e.g. mx31)
|
||||
options: board-specific options (e.g. integratorcp:CM1136)
|
||||
"""
|
||||
self.target = target
|
||||
self.arch = arch
|
||||
self.cpu = cpu
|
||||
self.board_name = board_name
|
||||
self.vendor = vendor
|
||||
self.soc = soc
|
||||
self.props = [self.target, self.arch, self.cpu, self.board_name,
|
||||
self.vendor, self.soc]
|
||||
self.options = options
|
||||
self.build_it = False
|
||||
|
||||
|
||||
class Boards:
|
||||
"""Manage a list of boards."""
|
||||
def __init__(self):
|
||||
# Use a simple list here, sinc OrderedDict requires Python 2.7
|
||||
self._boards = []
|
||||
|
||||
def AddBoard(self, board):
|
||||
"""Add a new board to the list.
|
||||
|
||||
The board's target member must not already exist in the board list.
|
||||
|
||||
Args:
|
||||
board: board to add
|
||||
"""
|
||||
self._boards.append(board)
|
||||
|
||||
def ReadBoards(self, fname):
|
||||
"""Read a list of boards from a board file.
|
||||
|
||||
Create a board object for each and add it to our _boards list.
|
||||
|
||||
Args:
|
||||
fname: Filename of boards.cfg file
|
||||
"""
|
||||
with open(fname, 'r') as fd:
|
||||
for line in fd:
|
||||
if line[0] == '#':
|
||||
continue
|
||||
fields = line.split()
|
||||
if not fields:
|
||||
continue
|
||||
for upto in range(len(fields)):
|
||||
if fields[upto] == '-':
|
||||
fields[upto] = ''
|
||||
while len(fields) < 7:
|
||||
fields.append('')
|
||||
|
||||
board = Board(*fields)
|
||||
self.AddBoard(board)
|
||||
|
||||
|
||||
def GetList(self):
|
||||
"""Return a list of available boards.
|
||||
|
||||
Returns:
|
||||
List of Board objects
|
||||
"""
|
||||
return self._boards
|
||||
|
||||
def GetDict(self):
|
||||
"""Build a dictionary containing all the boards.
|
||||
|
||||
Returns:
|
||||
Dictionary:
|
||||
key is board.target
|
||||
value is board
|
||||
"""
|
||||
board_dict = {}
|
||||
for board in self._boards:
|
||||
board_dict[board.target] = board
|
||||
return board_dict
|
||||
|
||||
def GetSelectedDict(self):
|
||||
"""Return a dictionary containing the selected boards
|
||||
|
||||
Returns:
|
||||
List of Board objects that are marked selected
|
||||
"""
|
||||
board_dict = {}
|
||||
for board in self._boards:
|
||||
if board.build_it:
|
||||
board_dict[board.target] = board
|
||||
return board_dict
|
||||
|
||||
def GetSelected(self):
|
||||
"""Return a list of selected boards
|
||||
|
||||
Returns:
|
||||
List of Board objects that are marked selected
|
||||
"""
|
||||
return [board for board in self._boards if board.build_it]
|
||||
|
||||
def GetSelectedNames(self):
|
||||
"""Return a list of selected boards
|
||||
|
||||
Returns:
|
||||
List of board names that are marked selected
|
||||
"""
|
||||
return [board.target for board in self._boards if board.build_it]
|
||||
|
||||
def SelectBoards(self, args):
|
||||
"""Mark boards selected based on args
|
||||
|
||||
Args:
|
||||
List of strings specifying boards to include, either named, or
|
||||
by their target, architecture, cpu, vendor or soc. If empty, all
|
||||
boards are selected.
|
||||
|
||||
Returns:
|
||||
Dictionary which holds the number of boards which were selected
|
||||
due to each argument, arranged by argument.
|
||||
"""
|
||||
result = {}
|
||||
for arg in args:
|
||||
result[arg] = 0
|
||||
result['all'] = 0
|
||||
|
||||
for board in self._boards:
|
||||
if args:
|
||||
for arg in args:
|
||||
if arg in board.props:
|
||||
if not board.build_it:
|
||||
board.build_it = True
|
||||
result[arg] += 1
|
||||
result['all'] += 1
|
||||
else:
|
||||
board.build_it = True
|
||||
result['all'] += 1
|
||||
|
||||
return result
|
60
tools/buildman/bsettings.py
Normal file
60
tools/buildman/bsettings.py
Normal file
@ -0,0 +1,60 @@
|
||||
# Copyright (c) 2012 The Chromium OS Authors.
|
||||
#
|
||||
# See file CREDITS for list of people who contributed to this
|
||||
# project.
|
||||
#
|
||||
# This program is free software; you can redistribute it and/or
|
||||
# modify it under the terms of the GNU General Public License as
|
||||
# published by the Free Software Foundation; either version 2 of
|
||||
# the License, or (at your option) any later version.
|
||||
#
|
||||
# This program is distributed in the hope that it will be useful,
|
||||
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||
# GNU General Public License for more details.
|
||||
#
|
||||
# You should have received a copy of the GNU General Public License
|
||||
# along with this program; if not, write to the Free Software
|
||||
# Foundation, Inc., 59 Temple Place, Suite 330, Boston,
|
||||
# MA 02111-1307 USA
|
||||
#
|
||||
|
||||
import ConfigParser
|
||||
import os
|
||||
|
||||
|
||||
def Setup(fname=''):
|
||||
"""Set up the buildman settings module by reading config files
|
||||
|
||||
Args:
|
||||
config_fname: Config filename to read ('' for default)
|
||||
"""
|
||||
global settings
|
||||
global config_fname
|
||||
|
||||
settings = ConfigParser.SafeConfigParser()
|
||||
config_fname = fname
|
||||
if config_fname == '':
|
||||
config_fname = '%s/.buildman' % os.getenv('HOME')
|
||||
if config_fname:
|
||||
settings.read(config_fname)
|
||||
|
||||
def GetItems(section):
|
||||
"""Get the items from a section of the config.
|
||||
|
||||
Args:
|
||||
section: name of section to retrieve
|
||||
|
||||
Returns:
|
||||
List of (name, value) tuples for the section
|
||||
"""
|
||||
try:
|
||||
return settings.items(section)
|
||||
except ConfigParser.NoSectionError as e:
|
||||
print e
|
||||
print ("Warning: No tool chains - please add a [toolchain] section "
|
||||
"to your buildman config file %s. See README for details" %
|
||||
config_fname)
|
||||
return []
|
||||
except:
|
||||
raise
|
1445
tools/buildman/builder.py
Normal file
1445
tools/buildman/builder.py
Normal file
File diff suppressed because it is too large
Load Diff
1
tools/buildman/buildman
Symbolic link
1
tools/buildman/buildman
Symbolic link
@ -0,0 +1 @@
|
||||
buildman.py
|
126
tools/buildman/buildman.py
Executable file
126
tools/buildman/buildman.py
Executable file
@ -0,0 +1,126 @@
|
||||
#!/usr/bin/python
|
||||
#
|
||||
# Copyright (c) 2012 The Chromium OS Authors.
|
||||
#
|
||||
# See file CREDITS for list of people who contributed to this
|
||||
# project.
|
||||
#
|
||||
# This program is free software; you can redistribute it and/or
|
||||
# modify it under the terms of the GNU General Public License as
|
||||
# published by the Free Software Foundation; either version 2 of
|
||||
# the License, or (at your option) any later version.
|
||||
#
|
||||
# This program is distributed in the hope that it will be useful,
|
||||
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||
# GNU General Public License for more details.
|
||||
#
|
||||
# You should have received a copy of the GNU General Public License
|
||||
# along with this program; if not, write to the Free Software
|
||||
# Foundation, Inc., 59 Temple Place, Suite 330, Boston,
|
||||
# MA 02111-1307 USA
|
||||
#
|
||||
|
||||
"""See README for more information"""
|
||||
|
||||
import multiprocessing
|
||||
from optparse import OptionParser
|
||||
import os
|
||||
import re
|
||||
import sys
|
||||
import unittest
|
||||
|
||||
# Bring in the patman libraries
|
||||
our_path = os.path.dirname(os.path.realpath(__file__))
|
||||
sys.path.append(os.path.join(our_path, '../patman'))
|
||||
|
||||
# Our modules
|
||||
import board
|
||||
import builder
|
||||
import checkpatch
|
||||
import command
|
||||
import control
|
||||
import doctest
|
||||
import gitutil
|
||||
import patchstream
|
||||
import terminal
|
||||
import toolchain
|
||||
|
||||
def RunTests():
|
||||
import test
|
||||
|
||||
sys.argv = [sys.argv[0]]
|
||||
suite = unittest.TestLoader().loadTestsFromTestCase(test.TestBuild)
|
||||
result = unittest.TestResult()
|
||||
suite.run(result)
|
||||
|
||||
# TODO: Surely we can just 'print' result?
|
||||
print result
|
||||
for test, err in result.errors:
|
||||
print err
|
||||
for test, err in result.failures:
|
||||
print err
|
||||
|
||||
|
||||
parser = OptionParser()
|
||||
parser.add_option('-b', '--branch', type='string',
|
||||
help='Branch name to build')
|
||||
parser.add_option('-B', '--bloat', dest='show_bloat',
|
||||
action='store_true', default=False,
|
||||
help='Show changes in function code size for each board')
|
||||
parser.add_option('-c', '--count', dest='count', type='int',
|
||||
default=-1, help='Run build on the top n commits')
|
||||
parser.add_option('-e', '--show_errors', action='store_true',
|
||||
default=False, help='Show errors and warnings')
|
||||
parser.add_option('-f', '--force-build', dest='force_build',
|
||||
action='store_true', default=False,
|
||||
help='Force build of boards even if already built')
|
||||
parser.add_option('-d', '--detail', dest='show_detail',
|
||||
action='store_true', default=False,
|
||||
help='Show detailed information for each board in summary')
|
||||
parser.add_option('-g', '--git', type='string',
|
||||
help='Git repo containing branch to build', default='.')
|
||||
parser.add_option('-H', '--full-help', action='store_true', dest='full_help',
|
||||
default=False, help='Display the README file')
|
||||
parser.add_option('-j', '--jobs', dest='jobs', type='int',
|
||||
default=None, help='Number of jobs to run at once (passed to make)')
|
||||
parser.add_option('-k', '--keep-outputs', action='store_true',
|
||||
default=False, help='Keep all build output files (e.g. binaries)')
|
||||
parser.add_option('--list-tool-chains', action='store_true', default=False,
|
||||
help='List available tool chains')
|
||||
parser.add_option('-n', '--dry-run', action='store_true', dest='dry_run',
|
||||
default=False, help="Do a try run (describe actions, but no nothing)")
|
||||
parser.add_option('-Q', '--quick', action='store_true',
|
||||
default=False, help='Do a rough build, with limited warning resolution')
|
||||
parser.add_option('-s', '--summary', action='store_true',
|
||||
default=False, help='Show a build summary')
|
||||
parser.add_option('-S', '--show-sizes', action='store_true',
|
||||
default=False, help='Show image size variation in summary')
|
||||
parser.add_option('--step', type='int',
|
||||
default=1, help='Only build every n commits (0=just first and last)')
|
||||
parser.add_option('-t', '--test', action='store_true', dest='test',
|
||||
default=False, help='run tests')
|
||||
parser.add_option('-T', '--threads', type='int',
|
||||
default=None, help='Number of builder threads to use')
|
||||
parser.add_option('-u', '--show_unknown', action='store_true',
|
||||
default=False, help='Show boards with unknown build result')
|
||||
|
||||
parser.usage = """buildman -b <branch> [options]
|
||||
|
||||
Build U-Boot for all commits in a branch. Use -n to do a dry run"""
|
||||
|
||||
(options, args) = parser.parse_args()
|
||||
|
||||
# Run our meagre tests
|
||||
if options.test:
|
||||
RunTests()
|
||||
elif options.full_help:
|
||||
pager = os.getenv('PAGER')
|
||||
if not pager:
|
||||
pager = 'more'
|
||||
fname = os.path.join(os.path.dirname(sys.argv[0]), 'README')
|
||||
command.Run(pager, fname)
|
||||
|
||||
# Build selected commits for selected boards
|
||||
else:
|
||||
control.DoBuildman(options, args)
|
181
tools/buildman/control.py
Normal file
181
tools/buildman/control.py
Normal file
@ -0,0 +1,181 @@
|
||||
# Copyright (c) 2013 The Chromium OS Authors.
|
||||
#
|
||||
# See file CREDITS for list of people who contributed to this
|
||||
# project.
|
||||
#
|
||||
# This program is free software; you can redistribute it and/or
|
||||
# modify it under the terms of the GNU General Public License as
|
||||
# published by the Free Software Foundation; either version 2 of
|
||||
# the License, or (at your option) any later version.
|
||||
#
|
||||
# This program is distributed in the hope that it will be useful,
|
||||
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||
# GNU General Public License for more details.
|
||||
#
|
||||
# You should have received a copy of the GNU General Public License
|
||||
# along with this program; if not, write to the Free Software
|
||||
# Foundation, Inc., 59 Temple Place, Suite 330, Boston,
|
||||
# MA 02111-1307 USA
|
||||
#
|
||||
|
||||
import multiprocessing
|
||||
import os
|
||||
import sys
|
||||
|
||||
import board
|
||||
import bsettings
|
||||
from builder import Builder
|
||||
import gitutil
|
||||
import patchstream
|
||||
import terminal
|
||||
import toolchain
|
||||
|
||||
def GetPlural(count):
|
||||
"""Returns a plural 's' if count is not 1"""
|
||||
return 's' if count != 1 else ''
|
||||
|
||||
def GetActionSummary(is_summary, count, selected, options):
|
||||
"""Return a string summarising the intended action.
|
||||
|
||||
Returns:
|
||||
Summary string.
|
||||
"""
|
||||
count = (count + options.step - 1) / options.step
|
||||
str = '%s %d commit%s for %d boards' % (
|
||||
'Summary of' if is_summary else 'Building', count, GetPlural(count),
|
||||
len(selected))
|
||||
str += ' (%d thread%s, %d job%s per thread)' % (options.threads,
|
||||
GetPlural(options.threads), options.jobs, GetPlural(options.jobs))
|
||||
return str
|
||||
|
||||
def ShowActions(series, why_selected, boards_selected, builder, options):
|
||||
"""Display a list of actions that we would take, if not a dry run.
|
||||
|
||||
Args:
|
||||
series: Series object
|
||||
why_selected: Dictionary where each key is a buildman argument
|
||||
provided by the user, and the value is the boards brought
|
||||
in by that argument. For example, 'arm' might bring in
|
||||
400 boards, so in this case the key would be 'arm' and
|
||||
the value would be a list of board names.
|
||||
boards_selected: Dict of selected boards, key is target name,
|
||||
value is Board object
|
||||
builder: The builder that will be used to build the commits
|
||||
options: Command line options object
|
||||
"""
|
||||
col = terminal.Color()
|
||||
print 'Dry run, so not doing much. But I would do this:'
|
||||
print
|
||||
print GetActionSummary(False, len(series.commits), boards_selected,
|
||||
options)
|
||||
print 'Build directory: %s' % builder.base_dir
|
||||
for upto in range(0, len(series.commits), options.step):
|
||||
commit = series.commits[upto]
|
||||
print ' ', col.Color(col.YELLOW, commit.hash, bright=False),
|
||||
print commit.subject
|
||||
print
|
||||
for arg in why_selected:
|
||||
if arg != 'all':
|
||||
print arg, ': %d boards' % why_selected[arg]
|
||||
print ('Total boards to build for each commit: %d\n' %
|
||||
why_selected['all'])
|
||||
|
||||
def DoBuildman(options, args):
|
||||
"""The main control code for buildman
|
||||
|
||||
Args:
|
||||
options: Command line options object
|
||||
args: Command line arguments (list of strings)
|
||||
"""
|
||||
gitutil.Setup()
|
||||
|
||||
bsettings.Setup()
|
||||
options.git_dir = os.path.join(options.git, '.git')
|
||||
|
||||
toolchains = toolchain.Toolchains()
|
||||
toolchains.Scan(options.list_tool_chains)
|
||||
if options.list_tool_chains:
|
||||
toolchains.List()
|
||||
print
|
||||
return
|
||||
|
||||
# Work out how many commits to build. We want to build everything on the
|
||||
# branch. We also build the upstream commit as a control so we can see
|
||||
# problems introduced by the first commit on the branch.
|
||||
col = terminal.Color()
|
||||
count = options.count
|
||||
if count == -1:
|
||||
if not options.branch:
|
||||
str = 'Please use -b to specify a branch to build'
|
||||
print col.Color(col.RED, str)
|
||||
sys.exit(1)
|
||||
count = gitutil.CountCommitsInBranch(options.git_dir, options.branch)
|
||||
count += 1 # Build upstream commit also
|
||||
|
||||
if not count:
|
||||
str = ("No commits found to process in branch '%s': "
|
||||
"set branch's upstream or use -c flag" % options.branch)
|
||||
print col.Color(col.RED, str)
|
||||
sys.exit(1)
|
||||
|
||||
# Work out what subset of the boards we are building
|
||||
boards = board.Boards()
|
||||
boards.ReadBoards(os.path.join(options.git, 'boards.cfg'))
|
||||
why_selected = boards.SelectBoards(args)
|
||||
selected = boards.GetSelected()
|
||||
if not len(selected):
|
||||
print col.Color(col.RED, 'No matching boards found')
|
||||
sys.exit(1)
|
||||
|
||||
# Read the metadata from the commits. First look at the upstream commit,
|
||||
# then the ones in the branch. We would like to do something like
|
||||
# upstream/master~..branch but that isn't possible if upstream/master is
|
||||
# a merge commit (it will list all the commits that form part of the
|
||||
# merge)
|
||||
range_expr = gitutil.GetRangeInBranch(options.git_dir, options.branch)
|
||||
upstream_commit = gitutil.GetUpstream(options.git_dir, options.branch)
|
||||
series = patchstream.GetMetaDataForList(upstream_commit, options.git_dir,
|
||||
1)
|
||||
series = patchstream.GetMetaDataForList(range_expr, options.git_dir, None,
|
||||
series)
|
||||
|
||||
# By default we have one thread per CPU. But if there are not enough jobs
|
||||
# we can have fewer threads and use a high '-j' value for make.
|
||||
if not options.threads:
|
||||
options.threads = min(multiprocessing.cpu_count(), len(selected))
|
||||
if not options.jobs:
|
||||
options.jobs = max(1, (multiprocessing.cpu_count() +
|
||||
len(selected) - 1) / len(selected))
|
||||
|
||||
if not options.step:
|
||||
options.step = len(series.commits) - 1
|
||||
|
||||
# Create a new builder with the selected options
|
||||
output_dir = os.path.join('..', options.branch)
|
||||
builder = Builder(toolchains, output_dir, options.git_dir,
|
||||
options.threads, options.jobs, checkout=True,
|
||||
show_unknown=options.show_unknown, step=options.step)
|
||||
builder.force_config_on_failure = not options.quick
|
||||
|
||||
# For a dry run, just show our actions as a sanity check
|
||||
if options.dry_run:
|
||||
ShowActions(series, why_selected, selected, builder, options)
|
||||
else:
|
||||
builder.force_build = options.force_build
|
||||
|
||||
# Work out which boards to build
|
||||
board_selected = boards.GetSelectedDict()
|
||||
|
||||
print GetActionSummary(options.summary, count, board_selected, options)
|
||||
|
||||
if options.summary:
|
||||
# We can't show function sizes without board details at present
|
||||
if options.show_bloat:
|
||||
options.show_detail = True
|
||||
builder.ShowSummary(series.commits, board_selected,
|
||||
options.show_errors, options.show_sizes,
|
||||
options.show_detail, options.show_bloat)
|
||||
else:
|
||||
builder.BuildBoards(series.commits, board_selected,
|
||||
options.show_errors, options.keep_outputs)
|
185
tools/buildman/test.py
Normal file
185
tools/buildman/test.py
Normal file
@ -0,0 +1,185 @@
|
||||
#
|
||||
# Copyright (c) 2012 The Chromium OS Authors.
|
||||
#
|
||||
# See file CREDITS for list of people who contributed to this
|
||||
# project.
|
||||
#
|
||||
# This program is free software; you can redistribute it and/or
|
||||
# modify it under the terms of the GNU General Public License as
|
||||
# published by the Free Software Foundation; either version 2 of
|
||||
# the License, or (at your option) any later version.
|
||||
#
|
||||
# This program is distributed in the hope that it will be useful,
|
||||
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||
# GNU General Public License for more details.
|
||||
#
|
||||
# You should have received a copy of the GNU General Public License
|
||||
# along with this program; if not, write to the Free Software
|
||||
# Foundation, Inc., 59 Temple Place, Suite 330, Boston,
|
||||
# MA 02111-1307 USA
|
||||
#
|
||||
|
||||
import os
|
||||
import shutil
|
||||
import sys
|
||||
import tempfile
|
||||
import time
|
||||
import unittest
|
||||
|
||||
# Bring in the patman libraries
|
||||
our_path = os.path.dirname(os.path.realpath(__file__))
|
||||
sys.path.append(os.path.join(our_path, '../patman'))
|
||||
|
||||
import board
|
||||
import bsettings
|
||||
import builder
|
||||
import control
|
||||
import command
|
||||
import commit
|
||||
import toolchain
|
||||
|
||||
errors = [
|
||||
'''main.c: In function 'main_loop':
|
||||
main.c:260:6: warning: unused variable 'joe' [-Wunused-variable]
|
||||
''',
|
||||
'''main.c: In function 'main_loop':
|
||||
main.c:295:2: error: 'fred' undeclared (first use in this function)
|
||||
main.c:295:2: note: each undeclared identifier is reported only once for each function it appears in
|
||||
make[1]: *** [main.o] Error 1
|
||||
make: *** [common/libcommon.o] Error 2
|
||||
Make failed
|
||||
''',
|
||||
'''main.c: In function 'main_loop':
|
||||
main.c:280:6: warning: unused variable 'mary' [-Wunused-variable]
|
||||
''',
|
||||
'''powerpc-linux-ld: warning: dot moved backwards before `.bss'
|
||||
powerpc-linux-ld: warning: dot moved backwards before `.bss'
|
||||
powerpc-linux-ld: u-boot: section .text lma 0xfffc0000 overlaps previous sections
|
||||
powerpc-linux-ld: u-boot: section .rodata lma 0xfffef3ec overlaps previous sections
|
||||
powerpc-linux-ld: u-boot: section .reloc lma 0xffffa400 overlaps previous sections
|
||||
powerpc-linux-ld: u-boot: section .data lma 0xffffcd38 overlaps previous sections
|
||||
powerpc-linux-ld: u-boot: section .u_boot_cmd lma 0xffffeb40 overlaps previous sections
|
||||
powerpc-linux-ld: u-boot: section .bootpg lma 0xfffff198 overlaps previous sections
|
||||
'''
|
||||
]
|
||||
|
||||
|
||||
# hash, subject, return code, list of errors/warnings
|
||||
commits = [
|
||||
['1234', 'upstream/master, ok', 0, []],
|
||||
['5678', 'Second commit, a warning', 0, errors[0:1]],
|
||||
['9012', 'Third commit, error', 1, errors[0:2]],
|
||||
['3456', 'Fourth commit, warning', 0, [errors[0], errors[2]]],
|
||||
['7890', 'Fifth commit, link errors', 1, [errors[0], errors[3]]],
|
||||
['abcd', 'Sixth commit, fixes all errors', 0, []]
|
||||
]
|
||||
|
||||
boards = [
|
||||
['board0', 'arm', 'armv7', 'ARM Board 1', 'Tester', '', ''],
|
||||
['board1', 'arm', 'armv7', 'ARM Board 2', 'Tester', '', ''],
|
||||
['board2', 'powerpc', 'powerpc', 'PowerPC board 1', 'Tester', '', ''],
|
||||
['board3', 'powerpc', 'mpc5xx', 'PowerPC board 2', 'Tester', '', ''],
|
||||
['board4', 'sandbox', 'sandbox', 'Sandbox board', 'Tester', '', '']
|
||||
]
|
||||
|
||||
class Options:
|
||||
"""Class that holds build options"""
|
||||
pass
|
||||
|
||||
class TestBuild(unittest.TestCase):
|
||||
"""Test buildman
|
||||
|
||||
TODO: Write tests for the rest of the functionality
|
||||
"""
|
||||
def setUp(self):
|
||||
# Set up commits to build
|
||||
self.commits = []
|
||||
sequence = 0
|
||||
for commit_info in commits:
|
||||
comm = commit.Commit(commit_info[0])
|
||||
comm.subject = commit_info[1]
|
||||
comm.return_code = commit_info[2]
|
||||
comm.error_list = commit_info[3]
|
||||
comm.sequence = sequence
|
||||
sequence += 1
|
||||
self.commits.append(comm)
|
||||
|
||||
# Set up boards to build
|
||||
self.boards = board.Boards()
|
||||
for brd in boards:
|
||||
self.boards.AddBoard(board.Board(*brd))
|
||||
self.boards.SelectBoards([])
|
||||
|
||||
# Set up the toolchains
|
||||
bsettings.Setup()
|
||||
self.toolchains = toolchain.Toolchains()
|
||||
self.toolchains.Add('arm-linux-gcc', test=False)
|
||||
self.toolchains.Add('sparc-linux-gcc', test=False)
|
||||
self.toolchains.Add('powerpc-linux-gcc', test=False)
|
||||
self.toolchains.Add('gcc', test=False)
|
||||
|
||||
def Make(self, commit, brd, stage, *args, **kwargs):
|
||||
result = command.CommandResult()
|
||||
boardnum = int(brd.target[-1])
|
||||
result.return_code = 0
|
||||
result.stderr = ''
|
||||
result.stdout = ('This is the test output for board %s, commit %s' %
|
||||
(brd.target, commit.hash))
|
||||
if boardnum >= 1 and boardnum >= commit.sequence:
|
||||
result.return_code = commit.return_code
|
||||
result.stderr = ''.join(commit.error_list)
|
||||
if stage == 'build':
|
||||
target_dir = None
|
||||
for arg in args:
|
||||
if arg.startswith('O='):
|
||||
target_dir = arg[2:]
|
||||
|
||||
if not os.path.isdir(target_dir):
|
||||
os.mkdir(target_dir)
|
||||
#time.sleep(.2 + boardnum * .2)
|
||||
|
||||
result.combined = result.stdout + result.stderr
|
||||
return result
|
||||
|
||||
def testBasic(self):
|
||||
"""Test basic builder operation"""
|
||||
output_dir = tempfile.mkdtemp()
|
||||
if not os.path.isdir(output_dir):
|
||||
os.mkdir(output_dir)
|
||||
build = builder.Builder(self.toolchains, output_dir, None, 1, 2,
|
||||
checkout=False, show_unknown=False)
|
||||
build.do_make = self.Make
|
||||
board_selected = self.boards.GetSelectedDict()
|
||||
|
||||
#build.BuildCommits(self.commits, board_selected, False)
|
||||
build.BuildBoards(self.commits, board_selected, False, False)
|
||||
build.ShowSummary(self.commits, board_selected, True, False,
|
||||
False, False)
|
||||
|
||||
def _testGit(self):
|
||||
"""Test basic builder operation by building a branch"""
|
||||
base_dir = tempfile.mkdtemp()
|
||||
if not os.path.isdir(base_dir):
|
||||
os.mkdir(base_dir)
|
||||
options = Options()
|
||||
options.git = os.getcwd()
|
||||
options.summary = False
|
||||
options.jobs = None
|
||||
options.dry_run = False
|
||||
#options.git = os.path.join(base_dir, 'repo')
|
||||
options.branch = 'test-buildman'
|
||||
options.force_build = False
|
||||
options.list_tool_chains = False
|
||||
options.count = -1
|
||||
options.git_dir = None
|
||||
options.threads = None
|
||||
options.show_unknown = False
|
||||
options.quick = False
|
||||
options.show_errors = False
|
||||
options.keep_outputs = False
|
||||
args = ['tegra20']
|
||||
control.DoBuildman(options, args)
|
||||
|
||||
if __name__ == "__main__":
|
||||
unittest.main()
|
185
tools/buildman/toolchain.py
Normal file
185
tools/buildman/toolchain.py
Normal file
@ -0,0 +1,185 @@
|
||||
# Copyright (c) 2012 The Chromium OS Authors.
|
||||
#
|
||||
# See file CREDITS for list of people who contributed to this
|
||||
# project.
|
||||
#
|
||||
# This program is free software; you can redistribute it and/or
|
||||
# modify it under the terms of the GNU General Public License as
|
||||
# published by the Free Software Foundation; either version 2 of
|
||||
# the License, or (at your option) any later version.
|
||||
#
|
||||
# This program is distributed in the hope that it will be useful,
|
||||
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||
# GNU General Public License for more details.
|
||||
#
|
||||
# You should have received a copy of the GNU General Public License
|
||||
# along with this program; if not, write to the Free Software
|
||||
# Foundation, Inc., 59 Temple Place, Suite 330, Boston,
|
||||
# MA 02111-1307 USA
|
||||
#
|
||||
|
||||
import glob
|
||||
import os
|
||||
|
||||
import bsettings
|
||||
import command
|
||||
|
||||
class Toolchain:
|
||||
"""A single toolchain
|
||||
|
||||
Public members:
|
||||
gcc: Full path to C compiler
|
||||
path: Directory path containing C compiler
|
||||
cross: Cross compile string, e.g. 'arm-linux-'
|
||||
arch: Architecture of toolchain as determined from the first
|
||||
component of the filename. E.g. arm-linux-gcc becomes arm
|
||||
"""
|
||||
|
||||
def __init__(self, fname, test, verbose=False):
|
||||
"""Create a new toolchain object.
|
||||
|
||||
Args:
|
||||
fname: Filename of the gcc component
|
||||
test: True to run the toolchain to test it
|
||||
"""
|
||||
self.gcc = fname
|
||||
self.path = os.path.dirname(fname)
|
||||
self.cross = os.path.basename(fname)[:-3]
|
||||
pos = self.cross.find('-')
|
||||
self.arch = self.cross[:pos] if pos != -1 else 'sandbox'
|
||||
|
||||
env = self.MakeEnvironment()
|
||||
|
||||
# As a basic sanity check, run the C compiler with --version
|
||||
cmd = [fname, '--version']
|
||||
if test:
|
||||
result = command.RunPipe([cmd], capture=True, env=env)
|
||||
self.ok = result.return_code == 0
|
||||
if verbose:
|
||||
print 'Tool chain test: ',
|
||||
if self.ok:
|
||||
print 'OK'
|
||||
else:
|
||||
print 'BAD'
|
||||
print 'Command: ', cmd
|
||||
print result.stdout
|
||||
print result.stderr
|
||||
else:
|
||||
self.ok = True
|
||||
self.priority = self.GetPriority(fname)
|
||||
|
||||
def GetPriority(self, fname):
|
||||
"""Return the priority of the toolchain.
|
||||
|
||||
Toolchains are ranked according to their suitability by their
|
||||
filename prefix.
|
||||
|
||||
Args:
|
||||
fname: Filename of toolchain
|
||||
Returns:
|
||||
Priority of toolchain, 0=highest, 20=lowest.
|
||||
"""
|
||||
priority_list = ['-elf', '-unknown-linux-gnu', '-linux', '-elf',
|
||||
'-none-linux-gnueabi', '-uclinux', '-none-eabi',
|
||||
'-gentoo-linux-gnu', '-linux-gnueabi', '-le-linux', '-uclinux']
|
||||
for prio in range(len(priority_list)):
|
||||
if priority_list[prio] in fname:
|
||||
return prio
|
||||
return prio
|
||||
|
||||
def MakeEnvironment(self):
|
||||
"""Returns an environment for using the toolchain.
|
||||
|
||||
Thie takes the current environment, adds CROSS_COMPILE and
|
||||
augments PATH so that the toolchain will operate correctly.
|
||||
"""
|
||||
env = dict(os.environ)
|
||||
env['CROSS_COMPILE'] = self.cross
|
||||
env['PATH'] += (':' + self.path)
|
||||
return env
|
||||
|
||||
|
||||
class Toolchains:
|
||||
"""Manage a list of toolchains for building U-Boot
|
||||
|
||||
We select one toolchain for each architecture type
|
||||
|
||||
Public members:
|
||||
toolchains: Dict of Toolchain objects, keyed by architecture name
|
||||
paths: List of paths to check for toolchains (may contain wildcards)
|
||||
"""
|
||||
|
||||
def __init__(self):
|
||||
self.toolchains = {}
|
||||
self.paths = []
|
||||
for name, value in bsettings.GetItems('toolchain'):
|
||||
if '*' in value:
|
||||
self.paths += glob.glob(value)
|
||||
else:
|
||||
self.paths.append(value)
|
||||
|
||||
|
||||
def Add(self, fname, test=True, verbose=False):
|
||||
"""Add a toolchain to our list
|
||||
|
||||
We select the given toolchain as our preferred one for its
|
||||
architecture if it is a higher priority than the others.
|
||||
|
||||
Args:
|
||||
fname: Filename of toolchain's gcc driver
|
||||
test: True to run the toolchain to test it
|
||||
"""
|
||||
toolchain = Toolchain(fname, test, verbose)
|
||||
add_it = toolchain.ok
|
||||
if toolchain.arch in self.toolchains:
|
||||
add_it = (toolchain.priority <
|
||||
self.toolchains[toolchain.arch].priority)
|
||||
if add_it:
|
||||
self.toolchains[toolchain.arch] = toolchain
|
||||
|
||||
def Scan(self, verbose):
|
||||
"""Scan for available toolchains and select the best for each arch.
|
||||
|
||||
We look for all the toolchains we can file, figure out the
|
||||
architecture for each, and whether it works. Then we select the
|
||||
highest priority toolchain for each arch.
|
||||
|
||||
Args:
|
||||
verbose: True to print out progress information
|
||||
"""
|
||||
if verbose: print 'Scanning for tool chains'
|
||||
for path in self.paths:
|
||||
if verbose: print " - scanning path '%s'" % path
|
||||
for subdir in ['.', 'bin', 'usr/bin']:
|
||||
dirname = os.path.join(path, subdir)
|
||||
if verbose: print " - looking in '%s'" % dirname
|
||||
for fname in glob.glob(dirname + '/*gcc'):
|
||||
if verbose: print " - found '%s'" % fname
|
||||
self.Add(fname, True, verbose)
|
||||
|
||||
def List(self):
|
||||
"""List out the selected toolchains for each architecture"""
|
||||
print 'List of available toolchains (%d):' % len(self.toolchains)
|
||||
if len(self.toolchains):
|
||||
for key, value in sorted(self.toolchains.iteritems()):
|
||||
print '%-10s: %s' % (key, value.gcc)
|
||||
else:
|
||||
print 'None'
|
||||
|
||||
def Select(self, arch):
|
||||
"""Returns the toolchain for a given architecture
|
||||
|
||||
Args:
|
||||
args: Name of architecture (e.g. 'arm', 'ppc_8xx')
|
||||
|
||||
returns:
|
||||
toolchain object, or None if none found
|
||||
"""
|
||||
for name, value in bsettings.GetItems('toolchain-alias'):
|
||||
if arch == name:
|
||||
arch = value
|
||||
|
||||
if not arch in self.toolchains:
|
||||
raise ValueError, ("No tool chain found for arch '%s'" % arch)
|
||||
return self.toolchains[arch]
|
Loading…
Reference in New Issue
Block a user