diff options
author | Jaegeuk Kim <jaegeuk@kernel.org> | 2014-09-10 18:04:03 -0400 |
---|---|---|
committer | Jaegeuk Kim <jaegeuk@kernel.org> | 2014-09-23 14:10:16 -0400 |
commit | 19c9c466e5c79402cbf8929f922011b02fbb59bd (patch) | |
tree | 632f1646864c395780dc360add826f1f24e33529 /fs | |
parent | 441ac5cb323a47b0a665f77f7cd6f76aacbdf21c (diff) |
f2fs: do not skip latest inode information
In f2fs_sync_file, if there is no written appended writes, it skips
to write its node blocks.
But, if there is up-to-date inode page, we should write it to update
its metadata during the roll-forward recovery.
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/f2fs/file.c | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c index 3035c791d934..a041c66fd611 100644 --- a/fs/f2fs/file.c +++ b/fs/f2fs/file.c | |||
@@ -169,12 +169,21 @@ int f2fs_sync_file(struct file *file, loff_t start, loff_t end, int datasync) | |||
169 | */ | 169 | */ |
170 | if (!is_inode_flag_set(fi, FI_APPEND_WRITE) && | 170 | if (!is_inode_flag_set(fi, FI_APPEND_WRITE) && |
171 | !exist_written_data(sbi, ino, APPEND_INO)) { | 171 | !exist_written_data(sbi, ino, APPEND_INO)) { |
172 | struct page *i = find_get_page(NODE_MAPPING(sbi), ino); | ||
173 | |||
174 | /* But we need to avoid that there are some inode updates */ | ||
175 | if ((i && PageDirty(i)) || need_inode_block_update(sbi, ino)) { | ||
176 | f2fs_put_page(i, 0); | ||
177 | goto go_write; | ||
178 | } | ||
179 | f2fs_put_page(i, 0); | ||
180 | |||
172 | if (is_inode_flag_set(fi, FI_UPDATE_WRITE) || | 181 | if (is_inode_flag_set(fi, FI_UPDATE_WRITE) || |
173 | exist_written_data(sbi, ino, UPDATE_INO)) | 182 | exist_written_data(sbi, ino, UPDATE_INO)) |
174 | goto flush_out; | 183 | goto flush_out; |
175 | goto out; | 184 | goto out; |
176 | } | 185 | } |
177 | 186 | go_write: | |
178 | /* guarantee free sections for fsync */ | 187 | /* guarantee free sections for fsync */ |
179 | f2fs_balance_fs(sbi); | 188 | f2fs_balance_fs(sbi); |
180 | 189 | ||