diff options
| author | Jaegeuk Kim <jaegeuk@kernel.org> | 2016-09-21 18:34:48 +0200 |
|---|---|---|
| committer | Jaegeuk Kim <jaegeuk@kernel.org> | 2016-10-01 02:34:25 +0200 |
| commit | f6fe2be3c6d6f0127742ae1cc2e3ffe9ad31ea8b (patch) | |
| tree | 7b9e3f25bad7aa53cdde1fb9b4d4f186e26be73b /fs/f2fs/gc.c | |
| parent | f2fs: assign return value in f2fs_gc (diff) | |
| download | linux-f6fe2be3c6d6f0127742ae1cc2e3ffe9ad31ea8b.tar.xz linux-f6fe2be3c6d6f0127742ae1cc2e3ffe9ad31ea8b.zip | |
f2fs: should put_page for summary page
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>
Diffstat (limited to 'fs/f2fs/gc.c')
| -rw-r--r-- | fs/f2fs/gc.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c index 400bc6d78372..a9a3c9f19032 100644 --- a/fs/f2fs/gc.c +++ b/fs/f2fs/gc.c @@ -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); } |