On 4/23/19 2:24 AM, AKASHI, Takahiro wrote:
Heinrich,
On Tue, 23 Apr 2019 at 03:16, Heinrich Schuchardt <[email protected]> wrote:
On 4/19/19 5:22 AM, AKASHI Takahiro wrote:
There are several reasons that I want to rework/refactor bootefi command
as well as bootmgr:
* Some previous commits on bootefi.c have made the code complicated
and a bit hard to understand.
* do_bootefi_exec() would better be implemented using load_image() along
with start_image() to be aligned with UEFI interfaces.
* Contrary to the other part, efi_selftest part of the code is unusual
in terms of loading/execution path in do_bootefi().
* When we will support "secure boot" in the future, EFI Boot Manager
is expected to be invoked as a standalone command without any arguments
to mitigate security surfaces.
In this patch set,
Patch#1 to #7 are preparatory patches for patch#8.
Patch#8 is a core part of reworking.
Patch#9 is for standalone boot manager.
Hello Takahiro,
your patches 1-8 are now (with some modifications) in the efi-2019-07
branch. I have added some more patches.
https://github.com/xypron2/u-boot/commits/efi-2019-07
Travis CI testing was ok:
https://travis-ci.org/xypron2/u-boot/builds/522947698
I have tested on real hardware without error:
* TinkerBoard (32 bit) boot via GRUB
* Pine64 A64 LTS (64 bit) boot via iPXE and GRUB
* Odroid C2 (64 bit) boot via GRUB
If you are ok with the adjustments to your patch series I will create a
pull request.
Thank you for the merge.
One question: how did you fix a grub error that you reported before?
I fixed the issue that iPXE could not find a chained script via
efi_loader: correctly split device path of loaded image
https://lists.denx.de/pipermail/u-boot/2019-April/365907.html
Furthermore I fixed a bug in Debian's QEMU by applying Alex's
https://github.com/qemu/qemu/commit/2d2a4549cc29850aab891495685a7b31f5254b12
Best regards
Heinrich
-Takahiro Akashi
Best regards
Heinrich
_______________________________________________
U-Boot mailing list
[email protected]
https://lists.denx.de/listinfo/u-boot
