Hi all: I'm a little confused about BH_Freed bit. The only place it is set is journal_unmap_buffer, which is called by jbd2_journal_invalidatepage when we want to truncate a file. Since jbd2_journal_invalidatepage is called outside of transaction, We can't make sure whether the "add to orphan" operation belongs to committing transaction or not, so we can't touch the buffer belongs to committing transaction, instead BH_Freed bit is set to indicate that this buffer can be discarded in running transaction. But i think we shouldn't clear BH_JBDdirty in jbd2_journal_commit_transaction, as following codes does: /* A buffer which has been freed while still being * journaled by a previous transaction may end up still * being dirty here, but we want to avoid writing back * that buffer in the future now that the last use has * been committed. That's not only a performance gain, * it also stops aliasing problems if the buffer is left * behind for writeback and gets reallocated for another * use in a different page. */ if (buffer_freed(bh)) { clear_buffer_freed(bh); clear_buffer_jbddirty(bh); } Note that, We can't make sure "current running transaction" can complete commit work. If we clear BH_JBDdirty bit here, this buffer may be freed here, the log space of older transaction may be freed before the "current running transaction" complete commit work, and if this happends and system crashed at this moment, filesystem will be inconsistent. Above is my analysis, please let me know if it's wrong, and if it's a bug, may be I can send a patch out, thanks. -- 丁定华 -- 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