diff mbox series

[f2fs-dev] f2fs: fix the wrong f2fs_bug_on condition in f2fs_do_replace_block

Message ID 20241020105752.1767730-1-weilongping@oppo.com (mailing list archive)
State Superseded
Headers show
Series [f2fs-dev] f2fs: fix the wrong f2fs_bug_on condition in f2fs_do_replace_block | expand

Commit Message

LongPing Wei Oct. 20, 2024, 10:57 a.m. UTC
This f2fs_bug_on was introduced by commit 2c1905042c8c3("f2fs: check segment type in __f2fs_replace_block") when there were only 6 curseg types.
After commit d0b9e42ab6155(f2fs: introduce inmem curseg) was introduced, the condition should be changed to checking curseg->seg_type.

Fixes: d0b9e42ab6155("f2fs: introduce inmem curseg")

Signed-off-by: LongPing Wei <weilongping@oppo.com>
---
 fs/f2fs/segment.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c
index 1766254279d2..7b54b1851d34 100644
--- a/fs/f2fs/segment.c
+++ b/fs/f2fs/segment.c
@@ -3977,8 +3977,8 @@  void f2fs_do_replace_block(struct f2fs_sb_info *sbi, struct f2fs_summary *sum,
 		}
 	}
 
-	f2fs_bug_on(sbi, !IS_DATASEG(type));
 	curseg = CURSEG_I(sbi, type);
+	f2fs_bug_on(sbi, !IS_DATASEG(curseg->seg_type));
 
 	mutex_lock(&curseg->curseg_mutex);
 	down_write(&sit_i->sentry_lock);