[PATCH 00/31] passage: Define a standard for firmware data flow
Simon Glass
sjg at chromium.org
Mon Nov 1 02:17:02 CET 2021
This series adds a standard way of passing information between different
firmware phases. This already exists in U-Boot at a very basic level, in
the form of a bloblist containing an spl_handoff structure, but the intent
here is to define something useful across projects.
The need for this is growing as firmware fragments into multiple binaries
each with its own purpose. Without any run-time connection, we must rely
on build-time settings which are brittle and painful to keep in sync.
This feature is named 'standard passage' since the name is more unique
than many others that could be chosen, it is a passage in the sense that
information is flowing from one place to another and it is standard,
because that is what we want to create.
The implementation is simply a pointer to a bloblist in a register, with
an extra register to point to a devicetree, for more complex data, if one
is present in the bloblist. This should cover all cases (small memory
footprint as well as complex data flow) and be easy enough to implement on
all architectures.
The core bloblist code is relicensed to BSD-3-Clause in case it is useful
in non-GPL projects but there is no requirement to use the same code.
This series includes tweaks to the bloblist implementation in U-Boot to
make it more suitable for the task, including:
- Allocate tags explicitly in the enum
- Put the magic number first
- Define a process for adding tags
The emphasis is on enabling open communcation between binaries, not
enabling passage of secret, undocumented data, although this is possible
in a private environment.
This series is built on the OF_BOARD series It is available at
u-boot-dm/pass-working or:
https://source.denx.de/u-boot/custodians/u-boot-dm/-/commit/073b5c156f222c69a98b8ebcaa563d1ff10eb217
Simon Glass (31):
Makefile: Correct TPL rule for OF_REAL
kconfig: Add support for conditional values
dm: core: Allow getting some basic stats
stddef: Avoid warning with clang with offsetof()
fdt: Drop SPL_BUILD macro
bloblist: Put the magic number first
bloblist: Rename the SPL tag
bloblist: Drop unused tags
bloblist: Use explicit numbering for the tags
bloblist: Support allocating the bloblist
bloblist: Use LOG_CATEGORY to simply logging
bloblist: Use 'phase' consistently for bloblists
bloblist: Refactor Kconfig to support alloc or fixed
arm: qemu: Add an SPL build
bloblist: Add functions to obtain base address and size
passage: Support an incoming passage
passage: Support a control devicetree
passage: arm: Accept a passage from the previous phase
passage: spl: Support adding the dtb to the passage bloblist
passage: spl: Support passing the passage to U-Boot
passage: Record where the devicetree came from
passage: Report the devicetree source
passage: Add a qemu test for ARM
bloblist: doc: Bring in the API documentation
bloblist: Relicense to allow BSD-3-Clause
sandbox: Add a way of checking structs for standard passage
passage: Add documentation
passage: Add docs for spl_handoff
x86: Move Intel GNVS file into the common include directory
passage: Add checks for pre-existing blobs
WIP: RFC: Add a gitlab test
.gitlab-ci.yml | 6 +
MAINTAINERS | 10 +
Makefile | 2 +-
arch/arm/cpu/armv7/start.S | 7 +-
arch/arm/dts/qemu-arm-u-boot.dtsi | 22 ++
arch/arm/lib/crt0.S | 4 +
arch/arm/mach-qemu/Kconfig | 9 +
arch/sandbox/cpu/spl.c | 2 +-
arch/x86/cpu/apollolake/acpi.c | 2 +-
arch/x86/cpu/broadwell/cpu_from_spl.c | 4 +-
arch/x86/cpu/intel_common/acpi.c | 2 +-
.../include/asm/arch-apollolake/global_nvs.h | 2 +-
arch/x86/lib/spl.c | 2 +-
arch/x86/lib/tpl.c | 2 +-
board/emulation/qemu-arm/Kconfig | 23 +-
board/emulation/qemu-arm/MAINTAINERS | 1 +
board/emulation/qemu-arm/Makefile | 1 +
board/emulation/qemu-arm/spl.c | 27 ++
board/google/chromebook_coral/coral.c | 2 +-
board/sandbox/Makefile | 3 +-
board/sandbox/stdpass_check.c | 107 ++++++
cmd/bdinfo.c | 2 +
common/Kconfig | 161 ++++++++-
common/bloblist.c | 124 +++++--
common/board_f.c | 48 ++-
common/board_r.c | 18 +
common/spl/spl.c | 74 +++-
configs/qemu_arm_spl_defconfig | 78 +++++
doc/board/emulation/qemu-arm.rst | 38 +++
doc/develop/bloblist.rst | 28 +-
doc/develop/index.rst | 1 +
doc/develop/std_passage.rst | 319 ++++++++++++++++++
drivers/core/device.c | 11 +
drivers/core/root.c | 7 +
drivers/core/uclass.c | 13 +
drivers/serial/serial-uclass.c | 3 +-
dts/Kconfig | 12 +
include/asm-generic/global_data.h | 35 ++
include/bloblist.h | 175 +++++++---
include/dm/device.h | 11 +-
include/dm/root.h | 8 +
include/dm/uclass-internal.h | 7 +
include/fdtdec.h | 40 ++-
include/handoff.h | 8 +-
.../x86/include/asm => include}/intel_gnvs.h | 0
include/linux/kconfig.h | 18 +
include/linux/stddef.h | 8 +-
include/spl.h | 15 +
include/stdpass/README | 4 +
include/stdpass/tpm2_eventlog.h | 42 +++
include/stdpass/vboot_ctx.h | 267 +++++++++++++++
lib/asm-offsets.c | 5 +
lib/fdtdec.c | 65 +++-
scripts/config_whitelist.txt | 1 +
test/bloblist.c | 21 +-
test/dm/core.c | 41 +++
test/py/tests/test_passage.py | 11 +
57 files changed, 1798 insertions(+), 161 deletions(-)
create mode 100644 arch/arm/dts/qemu-arm-u-boot.dtsi
create mode 100644 board/emulation/qemu-arm/spl.c
create mode 100644 board/sandbox/stdpass_check.c
create mode 100644 configs/qemu_arm_spl_defconfig
create mode 100644 doc/develop/std_passage.rst
rename {arch/x86/include/asm => include}/intel_gnvs.h (100%)
create mode 100644 include/stdpass/README
create mode 100644 include/stdpass/tpm2_eventlog.h
create mode 100644 include/stdpass/vboot_ctx.h
create mode 100644 test/py/tests/test_passage.py
--
2.33.1.1089.g2158813163f-goog
More information about the U-Boot
mailing list