grub-devel.gnu.org archive mirror
 help / color / mirror / Atom feed
From: Steve McIntyre <steve@einval.com>
To: The development of GNU GRUB <grub-devel@gnu.org>
Cc: Peter Palfrader <weasel@debian.org>, 858832-done@bugs.debian.org
Subject: Re: [weasel@debian.org: Bug#858832: calls efibootmgr with invalid options]
Date: Mon, 3 Apr 2017 13:01:38 +0100	[thread overview]
Message-ID: <20170403120138.GA5825@einval.com> (raw)
In-Reply-To: <cd9d9dd1-37e5-32c8-915e-7fb7e346bf73@gmail.com>

On Tue, Mar 28, 2017 at 08:59:30PM +0300, Andrei Borzenkov wrote:
>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.

ACK, good catch.

>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.

More than happy to help with that - it's something I've been wanting
for a while.

-- 
Steve McIntyre, Cambridge, UK.                                steve@einval.com
"I used to be the first kid on the block wanting a cranial implant,
 now I want to be the first with a cranial firewall. " -- Charlie Stross



  reply	other threads:[~2017-04-03 12:02 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
2017-04-03 12:01       ` Steve McIntyre [this message]
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=20170403120138.GA5825@einval.com \
    --to=steve@einval.com \
    --cc=858832-done@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).