Whamcloud - gitweb
e4defrag: adjust total count when files change during the run
[tools/e2fsprogs.git] / e2fsck / journal.c
index 6119cda..e83f3a9 100644 (file)
@@ -112,7 +112,7 @@ int journal_bmap(journal_t *journal, blk64_t block, unsigned long long *phys)
        retval= ext2fs_bmap2(inode->i_ctx->fs, inode->i_ino,
                             &inode->i_ext2, NULL, 0, block, 0, &pblk);
        *phys = pblk;
-       return (int) retval;
+       return -1 * ((int) retval);
 #endif
 }
 
@@ -153,7 +153,7 @@ int sync_blockdev(kdev_t kdev)
        else
                io = kdev->k_ctx->journal_io;
 
-       return io_channel_flush(io) ? EIO : 0;
+       return io_channel_flush(io) ? -EIO : 0;
 }
 
 void ll_rw_block(int rw, int nr, struct buffer_head *bhp[])
@@ -239,7 +239,7 @@ void wait_on_buffer(struct buffer_head *bh)
 
 static void e2fsck_clear_recover(e2fsck_t ctx, int error)
 {
-       ctx->fs->super->s_feature_incompat &= ~EXT3_FEATURE_INCOMPAT_RECOVER;
+       ext2fs_clear_feature_journal_needs_recovery(ctx->fs->super);
 
        /* if we had an error doing journal recovery, we need a full fsck */
        if (error)
@@ -289,6 +289,7 @@ static errcode_t e2fsck_get_journal(e2fsck_t ctx, journal_t **ret_journal)
        errcode_t               retval = 0;
        io_manager              io_ptr = 0;
        unsigned long long      start = 0;
+       int                     ret;
        int                     ext_journal = 0;
        int                     tried_backup_jnl = 0;
 
@@ -389,8 +390,10 @@ static errcode_t e2fsck_get_journal(e2fsck_t ctx, journal_t **ret_journal)
 #else
                journal->j_inode = j_inode;
                ctx->journal_io = ctx->fs->io;
-               if ((retval = (errcode_t) journal_bmap(journal, 0, &start)) != 0)
+               if ((ret = journal_bmap(journal, 0, &start)) != 0) {
+                       retval = (errcode_t) (-1 * ret);
                        goto errout;
+               }
 #endif
        } else {
                ext_journal = 1;
@@ -461,7 +464,7 @@ static errcode_t e2fsck_get_journal(e2fsck_t ctx, journal_t **ret_journal)
                        ext2fs_swap_super(&jsuper);
 #endif
                if (jsuper.s_magic != EXT2_SUPER_MAGIC ||
-                   !(jsuper.s_feature_incompat & EXT3_FEATURE_INCOMPAT_JOURNAL_DEV)) {
+                   !ext2fs_has_feature_journal_dev(&jsuper)) {
                        fix_problem(ctx, PR_0_EXT_JOURNAL_BAD_SUPER, &pctx);
                        retval = EXT2_ET_LOAD_EXT_JOURNAL;
                        brelse(bh);
@@ -477,8 +480,7 @@ static errcode_t e2fsck_get_journal(e2fsck_t ctx, journal_t **ret_journal)
                }
 
                /* Check the superblock checksum */
-               if (jsuper.s_feature_ro_compat &
-                   EXT4_FEATURE_RO_COMPAT_METADATA_CSUM) {
+               if (ext2fs_has_feature_metadata_csum(&jsuper)) {
                        struct struct_ext2_filsys fsx;
                        struct ext2_super_block superx;
                        void *p;
@@ -487,8 +489,7 @@ static errcode_t e2fsck_get_journal(e2fsck_t ctx, journal_t **ret_journal)
                        memcpy(&fsx, ctx->fs, sizeof(fsx));
                        memcpy(&superx, ctx->fs->super, sizeof(superx));
                        fsx.super = &superx;
-                       fsx.super->s_feature_ro_compat |=
-                                       EXT4_FEATURE_RO_COMPAT_METADATA_CSUM;
+                       ext2fs_set_feature_metadata_csum(fsx.super);
                        if (!ext2fs_superblock_csum_verify(&fsx, p) &&
                            fix_problem(ctx, PR_0_EXT_JOURNAL_SUPER_CSUM_INVALID,
                                        &pctx)) {
@@ -535,19 +536,16 @@ static errcode_t e2fsck_journal_fix_bad_inode(e2fsck_t ctx,
                                              struct problem_context *pctx)
 {
        struct ext2_super_block *sb = ctx->fs->super;
-       int recover = ctx->fs->super->s_feature_incompat &
-               EXT3_FEATURE_INCOMPAT_RECOVER;
-       int has_journal = ctx->fs->super->s_feature_compat &
-               EXT3_FEATURE_COMPAT_HAS_JOURNAL;
+       int recover = ext2fs_has_feature_journal_needs_recovery(ctx->fs->super);
+       int has_journal = ext2fs_has_feature_journal(ctx->fs->super);
 
        if (has_journal || sb->s_journal_inum) {
                /* The journal inode is bogus, remove and force full fsck */
                pctx->ino = sb->s_journal_inum;
                if (fix_problem(ctx, PR_0_JOURNAL_BAD_INODE, pctx)) {
                        if (has_journal && sb->s_journal_inum)
-                               printf("*** ext3 journal has been deleted - "
-                                      "filesystem is now ext2 only ***\n\n");
-                       sb->s_feature_compat &= ~EXT3_FEATURE_COMPAT_HAS_JOURNAL;
+                               printf("*** journal has been deleted ***\n\n");
+                       ext2fs_clear_feature_journal(sb);
                        sb->s_journal_inum = 0;
                        memset(sb->s_jnl_blocks, 0, sizeof(sb->s_jnl_blocks));
                        ctx->flags |= E2F_FLAG_JOURNAL_INODE;
@@ -555,7 +553,7 @@ static errcode_t e2fsck_journal_fix_bad_inode(e2fsck_t ctx,
                        e2fsck_clear_recover(ctx, 1);
                        return 0;
                }
-               return EXT2_ET_BAD_INODE_NUM;
+               return EXT2_ET_CORRUPT_JOURNAL_SB;
        } else if (recover) {
                if (fix_problem(ctx, PR_0_JOURNAL_RECOVER_SET, pctx)) {
                        e2fsck_clear_recover(ctx, 1);
@@ -577,6 +575,7 @@ static void clear_v2_journal_fields(journal_t *journal)
        if (!fix_problem(ctx, PR_0_CLEAR_V2_JOURNAL, &pctx))
                return;
 
+       ctx->flags |= E2F_FLAG_PROBLEMS_FIXED;
        memset(((char *) journal->j_superblock) + V1_SB_SIZE, 0,
               ctx->fs->blocksize-V1_SB_SIZE);
        mark_buffer_dirty(journal->j_sb_buffer);
@@ -632,7 +631,7 @@ static errcode_t e2fsck_journal_load(journal_t *journal)
        case JFS_DESCRIPTOR_BLOCK:
        case JFS_COMMIT_BLOCK:
        case JFS_REVOKE_BLOCK:
-               return EXT2_ET_CORRUPT_SUPERBLOCK;
+               return EXT2_ET_CORRUPT_JOURNAL_SB;
 
        /* If we don't understand the superblock major type, but there
         * is a magic number, then it is likely to be a new format we
@@ -648,17 +647,16 @@ static errcode_t e2fsck_journal_load(journal_t *journal)
                return EXT2_ET_RO_UNSUPP_FEATURE;
 
        /* Checksum v1-3 are mutually exclusive features. */
-       if (JFS_HAS_INCOMPAT_FEATURE(journal, JFS_FEATURE_INCOMPAT_CSUM_V2) &&
-           JFS_HAS_INCOMPAT_FEATURE(journal, JFS_FEATURE_INCOMPAT_CSUM_V3))
-               return EXT2_ET_CORRUPT_SUPERBLOCK;
+       if (jfs_has_feature_csum2(journal) && jfs_has_feature_csum3(journal))
+               return EXT2_ET_CORRUPT_JOURNAL_SB;
 
        if (journal_has_csum_v2or3(journal) &&
-           JFS_HAS_COMPAT_FEATURE(journal, JFS_FEATURE_COMPAT_CHECKSUM))
-               return EXT2_ET_CORRUPT_SUPERBLOCK;
+           jfs_has_feature_checksum(journal))
+               return EXT2_ET_CORRUPT_JOURNAL_SB;
 
        if (!e2fsck_journal_verify_csum_type(journal, jsb) ||
            !e2fsck_journal_sb_csum_verify(journal, jsb))
-               return EXT2_ET_CORRUPT_SUPERBLOCK;
+               return EXT2_ET_CORRUPT_JOURNAL_SB;
 
        if (journal_has_csum_v2or3(journal))
                journal->j_csum_seed = jbd2_chksum(journal, ~0, jsb->s_uuid,
@@ -668,19 +666,19 @@ static errcode_t e2fsck_journal_load(journal_t *journal)
         * format to be able to proceed safely, so any other checks that
         * fail we should attempt to recover from. */
        if (jsb->s_blocksize != htonl(journal->j_blocksize)) {
-               com_err(ctx->program_name, EXT2_ET_CORRUPT_SUPERBLOCK,
+               com_err(ctx->program_name, EXT2_ET_CORRUPT_JOURNAL_SB,
                        _("%s: no valid journal superblock found\n"),
                        ctx->device_name);
-               return EXT2_ET_CORRUPT_SUPERBLOCK;
+               return EXT2_ET_CORRUPT_JOURNAL_SB;
        }
 
        if (ntohl(jsb->s_maxlen) < journal->j_maxlen)
                journal->j_maxlen = ntohl(jsb->s_maxlen);
        else if (ntohl(jsb->s_maxlen) > journal->j_maxlen) {
-               com_err(ctx->program_name, EXT2_ET_CORRUPT_SUPERBLOCK,
+               com_err(ctx->program_name, EXT2_ET_CORRUPT_JOURNAL_SB,
                        _("%s: journal too short\n"),
                        ctx->device_name);
-               return EXT2_ET_CORRUPT_SUPERBLOCK;
+               return EXT2_ET_CORRUPT_JOURNAL_SB;
        }
 
        journal->j_tail_sequence = ntohl(jsb->s_sequence);
@@ -742,10 +740,9 @@ static errcode_t e2fsck_journal_fix_corrupt_super(e2fsck_t ctx,
                                                  struct problem_context *pctx)
 {
        struct ext2_super_block *sb = ctx->fs->super;
-       int recover = ctx->fs->super->s_feature_incompat &
-               EXT3_FEATURE_INCOMPAT_RECOVER;
+       int recover = ext2fs_has_feature_journal_needs_recovery(ctx->fs->super);
 
-       if (sb->s_feature_compat & EXT3_FEATURE_COMPAT_HAS_JOURNAL) {
+       if (ext2fs_has_feature_journal(sb)) {
                if (fix_problem(ctx, PR_0_JOURNAL_BAD_SUPER, pctx)) {
                        e2fsck_journal_reset_super(ctx, journal->j_superblock,
                                                   journal);
@@ -753,9 +750,9 @@ static errcode_t e2fsck_journal_fix_corrupt_super(e2fsck_t ctx,
                        e2fsck_clear_recover(ctx, recover);
                        return 0;
                }
-               return EXT2_ET_CORRUPT_SUPERBLOCK;
+               return EXT2_ET_CORRUPT_JOURNAL_SB;
        } else if (e2fsck_journal_fix_bad_inode(ctx, pctx))
-               return EXT2_ET_CORRUPT_SUPERBLOCK;
+               return EXT2_ET_CORRUPT_JOURNAL_SB;
 
        return 0;
 }
@@ -800,15 +797,14 @@ errcode_t e2fsck_check_ext3_journal(e2fsck_t ctx)
 {
        struct ext2_super_block *sb = ctx->fs->super;
        journal_t *journal;
-       int recover = ctx->fs->super->s_feature_incompat &
-               EXT3_FEATURE_INCOMPAT_RECOVER;
+       int recover = ext2fs_has_feature_journal_needs_recovery(ctx->fs->super);
        struct problem_context pctx;
        problem_t problem;
        int reset = 0, force_fsck = 0;
        errcode_t retval;
 
        /* If we don't have any journal features, don't do anything more */
-       if (!(sb->s_feature_compat & EXT3_FEATURE_COMPAT_HAS_JOURNAL) &&
+       if (!ext2fs_has_feature_journal(sb) &&
            !recover && sb->s_journal_inum == 0 && sb->s_journal_dev == 0 &&
            uuid_is_null(sb->s_journal_uuid))
                return 0;
@@ -828,7 +824,7 @@ errcode_t e2fsck_check_ext3_journal(e2fsck_t ctx)
 
        retval = e2fsck_journal_load(journal);
        if (retval) {
-               if ((retval == EXT2_ET_CORRUPT_SUPERBLOCK) ||
+               if ((retval == EXT2_ET_CORRUPT_JOURNAL_SB) ||
                    ((retval == EXT2_ET_UNSUPP_FEATURE) &&
                    (!fix_problem(ctx, PR_0_JOURNAL_UNSUPP_INCOMPAT,
                                  &pctx))) ||
@@ -849,8 +845,8 @@ errcode_t e2fsck_check_ext3_journal(e2fsck_t ctx)
         * with -y, -n, or -p, only if a user isn't making up their mind.
         */
 no_has_journal:
-       if (!(sb->s_feature_compat & EXT3_FEATURE_COMPAT_HAS_JOURNAL)) {
-               recover = sb->s_feature_incompat & EXT3_FEATURE_INCOMPAT_RECOVER;
+       if (!ext2fs_has_feature_journal(sb)) {
+               recover = ext2fs_has_feature_journal_needs_recovery(sb);
                if (fix_problem(ctx, PR_0_JOURNAL_HAS_JOURNAL, &pctx)) {
                        if (recover &&
                            !fix_problem(ctx, PR_0_JOURNAL_RECOVER_SET, &pctx))
@@ -868,14 +864,14 @@ no_has_journal:
                               sizeof(sb->s_journal_uuid));
                        e2fsck_clear_recover(ctx, force_fsck);
                } else if (!(ctx->options & E2F_OPT_READONLY)) {
-                       sb->s_feature_compat |= EXT3_FEATURE_COMPAT_HAS_JOURNAL;
+                       ext2fs_set_feature_journal(sb);
                        ctx->fs->flags &= ~EXT2_FLAG_MASTER_SB_ONLY;
                        ext2fs_mark_super_dirty(ctx->fs);
                }
        }
 
-       if (sb->s_feature_compat & EXT3_FEATURE_COMPAT_HAS_JOURNAL &&
-           !(sb->s_feature_incompat & EXT3_FEATURE_INCOMPAT_RECOVER) &&
+       if (ext2fs_has_feature_journal(sb) &&
+           !ext2fs_has_feature_journal_needs_recovery(sb) &&
            journal->j_superblock->s_start != 0) {
                /* Print status information */
                fix_problem(ctx, PR_0_JOURNAL_RECOVERY_CLEAR, &pctx);
@@ -885,8 +881,7 @@ no_has_journal:
                        problem = PR_0_JOURNAL_RUN;
                if (fix_problem(ctx, problem, &pctx)) {
                        ctx->options |= E2F_OPT_FORCE;
-                       sb->s_feature_incompat |=
-                               EXT3_FEATURE_INCOMPAT_RECOVER;
+                       ext2fs_set_feature_journal_needs_recovery(sb);
                        ext2fs_mark_super_dirty(ctx->fs);
                } else if (fix_problem(ctx,
                                       PR_0_JOURNAL_RESET_JOURNAL, &pctx)) {
@@ -912,7 +907,7 @@ no_has_journal:
         * the journal's errno is set; if so, we need to mark the file
         * system as being corrupt and clear the journal's s_errno.
         */
-       if (!(sb->s_feature_incompat & EXT3_FEATURE_INCOMPAT_RECOVER) &&
+       if (!ext2fs_has_feature_journal_needs_recovery(sb) &&
            journal->j_superblock->s_errno) {
                ctx->fs->super->s_state |= EXT2_ERROR_FS;
                ext2fs_mark_super_dirty(ctx->fs);
@@ -957,6 +952,8 @@ static errcode_t recover_ext3_journal(e2fsck_t ctx)
                mark_buffer_dirty(journal->j_sb_buffer);
        }
 
+       journal->j_tail_sequence = journal->j_transaction_sequence;
+
 errout:
        journal_destroy_revoke(journal);
        journal_destroy_revoke_caches();
@@ -1048,7 +1045,7 @@ void e2fsck_move_ext3_journal(e2fsck_t ctx)
         */
        if ((ctx->options & E2F_OPT_READONLY) ||
            (sb->s_journal_inum == 0) ||
-           !(sb->s_feature_compat & EXT3_FEATURE_COMPAT_HAS_JOURNAL))
+           !ext2fs_has_feature_journal(sb))
                return;
 
        /*
@@ -1160,7 +1157,7 @@ int e2fsck_fix_ext3_journal_hint(e2fsck_t ctx)
        char uuid[37], *journal_name;
        struct stat st;
 
-       if (!(sb->s_feature_compat & EXT3_FEATURE_COMPAT_HAS_JOURNAL) ||
+       if (!ext2fs_has_feature_journal(sb) ||
            uuid_is_null(sb->s_journal_uuid))
                return 0;