summary refs log tree commit diff
path: root/tools
diff options
context:
space:
mode:
authorJan Kara <jack@suse.cz>2019-11-05 17:44:21 +0100
committerTheodore Ts'o <tytso@mit.edu>2019-11-05 16:00:48 -0500
commit150549ed2fcf4be9bf3efedd99b72924dff26166 (patch)
tree556dbb10760d06168e671883201a12c481c67a32 /tools
parentdfaf5ffda227be3e867fee7c0f6a66749392fbd0 (diff)
downloadlinux-150549ed2fcf4be9bf3efedd99b72924dff26166.tar.gz
jbd2: Drop pointless check from jbd2_journal_stop()
If a transaction is larger than journal->j_max_transaction_buffers, that
is a bug and not a trigger for transaction commit. Also the very next
attempt to start new handle will start transaction commit anyway. So
just remove the pointless check. Arguably, we could start transaction
commit whenever the transaction size is *close* to
journal->j_max_transaction_buffers. This has a potential to reduce
latency of the next jbd2_journal_start() at the cost of somewhat smaller
transactions. However for this to have any effect, it would mean that
there isn't someone already waiting in jbd2_journal_start() which means
metadata load for the fs is pretty light anyway so probably this
optimization is not worth it.

Reviewed-by: Theodore Ts'o <tytso@mit.edu>
Signed-off-by: Jan Kara <jack@suse.cz>
Link: https://lore.kernel.org/r/20191105164437.32602-15-jack@suse.cz
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
Diffstat (limited to 'tools')
0 files changed, 0 insertions, 0 deletions