diff options
| author | Jaegeuk Kim <jaegeuk.kim@samsung.com> | 2013-05-15 16:12:18 +0900 |
|---|---|---|
| committer | Chanho Park <chanho61.park@samsung.com> | 2014-11-18 11:43:44 +0900 |
| commit | b3db19ea309c9f773d7c1b817495a913ea5b59fc (patch) | |
| tree | b9f2dc553b6af69b47cfab1f045c341fd847fe4b /fs | |
| parent | 22129ae8d8bceef075caa6cd7db6e8678f31d539 (diff) | |
| download | linux-3.10-b3db19ea309c9f773d7c1b817495a913ea5b59fc.tar.gz linux-3.10-b3db19ea309c9f773d7c1b817495a913ea5b59fc.tar.bz2 linux-3.10-b3db19ea309c9f773d7c1b817495a913ea5b59fc.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')
| -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 294198775f8..4d895149a6f 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; } |