[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/5] tests/uefi-test-tools: add build scripts
From: |
Laszlo Ersek |
Subject: |
Re: [Qemu-devel] [PATCH 4/5] tests/uefi-test-tools: add build scripts |
Date: |
Thu, 24 Jan 2019 18:05:33 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 |
On 01/23/19 17:13, Laszlo Ersek wrote:
> FWIW, I've asked
>
> - both on edk2-devel, about adding .NOTPARALLEL to the generated inner
> makefiles:
>
> "parallelism in the module-level, generated GNUmakefile's"
> https://lists.01.org/pipermail/edk2-devel/2019-January/035463.html
>
> - and on help-make, about safely filtering the job-related options from
> MAKEFLAGS, between the outer and inner "make" processes:
>
> "filtering job options from MAKEFLAGS, manually"
> http://lists.gnu.org/archive/html/help-make/2019-01/msg00003.html
Results:
- The "build" utility in BaseTools generates such GNUMakefiles that
dependencies between targets are fully described. This is an explicit
goal and if we notice any missing deps, that's a bug to be reported and
fixed. As a consequence, we can expect the recipes in those GNUMakefiles
to be fully parallelizable. Therefore we need not add .NOTPARALLEL to
them (in the edk2 project), or filter MAKEFLAGS, or append "-j1" to
MAKEFLAGS.
- The .NOTPARALLEL target ignores any pre-requisites it gets;
.NOTPARALLEL applies to the entire current invocation of make. Therefore
we just need to stick a bare .NOTPARALLEL into
"tests/uefi-test-tools/Makefile", to keep "build" itself isolated.
I'm going to post v2 with this one addition (i.e., .NOTPARALLEL).
Thanks,
Laszlo
- Re: [Qemu-devel] [PATCH 2/5] roms: build the EfiRom utility from the roms/edk2 submodule, (continued)
[Qemu-devel] [PATCH 4/5] tests/uefi-test-tools: add build scripts, Laszlo Ersek, 2019/01/18
[Qemu-devel] [PATCH 3/5] tests: introduce "uefi-test-tools" with the BiosTablesTest UEFI app, Laszlo Ersek, 2019/01/18
[Qemu-devel] [PATCH 5/5] tests/data: introduce "uefi-boot-images" with the "bios-tables-test" ISOs, Laszlo Ersek, 2019/01/18