u-boot.lists.denx.de archive mirror
 help / color / mirror / Atom feed
From: Ilias Apalodimas <ilias.apalodimas@linaro.org>
To: Janne Grunau <j@jannau.net>
Cc: Heinrich Schuchardt <xypron.glpk@gmx.de>,
	Mark Kettenis <mark.kettenis@xs4all.nl>,
	u-boot@lists.denx.de, sjg@chromium.org, marcan@marcan.st
Subject: Re: RFC: Handling of multiple EFI System Partitions
Date: Mon, 19 Dec 2022 12:52:30 +0200	[thread overview]
Message-ID: <Y6BCbq9OUlCD8qTZ@hades> (raw)
In-Reply-To: <20221219094817.GA4629@jannau.net>

Hi Janne, 

[...]

> > >function that can be called from board-specific code that sets the 
> > >UUID.
> > >
> > >Thoughts?  Would such a feature be useful on other hardware 
> > >platforms?
> > 
> > efi/boot/bootaarch64.efi is only a fallback if load options are not 
> > set up. Once the operating system has generated a load option it is 
> > not used anymore.
> 
> Setting load options from operation systems is currently not 
> implemented. The only readily available method to store variables is a 
> file in the ESP. This obviously can not be supported as UEFI runtime 
> service while the operation system is using the same disk.

Yes, but I'd skip the 'currently not implemented' part.  If you store your EFI
variables in a file on the ESP, we will *never* be able to (sanely) support SetVariable
at runtime. 

> It might be possible to use NOR flash as UEFI variable store. This could 
> cause issues with the primary boot loader iboot which we can not avoid 
> or with macOS in dual boot configurations.

It is possible.  In fact we already have code in U-Boot that stores the EFI
variables in an RPMB partition of the eMMC.  We also have (unfortunately
not yet upstreamed) code that stores the in an i2c eeprom in the secure
world. 

This is again situational though and none of these applies to MBPs.
SetVariable at runtime in an RPMB comes with it's own set of problems.
You basically need to replace the runtime services with OS provided ones...
The I2C one works fine.

But letting the implementation details aside, what we need to keep in mind,
is that being able to support SetVariable-RT primarily depends on the *hardware*
and there's gonna be hardware we'll never be able to support this. 

> 
> > The MacBooks only have one drive. Why would you want two ESPs on one 
> > drive?
> 
> The lack of support for setting boot variable form the operating system 
> makes it hard to support multiple OS with a single ESP. The systems 
> comes with an accessible graphical boot picker which should be preferred 
> over an u-boot based boot manager. Accessibility and the ability to boot 
> macOS are the most important advantages.
> 
> At the current stage an u-boot based boot manager is not a viable option 
> since the devicetree can not be guaranteed to be backwards compatible.  
> The most recent breaking change was the addition of a phy phandle 
> reference for USB controller nodes for USB3 support. Older kernel will 
> obviously miss a driver for the phy and at least on Linux the USB 
> controller will not be initialized. This breaks USB completely.
> 
> The devicetree is transformed by the loader before u-boot in a 
> non-trivial way. If u-boot were to use kernel version specific DTB 
> templates from the ESP it would require a non-trivial amount of code to 
> update the template by the information added by the loader.
> 
> > Why can't the Asahi team use the current UEFI bootflow? We should 
> > avoid unneeded deviations. Can the current deviations be removed in 
> > Asahi Linux?
> 
> See above, the main reason is the lack of support for setting UEFI 
> variables at runtime from operating systems. Even if that those were 
> supported it's impossible to upgrade all installed operating systems 
> when the devicetree changes in a non backward compatible way due to 
> additional hardware support.
> 

Ok fair enough,  I'll talk to Heinrich and see if The current idea from
Mark can be wired up without causing too much of a mess.

> At the current stage it is not possible to support UEFI bootflow.
> 
> Best regards
> 
> Janne


Cheers
/Ilias

  reply	other threads:[~2022-12-19 10:52 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-12-18 14:21 RFC: Handling of multiple EFI System Partitions Mark Kettenis
2022-12-18 19:40 ` Heinrich Schuchardt
2022-12-19  9:48   ` Janne Grunau
2022-12-19 10:52     ` Ilias Apalodimas [this message]
2022-12-19 11:18       ` Hector Martin
2022-12-19 14:29         ` Ilias Apalodimas
2022-12-19 10:17   ` Hector Martin
2022-12-19 10:51   ` Mark Kettenis

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=Y6BCbq9OUlCD8qTZ@hades \
    --to=ilias.apalodimas@linaro.org \
    --cc=j@jannau.net \
    --cc=marcan@marcan.st \
    --cc=mark.kettenis@xs4all.nl \
    --cc=sjg@chromium.org \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).