From: "Vladimir 'φ-coder/phcoder' Serbinenko" <phcoder@gmail.com>
To: The development of GNU GRUB <grub-devel@gnu.org>
Subject: Re: Problems with grub-reboot/savedefault/default=saved
Date: Tue, 05 Jan 2010 12:33:23 +0100 [thread overview]
Message-ID: <4B432383.10604@gmail.com> (raw)
In-Reply-To: <20100105110855.GD5847@riva.ucam.org>
[-- Attachment #1: Type: text/plain, Size: 1653 bytes --]
Colin Watson wrote:
>> 4: Even with the first grub-reboot fix the default is still not restored
>> when grubenv is not writable ( /boot on lvm for example ). Since
>> grub-reboot can't work without a writable grubenv it's at least safer to
>> boot into the "real" default instead of the "temporary" default. The
>> fourth patch sets default=$prev_saved_default if save_env fails.
>> Grub-reboot ( the utility ) should also complain when grubenv isn't
>> writable by grub. What is the best way to determine that grubenv likely is
>> or isn't writable by grub?
>>
>
> I don't understand why /boot on LVM should mean that grubenv is not
> writable. The point of grubenv is to be a short chunk of contiguous
> reserved disk space which can be written by GRUB without needing any
> special filesystem intelligence. Surely LVM doesn't split up those 1024
> bytes into different physical extents?
>
>
Consider a RAID-1 (mirror). Currently GRUB2 uses only one disk of the
mirror. It may even be possible that second disk is inaccessible through
currently loaded drivers. But writing would need to update all copies.
And what to do if second disk isn't accessible? Same problem is present
in other RAID levels too.
ZFS and btrfs have checksums. If you update a block you have to update
its checksum too, and checksum of block containg checksum and so on. So
at the end of the day the whole is more complicated that what we wanted.
And a mistake in writing code may lead to corruption of unrelated data.
IMO for long-term we need a better place for grubenv.
--
Regards
Vladimir 'φ-coder/phcoder' Serbinenko
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 293 bytes --]
next prev parent reply other threads:[~2010-01-05 11:33 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-12-16 22:28 Problems with grub-reboot/savedefault/default=saved Jordan Uggla
2009-12-20 8:37 ` Jordan Uggla
2010-01-05 11:23 ` Colin Watson
2010-01-05 18:30 ` Embedding area too small... (GRUB2 on software RAID1) Lapohos Tibor
2010-01-05 18:30 ` Lapohos Tibor
2009-12-23 2:03 ` Problems with grub-reboot/savedefault/default=saved Colin Watson
2010-01-05 11:08 ` Colin Watson
2010-01-05 11:33 ` Vladimir 'φ-coder/phcoder' Serbinenko [this message]
2010-01-05 11:48 ` Colin Watson
2010-01-07 5:22 ` Jordan Uggla
2010-02-25 13:38 ` Colin Watson
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=4B432383.10604@gmail.com \
--to=phcoder@gmail.com \
--cc=grub-devel@gnu.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 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.