From mboxrd@z Thu Jan  1 00:00:00 1970
From: Simon Glass <sjg@chromium.org>
Date: Tue, 18 Oct 2016 14:37:26 -0600
Subject: [U-Boot] [PATCH v3 7/8] x86: efi: Add a hello world test program
In-Reply-To: <5805CBE5.8050509@suse.de>
References: <1476757754-1220-1-git-send-email-sjg@chromium.org>
 <1476757754-1220-7-git-send-email-sjg@chromium.org> <5805CBE5.8050509@suse.de>
Message-ID: <CAPnjgZ1Mm5WohELej0ERPyZ86N-5zjhpbP0TRSRmd6RWcyQY9A@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Alex,

On 18 October 2016 at 01:14, Alexander Graf <agraf@suse.de> wrote:
> On 10/18/2016 04:29 AM, Simon Glass wrote:
>>
>> It is useful to have a basic sanity check for EFI loader support. Add a
>> 'bootefi hello' command which loads HelloWord.efi and runs it under
>> U-Boot.
>>
>> Signed-off-by: Simon Glass <sjg@chromium.org>
>> ---
>>
>> Changes in v3:
>> - Include a link to the program instead of adding it to the tree
>
>
> So, uh, where is the link?

I put it in the README (see the arm patch).

>
> I'm really not convinced this command buys us anything yet. I do agree that
> we want automated testing - but can't we get that using QEMU and a
> downloadable image file that we pass in as disk and have the distro boot do
> its magic?

That seems very heavyweight as a sanity check, although I agree it is useful.

Here I am just making sure that EFI programs can start, print output
and exit. It is a test that we can easily run without a lot of
overhead, much less than a full distro boot.

Regards,
Simon