diff options
| author | Chao Yu <chao2.yu@samsung.com> | 2013-10-24 15:08:28 +0800 |
|---|---|---|
| committer | Andreas Blaesius <skate4life@gmx.de> | 2016-06-05 21:20:46 +0200 |
| commit | 4ddcffa7aed0761bd9ac3d37a0ee92f52f6f596c (patch) | |
| tree | b20b6fc82524d1abb204c77f3c2cfbe8730c6067 | |
| parent | 1980219cb5c2565f92ad26a439a02b2a3a8716b9 (diff) | |
| download | kernel_samsung_espresso10-4ddcffa7aed0761bd9ac3d37a0ee92f52f6f596c.zip kernel_samsung_espresso10-4ddcffa7aed0761bd9ac3d37a0ee92f52f6f596c.tar.gz kernel_samsung_espresso10-4ddcffa7aed0761bd9ac3d37a0ee92f52f6f596c.tar.bz2 | |
f2fs: remove redundant set_page_dirty from write_compacted_summaries
Previously, set_page_dirty is called every time after writting one summary info
into compacted summary page,
To avoid redundant set_page_dirty, we only call set_page_dirty before release
page.
Signed-off-by: Yu Chao <chao2.yu@samsung.com>
Signed-off-by: Jaegeuk Kim <jaegeuk.kim@samsung.com>
| -rw-r--r-- | fs/f2fs/segment.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c index 8e5ca28..487af61 100644 --- a/fs/f2fs/segment.c +++ b/fs/f2fs/segment.c @@ -1138,8 +1138,6 @@ static void write_compacted_summaries(struct f2fs_sb_info *sbi, block_t blkaddr) SUM_JOURNAL_SIZE); written_size += SUM_JOURNAL_SIZE; - set_page_dirty(page); - /* Step 3: write summary entries */ for (i = CURSEG_HOT_DATA; i <= CURSEG_COLD_DATA; i++) { unsigned short blkoff; @@ -1158,18 +1156,20 @@ static void write_compacted_summaries(struct f2fs_sb_info *sbi, block_t blkaddr) summary = (struct f2fs_summary *)(kaddr + written_size); *summary = seg_i->sum_blk->entries[j]; written_size += SUMMARY_SIZE; - set_page_dirty(page); if (written_size + SUMMARY_SIZE <= PAGE_CACHE_SIZE - SUM_FOOTER_SIZE) continue; + set_page_dirty(page); f2fs_put_page(page, 1); page = NULL; } } - if (page) + if (page) { + set_page_dirty(page); f2fs_put_page(page, 1); + } } static void write_normal_summaries(struct f2fs_sb_info *sbi, |