aboutsummaryrefslogtreecommitdiffstats
path: root/fs/ext4
diff options
context:
space:
mode:
authorJan Kara <jack@suse.cz>2014-03-04 10:50:50 -0500
committerTheodore Ts'o <tytso@mit.edu>2014-03-04 10:50:50 -0500
commit10542c229a4e8e25b40357beea66abe9dacda2c0 (patch)
tree7ae7251bfd1da2cb97a53883ae5eb6490c18415e /fs/ext4
parent9eb79482a97152930b113b51dff530aba9e28c8e (diff)
ext4: Speedup WB_SYNC_ALL pass called from sync(2)
When doing filesystem wide sync, there's no need to force transaction commit (or synchronously write inode buffer) separately for each inode because ext4_sync_fs() takes care of forcing commit at the end (VFS takes care of flushing buffer cache, respectively). Most of the time this slowness doesn't manifest because previous WB_SYNC_NONE writeback doesn't leave much to write but when there are processes aggressively creating new files and several filesystems to sync, the sync slowness can be noticeable. In the following test script sync(1) takes around 6 minutes when there are two ext4 filesystems mounted on a standard SATA drive. After this patch sync takes a couple of seconds so we have about two orders of magnitude improvement. function run_writers { for (( i = 0; i < 10; i++ )); do mkdir $1/dir$i for (( j = 0; j < 40000; j++ )); do dd if=/dev/zero of=$1/dir$i/$j bs=4k count=4 &>/dev/null done & done } for dir in "$@"; do run_writers $dir done sleep 40 time sync Signed-off-by: Jan Kara <jack@suse.cz> Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs/ext4')
-rw-r--r--fs/ext4/inode.c13
1 files changed, 11 insertions, 2 deletions
diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c
index 5324a38d848d..ab3e8357929d 100644
--- a/fs/ext4/inode.c
+++ b/fs/ext4/inode.c
@@ -4455,7 +4455,12 @@ int ext4_write_inode(struct inode *inode, struct writeback_control *wbc)
4455 return -EIO; 4455 return -EIO;
4456 } 4456 }
4457 4457
4458 if (wbc->sync_mode != WB_SYNC_ALL) 4458 /*
4459 * No need to force transaction in WB_SYNC_NONE mode. Also
4460 * ext4_sync_fs() will force the commit after everything is
4461 * written.
4462 */
4463 if (wbc->sync_mode != WB_SYNC_ALL || wbc->for_sync)
4459 return 0; 4464 return 0;
4460 4465
4461 err = ext4_force_commit(inode->i_sb); 4466 err = ext4_force_commit(inode->i_sb);
@@ -4465,7 +4470,11 @@ int ext4_write_inode(struct inode *inode, struct writeback_control *wbc)
4465 err = __ext4_get_inode_loc(inode, &iloc, 0); 4470 err = __ext4_get_inode_loc(inode, &iloc, 0);
4466 if (err) 4471 if (err)
4467 return err; 4472 return err;
4468 if (wbc->sync_mode == WB_SYNC_ALL) 4473 /*
4474 * sync(2) will flush the whole buffer cache. No need to do
4475 * it here separately for each inode.
4476 */
4477 if (wbc->sync_mode == WB_SYNC_ALL && !wbc->for_sync)
4469 sync_dirty_buffer(iloc.bh); 4478 sync_dirty_buffer(iloc.bh);
4470 if (buffer_req(iloc.bh) && !buffer_uptodate(iloc.bh)) { 4479 if (buffer_req(iloc.bh) && !buffer_uptodate(iloc.bh)) {
4471 EXT4_ERROR_INODE_BLOCK(inode, iloc.bh->b_blocknr, 4480 EXT4_ERROR_INODE_BLOCK(inode, iloc.bh->b_blocknr,