f2fs: introduce f2fs_invalidate_internal_cache() for cleanup

Just cleanup, no logic changes.

Signed-off-by: Chao Yu <chao@kernel.org>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>

diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c
index d86419b01310ce633e112f8ee08795947859503c..27015b7875aed6da2cb6be2616465802f14ccb43 100644 (file)
--- a/fs/f2fs/data.c
+++ b/fs/f2fs/data.c
@@ -1484,11 +1484,9 @@ static int __allocate_data_block(struct dnode_of_data *dn, int seg_type)
        old_blkaddr = dn->data_blkaddr;
        f2fs_allocate_data_block(sbi, NULL, old_blkaddr, &dn->data_blkaddr,
                                &sum, seg_type, NULL);
-       if (GET_SEGNO(sbi, old_blkaddr) != NULL_SEGNO) {
-               invalidate_mapping_pages(META_MAPPING(sbi),
-                                       old_blkaddr, old_blkaddr);
-               f2fs_invalidate_compress_page(sbi, old_blkaddr);
-       }
+       if (GET_SEGNO(sbi, old_blkaddr) != NULL_SEGNO)
+               f2fs_invalidate_internal_cache(sbi, old_blkaddr);
+
        f2fs_update_data_blkaddr(dn, dn->data_blkaddr);
        return 0;
 }

diff --git a/fs/f2fs/f2fs.h b/fs/f2fs/f2fs.h
index 50e666ebd9877a70b194a3f7d2c642d31de910a2..65294e3b0bef880a424c480ba4f4997b23c6b59c 100644 (file)
--- a/fs/f2fs/f2fs.h
+++ b/fs/f2fs/f2fs.h
@@ -4613,6 +4613,13 @@ static inline bool f2fs_is_readonly(struct f2fs_sb_info *sbi)
        return f2fs_sb_has_readonly(sbi) || f2fs_readonly(sbi->sb);
 }
 
+static inline void f2fs_invalidate_internal_cache(struct f2fs_sb_info *sbi,
+                                                               block_t blkaddr)
+{
+       invalidate_mapping_pages(META_MAPPING(sbi), blkaddr, blkaddr);
+       f2fs_invalidate_compress_page(sbi, blkaddr);
+}
+
 #define EFSBADCRC      EBADMSG         /* Bad CRC detected */
 #define EFSCORRUPTED   EUCLEAN         /* Filesystem is corrupted */
 

diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c
index 2fbe16ad726f457dee71e7d1b5d67893d04aa884..405a6077bd83bd90773a7efb02b6684a3aaa6ff2 100644 (file)
--- a/fs/f2fs/gc.c
+++ b/fs/f2fs/gc.c
@@ -1380,9 +1380,8 @@ static int move_data_block(struct inode *inode, block_t bidx,
        memcpy(page_address(fio.encrypted_page),
                                page_address(mpage), PAGE_SIZE);
        f2fs_put_page(mpage, 1);
-       invalidate_mapping_pages(META_MAPPING(fio.sbi),
-                               fio.old_blkaddr, fio.old_blkaddr);
-       f2fs_invalidate_compress_page(fio.sbi, fio.old_blkaddr);
+
+       f2fs_invalidate_internal_cache(fio.sbi, fio.old_blkaddr);
 
        set_page_dirty(fio.encrypted_page);
        if (clear_page_dirty_for_io(fio.encrypted_page))

diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c
index 007ebb10723662a859dabb0c81a7f61348d6100a..61da26eb61cc6d35344e33217e1e041dce730d73 100644 (file)
--- a/fs/f2fs/segment.c
+++ b/fs/f2fs/segment.c
@@ -2500,8 +2500,7 @@ void f2fs_invalidate_blocks(struct f2fs_sb_info *sbi, block_t addr)
        if (addr == NEW_ADDR || addr == COMPRESS_ADDR)
                return;
 
-       invalidate_mapping_pages(META_MAPPING(sbi), addr, addr);
-       f2fs_invalidate_compress_page(sbi, addr);
+       f2fs_invalidate_internal_cache(sbi, addr);
 
        /* add it into sit main buffer */
        down_write(&sit_i->sentry_lock);
@@ -3562,11 +3561,8 @@ static void do_write_page(struct f2fs_summary *sum, struct f2fs_io_info *fio)
 reallocate:
        f2fs_allocate_data_block(fio->sbi, fio->page, fio->old_blkaddr,
                        &fio->new_blkaddr, sum, type, fio);
-       if (GET_SEGNO(fio->sbi, fio->old_blkaddr) != NULL_SEGNO) {
-               invalidate_mapping_pages(META_MAPPING(fio->sbi),
-                                       fio->old_blkaddr, fio->old_blkaddr);
-               f2fs_invalidate_compress_page(fio->sbi, fio->old_blkaddr);
-       }
+       if (GET_SEGNO(fio->sbi, fio->old_blkaddr) != NULL_SEGNO)
+               f2fs_invalidate_internal_cache(fio->sbi, fio->old_blkaddr);
 
        /* writeout dirty page into bdev */
        f2fs_submit_page_write(fio);
@@ -3762,9 +3758,7 @@ void f2fs_do_replace_block(struct f2fs_sb_info *sbi, struct f2fs_summary *sum,
                update_sit_entry(sbi, new_blkaddr, 1);
        }
        if (GET_SEGNO(sbi, old_blkaddr) != NULL_SEGNO) {
-               invalidate_mapping_pages(META_MAPPING(sbi),
-                                       old_blkaddr, old_blkaddr);
-               f2fs_invalidate_compress_page(sbi, old_blkaddr);
+               f2fs_invalidate_internal_cache(sbi, old_blkaddr);
                if (!from_gc)
                        update_segment_mtime(sbi, old_blkaddr, 0);
                update_sit_entry(sbi, old_blkaddr, -1);