grub-devel.gnu.org archive mirror
 help / color / mirror / Atom feed
From: Andrei Borzenkov <arvidjaar@gmail.com>
To: Peter Palfrader <weasel@debian.org>
Cc: The development of GNU GRUB <grub-devel@gnu.org>, 858832@bugs.debian.org
Subject: Re: [weasel@debian.org: Bug#858832: calls efibootmgr with invalid options]
Date: Tue, 28 Mar 2017 20:59:30 +0300	[thread overview]
Message-ID: <cd9d9dd1-37e5-32c8-915e-7fb7e346bf73@gmail.com> (raw)
In-Reply-To: <20170328063411.GJ10005@sarek.noreply.org>

28.03.2017 09:34, Peter Palfrader пишет:
> } /dev/md2        953M  176K  953M   1% /boot/efi

Sorry, that's not going to work. Even assuming that grub can map from
Linux MD to underlying physical device + partition (as that is what
efibootmgr needs, we cannot simply pass /dev/md2 to it), this will
obviously fail unless metadata is 0.90 or 1.0 (and current mdadm default
is 1.2). And even if you are careful enough - EFI application is free to
write to ESP, so unless you have EFI driver for Linux MD it is no go to
try to mirror it, as you are never sure whether mirrors are identical
after boot. And if you have EFI driver for Linux MD you need to modify
efibootmgr to understand it.

We need to return better error message here to explain what happens
though, that's the actual bug.

The (IMHO) correct way to handle ESP redundancy is to call "grub-install
/path/to/first/ESP /path/to/second/ESP". I have half-finished patch
allowing this, but it will need distributions cooperation to actually be
useful.


  reply	other threads:[~2017-03-28 17:59 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-27 12:41 [weasel@debian.org: Bug#858832: calls efibootmgr with invalid options] Colin Watson
2017-03-27 13:08 ` Andrei Borzenkov
2017-03-27 13:09   ` Peter Palfrader
2017-03-27 16:25 ` Andrei Borzenkov
2017-03-28  6:34   ` Peter Palfrader
2017-03-28 17:59     ` Andrei Borzenkov [this message]
2017-04-03 12:01       ` Steve McIntyre
2017-04-03 20:22       ` Patch to support ESP redundancy [was: Re: [weasel@debian.org: Bug#858832: calls efibootmgr with invalid options]] Francesco Poli

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=cd9d9dd1-37e5-32c8-915e-7fb7e346bf73@gmail.com \
    --to=arvidjaar@gmail.com \
    --cc=858832@bugs.debian.org \
    --cc=grub-devel@gnu.org \
    --cc=weasel@debian.org \
    /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).