[PATCH 0/7] efi: Partial attempt at a test for EFI bootmeth

[Simon Glass]

Hi Heinrich,

On Tue, 21 Nov 2023 at 04:36, Simon Glass <sjg at chromium.org> wrote:
>
> The test coverage for the EFI bootmeth is incomplete since it does not
> actually boot the application.
>
> This series makes an attempt at this.
>
> However there are at least two problems:
> 1. The test does not set up the loaded image properly, so it isn't
> found by the app.
> 2. The test needs to use USB instead of mmc, so that usb_stop() is
> called

I think you may have missed the point of this series. I am not sure
how to fix problem #1 above. Do you know what is going on, or could
you help figure it out, please?

>
>
> Simon Glass (7):
>   efi: Use puts() in cout so that console recording works
>   efi: Show the vendor in helloworld
>   efi: test: Create a disk image with and EFI app in it
>   WIP: efi: Disable ANSI output
>   WIP: efi: Add a test for the EFI bootmeth
>   WIP: efi: Allow helloworld to exit boot services
>   WIP: efi: debugging
>
>  arch/sandbox/dts/test.dts     | 10 ++++++-
>  boot/bootflow.c               |  2 ++
>  boot/bootmeth_efi.c           |  1 +
>  cmd/bootefi.c                 |  1 +
>  include/efi_loader.h          | 14 +++++++--
>  lib/efi_loader/efi_boottime.c |  5 ++++
>  lib/efi_loader/efi_console.c  | 19 +++++-------
>  lib/efi_loader/efi_setup.c    |  2 +-
>  lib/efi_loader/helloworld.c   | 39 ++++++++++++++++++++++++-
>  test/boot/bootflow.c          | 55 ++++++++++++++++++++++++++++++++++-
>  test/py/tests/test_ut.py      | 32 ++++++++++++++++++++
>  11 files changed, 163 insertions(+), 17 deletions(-)
>
> --
> 2.43.0.rc1.413.gea7ed67945-goog
>

Regards,
Simon