From: Tom Rini <trini@konsulko.com>
To: Simon Glass <sjg@chromium.org>
Cc: U-Boot Mailing List <u-boot@lists.denx.de>,
Ilias Apalodimas <ilias.apalodimas@linaro.org>,
Steffen Jaeckel <jaeckel-floss@eyet-services.de>,
Michal Simek <michal.simek@xilinx.com>,
Dennis Gilmore <dennis@ausil.us>,
Daniel Schwierzeck <daniel.schwierzeck@gmail.com>,
Lukas Auer <lukas.auer@aisec.fraunhofer.de>,
Jaehoon Chung <jh80.chung@samsung.com>,
Matthias Brugger <mbrugger@suse.com>, Peng Fan <peng.fan@nxp.com>,
Stephen Warren <swarren@nvidia.com>,
Stephen Warren <swarren@wwwdotorg.org>
Subject: Re: [PATCH 00/28] Initial implementation of bootmethod/bootflow
Date: Thu, 19 Aug 2021 09:59:44 -0400 [thread overview]
Message-ID: <20210819135944.GX858@bill-the-cat> (raw)
In-Reply-To: <20210819034601.1618773-1-sjg@chromium.org>
[-- Attachment #1: Type: text/plain, Size: 3039 bytes --]
On Wed, Aug 18, 2021 at 09:45:33PM -0600, Simon Glass wrote:
> Bootmethod and bootflow provide a built-in way for U-Boot to automatically boot
> an Operating System without custom scripting and other customisation:
>
> - bootmethod - a method to scan a device to find bootflows (owned by U-Boot)
> - bootflow - a description of how to boot (owned by the distro)
>
> This series provides an initial implementation of these, enable to scan
> for bootflows from MMC and Ethernet. The only bootflow supported is
> distro boot, i.e. an extlinux.conf file included on a filesystem or
> tftp server. It works similiarly to the existing script-based approach,
> but is native to U-Boot.
>
> With this we can boot on a Raspberry Pi 3 with just one command:
>
> bootflow scan -lb
>
> which means to scan, listing (-l) each bootflow and trying to boot each
> one (-b). The final patch shows this.
>
> It is intended that this approach be expanded to support mechanisms other
> than distro boot, including EFI-related ones. With a standard way to
> identify boot devices, these features become easier. It also should
> support U-Boot scripts, for backwards compatibility only.
>
> The first patch of this series moves boot-related code out of common/ and
> into a new boot/ directory. This helps to collect these related files
> in one place, as common/ is quite large.
>
> Like sysboot, this feature makes use of the existing PXE implementation.
> Much of this series consists of cleaning up that code and refactoring it
> into something closer to a module that can be called, teasing apart its
> reliance on the command-line interpreter to access filesystems and the
> like. Also it now uses function arguments and its own context struct
> internally rather than environment variables, which is very hard to
> follow. No core functional change is included in the included PXE patches.
>
> For documentation, see the 'doc' patch.
>
> There is quite a long list of future work included in the documentation.
> One question is the choice of naming. Since this is a bootloader, should
> we just call this a 'method' and a 'flow' ? The 'boot' prefix is already
> shared by other commands like bootm, booti, etc.
>
> The design is described here:
>
> https://drive.google.com/file/d/1ggW0KJpUOR__vBkj3l61L2dav4ZkNC12/view?usp=sharing
>
> The series is available at u-boot-dm/bmea-working
My question / concern is this. Would the next step here be to
implement the generic UEFI boot path? Today, I can write Fedora 34 for
AArch64 to a USB stick, boot U-Boot off of uSD card and the installer
automatically boots. I'm sure I could do the same with the BSDs.
Reading the documentation left me with the impression that every OSV
would be expected to write something, so that their installer / OS boot.
But there's already standards for that, which they do, and we should be
implementing (and do, via the current distro_boot) or making easier to
enable. Thanks!
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
next prev parent reply other threads:[~2021-08-19 13:59 UTC|newest]
Thread overview: 60+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-08-19 3:45 [PATCH 00/28] Initial implementation of bootmethod/bootflow Simon Glass
2021-08-19 3:45 ` [PATCH 01/28] Create a new boot/ directory Simon Glass
2021-08-19 3:45 ` [PATCH 02/28] pxe: Move API comments to the header files Simon Glass
2021-08-19 3:45 ` [PATCH 03/28] pxe: Use a context pointer Simon Glass
2021-08-19 3:45 ` [PATCH 04/28] pxe: Move do_getfile() into the context Simon Glass
2021-08-19 3:45 ` [PATCH 05/28] pxe: Add a userdata field to " Simon Glass
2021-08-19 3:45 ` [PATCH 06/28] pxe: Tidy up the is_pxe global Simon Glass
2021-08-19 3:45 ` [PATCH 07/28] pxe: Move pxe_utils files Simon Glass
2021-08-19 3:45 ` [PATCH 08/28] pxe: Tidy up some comments in pxe_utils Simon Glass
2021-08-19 3:45 ` [PATCH 09/28] pxe: Tidy up code style a little " Simon Glass
2021-08-19 3:45 ` [PATCH 10/28] pxe: Move common parsing coding into pxe_util Simon Glass
2021-08-19 3:45 ` [PATCH 11/28] pxe: Clean up the use of bootfile Simon Glass
2021-08-19 3:45 ` [PATCH 12/28] pxe: Drop get_bootfile_path() Simon Glass
2021-08-19 3:45 ` [PATCH 13/28] lib: Add tests for simple_itoa() Simon Glass
2021-08-19 3:45 ` [PATCH 14/28] lib: Add a function to convert a string to a hex value Simon Glass
2021-08-19 3:45 ` [PATCH 15/28] pxe: Return the file size from the getfile() function Simon Glass
2021-08-19 3:45 ` [PATCH 16/28] pxe: Refactor sysboot to have one helper Simon Glass
2021-08-19 3:45 ` [PATCH 17/28] doc: Move distro boot doc to rST Simon Glass
2021-08-19 3:45 ` [PATCH 18/28] pxe: Allow calling the pxe_get logic directly Simon Glass
2021-08-19 3:45 ` [PATCH 19/28] bootmethod: Add the uclass and core implementation Simon Glass
2021-08-19 3:45 ` [PATCH 20/28] bootmethod: Add an implementation of distro boot Simon Glass
2021-08-19 3:45 ` [PATCH 21/28] bootmethod: Add a command Simon Glass
2021-08-19 3:45 ` [PATCH 22/28] bootflow: " Simon Glass
2021-08-19 3:45 ` [PATCH 23/28] bootmethod: Add tests for bootmethod and bootflow Simon Glass
2021-08-19 3:45 ` [PATCH 24/28] bootmethod: doc: Add documentation Simon Glass
2021-08-19 3:45 ` [PATCH 25/28] mmc: Allow for children other than the block device Simon Glass
2021-08-19 3:45 ` [PATCH 26/28] mmc: Add a bootmethod Simon Glass
2021-08-19 3:46 ` [PATCH 27/28] ethernet: " Simon Glass
2021-08-19 3:46 ` [PATCH 28/28] RFC: rpi: Switch over to use bootflow Simon Glass
2021-08-19 13:59 ` Tom Rini [this message]
2021-08-19 14:25 ` [PATCH 00/28] Initial implementation of bootmethod/bootflow Simon Glass
2021-08-19 17:27 ` Tom Rini
2021-08-23 12:35 ` Ilias Apalodimas
2021-08-23 17:25 ` Simon Glass
2021-08-23 20:08 ` Tom Rini
2021-08-24 9:29 ` Ilias Apalodimas
2021-08-25 13:11 ` Simon Glass
2021-08-25 13:29 ` Peter Robinson
2021-08-25 21:34 ` Mark Kettenis
2021-08-25 21:58 ` Tom Rini
2021-08-20 3:15 ` AKASHI Takahiro
2021-08-20 18:22 ` Simon Glass
2021-08-23 0:46 ` AKASHI Takahiro
2021-08-23 11:54 ` Mark Kettenis
2021-08-23 17:25 ` Simon Glass
2021-08-23 20:01 ` Tom Rini
2021-08-24 10:22 ` Mark Kettenis
2021-08-25 10:45 ` Emmanuel Vadot
2021-08-25 13:11 ` Simon Glass
2021-08-25 14:42 ` AKASHI Takahiro
2021-08-25 14:56 ` Tom Rini
2021-08-25 21:54 ` Mark Kettenis
2021-08-25 22:06 ` Tom Rini
2021-08-26 6:33 ` AKASHI Takahiro
2021-08-26 13:03 ` Tom Rini
2021-08-26 12:01 ` Mark Kettenis
2021-08-26 13:00 ` Tom Rini
2021-08-26 13:32 ` Mark Kettenis
2021-08-26 13:50 ` Ilias Apalodimas
2021-08-26 11:55 ` Peter Robinson
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=20210819135944.GX858@bill-the-cat \
--to=trini@konsulko.com \
--cc=daniel.schwierzeck@gmail.com \
--cc=dennis@ausil.us \
--cc=ilias.apalodimas@linaro.org \
--cc=jaeckel-floss@eyet-services.de \
--cc=jh80.chung@samsung.com \
--cc=lukas.auer@aisec.fraunhofer.de \
--cc=mbrugger@suse.com \
--cc=michal.simek@xilinx.com \
--cc=peng.fan@nxp.com \
--cc=sjg@chromium.org \
--cc=swarren@nvidia.com \
--cc=swarren@wwwdotorg.org \
--cc=u-boot@lists.denx.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox