@@ -331,6 +331,9 @@ void jbd2_journal_commit_transaction(journal_t *journal)
struct buffer_head *cbh = NULL; /* For transactional checksums */
__u32 crc32_sum = ~0;
struct blk_plug plug;
+ /* Tail of the journal */
+ unsigned long first_block;
+ tid_t first_tid;
/*
* First job: lock down the current transaction and wait for
@@ -675,6 +678,14 @@ start_journal_io:
J_ASSERT(commit_transaction->t_state == T_COMMIT);
commit_transaction->t_state = T_COMMIT_DFLUSH;
write_unlock(&journal->j_state_lock);
+ /*
+ * Get current oldest transaction in the log before we issue flush
+ * to the filesystem device. After the flush we can be sure that
+ * blocks of all older transactions are checkpointed to persistent
+ * storage and we will be safe to update journal start in the
+ * superblock with the numbers we get here.
+ */
+ jbd2_journal_get_log_tail(journal, &first_tid, &first_block);
/*
* If the journal is not located on the file system device,
* then we must flush the file system device before we issue
@@ -825,6 +836,14 @@ wait_for_iobuf:
if (err)
jbd2_journal_abort(journal, err);
+ /*
+ * Now disk caches are flushed so we are safe to erase checkpointed
+ * transactions from the log by updating journal superblock.
+ */
+ if (jbd2_update_log_tail(journal, first_tid, first_block) &&
+ !(journal->j_flags & JBD2_ABORT))
+ jbd2_journal_update_superblock(journal, 1);
+
/* End of a transaction! Finally, we can do checkpoint
processing: any buffers committed as a result of this
transaction can be removed from any checkpoint list it was on
Normally, we have to issue a cache flush before we can update journal tail in journal superblock, effectively wiping out old transactions from the journal. So use the fact that during transaction commit we issue cache flush anyway and opportunistically push journal tail as far as we can. Signed-off-by: Jan Kara <jack@suse.cz> --- fs/jbd2/commit.c | 19 +++++++++++++++++++ 1 files changed, 19 insertions(+), 0 deletions(-)