[PATCH 6/7] WIP: efi: Allow helloworld to exit boot services

[Simon Glass]

Hi Ilias,

On Wed, 22 Nov 2023 at 00:39, Ilias Apalodimas
<ilias.apalodimas at linaro.org> wrote:
>
> Hi Simon,
>
> On Wed, 22 Nov 2023 at 00:10, Simon Glass <sjg at chromium.org> wrote:
> >
> > Hi Heinrich,
> >
> > On Tue, 21 Nov 2023 at 10:17, Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
> > >
> > > On 11/21/23 12:35, Simon Glass wrote:
> > > > This allows testing of the exit_boot_services call, providing more
> > > > coverage of the EFI bootmeth.
> > > >
> > > > Signed-off-by: Simon Glass <sjg at chromium.org>
> > >
> > > I would prefer to keep helloworld.efi benign.
> > >
> > > Please, create a new EFI binary for testing ExitBootServices().
> >
> > Eek...isn't helloworld already used in tets?\
>
> Yes, but calling exitboot services from it means we need to be really
> careful when/how we call it, since once that app runs the boottime
> services along with any attached disks will disappear.
> Instead having a single EFI app that calls that and then tests for any
> runtime services is more scalable

OK I can add a new app.

I am still stuck on the actual bug mentioned in the cover letter, though.

Regards,
Simon