From: Chris Mason <clm@fb.com>
To: Miao Xie <miaox@cn.fujitsu.com>, <linux-btrfs@vger.kernel.org>
Subject: Re: [PATCH RFC 0/5] Scan all devices to build fs device list
Date: Sat, 6 Sep 2014 16:05:20 -0400 [thread overview]
Message-ID: <540B6900.7000307@fb.com> (raw)
In-Reply-To: <1409751393-5403-1-git-send-email-miaox@cn.fujitsu.com>
On 09/03/2014 09:36 AM, Miao Xie wrote:
> This patchset implements device list automatic building function. As we
> know, currently we need scan the devices to build device list by a user tool
> before mounting the filesystem, especially mount the filesystem after
> we re-install btrfs module. It is not convenient. This patchset can improve
> that problem. With this patchset, we will scan all the devices in the
> system to build the device list if we find the number of the devices
> is not right when we mount the filesystem. By this way, we needn't scan
> the device by the user tool and reduce the mount failure probability due
> to the incomplete device list.
Thanks for working on these patches, but I really prefer that we do
these scans from userspace. I know it introduces a few problems, but it
also solves a lot of complexity at the same time. And udev already has
most of the info we need, so I think our time is best spent fixing any
problems with the udev integration.
-chris
next prev parent reply other threads:[~2014-09-06 20:05 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-03 13:36 [PATCH RFC 0/5] Scan all devices to build fs device list Miao Xie
2014-09-03 13:36 ` [PATCH 1/5] block: export disk_class and disk_type for btrfs Miao Xie
2014-09-03 13:36 ` [PATCH 2/5] Btrfs: don't return btrfs_fs_devices if the caller doesn't want it Miao Xie
2014-09-03 13:36 ` [PATCH 3/5] Btrfs: restructure btrfs_scan_one_device Miao Xie
2014-09-03 13:36 ` [PATCH 4/5] Btrfs: restructure btrfs_get_bdev_and_sb and pick up some code used later Miao Xie
2014-09-03 13:36 ` [PATCH 5/5] Btrfs: scan all the devices and build the fs device list by btrfs's self Miao Xie
2014-09-06 11:48 ` Goffredo Baroncelli
2014-09-09 4:06 ` Miao Xie
2014-09-06 20:05 ` Chris Mason [this message]
2014-09-08 9:09 ` [PATCH RFC 0/5] Scan all devices to build fs device list David Sterba
2014-09-08 11:04 ` Anand Jain
2014-09-08 17:15 ` Goffredo Baroncelli
2014-09-09 3:26 ` Anand Jain
2014-09-09 20:31 ` Goffredo Baroncelli
2014-09-10 6:06 ` Anand Jain
2014-09-08 16:59 ` Goffredo Baroncelli
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=540B6900.7000307@fb.com \
--to=clm@fb.com \
--cc=linux-btrfs@vger.kernel.org \
--cc=miaox@cn.fujitsu.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 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.