LU-8465 e2fsck: clear icache when using multi-thread fsck

icache of fs will be rebuilt when needed, so after copying
fs, icache can be inited to NULL.

Signed-off-by: Li Xi <lixi@ddn.com>
Signed-off-by: Wang Shilong <wshilong@ddn.com>
Change-Id: If153d20c4fb10163a7759facb14d1c065c7037fa
Reviewed-on: https://review.whamcloud.com/40014
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>

diff --git a/e2fsck/pass1.c b/e2fsck/pass1.c
index c601b6d..4f9f64a 100644 (file)
--- a/e2fsck/pass1.c
+++ b/e2fsck/pass1.c
@@ -2344,8 +2344,10 @@ endit:
                ctx->invalid_bitmaps++;
 }
 
-static void e2fsck_pass1_copy_fs(ext2_filsys dest, ext2_filsys src)
+static errcode_t e2fsck_pass1_copy_fs(ext2_filsys dest, ext2_filsys src)
 {
+       errcode_t       retval;
+
        memcpy(dest, src, sizeof(struct struct_ext2_filsys));
        if (dest->dblist)
                dest->dblist->fs = dest;
@@ -2353,6 +2355,29 @@ static void e2fsck_pass1_copy_fs(ext2_filsys dest, ext2_filsys src)
                dest->inode_map->fs = dest;
        if (dest->block_map)
                dest->block_map->fs = dest;
+
+       /* icache will be rebuilt if needed, so do not copy from @src */
+       src->icache = NULL;
+       return 0;
+}
+
+static void e2fsck_pass1_merge_fs(ext2_filsys dest, ext2_filsys src)
+{
+       struct ext2_inode_cache *icache = dest->icache;
+
+       memcpy(dest, src, sizeof(struct struct_ext2_filsys));
+       if (dest->dblist)
+               dest->dblist->fs = dest;
+       if (dest->inode_map)
+               dest->inode_map->fs = dest;
+       if (dest->block_map)
+               dest->block_map->fs = dest;
+       dest->icache = icache;
+
+       if (src->icache) {
+               ext2fs_free_inode_cache(src->icache);
+               src->icache = NULL;
+       }
 }
 
 static errcode_t e2fsck_pass1_thread_prepare(e2fsck_t global_ctx, e2fsck_t *thread_ctx)
@@ -2376,12 +2401,18 @@ static errcode_t e2fsck_pass1_thread_prepare(e2fsck_t global_ctx, e2fsck_t *thre
                goto out_context;
        }
 
-       e2fsck_pass1_copy_fs(thread_fs, global_fs);
+       retval = e2fsck_pass1_copy_fs(thread_fs, global_fs);
+       if (retval) {
+               com_err(global_ctx->program_name, retval, "while copying fs");
+               goto out_fs;
+       }
        thread_fs->priv_data = thread_context;
 
        thread_context->fs = thread_fs;
        *thread_ctx = thread_context;
        return 0;
+out_fs:
+       ext2fs_free_mem(&thread_fs);
 out_context:
        ext2fs_free_mem(&thread_context);
        return retval;
@@ -2405,7 +2436,7 @@ static int e2fsck_pass1_thread_join(e2fsck_t global_ctx, e2fsck_t thread_ctx)
        global_ctx->flags |= (flags & E2F_FLAG_SIGNAL_MASK) |
                             (global_ctx->flags & E2F_FLAG_SIGNAL_MASK);
 
-       e2fsck_pass1_copy_fs(global_fs, thread_fs);
+       e2fsck_pass1_merge_fs(global_fs, thread_fs);
        global_fs->priv_data = global_ctx;
        global_ctx->fs = global_fs;