On Thu, Oct 08, 2009 at 06:13:12PM -0700, Mingming wrote: > @@ -2827,8 +2828,8 @@ static int ext4_split_unwritten_extents( > * the size of extent to write, there is no need to split > * uninitialized extent > */ > - if (allocated <= max_blocks) > - return ret; > + if (iblock == ee_block && allocated <= max_blocks) > + return allocated; The change to add "iblock == ee_block" isn't explained in the patch description and it makes the comment above the conditional no longer accurate. Can you add an explanation why it's necessary? Thanks, - 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