From: Eric Sandeen <sandeen@redhat.com>
To: "Ricardo M. Correia" <Ricardo.M.Correia@Sun.COM>
Cc: Andreas Dilger <adilger@Sun.COM>, "Theodore Ts'o" <tytso@mit.edu>,
linux-ext4@vger.kernel.org, Karel Zak <kzak@redhat.com>
Subject: Re: [PATCH e2fsprogs] Add ZFS detection to libblkid
Date: Mon, 13 Apr 2009 14:27:19 -0500 [thread overview]
Message-ID: <49E39217.60200@redhat.com> (raw)
In-Reply-To: <1239650325.18123.35.camel@localhost>
Ricardo M. Correia wrote:
> Hi Eric,
>
> On Seg, 2009-04-06 at 13:13 -0700, Eric Sandeen wrote:
>> Can you perhaps just chime in on these bugs & ask? you speak zfs better
>> than I do...
>>
>> https://bugzilla.redhat.com/show_bug.cgi?id=494070
>> https://bugzilla.redhat.com/show_bug.cgi?id=490795
>
> Sorry for taking a while to get back to you.
>
> I've just looked at these bugs and everything seems to make more sense
> now. Those partitions are actually Solaris partitions, which contain
> their own partition table inside.
>
> So in bug 494070, the /dev/sda2 partition is internally subdivided into
> what Solaris calls "slices" (which are similar to partitions) and then
> the root ZFS pool/filesystem is stored inside one of these slices. So in
> fact, the ZFS pool is not directly in /dev/sda2, it's somewhere inside
> it. This is why the ZFS magic numbers don't seem to be in the right
> place.
>
> These slices don't seem to be visible to that Linux system, which I
> suspect is due to the kernel not being compiled with Solaris partition
> table support. If it were, other partitions (including the ZFS one)
> would show up (if my assumption is correct).
It actually is compiled with that... but then we didn't look at
/proc/partitions (the kernel's view) in the bug but rather fdisk output,
which probably doesn't understand this at all.
> So from looking at the libblkid magic offsets, it seems that ext3 magic
> value is stored between 1K-2K, which means that it will fall into the
> boot slice, not in the ZFS slice. So AFAICS this bug doesn't have
> anything to do with ZFS, i.e., the same thing would happen if the root
> filesystem of the OpenSolaris installation was UFS.
ok, makes sense. (maybe blkid should recognize sda2 as being this
special sort of partition and stop there...)
> It'd be nice if OpenSolaris zeroed the boot slice when it is installed,
> but on the other hand, should the Anaconda installer fail just because
> it can't mount a (possibly corrupted/leftover) filesystem?
Generally, no; and anaconda now will not fail if the mount simply fails.
But if the mount attempt results in a kernel oops there's not much
anaconda can do... the filesystem that attempts the mount should be
robust enough to not oops as well, of course...
> I can file a bug for OpenSolaris if you feel strongly about this.
Well, it's always nice to be able to recognize a partition; blkid just
can't do this reliably if some tools leave old signatures lying around.
So zeroing it out would be the "polite" thing to do in any case. :)
Thanks,
-Eric
> Thanks,
> Ricardo
>
>
next prev parent reply other threads:[~2009-04-13 19:27 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-30 18:20 [PATCH e2fsprogs] Add ZFS detection to libblkid Ricardo M. Correia
2008-06-02 7:41 ` Karel Zak
2008-06-02 15:03 ` Ricardo M. Correia
[not found] ` <1212418863.7337.34.camel@localhost>
2008-06-02 20:58 ` Karel Zak
2008-06-02 21:38 ` Andreas Dilger
2008-06-02 22:31 ` Karel Zak
2008-06-02 23:11 ` Andreas Dilger
2008-06-03 0:05 ` Theodore Tso
2008-06-03 1:11 ` Theodore Tso
2009-04-04 2:39 ` Eric Sandeen
2009-04-04 13:04 ` Eric Sandeen
2009-04-04 21:25 ` Andreas Dilger
2009-04-04 21:46 ` Eric Sandeen
2009-04-06 6:25 ` Andreas Dilger
2009-04-06 19:22 ` Ricardo M. Correia
2009-04-06 20:13 ` Eric Sandeen
2009-04-13 19:18 ` Ricardo M. Correia
2009-04-13 19:27 ` Eric Sandeen [this message]
2009-04-06 20:35 ` Karel Zak
2009-04-07 7:40 ` Andreas Dilger
2009-04-13 19:29 ` Ricardo M. Correia
2009-04-17 9:51 ` Karel Zak
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=49E39217.60200@redhat.com \
--to=sandeen@redhat.com \
--cc=Ricardo.M.Correia@Sun.COM \
--cc=adilger@Sun.COM \
--cc=kzak@redhat.com \
--cc=linux-ext4@vger.kernel.org \
--cc=tytso@mit.edu \
/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.