f2fs: should put_page for summary page
authorJaegeuk Kim <jaegeuk@kernel.org>
Wed, 21 Sep 2016 16:34:48 +0000 (09:34 -0700)
committerJaegeuk Kim <jaegeuk@kernel.org>
Sat, 1 Oct 2016 00:34:25 +0000 (17:34 -0700)
We should call put_page for preloaded summary pages in do_garbage_collect.

Reviewed-by: Chao Yu <yuchao0@huawei.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
fs/f2fs/gc.c

index 400bc6d78372237f69d77cc9df47c71d15a10666..a9a3c9f190326fed1b776ef32909e8078401f09d 100644 (file)
@@ -848,7 +848,7 @@ static int do_garbage_collect(struct f2fs_sb_info *sbi,
        for (segno = start_segno; segno < end_segno; segno++) {
 
                if (get_valid_blocks(sbi, segno, 1) == 0)
-                       continue;
+                       goto next;
 
                /* find segment summary of victim */
                sum_page = find_get_page(META_MAPPING(sbi),
@@ -874,7 +874,7 @@ static int do_garbage_collect(struct f2fs_sb_info *sbi,
                                                                gc_type);
 
                stat_inc_seg_count(sbi, type, gc_type);
-
+next:
                f2fs_put_page(sum_page, 0);
        }