From: "Darrick J. Wong" Subject: Re: [PATCH 11/34] dumpe2fs: output cleanup Date: Mon, 13 Oct 2014 11:04:33 -0700 Message-ID: <20141013180433.GF12009@birch.djwong.org> References: <20140913221112.13646.3873.stgit@birch.djwong.org> <20140913221226.13646.37351.stgit@birch.djwong.org> <20140919162200.GO26995@thunk.org> <20140919200005.GI10150@birch.djwong.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: linux-ext4@vger.kernel.org, TR Reardon To: "Theodore Ts'o" Return-path: Received: from userp1040.oracle.com ([156.151.31.81]:47508 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754102AbaJMSEk (ORCPT ); Mon, 13 Oct 2014 14:04:40 -0400 Content-Disposition: inline In-Reply-To: <20140919200005.GI10150@birch.djwong.org> Sender: linux-ext4-owner@vger.kernel.org List-ID: On Fri, Sep 19, 2014 at 01:00:05PM -0700, Darrick J. Wong wrote: > On Fri, Sep 19, 2014 at 12:22:00PM -0400, Theodore Ts'o wrote: > > On Sat, Sep 13, 2014 at 03:12:26PM -0700, Darrick J. Wong wrote: > > > Don't display unused inodes twice, and make it clear that we're > > > printing a descriptor checksum. > > > > > > Signed-off-by: Darrick J. Wong > > > Cc: TR Reardon > > > > One problem with the current output format is that exceeds the 80 > > character line limit pretty blatently: > > > > Group 3: (Blocks 24577-32768) [INODE_UNINIT, BLOCK_UNINIT, ITABLE_ZEROED] > > Checksum 0x5bd9, unused inodes 2048 > > Backup superblock at 24577, Group descriptors at 24578-24578 > > Reserved GDT blocks at 24579-24833 > > Block bitmap at 261 (bg #0 + 260), csum 0x00000000, Inode bitmap at 269 (bg #0 + 268), csum 0x00000000 > > Inode table at 1042-1297 (bg #0 + 1041) > > 7935 free blocks, 2048 free inodes, 0 directories, 2048 unused inodes > > Free blocks: 24834-32768 > > Free inodes: 6145-8192 > > > > If we are printing the checksum, we probably need to insert a line > > break and indent before printing the Inode bitmap. Does that seem > > reasonable? > > Seems fine to me, since other BG fields get their own line anyway. Do you want > me to make a(nother) patch, or have you already fixed this up in git? Done, but it's a huge patch (~4700 lines) on account of having to change a lot of testcases' expect files. I could change the patch to emit the newline before the inode bitmap line only if metadata_csum is present. It would make the output less consistent, though. What do you think, Ted? --D > > --D > > > > - Ted > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-ext4" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- > To unsubscribe from this list: send the line "unsubscribe linux-ext4" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html