diff mbox series

[f2fs-dev] fsck.f2fs: run full scan if checkpoint is disabled

Message ID 20231204180551.927014-1-jaegeuk@kernel.org (mailing list archive)
State New
Headers show
Series [f2fs-dev] fsck.f2fs: run full scan if checkpoint is disabled | expand

Commit Message

Jaegeuk Kim Dec. 4, 2023, 6:05 p.m. UTC
Let's fix any inconsistency until checkpint being enabled back.

Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
---
 fsck/mount.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Daeho Jeong Dec. 4, 2023, 7:12 p.m. UTC | #1
LGTM

On Mon, Dec 4, 2023 at 10:07 AM Jaegeuk Kim <jaegeuk@kernel.org> wrote:
>
> Let's fix any inconsistency until checkpint being enabled back.
>
> Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
> ---
>  fsck/mount.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/fsck/mount.c b/fsck/mount.c
> index e957904494ef..30c62280b281 100644
> --- a/fsck/mount.c
> +++ b/fsck/mount.c
> @@ -1435,6 +1435,7 @@ static int f2fs_should_proceed(struct f2fs_super_block *sb, u32 flag)
>  {
>         if (!c.fix_on && (c.auto_fix || c.preen_mode)) {
>                 if (flag & CP_FSCK_FLAG ||
> +                       flag & CP_DISABLED_FLAG ||
>                         flag & CP_QUOTA_NEED_FSCK_FLAG ||
>                         c.abnormal_stop || c.fs_errors ||
>                         (exist_qf_ino(sb) && (flag & CP_ERROR_FLAG))) {
> --
> 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
Jaegeuk Kim Dec. 5, 2023, 1:16 a.m. UTC | #2
Thanks. Let me add reviewed-by. :)

On 12/04, Daeho Jeong wrote:
> LGTM
> 
> On Mon, Dec 4, 2023 at 10:07 AM Jaegeuk Kim <jaegeuk@kernel.org> wrote:
> >
> > Let's fix any inconsistency until checkpint being enabled back.
> >
> > Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
> > ---
> >  fsck/mount.c | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/fsck/mount.c b/fsck/mount.c
> > index e957904494ef..30c62280b281 100644
> > --- a/fsck/mount.c
> > +++ b/fsck/mount.c
> > @@ -1435,6 +1435,7 @@ static int f2fs_should_proceed(struct f2fs_super_block *sb, u32 flag)
> >  {
> >         if (!c.fix_on && (c.auto_fix || c.preen_mode)) {
> >                 if (flag & CP_FSCK_FLAG ||
> > +                       flag & CP_DISABLED_FLAG ||
> >                         flag & CP_QUOTA_NEED_FSCK_FLAG ||
> >                         c.abnormal_stop || c.fs_errors ||
> >                         (exist_qf_ino(sb) && (flag & CP_ERROR_FLAG))) {
> > --
> > 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 mbox series

Patch

diff --git a/fsck/mount.c b/fsck/mount.c
index e957904494ef..30c62280b281 100644
--- a/fsck/mount.c
+++ b/fsck/mount.c
@@ -1435,6 +1435,7 @@  static int f2fs_should_proceed(struct f2fs_super_block *sb, u32 flag)
 {
 	if (!c.fix_on && (c.auto_fix || c.preen_mode)) {
 		if (flag & CP_FSCK_FLAG ||
+			flag & CP_DISABLED_FLAG ||
 			flag & CP_QUOTA_NEED_FSCK_FLAG ||
 			c.abnormal_stop || c.fs_errors ||
 			(exist_qf_ino(sb) && (flag & CP_ERROR_FLAG))) {