Please add simple description here to explain why we need to add this, even though this change is very simple. On 2018/1/4 11:48, Yunlong Song wrote: > Signed-off-by: Yunlong Song <yunlong.song@xxxxxxxxxx> > --- > fs/f2fs/segment.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c > index 890d483..e3bbabf 100644 > --- a/fs/f2fs/segment.c > +++ b/fs/f2fs/segment.c > @@ -2719,6 +2719,8 @@ void __f2fs_replace_block(struct f2fs_sb_info *sbi, struct f2fs_summary *sum, > se = get_seg_entry(sbi, segno); > type = se->type; > > + f2fs_bug_on(sbi, se->valid_blocks && IS_NODESEG(type)); > + > down_write(&SM_I(sbi)->curseg_lock); > > if (!recover_curseg) { >