f2fs: remove struct segment_allocation default_salloc_ops
authorChristoph Hellwig <hch@lst.de>
Mon, 28 Nov 2022 09:43:44 +0000 (10:43 +0100)
committerJaegeuk Kim <jaegeuk@kernel.org>
Thu, 8 Dec 2022 17:32:10 +0000 (09:32 -0800)
There is only  single instance of these ops, so remove the indirection
and call allocate_segment_by_default directly.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Chao Yu <chao@kernel.org>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
fs/f2fs/segment.c
fs/f2fs/segment.h

index 0ff451e..bbe6556 100644 (file)
@@ -2926,7 +2926,7 @@ static void __allocate_new_segment(struct f2fs_sb_info *sbi, int type,
                return;
 alloc:
        old_segno = curseg->segno;
-       SIT_I(sbi)->s_ops->allocate_segment(sbi, type, true);
+       allocate_segment_by_default(sbi, type, true);
        locate_dirty_segment(sbi, old_segno);
 }
 
@@ -2957,10 +2957,6 @@ void f2fs_allocate_new_segments(struct f2fs_sb_info *sbi)
        f2fs_up_read(&SM_I(sbi)->curseg_lock);
 }
 
-static const struct segment_allocation default_salloc_ops = {
-       .allocate_segment = allocate_segment_by_default,
-};
-
 bool f2fs_exist_trim_candidates(struct f2fs_sb_info *sbi,
                                                struct cp_control *cpc)
 {
@@ -3284,7 +3280,7 @@ void f2fs_allocate_data_block(struct f2fs_sb_info *sbi, struct page *page,
                        get_atssr_segment(sbi, type, se->type,
                                                AT_SSR, se->mtime);
                else
-                       sit_i->s_ops->allocate_segment(sbi, type, false);
+                       allocate_segment_by_default(sbi, type, false);
        }
        /*
         * segment dirty status should be updated after segment allocation,
@@ -4270,9 +4266,6 @@ static int build_sit_info(struct f2fs_sb_info *sbi)
                return -ENOMEM;
 #endif
 
-       /* init SIT information */
-       sit_i->s_ops = &default_salloc_ops;
-
        sit_i->sit_base_addr = le32_to_cpu(raw_super->sit_blkaddr);
        sit_i->sit_blocks = sit_segs << sbi->log_blocks_per_seg;
        sit_i->written_valid_blocks = 0;
index be8f2d7..3ad1b7b 100644 (file)
@@ -222,10 +222,6 @@ struct sec_entry {
        unsigned int valid_blocks;      /* # of valid blocks in a section */
 };
 
-struct segment_allocation {
-       void (*allocate_segment)(struct f2fs_sb_info *, int, bool);
-};
-
 #define MAX_SKIP_GC_COUNT                      16
 
 struct revoke_entry {
@@ -235,8 +231,6 @@ struct revoke_entry {
 };
 
 struct sit_info {
-       const struct segment_allocation *s_ops;
-
        block_t sit_base_addr;          /* start block address of SIT area */
        block_t sit_blocks;             /* # of blocks used by SIT area */
        block_t written_valid_blocks;   /* # of valid blocks in main area */