All of lore.kernel.org
 help / color / mirror / Atom feed
From: Simon Glass <sjg@chromium.org>
To: Ilias Apalodimas <ilias.apalodimas@linaro.org>
Cc: Tom Rini <trini@konsulko.com>,
	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: Wed, 25 Aug 2021 07:11:46 -0600	[thread overview]
Message-ID: <CAPnjgZ3rjLhhyd_eeuMG2PNtd7uaJ5Cr6KVWdVRxUY1KX_5JhA@mail.gmail.com> (raw)
In-Reply-To: <YSS8DpeUD2lQr/ty@Iliass-MBP>

Hi Ilias,

On Tue, 24 Aug 2021 at 03:30, Ilias Apalodimas
<ilias.apalodimas@linaro.org> wrote:
>
> 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.

I had heard that RISC-V was just going to use UEFI (and not U-Boot),
but perhaps that is not correct. So far I have not really done
anything with RISC-V so am not familiar with it.

Regards,
Simon

  reply	other threads:[~2021-08-25 13:12 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
2021-08-25 13:11               ` Simon Glass [this message]
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=CAPnjgZ3rjLhhyd_eeuMG2PNtd7uaJ5Cr6KVWdVRxUY1KX_5JhA@mail.gmail.com \
    --to=sjg@chromium.org \
    --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=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 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.