| Message ID | 20231204180428.925779-3-jaegeuk@kernel.org (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | aca90eea8a90b1abc844504109b6f919dfbd4259 |
| Headers | show |
| Series | [f2fs-dev,1/4] f2fs: allocate new section if it's not new | expand |
LGTM On Mon, Dec 4, 2023 at 10:06 AM Jaegeuk Kim <jaegeuk@kernel.org> wrote: > > Even if f2fs was rebooted as staying checkpoint=disable, let's match the write > pointers all the time. > > Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org> > --- > fs/f2fs/super.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c > index 617340e9ea7f..9a874b4d1501 100644 > --- a/fs/f2fs/super.c > +++ b/fs/f2fs/super.c > @@ -4741,7 +4741,7 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent) > #ifdef CONFIG_QUOTA > f2fs_recover_quota_end(sbi, quota_enabled); > #endif > - > +reset_checkpoint: > /* > * If the f2fs is not readonly and fsync data recovery succeeds, > * check zoned block devices' write pointer consistency. > @@ -4752,7 +4752,6 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent) > goto free_meta; > } > > -reset_checkpoint: > f2fs_init_inmem_curseg(sbi); > > /* f2fs_recover_fsync_data() cleared this already */ > -- > 2.43.0.rc2.451.g8631bc7472-goog > > > > _______________________________________________ > Linux-f2fs-devel mailing list > Linux-f2fs-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c index 617340e9ea7f..9a874b4d1501 100644 --- a/fs/f2fs/super.c +++ b/fs/f2fs/super.c @@ -4741,7 +4741,7 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent) #ifdef CONFIG_QUOTA f2fs_recover_quota_end(sbi, quota_enabled); #endif - +reset_checkpoint: /* * If the f2fs is not readonly and fsync data recovery succeeds, * check zoned block devices' write pointer consistency. @@ -4752,7 +4752,6 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent) goto free_meta; } -reset_checkpoint: f2fs_init_inmem_curseg(sbi); /* f2fs_recover_fsync_data() cleared this already */
Even if f2fs was rebooted as staying checkpoint=disable, let's match the write pointers all the time. Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org> --- fs/f2fs/super.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)