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 <darrick.wong@xxxxxxxxxx> > > > Cc: TR Reardon <thomas_reardon@xxxxxxxxxxx> > > > > 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@xxxxxxxxxxxxxxx > > 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@xxxxxxxxxxxxxxx > 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@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html