On Thu, May 17, 2012 at 03:16:07PM +0800, Tao Ma wrote: > From: Tao Ma <boyu.mt@xxxxxxxxxx> > > In commit "e2fsck: Let end_blk to be the maximum value of u32." > we fix a bug where a normal fallocate will cause e2fsck complain. > So add it to the regression test suite. > > Signed-off-by: Tao Ma <boyu.mt@xxxxxxxxxx> I've changed the description of this commit to reflect the change in how I propose to fix the problem.... - Ted commit 8d12c46a22965179cae1e3b47778fdee5efeb513 Author: Tao Ma <boyu.mt@xxxxxxxxxx> Date: Sun Jun 10 23:56:30 2012 -0400 tests: add new test f_zero_extent_length If all of the extents in the last extent tree block (ETB) in a non-trivial extent tree contain uninitialized extents which are after the end of the file as defined by i_size, the hueristics will incorrectly estimate the last entry (and hence the node's e_len field) in the last entry of each level of the extent tree. As Tao Ma has noted, since e2fsck was requiring that the length (e_len) field of interior nodes be non-zero, this was causing false failures where e2fsck would declare that the extent tree was corrupted. This was fixed in commit 9c40d14841: "e2fsck: only check for zero-length leaf extents". Add a regression test case to ensure that this issue remains fixed. Signed-off-by: Tao Ma <boyu.mt@xxxxxxxxxx> Signed-off-by: Theodore Ts'o <tytso@xxxxxxx> -- 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