f2fs: introduce struct flush_cmd_control to wrap the flush_merge fields
Split the flush_merge fields from sm_i, and use the new struct flush_cmd_control
to wrap it, so that we can igonre these fileds if flush_merge is disable, and
it alse can the structs more neat.
Signed-off-by: Gu Zheng <guz.fnst@cn.fujitsu.com>
Signed-off-by: Jaegeuk Kim <jaegeuk.kim@samsung.com>
diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c
index bea642a..a7ed92e 100644
--- a/fs/f2fs/super.c
+++ b/fs/f2fs/super.c
@@ -641,29 +641,33 @@
* or if flush_merge is not passed in mount option.
*/
if ((*flags & MS_RDONLY) || !test_opt(sbi, FLUSH_MERGE)) {
- struct f2fs_sm_info *sm_info = sbi->sm_info;
+ struct flush_cmd_control *fcc =
+ sbi->sm_info->cmd_control_info;
- if (sm_info->f2fs_issue_flush)
- kthread_stop(sm_info->f2fs_issue_flush);
- sm_info->issue_list = sm_info->dispatch_list = NULL;
- sm_info->f2fs_issue_flush = NULL;
- } else if (test_opt(sbi, FLUSH_MERGE)) {
- struct f2fs_sm_info *sm_info = sbi->sm_info;
+ if (fcc && fcc->f2fs_issue_flush)
+ kthread_stop(fcc->f2fs_issue_flush);
+ kfree(fcc);
+ sbi->sm_info->cmd_control_info = NULL;
+ } else if (test_opt(sbi, FLUSH_MERGE) &&
+ !sbi->sm_info->cmd_control_info) {
+ dev_t dev = sbi->sb->s_bdev->bd_dev;
+ struct flush_cmd_control *fcc =
+ kzalloc(sizeof(struct flush_cmd_control), GFP_KERNEL);
- if (!sm_info->f2fs_issue_flush) {
- dev_t dev = sbi->sb->s_bdev->bd_dev;
-
- spin_lock_init(&sm_info->issue_lock);
- init_waitqueue_head(&sm_info->flush_wait_queue);
- sm_info->f2fs_issue_flush =
- kthread_run(issue_flush_thread, sbi,
- "f2fs_flush-%u:%u", MAJOR(dev), MINOR(dev));
- if (IS_ERR(sm_info->f2fs_issue_flush)) {
- err = PTR_ERR(sm_info->f2fs_issue_flush);
- sm_info->f2fs_issue_flush = NULL;
- goto restore_gc;
- }
+ if (!fcc) {
+ err = -ENOMEM;
+ goto restore_gc;
}
+ spin_lock_init(&fcc->issue_lock);
+ init_waitqueue_head(&fcc->flush_wait_queue);
+ fcc->f2fs_issue_flush = kthread_run(issue_flush_thread, sbi,
+ "f2fs_flush-%u:%u", MAJOR(dev), MINOR(dev));
+ if (IS_ERR(fcc->f2fs_issue_flush)) {
+ err = PTR_ERR(fcc->f2fs_issue_flush);
+ kfree(fcc);
+ goto restore_gc;
+ }
+ sbi->sm_info->cmd_control_info = fcc;
}
skip:
/* Update the POSIXACL Flag */