[U-Boot] [PULL] efi patch queue 2019-01-28
Hi Tom,
This is my current patch queue for efi. Please pull.
Alex
The following changes since commit e964df1e2ae7b2c041a9d767f03ad2b72a3f2ac7:
Merge branch '2019-01-16-master-imports' (2019-01-17 17:42:03 -0500)
are available in the git repository at:
git://github.com/agraf/u-boot.git tags/signed-efi-next
for you to fetch changes up to b90a387650970fa9dbda7403070dbbf564cf81b8:
efi_loader: debug output for HII protocols (2019-01-26 11:48:18 +0100)
---------------------------------------------------------------- Patch queue for efi - 2019-01-28
Goodness this time around:
- HII protocol, finally allows us to run the UEFI Shell! - efi selftest now available on Cortex-M - NVMe support for distro boot - Lots of code cleanup
---------------------------------------------------------------- AKASHI Takahiro (8): distro: not taint environment variables if possible efi_loader: move efi_init_obj_list() to a new efi_setup.c efi_loader: implement GetNextVariableName() efi_selftest: fix variables test for GetNextVariableName() efi: hii: add guid package support efi: hii: add keyboard layout package support efi: hii: add HII config routing/access protocols efi_selftest: add HII database protocols test
Akashi, Takahiro (1): lib: add u16_strcpy/strdup functions
Heinrich Schuchardt (36): efi_loader: efi_guid_t must be 64-bit aligned efi_loader: struct efi_configuration_table test: tests for u16_strdup() and u16_strcpy() distro_bootcmd: add NVME support efi_loader: efi_add_runtime_mmio() efi_loader: efi_connect_controller() use %pD efi_loader: use named constant for efi_dp_from_mem() efi_loader: CMD_BOOTEFI_HELLO_COMPILE in configs efi_loader: fix memory allocation on sandbox efi_loader: avoid unnecessary pointer to long conversion efi_loader: signature of StartImage and Exit efi_selftest: allow building on ARMv7-M doc: README.uefi: fix typos efi_loader: use const efi_guid_t * for variable services efi_loader: efi_set_variable use const void * efi_selftest: SNP unit test on sandbox efi_selftest: events unit test, check return values efi_selftest: tpl unit test, check return values efi_driver: simplify error message efi_loader: refactor switch to non-secure mode efi_loader: fix CopyMem() efi_loader: use u16* for file name efi_loader: consistent build flags for EFI applications efi_loader: remove duplicate GUID definition efi_loader: eliminate duplicate function hex2mem() efi_loader: remove duplicate function mem2hex() efi_loader: comments for variable services efi_loader: provide freestanding library efi_loader: use freestanding library for efi apps efi_loader: use library memcpy() in helloworld.efi efi_loader: do not use symbolic links efi_loader: fix GetNextVariableName efi_loader: GetNextVariableName() relies on REGEX x86: do not use i386 code for x86_64 memory functions efi_selftest: fix HII tests efi_loader: debug output for HII protocols
Leif Lindholm (1): efi_loader: Initial HII database protocols
arch/arm/cpu/armv7/Makefile | 1 + arch/arm/cpu/armv7/exception_level.c | 56 ++ arch/arm/cpu/armv8/Makefile | 1 + arch/arm/cpu/armv8/exception_level.c | 55 ++ arch/arm/lib/Makefile | 6 +- arch/x86/include/asm/string.h | 31 +- arch/x86/lib/Makefile | 2 +- cmd/Kconfig | 2 +- cmd/bootefi.c | 152 +--- common/bootm.c | 10 + configs/chromebook_link64_defconfig | 1 - configs/kp_imx6q_tpc_defconfig | 1 - configs/qemu-x86_64_defconfig | 1 - configs/stm32f429-discovery_defconfig | 1 - configs/stm32f429-evaluation_defconfig | 1 - configs/stm32f469-discovery_defconfig | 1 - doc/README.uefi | 10 +- include/bootm.h | 5 + include/charset.h | 23 + include/config_distro_bootcmd.h | 40 +- include/efi.h | 2 +- include/efi_api.h | 445 ++++++++++- include/efi_loader.h | 24 +- lib/charset.c | 29 + lib/efi_driver/efi_uclass.c | 3 +- lib/efi_loader/Kconfig | 1 + lib/efi_loader/Makefile | 4 +- lib/efi_loader/efi_boottime.c | 32 +- lib/efi_loader/efi_file.c | 14 +- lib/efi_loader/efi_freestanding.c | 90 +++ lib/efi_loader/efi_hii.c | 1095 ++++++++++++++++++++++++++ lib/efi_loader/efi_hii_config.c | 146 ++++ lib/efi_loader/efi_memory.c | 6 + lib/efi_loader/efi_runtime.c | 3 +- lib/efi_loader/efi_setup.c | 86 ++ lib/efi_loader/efi_variable.c | 300 +++++-- lib/efi_loader/helloworld.c | 36 +- lib/efi_selftest/Makefile | 11 +- lib/efi_selftest/efi_selftest_block_device.c | 6 +- lib/efi_selftest/efi_selftest_events.c | 6 +- lib/efi_selftest/efi_selftest_hii.c | 1035 ++++++++++++++++++++++++ lib/efi_selftest/efi_selftest_hii_data.c | 453 +++++++++++ lib/efi_selftest/efi_selftest_snp.c | 8 + lib/efi_selftest/efi_selftest_tpl.c | 4 +- lib/efi_selftest/efi_selftest_variables.c | 17 +- scripts/Makefile.lib | 2 +- test/unicode_ut.c | 23 + 47 files changed, 3948 insertions(+), 333 deletions(-) create mode 100644 arch/arm/cpu/armv7/exception_level.c create mode 100644 arch/arm/cpu/armv8/exception_level.c create mode 100644 lib/efi_loader/efi_freestanding.c create mode 100644 lib/efi_loader/efi_hii.c create mode 100644 lib/efi_loader/efi_hii_config.c create mode 100644 lib/efi_loader/efi_setup.c create mode 100644 lib/efi_selftest/efi_selftest_hii.c create mode 100644 lib/efi_selftest/efi_selftest_hii_data.c
On Mon, Jan 28, 2019 at 12:52:28PM +0100, Alexander Graf wrote:
Hi Tom,
This is my current patch queue for efi. Please pull.
Alex
The following changes since commit e964df1e2ae7b2c041a9d767f03ad2b72a3f2ac7:
Merge branch '2019-01-16-master-imports' (2019-01-17 17:42:03 -0500)
are available in the git repository at:
git://github.com/agraf/u-boot.git tags/signed-efi-next
for you to fetch changes up to b90a387650970fa9dbda7403070dbbf564cf81b8:
efi_loader: debug output for HII protocols (2019-01-26 11:48:18 +0100)
I'm adding a few people to the mail here. First, we have: commit ae4eb8232d927e4f630408f2533f06a5fb1b32a4 Author: Leif Lindholm leif.lindholm@linaro.org Date: Mon Jan 21 12:12:57 2019 +0900
efi_loader: Initial HII database protocols
which comes in without his, but with others Signed-off-by lines. This is just an oversight on Leif's part and this code can come in just fine and we don't need to revert things, right? This is a good bit more code than is in the occasional bugfix that I let in that's missing the SoB line.
Next, we have: https://travis-ci.org/trini/u-boot/jobs/485695447 which is because, oh, right, we need both: https://patchwork.ozlabs.org/patch/1031448/ which is a "me" patch and Stephen was OK with too but also: https://patchwork.ozlabs.org/patch/1030669/ which is an x86 patch I assigned to Bin and since he's not acked/reviewed it I don't feel comfortable grabbing it directly.
So for the moment, this PR is on hold.
On 01/29/2019 03:39 PM, Tom Rini wrote:
On Mon, Jan 28, 2019 at 12:52:28PM +0100, Alexander Graf wrote:
Hi Tom,
This is my current patch queue for efi. Please pull.
Alex
The following changes since commit e964df1e2ae7b2c041a9d767f03ad2b72a3f2ac7:
Merge branch '2019-01-16-master-imports' (2019-01-17 17:42:03 -0500)
are available in the git repository at:
git://github.com/agraf/u-boot.git tags/signed-efi-next
for you to fetch changes up to b90a387650970fa9dbda7403070dbbf564cf81b8:
efi_loader: debug output for HII protocols (2019-01-26 11:48:18 +0100)
I'm adding a few people to the mail here. First, we have: commit ae4eb8232d927e4f630408f2533f06a5fb1b32a4 Author: Leif Lindholm leif.lindholm@linaro.org Date: Mon Jan 21 12:12:57 2019 +0900
efi_loader: Initial HII database protocolswhich comes in without his, but with others Signed-off-by lines. This is just an oversight on Leif's part and this code can come in just fine and we don't need to revert things, right? This is a good bit more code than is in the occasional bugfix that I let in that's missing the SoB line.
We should fix that regardless. IIRC Leif just omitted the SoB because it was more of an RFC patch than a real submission proposal.
Leif, could you please give me your SoB on that patch so I can add it to the commit message?
Next, we have: https://travis-ci.org/trini/u-boot/jobs/485695447 which is because, oh, right, we need both: https://patchwork.ozlabs.org/patch/1031448/ which is a "me" patch and Stephen was OK with too but also: https://patchwork.ozlabs.org/patch/1030669/ which is an x86 patch I assigned to Bin and since he's not acked/reviewed it I don't feel comfortable grabbing it directly.
So for the moment, this PR is on hold.
Ok, please notify me when the x86 patches are sorted out. I'm not sure why we're suddenly hitting all those travis failures. The bugs in question have been around forever ...
Alex
Hi Tom, Alex,
On Tue, Jan 29, 2019 at 09:39:50AM -0500, Tom Rini wrote:
for you to fetch changes up to b90a387650970fa9dbda7403070dbbf564cf81b8:
efi_loader: debug output for HII protocols (2019-01-26 11:48:18 +0100)
I'm adding a few people to the mail here. First, we have: commit ae4eb8232d927e4f630408f2533f06a5fb1b32a4 Author: Leif Lindholm leif.lindholm@linaro.org Date: Mon Jan 21 12:12:57 2019 +0900
efi_loader: Initial HII database protocolswhich comes in without his, but with others Signed-off-by lines. This is just an oversight on Leif's part and this code can come in just fine and we don't need to revert things, right? This is a good bit more code than is in the occasional bugfix that I let in that's missing the SoB line.
My contribution was a bunch of structs, enums, #defines and stub functions to demonstrate what needed to be implemented in order for the EDK2 UEFI Shell to run. Which the others fixed up and filled in with actual functionality. So, fwiw, I'm happy for my SoB to be added (and I wouldn't be in the least bit offended if someone wanted to reassign the authorship).
(for ae4eb8232d927e4f630408f2533f06a5fb1b32a4: Signed-off-by: Leif Lindholm leif.lindholm@linaro.org)
/ Leif
Next, we have: https://travis-ci.org/trini/u-boot/jobs/485695447 which is because, oh, right, we need both: https://patchwork.ozlabs.org/patch/1031448/ which is a "me" patch and Stephen was OK with too but also: https://patchwork.ozlabs.org/patch/1030669/ which is an x86 patch I assigned to Bin and since he's not acked/reviewed it I don't feel comfortable grabbing it directly.
So for the moment, this PR is on hold.
-- Tom
participants (3)
-
Alexander Graf -
Leif Lindholm -
Tom Rini