[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1229391447.27573.6.camel@think.oraclecorp.com>
Date: Mon, 15 Dec 2008 20:37:27 -0500
From: Chris Mason <chris.mason@...cle.com>
To: Kay Sievers <kay.sievers@...y.org>
Cc: Andreas Dilger <adilger@....com>,
Andrew Morton <akpm@...ux-foundation.org>,
Stephen Rothwell <sfr@...b.auug.org.au>,
linux-kernel@...r.kernel.org, linux-btrfs@...r.kernel.org,
linux-fsdevel <linux-fsdevel@...r.kernel.org>
Subject: Re: Btrfs trees for linux-next
On Mon, 2008-12-15 at 23:55 +0100, Kay Sievers wrote:
> On Mon, Dec 15, 2008 at 22:03, Andreas Dilger <adilger@....com> wrote:
> > On Dec 11, 2008 09:43 -0500, Chris Mason wrote:
> >> The multi-device code uses a very simple brute force scan from userland
> >> to populate the list of devices that belong to a given FS. Kay Sievers
> >> has some ideas on hotplug magic to make this less dumb. (The scan isn't
> >> required for single device filesystems).
> >
> > This should use libblkid to do the scanning of the devices, and it can
> > cache the results for efficiency. Best would be to have the same LABEL+UUID
> > for all devices in the same filesystem, and then once any of these devices
> > are found the mount.btrfs code can query the rest of the devices to find
> > the remaining parts of the filesystem.
>
> Which is another way to do something you should not do that way in the
> first place, just with a library instead of your own code.
>
Well, its the same library everyone else is using to do things they
shouldn't be doing ;)
I'm very interested in your new scheme for device discovery. It seems
like the best and most reasonable way to go forward.
But, tossing things into libblkid also makes sense. It isn't much work,
and it allows btrfs to fit in with the established norms while we
experiment with new and better ways to hook it all together.
-chris
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists