On 05/17, Zhiguo Niu wrote: > commit 245930617c9b ("f2fs: fix to handle error paths of > {new,change}_curseg()") > missed this allocated path, fix it. > > Signed-off-by: Zhiguo Niu <zhiguo....@unisoc.com> > --- > fs/f2fs/segment.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c > index a0ce3d0..71dc8042 100644 > --- a/fs/f2fs/segment.c > +++ b/fs/f2fs/segment.c > @@ -5190,7 +5190,9 @@ static int fix_curseg_write_pointer(struct f2fs_sb_info > *sbi, int type) > if (cs->next_blkoff) { > unsigned int old_segno = cs->segno, old_blkoff = > cs->next_blkoff; > > - f2fs_allocate_new_section(sbi, type, true); > + err = f2fs_allocate_new_section(sbi, type, true); > + if (err) > + return err;
I hesitate to apply this, since this may give mount failures forever. Do you see a real issue with this? > f2fs_notice(sbi, "Assign new section to curseg[%d]: " > "[0x%x,0x%x] -> [0x%x,0x%x]", > type, old_segno, old_blkoff, > -- > 1.9.1 _______________________________________________ Linux-f2fs-devel mailing list Linux-f2fs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel