Re: [PATCH 1/3] btrfs-progs: random fixes of btrfs-filesystem documentation

2014-08-20 Thread David Sterba
On Tue, Aug 19, 2014 at 10:33:44AM -0500, Eric Sandeen wrote: > Seems like using /proc/partitions would make more sense in that case > than a recursive scan of every file under /dev, wouldn't it? > Any details on those reports? It does make sense. > I'm just wondering when you might possibly have

Re: [PATCH 1/3] btrfs-progs: random fixes of btrfs-filesystem documentation

2014-08-19 Thread Eric Sandeen
On 8/19/14, 10:10 AM, David Sterba wrote: > On Mon, Aug 11, 2014 at 10:05:52AM -0700, Eric Sandeen wrote: >> (What seems to be missing, though, is why would the user ever choose to use >> '-d?') > > That's a fallback method if blkid or udev are not available. We've had > reports in the past that

Re: [PATCH 1/3] btrfs-progs: random fixes of btrfs-filesystem documentation

2014-08-19 Thread David Sterba
On Mon, Aug 11, 2014 at 10:05:52AM -0700, Eric Sandeen wrote: > (What seems to be missing, though, is why would the user ever choose to use > '-d?') That's a fallback method if blkid or udev are not available. We've had reports in the past that this functionality should not be dropped. -- To unsu

Re: [PATCH 1/3] btrfs-progs: random fixes of btrfs-filesystem documentation

2014-08-11 Thread Eric Sandeen
> On Aug 11, 2014, at 4:51 PM, Satoru Takeuchi > wrote: > > Hi Eric, > ... > > > OK, I'll fix my patch based on your comment. > # Of course, I'll replace "(btrfs?)" with something proper words. I assumed it only scans btrfs but didn't know for sure :) > Can I add your "Signed-off-by" to m

Re: [PATCH 1/3] btrfs-progs: random fixes of btrfs-filesystem documentation

2014-08-11 Thread Satoru Takeuchi
Hi Eric, (2014/08/12 2:14), Eric Sandeen wrote: > On 8/11/14, 10:05 AM, Eric Sandeen wrote: >> On 8/11/14, 2:11 AM, Satoru Takeuchi wrote: >>> From: Satoru Takeuchi >>> >>> - Simplify and unify the description of both man and usage. >>> - Fix to show -m and -d is not exclusive >>> with "|

Re: [PATCH 1/3] btrfs-progs: random fixes of btrfs-filesystem documentation

2014-08-11 Thread Satoru Takeuchi
Hi Eric, (2014/08/12 2:05), Eric Sandeen wrote: > On 8/11/14, 2:11 AM, Satoru Takeuchi wrote: >> From: Satoru Takeuchi >> >> - Simplify and unify the description of both man and usage. >> - Fix to show -m and -d is not exclusive >> with "|||". >> - Add the description about short option

Re: [PATCH 1/3] btrfs-progs: random fixes of btrfs-filesystem documentation

2014-08-11 Thread Eric Sandeen
On 8/11/14, 10:05 AM, Eric Sandeen wrote: > On 8/11/14, 2:11 AM, Satoru Takeuchi wrote: >> From: Satoru Takeuchi >> >> - Simplify and unify the description of both man and usage. >> - Fix to show -m and -d is not exclusive >>with "|||". >> - Add the description about short options for "--mo

Re: [PATCH 1/3] btrfs-progs: random fixes of btrfs-filesystem documentation

2014-08-11 Thread Eric Sandeen
On 8/11/14, 2:11 AM, Satoru Takeuchi wrote: > From: Satoru Takeuchi > > - Simplify and unify the description of both man and usage. > - Fix to show -m and -d is not exclusive >with "|||". > - Add the description about short options for "--mounted" and >"--all-devices", "-m" and "-d" re

[PATCH 1/3] btrfs-progs: random fixes of btrfs-filesystem documentation

2014-08-11 Thread Satoru Takeuchi
From: Satoru Takeuchi - Simplify and unify the description of both man and usage. - Fix to show -m and -d is not exclusive with "|||". - Add the description about short options for "--mounted" and "--all-devices", "-m" and "-d" respectively. - Move the descriptions of options to "Option