[coreboot-gerrit] New patch to review for coreboot: util/xcompile: parallelize compiler checks
Patrick Georgi (pgeorgi@google.com)
gerrit at coreboot.org
Mon Jan 30 15:32:22 CET 2017
Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18262
-gerrit
commit fdb63e7cf38c114ced67a6f3d03fbeb442376876
Author: Patrick Georgi <pgeorgi at google.com>
Date: Mon Jan 30 15:27:35 2017 +0100
util/xcompile: parallelize compiler checks
Speed up the execution of this script from ~6 seconds to ~1 on my
system.
There are some changes to its output, but they're actually _more_
correct: so far, architectures without compiler support kept compiler
options for architectures that ran successfully earlier.
Change-Id: I0532ea2178fbedb114a75cfd5ba39301e534e742
Signed-off-by: Patrick Georgi <pgeorgi at google.com>
---
util/xcompile/xcompile | 17 +++++++++++++++--
1 file changed, 15 insertions(+), 2 deletions(-)
diff --git a/util/xcompile/xcompile b/util/xcompile/xcompile
index 90220f6..ec295a9 100755
--- a/util/xcompile/xcompile
+++ b/util/xcompile/xcompile
@@ -58,7 +58,7 @@ clean_up() {
# Create temporary file(s).
TMPFILE="$(mktemp /tmp/temp.XXXXXX 2>/dev/null || echo /tmp/temp.coreboot.$RANDOM)"
-touch "$TMPFILE"
+rm -f "$TMPFILE"
trap clean_up EXIT
@@ -438,11 +438,24 @@ test_architecture() {
fi
}
-# This loops over all supported architectures.
+OUT="$(mktemp /tmp/temp.XXXXXX 2>/dev/null || echo /tmp/temp.coreboot.$RANDOM)"
+rm -f $OUT
+
for architecture in $SUPPORTED_ARCHITECTURES; do
+ (
+ TMPFILE="$(mktemp /tmp/temp.XXXXXX 2>/dev/null || echo /tmp/temp.coreboot.$RANDOM)"
+ touch $TMPFILE
test_architecture "$architecture"
detect_special_flags "$architecture"
detect_compiler_runtime "$architecture"
report_arch_toolchain
+ clean_up
+ ) > $OUT.$architecture &
+done
+wait
+
+for architecture in $SUPPORTED_ARCHITECTURES; do
+ cat $OUT.$architecture
+ rm -f $OUT.$architecture
done
echo XCOMPILE_COMPLETE:=1
More information about the coreboot-gerrit
mailing list