[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 07/10] roms: build edk2 firmware binaries and va
From: |
Laszlo Ersek |
Subject: |
Re: [Qemu-devel] [PATCH 07/10] roms: build edk2 firmware binaries and variable store templates |
Date: |
Tue, 12 Mar 2019 16:25:12 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 |
On 03/10/19 12:26, Philippe Mathieu-Daudé wrote:
> Hi Laszlo,
>
> On 3/9/19 1:48 AM, Laszlo Ersek wrote:
>> Add the "efi" target to "Makefile".
>>
>> Introduce "Makefile.edk2" for building and cleaning the firmware images
>> and varstore templates.
>>
>> Collect the common bits from the recipes in the helper script
>> "edk2-build.sh".
>>
>> Signed-off-by: Laszlo Ersek <address@hidden>
>> ---
>> roms/Makefile | 5 +
>> roms/Makefile.edk2 | 138 ++++++++++++++++++++
>> roms/edk2-build.sh | 55 ++++++++
>> 3 files changed, 198 insertions(+)
>>
>> diff --git a/roms/Makefile b/roms/Makefile
>> index 2e83ececa25a..054b432834ba 100644
>> --- a/roms/Makefile
>> +++ b/roms/Makefile
>> @@ -61,6 +61,7 @@ default:
>> @echo " skiboot -- update skiboot.lid"
>> @echo " u-boot.e500 -- update u-boot.e500"
>> @echo " u-boot.sam460 -- update u-boot.sam460"
>> + @echo " efi -- update UEFI (edk2) platform firmware"
>>
>> bios: build-seabios-config-seabios-128k build-seabios-config-seabios-256k
>> cp seabios/builds/seabios-128k/bios.bin ../pc-bios/bios.bin
>> @@ -143,6 +144,9 @@ skiboot:
>> $(MAKE) -C skiboot CROSS=$(powerpc64_cross_prefix)
>> cp skiboot/skiboot.lid ../pc-bios/skiboot.lid
>>
>> +efi: edk2-basetools
>> + $(MAKE) -f Makefile.edk2
>> +
>> clean:
>> rm -rf seabios/.config seabios/out seabios/builds
>> $(MAKE) -C sgabios clean
>> @@ -153,3 +157,4 @@ clean:
>> rm -rf u-boot/build.e500
>> $(MAKE) -C u-boot-sam460ex distclean
>> $(MAKE) -C skiboot clean
>> + $(MAKE) -f Makefile.edk2 clean
>> diff --git a/roms/Makefile.edk2 b/roms/Makefile.edk2
>> new file mode 100644
>> index 000000000000..ad6fff044cd6
>> --- /dev/null
>> +++ b/roms/Makefile.edk2
>> @@ -0,0 +1,138 @@
>> +# Makefile for building firmware binaries and variable store templates for a
>> +# number of virtual platforms in edk2.
>> +#
>> +# Copyright (C) 2019, Red Hat, Inc.
>> +#
>> +# This program and the accompanying materials are licensed and made
>> available
>> +# under the terms and conditions of the BSD License that accompanies this
>> +# distribution. The full text of the license may be found at
>> +# <http://opensource.org/licenses/bsd-license.php>.
>> +#
>> +# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
>> WITHOUT
>> +# WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
>> +
>> +toolchain = $(shell source ./edk2-funcs.sh && qemu_edk2_get_toolchain $(1))
>> +
>> +licenses := \
>> + edk2/License.txt \
>> + edk2/OvmfPkg/License.txt \
>> + edk2/CryptoPkg/Library/OpensslLib/openssl/LICENSE
>> +
>> +# The "edk2-arm-vars.fd" varstore template is suitable for aarch64 as well.
>> +# Similarly, the "edk2-i386-vars.fd" varstore template is suitable for
>> x86_64
>> +# as well, independently of "secure" too.
>> +all: \
>> + ../pc-bios/edk2-aarch64-code.fd \
>> + ../pc-bios/edk2-arm-code.fd \
>> + ../pc-bios/edk2-i386-code.fd \
>> + ../pc-bios/edk2-i386-secure-code.fd \
>> + ../pc-bios/edk2-x86_64-code.fd \
>> + ../pc-bios/edk2-x86_64-secure-code.fd \
>> + \
>> + ../pc-bios/edk2-arm-vars.fd \
>> + ../pc-bios/edk2-i386-vars.fd \
>> + \
>> + ../pc-bios/edk2-licenses.txt
>> +
>> +submodules:
>> + cd edk2 && git submodule update --init --force
>> +
>> +# See notes on the ".NOTPARALLEL" target and the "+" indicator in
>> +# "tests/uefi-test-tools/Makefile".
>> +.NOTPARALLEL:
>> +
>> +../pc-bios/edk2-aarch64-code.fd: submodules
>> + +./edk2-build.sh \
>> + aarch64 \
>> + --arch=AARCH64 \
>> + --platform=ArmVirtPkg/ArmVirtQemu.dsc \
>> + -D NETWORK_IP6_ENABLE \
>> + -D HTTP_BOOT_ENABLE
>> + cp edk2/Build/ArmVirtQemu-AARCH64/DEBUG_$(call
>> toolchain,aarch64)/FV/QEMU_EFI.fd \
>> + $@
>> + truncate --size=64M $@
>> +
>> +../pc-bios/edk2-arm-code.fd: submodules
>> + +./edk2-build.sh \
>> + arm \
>> + --arch=ARM \
>> + --platform=ArmVirtPkg/ArmVirtQemu.dsc \
>> + -D NETWORK_IP6_ENABLE \
>> + -D HTTP_BOOT_ENABLE
>> + cp edk2/Build/ArmVirtQemu-ARM/DEBUG_$(call
>> toolchain,arm)/FV/QEMU_EFI.fd \
>> + $@
>> + truncate --size=64M $@
>> +
>> +../pc-bios/edk2-i386-code.fd: submodules
>> + +./edk2-build.sh \
>> + i386 \
>> + --arch=IA32 \
>> + --platform=OvmfPkg/OvmfPkgIa32.dsc \
>> + -D NETWORK_IP6_ENABLE \
>> + -D HTTP_BOOT_ENABLE \
>> + -D TLS_ENABLE \
>> + -D TPM2_ENABLE \
>> + -D TPM2_CONFIG_ENABLE
>> + cp edk2/Build/OvmfIa32/DEBUG_$(call toolchain,i386)/FV/OVMF_CODE.fd $@
>> +
>> +../pc-bios/edk2-i386-secure-code.fd: submodules
>> + +./edk2-build.sh \
>> + i386 \
>> + --arch=IA32 \
>> + --platform=OvmfPkg/OvmfPkgIa32.dsc \
>> + -D NETWORK_IP6_ENABLE \
>> + -D HTTP_BOOT_ENABLE \
>> + -D TLS_ENABLE \
>> + -D TPM2_ENABLE \
>> + -D TPM2_CONFIG_ENABLE \
>> + -D SECURE_BOOT_ENABLE \
>> + -D SMM_REQUIRE
>> + cp edk2/Build/OvmfIa32/DEBUG_$(call toolchain,i386)/FV/OVMF_CODE.fd $@
>> +
>> +../pc-bios/edk2-x86_64-code.fd: submodules
>> + +./edk2-build.sh \
>> + x86_64 \
>> + --arch=X64 \
>> + --platform=OvmfPkg/OvmfPkgX64.dsc \
>> + -D NETWORK_IP6_ENABLE \
>> + -D HTTP_BOOT_ENABLE \
>> + -D TLS_ENABLE \
>> + -D TPM2_ENABLE \
>> + -D TPM2_CONFIG_ENABLE
>> + cp edk2/Build/OvmfX64/DEBUG_$(call toolchain,x86_64)/FV/OVMF_CODE.fd $@
>> +
>> +../pc-bios/edk2-x86_64-secure-code.fd: submodules
>> + +./edk2-build.sh \
>> + x86_64 \
>> + --arch=IA32 \
>> + --arch=X64 \
>> + --platform=OvmfPkg/OvmfPkgIa32X64.dsc \
>> + -D NETWORK_IP6_ENABLE \
>> + -D HTTP_BOOT_ENABLE \
>> + -D TLS_ENABLE \
>> + -D TPM2_ENABLE \
>> + -D TPM2_CONFIG_ENABLE \
>> + -D SECURE_BOOT_ENABLE \
>> + -D SMM_REQUIRE
>> + cp edk2/Build/Ovmf3264/DEBUG_$(call toolchain,x86_64)/FV/OVMF_CODE.fd $@
>
> Do you mind adding a $EDK2_BUILD_OPTIONS variable to optionally add
> arguments from the environment?
Hm. Not really a fan of easily hooking random stuff into these command
lines.
> I figured it is useful for CI
> integration to quiet the build output, else the CI stdout limit is
> reached quickly.
That sounds really bad. "CI stdout limit" is something that should not
exist, or should be flexibly configurable. Where else do they want us to
read the build log, for example?
Anyway, the "build" utility can take a "--log" option. IMO it does not
belong in "Makefile.edk2", but in the "edk2-build.sh" script.
> I'll suggest a patch you can amend or improve :)
I'm extremely starved for cycles on this. Please let's stick to the
minimum acceptable content here, and work with incremental patches on
top (maybe in the next development cycle).
Thanks,
Laszlo
>
>> +
>> +../pc-bios/edk2-arm-vars.fd: ../pc-bios/edk2-arm-code.fd
>> + cp edk2/Build/ArmVirtQemu-ARM/DEBUG_$(call
>> toolchain,arm)/FV/QEMU_VARS.fd \
>> + $@
>> + truncate --size=64M $@
>> +
>> +../pc-bios/edk2-i386-vars.fd: ../pc-bios/edk2-i386-code.fd
>> + cp edk2/Build/OvmfIa32/DEBUG_$(call toolchain,i386)/FV/OVMF_VARS.fd $@
>> +
>> +# The license file accumulates several individual licenses from under edk2,
>> +# prefixing each individual license with a header (generated by "tail") that
>> +# states its pathname.
>> +../pc-bios/edk2-licenses.txt: submodules
>> + tail -n $(shell cat $(licenses) | wc -l) $(licenses) > $@
>> + dos2unix $@
>> +
>> +clean:
>> + rm -rf edk2/Build
>> + cd edk2/Conf && \
>> + rm -rf .cache BuildEnv.sh build_rule.txt target.txt \
>> + tools_def.txt
>> diff --git a/roms/edk2-build.sh b/roms/edk2-build.sh
>> new file mode 100755
>> index 000000000000..936d2c874a22
>> --- /dev/null
>> +++ b/roms/edk2-build.sh
>> @@ -0,0 +1,55 @@
>> +#!/bin/bash
>> +
>> +# Wrapper shell script for building a virtual platform firmware in edk2.
>> +#
>> +# Copyright (C) 2019, Red Hat, Inc.
>> +#
>> +# This program and the accompanying materials are licensed and made
>> available
>> +# under the terms and conditions of the BSD License that accompanies this
>> +# distribution. The full text of the license may be found at
>> +# <http://opensource.org/licenses/bsd-license.php>.
>> +#
>> +# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
>> WITHOUT
>> +# WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
>> +
>> +set -e -u -C
>> +
>> +# Save the command line arguments. We need to reset $# to 0 before sourcing
>> +# "edksetup.sh", as it will inherit address@hidden
>> +emulation_target=$1
>> +shift
>> +num_args=0
>> +args=()
>> +for arg in "$@"; do
>> + args[num_args++]="$arg"
>> +done
>> +shift $num_args
>> +
>> +cd edk2
>> +
>> +# Work around <https://bugzilla.tianocore.org/show_bug.cgi?id=1607>.
>> +export PYTHON_COMMAND=python2
>> +
>> +# Source "edksetup.sh" carefully.
>> +set +e +u +C
>> +source ./edksetup.sh
>> +ret=$?
>> +set -e -u -C
>> +if [ $ret -ne 0 ]; then
>> + exit $ret
>> +fi
>> +
>> +# Fetch some option arguments, and set the cross-compilation environment (if
>> +# any), for the edk2 "build" utility.
>> +source ../edk2-funcs.sh
>> +edk2_toolchain=$(qemu_edk2_get_toolchain "$emulation_target")
>> +edk2_thread_count=$(qemu_edk2_get_thread_count "$MAKEFLAGS")
>> +qemu_edk2_set_cross_env "$emulation_target"
>> +
>> +# Build the platform firmware.
>> +build \
>> + --cmd-len=65536 \
>> + -n "$edk2_thread_count" \
>> + --buildtarget=DEBUG \
>> + --tagname="$edk2_toolchain" \
>> + "address@hidden"
>>
- Re: [Qemu-devel] [PATCH 06/10] roms/Makefile: replace the $(EFIROM) target with "edk2-basetools", (continued)
[Qemu-devel] [PATCH 07/10] roms: build edk2 firmware binaries and variable store templates, Laszlo Ersek, 2019/03/08
Re: [Qemu-devel] [PATCH 07/10] roms: build edk2 firmware binaries and variable store templates, Philippe Mathieu-Daudé, 2019/03/10
- Re: [Qemu-devel] [PATCH 07/10] roms: build edk2 firmware binaries and variable store templates,
Laszlo Ersek <=
[Qemu-devel] [PATCH 08/10] pc-bios: add edk2 firmware binaries and variable store templates, Laszlo Ersek, 2019/03/08
[Qemu-devel] [PATCH 09/10] pc-bios: document the edk2 firmware images; add firmware descriptors, Laszlo Ersek, 2019/03/08
[Qemu-devel] [PATCH 10/10] Makefile: install the edk2 firmware images and their descriptors, Laszlo Ersek, 2019/03/08
Re: [Qemu-devel] [PATCH 00/10] bundle edk2 platform firmware with QEMU, Philippe Mathieu-Daudé, 2019/03/08
Re: [Qemu-devel] [PATCH 00/10] bundle edk2 platform firmware with QEMU, Philippe Mathieu-Daudé, 2019/03/11