diff options
author | Jaegeuk Kim <jaegeuk.kim@samsung.com> | 2013-05-15 03:12:18 -0400 |
---|---|---|
committer | Jaegeuk Kim <jaegeuk.kim@samsung.com> | 2013-05-28 02:03:01 -0400 |
commit | 8c26d7d5717adf7f06d98c4416852d09566edd7c (patch) | |
tree | b52f02643b816e135df23383b117995fd01c8c13 /fs/f2fs | |
parent | addbe45b005d73f876d55bcfc16f4a6ce52a55e3 (diff) |
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')
-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) | |||
381 | INIT_LIST_HEAD(&inode_list); | 381 | INIT_LIST_HEAD(&inode_list); |
382 | 382 | ||
383 | /* step #1: find fsynced inode numbers */ | 383 | /* step #1: find fsynced inode numbers */ |
384 | sbi->por_doing = 1; | ||
384 | err = find_fsync_dnodes(sbi, &inode_list); | 385 | err = find_fsync_dnodes(sbi, &inode_list); |
385 | if (err) | 386 | if (err) |
386 | goto out; | 387 | goto out; |
@@ -389,13 +390,12 @@ int recover_fsync_data(struct f2fs_sb_info *sbi) | |||
389 | goto out; | 390 | goto out; |
390 | 391 | ||
391 | /* step #2: recover data */ | 392 | /* step #2: recover data */ |
392 | sbi->por_doing = 1; | ||
393 | err = recover_data(sbi, &inode_list, CURSEG_WARM_NODE); | 393 | err = recover_data(sbi, &inode_list, CURSEG_WARM_NODE); |
394 | sbi->por_doing = 0; | ||
395 | BUG_ON(!list_empty(&inode_list)); | 394 | BUG_ON(!list_empty(&inode_list)); |
396 | out: | 395 | out: |
397 | destroy_fsync_dnodes(sbi, &inode_list); | 396 | destroy_fsync_dnodes(sbi, &inode_list); |
398 | kmem_cache_destroy(fsync_entry_slab); | 397 | kmem_cache_destroy(fsync_entry_slab); |
398 | sbi->por_doing = 0; | ||
399 | write_checkpoint(sbi, false); | 399 | write_checkpoint(sbi, false); |
400 | return err; | 400 | return err; |
401 | } | 401 | } |