From: "Javier Martín" <lordhabbit@gmail.com>
To: The development of GRUB 2 <grub-devel@gnu.org>
Subject: Re: grub-probe detects ext4 wronly as ext2
Date: Fri, 04 Jul 2008 12:34:50 +0200 [thread overview]
Message-ID: <1215167690.26019.78.camel@localhost> (raw)
In-Reply-To: <ca0f59980807032349w600fba75ncf58f8302d0052cd@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 2881 bytes --]
I know I promised not to keep posting but... I just cannot xD
El vie, 04-07-2008 a las 14:49 +0800, Bean escribió:
> Hi,
>
> This is an example of good practice, which check the driver
> by function, not by flags.
This will find and check core.img. As my previous post showed, one file
being readable in a partition with an incompatible flag does not
guarantee that every file will be. Thus, the conservative policy for
grub-probe would be to also check that the FS is "theoretically"
accesible (i.e. the ext2/whatever driver is fine with its flags) in
addition to the current "practical" check.
>
> The flags are a little vague, for example,
> if they change btree structure in the future, this would introduce
> another flag, but we don't need to worry about it, because we don't
> use it at all.
The btree flag (and any of its modifications) is ROCOMPAT, which indeed
allows us not to worry about it at all, because our driver is read only.
The proposed patch addresses only INCOMPAT flags, and it even adds a
mechanism to ignore some of them because we deem them not to create real
incompatibilities (like needs-recovery).
>
> If we can ensure the boot partition can be accessed, then the problem
> is solved. Yes, there may be other partition using ext4 that we can't
> access, but remember that linux loader will check for signatures as
> well, a corrupted kernel would not be loaded. So the difference is
> merely the error message users sees, unknown filesystem or invalid
> kernel.
I didn't know the "linux" loader checked signatures. Do the "multiboot",
"multiboot2" and "bsd" loaders do the same? In the platforms where it is
able to load a file, does the "chainloader" loader perform any checks?
So in that cases difference would be between "unknown filesystem" and
"the FSM knows what".
>
> And, grub WILL follow the evolution the extN, because it's the primary
> boot loader for linux. The only reason we don't have ext4 support at
> present is because it's not stable. If major distro starts to use it
> as default, we would have to support it as well.
Please... ReiserFS was used for long in many distros and GRUB2 didn't
support it until 1.96 - even with GRUB Legacy having implemented it long
ago. I literally waited years for it to be included! Besides, as I
already said, we cannot win in a race against the future: new features,
some of them incompatible, are introduced ""constantly"" (every few
years) in extN, and until we get to know about them and at least decide
whether they can be safely ignored, the sane behavior is to obey them
and reject access (except if the user override is enabled). Yes, we can
implement ext4 and possibly even before it's released as stable, but
could you please start implementing ext7 so that we don't have to worry
about its incompatibilities when it comes?
[-- Attachment #2: Esta parte del mensaje está firmada digitalmente --]
[-- Type: application/pgp-signature, Size: 827 bytes --]
next prev parent reply other threads:[~2008-07-04 10:35 UTC|newest]
Thread overview: 68+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-06-29 18:11 grub-probe detects ext4 wronly as ext2 Felix Zielcke
2008-06-29 18:46 ` Javier Martín
2008-06-29 19:17 ` Bean
2008-06-29 19:53 ` Javier Martín
2008-06-29 21:19 ` Robert Millan
2008-06-30 3:02 ` Javier Martín
2008-06-30 7:10 ` Felix Zielcke
2008-06-30 11:14 ` Isaac Dupree
2008-06-30 12:12 ` Javier Martín
2008-06-30 12:27 ` Bean
2008-06-30 12:43 ` Javier Martín
2008-07-01 16:08 ` Robert Millan
2008-07-01 16:25 ` Pavel Roskin
2008-07-01 18:42 ` Javier Martín
2008-07-01 19:01 ` Pavel Roskin
2008-07-01 20:48 ` Robert Millan
2008-07-01 23:05 ` Javier Martín
2008-07-01 23:28 ` Javier Martín
2008-07-02 14:22 ` Robert Millan
2008-07-02 16:03 ` Pavel Roskin
2008-07-02 19:32 ` Javier Martín
2008-07-03 14:02 ` Robert Millan
2008-07-03 14:21 ` Isaac Dupree
2008-07-03 17:07 ` Javier Martín
2008-07-04 0:08 ` Robert Millan
2008-07-04 1:20 ` Javier Martín
2008-08-05 17:23 ` Felix Zielcke
2008-08-06 10:36 ` Felix Zielcke
2008-08-11 0:35 ` Javier Martín
2008-08-11 7:56 ` Felix Zielcke
2008-07-04 1:32 ` Javier Martín
2008-07-04 6:49 ` Bean
2008-07-04 8:33 ` Felix Zielcke
2008-07-04 10:34 ` Javier Martín [this message]
2008-07-04 11:29 ` Bean
2008-07-04 12:00 ` Javier Martín
2008-07-04 14:09 ` Robert Millan
2008-07-04 14:33 ` Javier Martín
2008-07-04 14:11 ` Bean
2008-07-04 14:34 ` Javier Martín
2008-07-04 14:04 ` Robert Millan
2008-07-04 14:23 ` Robert Millan
2008-07-04 14:21 ` Robert Millan
2008-07-04 14:45 ` Javier Martín
2008-07-04 18:57 ` Robert Millan
2008-07-04 20:41 ` Javier Martín
2008-07-05 12:07 ` Robert Millan
2008-07-05 18:36 ` Javier Martín
2008-07-16 15:09 ` Javier Martín
2008-07-16 15:27 ` Felix Zielcke
2008-07-16 16:38 ` Javier Martín
2008-07-16 17:13 ` Felix Zielcke
2008-07-16 17:21 ` Felix Zielcke
2008-07-16 17:44 ` Felix Zielcke
2008-07-16 19:07 ` Javier Martín
2008-07-16 19:33 ` Felix Zielcke
2008-07-19 14:27 ` Robert Millan
2008-08-11 14:14 ` Javier Martín
2008-08-27 13:58 ` Felix Zielcke
2008-08-30 11:17 ` Robert Millan
2008-08-30 21:28 ` Javier Martín
2008-09-24 17:05 ` Javier Martín
2009-02-04 7:41 ` Felix Zielcke
2009-02-04 13:08 ` Javier Martín
2009-02-07 19:30 ` Felix Zielcke
2009-02-07 23:54 ` Javier Martín
2009-02-08 0:28 ` Robert Millan
2008-07-01 16:03 ` Robert Millan
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=1215167690.26019.78.camel@localhost \
--to=lordhabbit@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.