diff options
| author | Jaegeuk Kim <jaegeuk@kernel.org> | 2016-05-21 01:32:49 +0200 |
|---|---|---|
| committer | Jaegeuk Kim <jaegeuk@kernel.org> | 2016-06-03 03:05:12 +0200 |
| commit | ee6d182f2a19d5d44607b5ae4bec523726d76a99 (patch) | |
| tree | 514092851eb0e974c6ac8add90fed10056b84998 /fs/f2fs/recovery.c | |
| parent | f2fs: flush inode metadata when checkpoint is doing (diff) | |
| download | linux-ee6d182f2a19d5d44607b5ae4bec523726d76a99.tar.xz linux-ee6d182f2a19d5d44607b5ae4bec523726d76a99.zip | |
f2fs: remove syncing inode page in all the cases
This patch reduces to call them across the whole tree.
- sync_inode_page()
- update_inode_page()
- update_inode()
- f2fs_write_inode()
Instead, checkpoint will flush all the dirty inode metadata before syncing
node pages.
Note that, this is doable, since we call mark_inode_dirty_sync() for all
inode's field change which needs to update on-disk inode as well.
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Diffstat (limited to 'fs/f2fs/recovery.c')
| -rw-r--r-- | fs/f2fs/recovery.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/fs/f2fs/recovery.c b/fs/f2fs/recovery.c index 2500b6a5daf0..68c433f17ab5 100644 --- a/fs/f2fs/recovery.c +++ b/fs/f2fs/recovery.c @@ -490,9 +490,6 @@ static int do_recover_data(struct f2fs_sb_info *sbi, struct inode *inode, } } - if (IS_INODE(dn.node_page)) - sync_inode_page(&dn); - copy_node_footer(dn.node_page, page); fill_node_footer(dn.node_page, dn.nid, ni.ino, ofs_of_node(page), false); |