diff options
| author |   Jaegeuk Kim <jaegeuk@kernel.org> | 2015-09-25 19:34:50 -0700 |
|---|---|---|
| committer | Jaegeuk Kim <jaegeuk@kernel.org> | 2015-10-09 16:20:54 -0700 |
| commit | 90b803e6fb6243922bff9ddd8a6205c17cb93b31 (patch) | |
| tree | 197cfe1aca85ed407167d7bcbbbde9c7fe96f15f /fs/f2fs/data.c | |
| parent | f2fs: remove unneeded f2fs_{,un}lock_op in do_recover_data() (diff) | |
| download | linux-dev-90b803e6fb6243922bff9ddd8a6205c17cb93b31.tar.xz linux-dev-90b803e6fb6243922bff9ddd8a6205c17cb93b31.zip | |
f2fs: do not skip dentry block writes
Previously, we skip dentry block writes when wbc is SYNC_NONE with no memory
pressure and the number of dirty pages is pretty small.
But, we didn't skip for normal data writes, which gives us not much big impact
on overall performance.
Moreover, by skipping some data writes, kworker falls into infinite loop to try
to write blocks, when many dir inodes have only one dentry block.
So, this patch removes skipping data writes.
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Diffstat (limited to 'fs/f2fs/data.c')
| -rw-r--r-- | fs/f2fs/data.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c index bc04e9201fd6..a903423e4cd5 100644 --- a/fs/f2fs/data.c +++ b/fs/f2fs/data.c @@ -1340,11 +1340,6 @@ static int f2fs_write_data_pages(struct address_space *mapping, if (!get_dirty_pages(inode) && wbc->sync_mode == WB_SYNC_NONE) return 0; - if (S_ISDIR(inode->i_mode) && wbc->sync_mode == WB_SYNC_NONE && - get_dirty_pages(inode) < nr_pages_to_skip(sbi, DATA) && - available_free_memory(sbi, DIRTY_DENTS)) - goto skip_write; - /* during POR, we don't need to trigger writepage at all. */ if (unlikely(is_sbi_flag_set(sbi, SBI_POR_DOING))) goto skip_write; |