public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Ilias Apalodimas <ilias.apalodimas@linaro.org>
To: Tom Rini <trini@konsulko.com>
Cc: Simon Glass <sjg@chromium.org>,
	U-Boot Mailing List <u-boot@lists.denx.de>,
	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: Tue, 24 Aug 2021 12:29:50 +0300	[thread overview]
Message-ID: <YSS8DpeUD2lQr/ty@Iliass-MBP> (raw)
In-Reply-To: <20210823200854.GH858@bill-the-cat>

Hi Tom,

> > > > > > >

[...]

> > > > > > > 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!
> > > > >
> > > > > Here you are talking about scanning for a bootflow. It is not actually
> > > > > OS-specific. If it were, there would be no point to this :-)
> > > > >
> > > > > If you look in the distro scripts you will see 'scan_dev_for_efi' (and
> > > > > also scan_dev_for_scrips). At least the first needs to be implemented
> > > > > a bit like the distro boot is at present. So far I have only
> > > > > implemented scan_dev_for_extlinux (plus pxe) as it is enough to show
> > > > > the concept.
> > > > >
> > > > > Adding EFI it's likely to be about the same amount of code as distro.c
> > > > > at present, perhaps a little less since we don't have the network
> > > > > case. It is used by Fedora 34, I believe, so is easy enough for me to
> > > > > do.  But I wanted to get something out as the concept is visible in
> > > > > this series.
> > > >
> > > > OK, good.  I'd certainly like to see how this looks with EFI added.
> > > >
> > >
> > > What would be the order preference after scanning?
> > 
> > (from other thread)
> > With distro boot this is done with an environment variable, as I
> > understand it. That is not implemented in this series, but is easy to
> > do and is in the TODO. For now it can only be done with aliases to set
> > the order of the bootmethods and that requires adding to the DT, so I
> > don't think it scales well. I'm open to ideas though.
> > 
> > >
> > > Keep in mind that our efibootmgr is pretty complete wrt to booting.
> > > It can even support booting multiple OS'es without GRUB2 (even loading
> > > different initrds is supported).  Ideally (and assuming we want to support
> > > EFI booting for more devices), I would map existing extlinux configs into
> > > efibootmgr entries.
> > 
> > Yes I understand. I believe distro boot supports multiple OSes too,
> > but perhaps only on different media? I'm not sure. Anywayt ere are
> > always going to be people who don't want or need to use EFI (or grub)
> 
> Well, here's where I'm a bit curious.  I have seen at least twice "wait,
> EFI stub in the linux kernel adds how much?" type commentary.  I don't
> know how prevalent that type of concern will really be given just how
> often I don't see the Linux kernel config shrunk down from what the
> reference platform started with.  And as Ilias is pointing out, you can
> do _a_lot_ with efibootmgr and without grub/whatever.  And both Arm (the
> company) and RISC-V (the overarching organization) are both pushing to
> standardize around the idea that if you're on something bigger than an
> MCU, EFI-the-ABI should get you up and running.

Exactly. Keep in mind that RISC-V is looking into EBBR as well, so this is
far from an 'Arm thing'. Moreover, the efi stub side of the kernel for
risc-v, will only load an initrd using the EFI_LOAD_FILE2 protocol we added
support for. So right now the only way to properly boot a RISC-V with EFI is
through the manager.

Regards
/Ilias
> 
> -- 
> Tom



  reply	other threads:[~2021-08-24  9:30 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 ` [PATCH 00/28] Initial implementation of bootmethod/bootflow Tom Rini
2021-08-19 14:25   ` 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 [this message]
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=YSS8DpeUD2lQr/ty@Iliass-MBP \
    --to=ilias.apalodimas@linaro.org \
    --cc=daniel.schwierzeck@gmail.com \
    --cc=dennis@ausil.us \
    --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=trini@konsulko.com \
    --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