All of lore.kernel.org
 help / color / mirror / Atom feed
From: Simon Glass <sjg@chromium.org>
To: Heinrich Schuchardt <xypron.glpk@gmx.de>
Cc: Ilias Apalodimas <ilias.apalodimas@linaro.org>,
	 Christian Melki <christian.melki@t2data.com>,
	Bin Meng <bmeng.cn@gmail.com>,  Tom Rini <trini@konsulko.com>,
	U-Boot Mailing List <u-boot@lists.denx.de>,
	 Alexander Graf <agraf@csgraf.de>
Subject: Re: [PATCH v3 01/35] efi: Add a script for building and testing U-Boot on UEFI
Date: Thu, 28 Oct 2021 08:19:06 -0600	[thread overview]
Message-ID: <CAPnjgZ2o9UYw6S2ZW4zRtxkk3MPR2nCVsfQ3q3-COiPJSKh3Jg@mail.gmail.com> (raw)
In-Reply-To: <4cc3370d-1d52-8c4e-1726-1db4d5f2e6d6@gmx.de>

Hi Heinrich,

On Wed, 27 Oct 2021 at 23:38, Heinrich Schuchardt <xypron.glpk@gmx.de> wrote:
>
> On 10/26/21 5:30 AM, Simon Glass wrote:
> > It is quite complicated to run U-Boot on QEMU since we have four
> > different builds and they must use different versions of qemu and the
> > UEFI binaries.
> >
> > Add a script to help.
> >
> > Signed-off-by: Simon Glass <sjg@chromium.org>
>
> It seems that you missed some of my review comments:
>
> * The script does not work:
>
> $ scripts/build-efi.sh -wsPr
> Packaging efi-x86_app32
> [sudo] password for user:
> cp: cannot stat '/tmp/b//efi-x86_app32/u-boot-app.efi': No such file or
> directory
>
> * Do not use fixed directories in /tmp but use 'mktemp -d' to create a
> unique directory.

Did you see my response when you asked about that last time? To
repeat, you need to build U-Boot somewhere and have a build directory.
If this script creates a tmpdir, what is the point? We still need the
build dir.

Also:

> > +# Set ubdir to the build directory where you build U-Boot out-of-tree
> > +# We avoid in-tree build because it gets confusing trying different builds


Also documented at the very top of the script:

[..]


> > +# Script to build an EFI thing suitable for booting with QEMU, possibly running
> > +# it also.
> > +
> > +# This just an example. It assumes that
> > +
> > +# - you build U-Boot in ${ubdir}/<name> where <name> is the U-Boot board config
> > +# - /mnt/x is a directory used for mounting
> > +# - you have access to the 'pure UEFI' builds for QEMU
> > +#
> > +# UEFI binaries for QEMU used for testing this script:
> > +#
> > +# OVMF-pure-efi.i386.fd at
> > +# https://drive.google.com/file/d/1jWzOAZfQqMmS2_dAK2G518GhIgj9r2RY/view?usp=sharing
> > +
> > +# OVMF-pure-efi.x64.fd at
> > +# https://drive.google.com/file/d/1c39YI9QtpByGQ4V0UNNQtGqttEzS-eFV/view?usp=sharing
> > +

Regards,
Simon

  reply	other threads:[~2021-10-28 14:19 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-26  3:30 [PATCH v3 00/35] efi: Improvements to U-Boot running on top of UEFI Simon Glass
2021-10-26  3:30 ` [PATCH v3 01/35] efi: Add a script for building and testing U-Boot on UEFI Simon Glass
2021-10-28  5:33   ` Heinrich Schuchardt
2021-10-28 14:19     ` Simon Glass [this message]
2021-10-28 15:27       ` Heinrich Schuchardt
2021-10-28 15:38         ` Simon Glass
2021-10-26  3:30 ` [PATCH v3 02/35] efi: Enable DM_ETH for the app Simon Glass
2021-10-28  5:40   ` Heinrich Schuchardt
2021-11-04  2:12     ` Simon Glass
2021-10-26  3:30 ` [PATCH v3 03/35] efi: Drop the OF_EMBED warning for EFI Simon Glass
2021-10-28  5:43   ` Heinrich Schuchardt
2021-10-26  3:30 ` [PATCH v3 04/35] x86: Create a 32/64-bit selection for the app Simon Glass
2021-10-26  3:30 ` [PATCH v3 05/35] efi: Create a 64-bit app Simon Glass
2021-10-28  5:53   ` Heinrich Schuchardt
2021-10-28 14:19     ` Simon Glass
2021-10-26  3:30 ` [PATCH v3 06/35] x86: Don't duplicate global_ptr in 64-bit EFI app Simon Glass
2021-10-26  3:30 ` [PATCH v3 07/35] efi: Add a way to obtain boot services in the app Simon Glass
2021-10-26  3:30 ` [PATCH v3 08/35] efi: Add video support to " Simon Glass
2021-10-26  3:30 ` [PATCH v3 09/35] efi: Add EFI uclass for media Simon Glass
2021-10-26  3:30 ` [PATCH v3 10/35] efi: Add a media/block driver for EFI block devices Simon Glass
2021-10-26  3:30 ` [PATCH v3 11/35] efi: Locate all block devices in the app Simon Glass
2021-10-26  3:30 ` [PATCH v3 12/35] patman: Use a ValueError exception if tools.Run() fails Simon Glass
2021-10-26  3:30 ` [PATCH v3 13/35] binman: Report an error if test files fail to compile Simon Glass
2021-10-26  3:30 ` [PATCH v3 14/35] binman: Support reading the offset of an ELF-file symbol Simon Glass
2021-10-26  3:30 ` [PATCH v3 15/35] binman: Allow timeout to occur in the image or its section Simon Glass
2021-10-26  3:30 ` [PATCH v3 16/35] binman: Tidy up comments on _DoTestFile() Simon Glass
2021-10-26  3:30 ` [PATCH v3 17/35] binman: Support updating the dtb in an ELF file Simon Glass
2021-10-26  3:30 ` [PATCH v3 18/35] efi: serial: Support arrow keys Simon Glass
2021-10-26  3:30 ` [PATCH v3 19/35] bloblist: Support allocating the bloblist Simon Glass
2021-10-26  3:30 ` [PATCH v3 20/35] x86: Allow booting a kernel from the EFI app Simon Glass
2021-10-26  3:30 ` [PATCH v3 21/35] x86: Don't process the kernel command line unless enabled Simon Glass
2021-10-26  3:30 ` [PATCH v3 22/35] x86: efi: Add room for the binman definition in the dtb Simon Glass
2021-10-26  3:30 ` [PATCH v3 23/35] efi: Drop device_path from struct efi_priv Simon Glass
2021-10-26  3:30 ` [PATCH v3 24/35] efi: Add comments to " Simon Glass
2021-10-26  3:30 ` [PATCH v3 25/35] efi: Fix ll_boot_init() operation with the app Simon Glass
2021-10-26  3:30 ` [PATCH v3 26/35] efi: Add a few comments to the stub Simon Glass
2021-10-26  3:30 ` [PATCH v3 27/35] efi: Share struct efi_priv between the app and stub code Simon Glass
2021-10-26  3:30 ` [PATCH v3 28/35] efi: Move exit_boot_services into a function Simon Glass
2021-10-26  3:30 ` [PATCH v3 29/35] efi: Check for failure when initing the app Simon Glass
2021-10-26  3:30 ` [PATCH v3 30/35] efi: Mention that efi_info_get() is only used in the stub Simon Glass
2021-10-26  3:30 ` [PATCH v3 31/35] efi: Show when allocated pages are used Simon Glass
2021-10-26  3:30 ` [PATCH v3 32/35] efi: Allow easy selection of serial-only operation Simon Glass
2021-10-26  3:30 ` [PATCH v3 33/35] efi: Update efi_get_next_mem_desc() to avoid needing a map Simon Glass
2021-10-26  3:30 ` [PATCH v3 34/35] efi: Support the efi command in the app Simon Glass
2021-10-26  3:30 ` [PATCH v3 35/35] efi: Show the system-table revision Simon Glass

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAPnjgZ2o9UYw6S2ZW4zRtxkk3MPR2nCVsfQ3q3-COiPJSKh3Jg@mail.gmail.com \
    --to=sjg@chromium.org \
    --cc=agraf@csgraf.de \
    --cc=bmeng.cn@gmail.com \
    --cc=christian.melki@t2data.com \
    --cc=ilias.apalodimas@linaro.org \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    --cc=xypron.glpk@gmx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.