diff options
| author | Jaegeuk Kim <jaegeuk.kim@samsung.com> | 2013-05-15 09:12:18 +0200 |
|---|---|---|
| committer | Jaegeuk Kim <jaegeuk.kim@samsung.com> | 2013-05-28 08:03:01 +0200 |
| commit | 8c26d7d5717adf7f06d98c4416852d09566edd7c (patch) | |
| tree | b52f02643b816e135df23383b117995fd01c8c13 /fs/f2fs/recovery.c | |
| parent | f2fs: remove redundant assignment (diff) | |
| download | linux-8c26d7d5717adf7f06d98c4416852d09566edd7c.tar.xz linux-8c26d7d5717adf7f06d98c4416852d09566edd7c.zip | |
f2fs: fix por_doing variable coverage
The reason of using sbi->por_doing is to alleviate data writes during the
recovery.
The find_fsync_dnodes() produces some dirty dentry pages, so we should
cover it too with sbi->por_doing.
Signed-off-by: Jaegeuk Kim <jaegeuk.kim@samsung.com>
Diffstat (limited to 'fs/f2fs/recovery.c')
| -rw-r--r-- | fs/f2fs/recovery.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/f2fs/recovery.c b/fs/f2fs/recovery.c index 294198775f8b..4d895149a6f0 100644 --- a/fs/f2fs/recovery.c +++ b/fs/f2fs/recovery.c @@ -381,6 +381,7 @@ int recover_fsync_data(struct f2fs_sb_info *sbi) INIT_LIST_HEAD(&inode_list); /* step #1: find fsynced inode numbers */ + sbi->por_doing = 1; err = find_fsync_dnodes(sbi, &inode_list); if (err) goto out; @@ -389,13 +390,12 @@ int recover_fsync_data(struct f2fs_sb_info *sbi) goto out; /* step #2: recover data */ - sbi->por_doing = 1; err = recover_data(sbi, &inode_list, CURSEG_WARM_NODE); - sbi->por_doing = 0; BUG_ON(!list_empty(&inode_list)); out: destroy_fsync_dnodes(sbi, &inode_list); kmem_cache_destroy(fsync_entry_slab); + sbi->por_doing = 0; write_checkpoint(sbi, false); return err; } |