Re: [PATCH 7/7] mke2fs: only print the low-level file system stats in verbose mode

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 30 Apr 2014, Theodore Ts'o wrote:

> Date: Wed, 30 Apr 2014 10:01:50 -0400
> From: Theodore Ts'o <tytso@xxxxxxx>
> To: Lukáš Czerner <lczerner@xxxxxxxxxx>
> Cc: Ext4 Developers List <linux-ext4@xxxxxxxxxxxxxxx>
> Subject: Re: [PATCH 7/7] mke2fs: only print the low-level file system stats in
>      verbose mode
> 
> On Wed, Apr 30, 2014 at 01:22:52PM +0200, Lukáš Czerner wrote:
> > On Sat, 26 Apr 2014, Theodore Ts'o wrote:
> > 
> > > Date: Sat, 26 Apr 2014 20:00:34 -0400
> > > From: Theodore Ts'o <tytso@xxxxxxx>
> > > To: Ext4 Developers List <linux-ext4@xxxxxxxxxxxxxxx>
> > > Cc: Theodore Ts'o <tytso@xxxxxxx>
> > > Subject: [PATCH 7/7] mke2fs: only print the low-level file system stats in
> > >     verbose mode
> > 
> > While I kind of like this, because mke2fs is really quite verbose as
> > it is. However I am afraid that this will break scripts for people.
> 
> Are there scripts that are really trying to parse the output of
> mke2fs?  The output hasn't really been _that_ stable.  I'm sure we've
> added stuff in the past, althoughg admittedly the this is the first
> time that we would be removing stuff.

Yes, that's what I worry about. Addition would most likely not break
grep, but removal will surely do. I am sure that there are admins
doing that.

It also does not help that other file systems are printing out
various "low level" information.

Despite the fact that parsing mkfs output is not the best thing to
do I am on the fence whether we want to break it or not.

> 
> > Also there are actually some useful information in that output like
> > block size, size of the file system and file system label if
> > specified. Also maybe having UUID in there will be also useful.
> 
> The block sizs is pretty much always 4k, and the file system label is
> only there if the user specified one on the command line.  I can see
> how the size and UUID might be useful, though.
> 
> How about if we just print the size and UUID?

Sounds good to me.

-Lukas

> 
> 						- Ted
> 

[Index of Archives]     [Reiser Filesystem Development]     [Ceph FS]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite National Park]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Device Mapper]     [Linux Media]

  Powered by Linux