diff options
| author | Jaegeuk Kim <jaegeuk@kernel.org> | 2016-09-21 09:34:48 -0700 |
|---|---|---|
| committer | Jaegeuk Kim <jaegeuk@kernel.org> | 2016-09-30 17:34:25 -0700 |
| commit | f6fe2be3c6d6f0127742ae1cc2e3ffe9ad31ea8b (patch) | |
| tree | 7b9e3f25bad7aa53cdde1fb9b4d4f186e26be73b | |
| parent | 2956e450fa08669ebf1541acb07843b5aa6acf96 (diff) | |
| download | linux-stable-f6fe2be3c6d6f0127742ae1cc2e3ffe9ad31ea8b.tar.gz linux-stable-f6fe2be3c6d6f0127742ae1cc2e3ffe9ad31ea8b.tar.bz2 linux-stable-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>
| -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); } |