All of lore.kernel.org
 help / color / mirror / Atom feed
From: Daniel Kiper <dkiper@net-space.pl>
To: Chris Murphy <lists@colorremedies.com>
Cc: grub-devel@gnu.org, behlendorf1@llnl.gov,
	chris.coulson@canonical.com, george.wilson@delphix.com,
	javierm@redhat.com, konrad.wilk@oracle.com,
	lists@colorremedies.com, luizluca@gmail.com, mahrens@delphix.com,
	mchang@suse.com, pcd@delphix.com, phcoder@gmail.com,
	pjones@redhat.com, sebastien.roy@delphix.com, tsoome@me.com,
	xnox@ubuntu.com
Subject: Re: RFC: A partition for grubenv, etc.
Date: Wed, 28 Jul 2021 14:23:34 +0200	[thread overview]
Message-ID: <20210728122334.zl5obnjymlq626zy@tomti.i.net-space.pl> (raw)
In-Reply-To: <CAJCQCtSRspHMcw0CG06F3VL5uheJr7rC+3=h4B5wk5AW=9KVLw@mail.gmail.com>

CC-ing a few folks who may be interested in this thread...

First of all, sorry for late reply but I am really busy. I hope I will
be able to reply in more timely manner starting from now.

On Thu, May 27, 2021 at 04:08:45PM -0600, Chris Murphy wrote:
> On Thu, May 27, 2021 at 2:59 AM Michael Chang <mchang@suse.com> wrote:
> >
> > On Wed, May 26, 2021 at 08:36:05PM -0600, Chris Murphy wrote:
> > > So is the next era going to be we recommend /boot on FAT?
> >
> > No. I meant a new partition type only for grubenv files and keep
> > everything else as is. The filesystem can be extX or FAT whichever is
> > writable by grub.  For efi, we probably don't need to define new
> > partition, as we can reuse the efi system partition.
>
> Or use the same partition for consistency instead of yet another exception?
>
> But also, GRUB strictly speaking doesn't write to any file system.
> It's overwriting a sector or two, no file system metadata is updated,
> hence the problem.
>
> > It is easier to handle in script as we only need to point to the new
> > writable location if available, the concern to admin is that we also
> > have to define mount point for linux (eg /boot/grubenv).
>
> That'd be three partitions needed by BIOS GRUB: BIOS Boot, grubenv,
> and /boot. Plus ~440 bytes on LBA 0, quasi-partition. Seems like a
> lot.
>
> What about a FAT formatted file system at /boot/grub and everything
> just lives there? Deprecate BIOS Boot, use blocklists instead. Grubenv
> partition proposal isn't needed, it can just stay where it is,
> unchanged and can be reliably written to. And whether UEFI or BIOS,
> grub looks to /boot/grub/ for all modules and configuration files?

At this point I am not fully convinced the /boot/grub should be on
a separate filesystem. Though after going through this thread it seems
to me we should add full write support to the GRUB for a simple file
system. The FAT looks like a good candidate. Does not it? This way we
will be able to support writable grubenv without any limitations and have
flexibility to add other writable files without bigger hassle if needed.

Additionally, it is worth mentioning the [1] work which adds a save_env
redirection layer. It allows us to choose target for grubenv. However,
if we want make it more generic then we could redirect any writes to any
target which can be potentially useful.

Last but not least, I think we do not need to change anything in current
partitions usage. In particular it seems to me the BIOS Boot Partition
usage should stay as is.

Daniel

[1] https://lists.gnu.org/archive/html/grub-devel/2020-03/msg00126.html


  reply	other threads:[~2021-07-28 12:23 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-05-25 22:58 RFC: A partition for grubenv, etc Chris Murphy
2021-05-26  8:07 ` Konrad Rzeszutek Wilk
2021-05-26  8:35   ` Toomas Soome
2021-05-27  2:23   ` Chris Murphy
2021-05-26  9:16 ` Michael Chang
2021-05-27  2:36   ` Chris Murphy
2021-05-27  8:59     ` Michael Chang
2021-05-27 18:49       ` Luiz Angelo Daros de Luca
2021-05-27 23:10         ` Chris Murphy
2021-05-28  3:42         ` Michael Chang
2021-05-27 22:08       ` Chris Murphy
2021-07-28 12:23         ` Daniel Kiper [this message]
2021-09-24  2:59           ` Chris Murphy
2021-10-26 14:22             ` Daniel Kiper

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=20210728122334.zl5obnjymlq626zy@tomti.i.net-space.pl \
    --to=dkiper@net-space.pl \
    --cc=behlendorf1@llnl.gov \
    --cc=chris.coulson@canonical.com \
    --cc=george.wilson@delphix.com \
    --cc=grub-devel@gnu.org \
    --cc=javierm@redhat.com \
    --cc=konrad.wilk@oracle.com \
    --cc=lists@colorremedies.com \
    --cc=luizluca@gmail.com \
    --cc=mahrens@delphix.com \
    --cc=mchang@suse.com \
    --cc=pcd@delphix.com \
    --cc=phcoder@gmail.com \
    --cc=pjones@redhat.com \
    --cc=sebastien.roy@delphix.com \
    --cc=tsoome@me.com \
    --cc=xnox@ubuntu.com \
    /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.