From: "Austin S. Hemmelgarn" <ahferroin7@gmail.com>
To: Andrei Borzenkov <arvidjaar@gmail.com>,
Chris Murphy <lists@colorremedies.com>
Cc: Btrfs BTRFS <linux-btrfs@vger.kernel.org>
Subject: Re: GRUB writing to grubenv outside of kernel fs code
Date: Tue, 18 Sep 2018 15:11:55 -0400 [thread overview]
Message-ID: <26da8df8-45c9-f8e1-897c-8940898a4efd@gmail.com> (raw)
In-Reply-To: <55ee4d00-86ca-da77-8fe3-007010a89fca@gmail.com>
On 2018-09-18 14:38, Andrei Borzenkov wrote:
> 18.09.2018 21:25, Austin S. Hemmelgarn пишет:
>> On 2018-09-18 14:16, Andrei Borzenkov wrote:
>>> 18.09.2018 08:37, Chris Murphy пишет:
>>>> On Mon, Sep 17, 2018 at 11:24 PM, Andrei Borzenkov
>>>> <arvidjaar@gmail.com> wrote:
>>>>> 18.09.2018 07:21, Chris Murphy пишет:
>>>>>> On Mon, Sep 17, 2018 at 9:44 PM, Chris Murphy
>>>>>> <lists@colorremedies.com> wrote:
> ...
>>>>>>
>>>>>> There are a couple of reserve locations in Btrfs at the start and I
>>>>>> think after the first superblock, for bootloader embedding. Possibly
>>>>>> one or both of those areas could be used for this so it's outside the
>>>>>> file system. But other implementations are going to run into this
>>>>>> problem too.
>>>>>>
>>>>>
>>>>> That's what SUSE grub2 version does - it includes patches to redirect
>>>>> writes on btrfs to reserved area. I am not sure how it behaves in case
>>>>> of multi-device btrfs though.
>>>>
>>>> The patches aren't upstream yet? Will they be?
>>>>
>>>
>>> I do not know. Personally I think much easier is to make grub location
>>> independent of /boot, allowing grub be installed in separate partition.
>>> This automatically covers all other cases (like MD, LVM etc).
>> It actually is independent of /boot already. I've got it running just
>> fine on my laptop off of the EFI system partition (which is independent
>> of my /boot partition), and thus have no issues with handling of the
>> grubenv file. The problem is that all the big distros assume you want
>> it in /boot, so they have no option for putting it anywhere else.
>>
>
> This requires more than just explicit --boot-directory. With current
> monolithic configuration file listing all available kernels this file
> cannot be in the same location, it must be together with kernels (think
> about rollback to snapshot with completely different content). Or some
> different, more flexible configuration is needed.
Uh, no, it doesn't need to be with the kernels. Fedora stores it on the
ESP separate from the kernels (which are still on the boot partition) if
you use Secure Boot, and I'm doing the same (without secure boot)
without issue. You do have to explicitly set the `root` variable
correctly in the config though to get it to work though, and the default
upstream 'easy configuration' arrangement does not do this consistently.
It's not too hard to hack in though, and it's positively trivial if
you just write your own configuration files by hand like I do (no, I'm
not crazy, the default configuration generator just produces a
brobdingnagian monstrosity of a config that has tons of stuff I don't
need and makes invalid assumptions about how I want things invoked, and
the config syntax is actually not that hard).
>
> As is now grub silently assumes everything is under /boot. This turned
> out to be oversimplified.
No, it assumes everything is under whatever you told GRUB to set the
default value of the `prefix` variable to when you built the GRUB image,
which is automatically set to the path you pass to `--boot-directory`
when you use grub-install. This persists until you explicitly set that
variable to a different location, or change the `root` variable (but
GRUB still uses `prefix` for module look-ups if you just change the
`root` variable).
next prev parent reply other threads:[~2018-09-19 0:45 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-09-18 3:44 GRUB writing to grubenv outside of kernel fs code Chris Murphy
2018-09-18 4:21 ` Chris Murphy
2018-09-18 5:24 ` Andrei Borzenkov
2018-09-18 5:37 ` Chris Murphy
2018-09-18 18:16 ` Andrei Borzenkov
2018-09-18 18:25 ` Austin S. Hemmelgarn
2018-09-18 18:38 ` Andrei Borzenkov
2018-09-18 19:11 ` Austin S. Hemmelgarn [this message]
2018-09-19 5:10 ` Andrei Borzenkov
2018-09-18 19:00 ` Chris Murphy
2018-09-18 19:14 ` Austin S. Hemmelgarn
2018-09-18 18:57 ` Chris Murphy
2018-09-18 19:01 ` Andrei Borzenkov
2018-09-18 19:22 ` Chris Murphy
2018-09-18 19:13 ` Austin S. Hemmelgarn
2018-09-18 17:15 ` Goffredo Baroncelli
2018-09-18 18:52 ` Chris Murphy
2018-09-18 19:11 ` Goffredo Baroncelli
2018-09-18 19:34 ` Chris Murphy
2018-09-19 5:03 ` Duncan
2018-09-19 19:08 ` Goffredo Baroncelli
2018-09-19 19:34 ` Austin S. Hemmelgarn
2018-09-18 19:45 ` Chris Murphy
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=26da8df8-45c9-f8e1-897c-8940898a4efd@gmail.com \
--to=ahferroin7@gmail.com \
--cc=arvidjaar@gmail.com \
--cc=linux-btrfs@vger.kernel.org \
--cc=lists@colorremedies.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox