On Thu, Mar 15, 2012 at 09:59:35AM +0100, Jan Kara wrote: > On Tue 13-03-12 22:17:45, Ted Tso wrote: > > I'm not sure why you pulled out the code for > > jbd2_journal_get_log_tail(), since there's no other usage of the > > function, but it did make this commit harder to review. It's good for > > code movement patches to be segregated to their own commit, just to > > make it easier to review.... > I pulled the code out because the last patch in the series uses it from > commit code. So it was not a deliberate code movement. But I agree it might > have been easier to review if the code movement was a separate patch. > Should I do that or have you already coped with the patch as is? No, it's fine as it is. I've merged your patches into the ext4 tree for the upcoming merge window. 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
