f2fs: drop any block plugging
authorJaegeuk Kim <jaegeuk@kernel.org>
Sat, 4 Jun 2016 21:25:24 +0000 (14:25 -0700)
committerJaegeuk Kim <jaegeuk@kernel.org>
Wed, 8 Jun 2016 17:25:51 +0000 (10:25 -0700)
In f2fs, we don't need to keep block plugging for NODE and DATA writes, since
we already merged bios as much as possible.

Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
fs/f2fs/checkpoint.c
fs/f2fs/data.c
fs/f2fs/gc.c
fs/f2fs/segment.c

index 5ddd15cd6c6cbdf4976e81cc9288069909e5cff0..4179c7b971fc62c8e66194e97f040f9bcf03824b 100644 (file)
@@ -897,11 +897,8 @@ static int block_operations(struct f2fs_sb_info *sbi)
                .nr_to_write = LONG_MAX,
                .for_reclaim = 0,
        };
-       struct blk_plug plug;
        int err = 0;
 
-       blk_start_plug(&plug);
-
 retry_flush_dents:
        f2fs_lock_all(sbi);
        /* write all the dirty dentry pages */
@@ -938,7 +935,6 @@ retry_flush_nodes:
                goto retry_flush_nodes;
        }
 out:
-       blk_finish_plug(&plug);
        return err;
 }
 
index 30dc448cae6073e3ffc0a42de3281ede98bba000..5f655d0c5b1f9db4cc8d578d7b1f9a9b8b5afd5a 100644 (file)
@@ -98,10 +98,13 @@ static struct bio *__bio_alloc(struct f2fs_sb_info *sbi, block_t blk_addr,
 }
 
 static inline void __submit_bio(struct f2fs_sb_info *sbi, int rw,
-                                               struct bio *bio)
+                       struct bio *bio, enum page_type type)
 {
-       if (!is_read_io(rw))
+       if (!is_read_io(rw)) {
                atomic_inc(&sbi->nr_wb_bios);
+               if (current->plug && (type == DATA || type == NODE))
+                       blk_finish_plug(current->plug);
+       }
        submit_bio(rw, bio);
 }
 
@@ -117,7 +120,7 @@ static void __submit_merged_bio(struct f2fs_bio_info *io)
        else
                trace_f2fs_submit_write_bio(io->sbi->sb, fio, io->bio);
 
-       __submit_bio(io->sbi, fio->rw, io->bio);
+       __submit_bio(io->sbi, fio->rw, io->bio, fio->type);
        io->bio = NULL;
 }
 
@@ -235,7 +238,7 @@ int f2fs_submit_page_bio(struct f2fs_io_info *fio)
                return -EFAULT;
        }
 
-       __submit_bio(fio->sbi, fio->rw, bio);
+       __submit_bio(fio->sbi, fio->rw, bio, fio->type);
        return 0;
 }
 
@@ -1040,7 +1043,7 @@ got_it:
                 */
                if (bio && (last_block_in_bio != block_nr - 1)) {
 submit_and_realloc:
-                       __submit_bio(F2FS_I_SB(inode), READ, bio);
+                       __submit_bio(F2FS_I_SB(inode), READ, bio, DATA);
                        bio = NULL;
                }
                if (bio == NULL) {
@@ -1083,7 +1086,7 @@ set_error_page:
                goto next_page;
 confused:
                if (bio) {
-                       __submit_bio(F2FS_I_SB(inode), READ, bio);
+                       __submit_bio(F2FS_I_SB(inode), READ, bio, DATA);
                        bio = NULL;
                }
                unlock_page(page);
@@ -1093,7 +1096,7 @@ next_page:
        }
        BUG_ON(pages && !list_empty(pages));
        if (bio)
-               __submit_bio(F2FS_I_SB(inode), READ, bio);
+               __submit_bio(F2FS_I_SB(inode), READ, bio, DATA);
        return 0;
 }
 
index 4a03076074af239d7c4cd0a054cc0ebcfd422087..67fd2855ccc92216f757a32dbfef57311fd91322 100644 (file)
@@ -777,7 +777,6 @@ static int do_garbage_collect(struct f2fs_sb_info *sbi,
 {
        struct page *sum_page;
        struct f2fs_summary_block *sum;
-       struct blk_plug plug;
        unsigned int segno = start_segno;
        unsigned int end_segno = start_segno + sbi->segs_per_sec;
        int seg_freed = 0;
@@ -795,8 +794,6 @@ static int do_garbage_collect(struct f2fs_sb_info *sbi,
                unlock_page(sum_page);
        }
 
-       blk_start_plug(&plug);
-
        for (segno = start_segno; segno < end_segno; segno++) {
                /* find segment summary of victim */
                sum_page = find_get_page(META_MAPPING(sbi),
@@ -830,8 +827,6 @@ static int do_garbage_collect(struct f2fs_sb_info *sbi,
                f2fs_submit_merged_bio(sbi,
                                (type == SUM_TYPE_NODE) ? NODE : DATA, WRITE);
 
-       blk_finish_plug(&plug);
-
        if (gc_type == FG_GC) {
                while (start_segno < end_segno)
                        if (get_valid_blocks(sbi, start_segno++, 1) == 0)
index 7b58bfbd84a340ea00c9bf47a1150cbf97477222..eff046a792adcf8f50e66f26a7c47025e0ac6932 100644 (file)
@@ -379,13 +379,8 @@ void f2fs_balance_fs_bg(struct f2fs_sb_info *sbi)
                        excess_prefree_segs(sbi) ||
                        excess_dirty_nats(sbi) ||
                        (is_idle(sbi) && f2fs_time_over(sbi, CP_TIME))) {
-               if (test_opt(sbi, DATA_FLUSH)) {
-                       struct blk_plug plug;
-
-                       blk_start_plug(&plug);
+               if (test_opt(sbi, DATA_FLUSH))
                        sync_dirty_inodes(sbi, FILE_INODE);
-                       blk_finish_plug(&plug);
-               }
                f2fs_sync_fs(sbi->sb, true);
                stat_inc_bg_cp_count(sbi->stat_info);
        }