Whamcloud - gitweb
e2fsck: create logs for mult-threads
authorLi Xi <lixi@ddn.com>
Tue, 13 Aug 2019 03:55:27 +0000 (11:55 +0800)
committerLi Dongyang <dongyangli@ddn.com>
Fri, 28 Apr 2023 00:35:54 +0000 (10:35 +1000)
When multi-threads are used, different logs should be created
for different threads. Each thread has log files with suffix
of ".$THREAD_INDEX".

And this patch adds f_multithread_logfile test case.

E2fsprogs-commit: 2643de7d00dc8da9f5be79fab768fca7a27f3745

Change-Id: I86982866b9e8e2d2cc5ac2ec9ac44a3b802c21e0
Signed-off-by: Li Xi <lixi@ddn.com>
Signed-off-by: Wang Shilong <wshilong@ddn.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
e2fsck/e2fsck.h
e2fsck/logfile.c
e2fsck/pass1.c
tests/f_multithread_logfile/expect.1 [new file with mode: 0644]
tests/f_multithread_logfile/image.gz [new symlink]
tests/f_multithread_logfile/name [new file with mode: 0644]
tests/f_multithread_logfile/script [new file with mode: 0644]

index a1d21c0..b45912e 100644 (file)
@@ -343,8 +343,10 @@ struct e2fsck_struct {
        /*
         * For pass1_check_directory and pass1_get_blocks
         */
-       ext2_ino_t stashed_ino;
-       struct ext2_inode *stashed_inode;
+       ext2_ino_t              stashed_ino;
+       struct ext2_inode       *stashed_inode;
+       /* Thread index, if global_ctx is null, this field is unused */
+       int                     thread_index;
 
        /*
         * Location of the lost and found directory
index 9d79eed..83dbcef 100644 (file)
@@ -16,6 +16,7 @@
 #include <sys/types.h>
 #include <sys/stat.h>
 #include <fcntl.h>
+#include <assert.h>
 
 #include "e2fsck.h"
 #include <pwd.h>
@@ -294,6 +295,8 @@ static FILE *set_up_log_file(e2fsck_t ctx, const char *key, const char *fn)
        struct string s, s1, s2;
        char *s0 = 0, *log_dir = 0, *log_fn = 0;
        int log_dir_wait = 0;
+       int string_size;
+       char string_index[10];
 
        s.s = s1.s = s2.s = 0;
 
@@ -310,6 +313,12 @@ static FILE *set_up_log_file(e2fsck_t ctx, const char *key, const char *fn)
                goto out;
 
        expand_logfn(ctx, log_fn, &s);
+       if (ctx->global_ctx) {
+               sprintf(string_index, "%d", ctx->thread_index);
+               append_string(&s, ".", 1);
+               append_string(&s, string_index, 0);
+       }
+
        if ((log_fn[0] == '/') || !log_dir || !log_dir[0])
                s0 = s.s;
 
@@ -328,7 +337,6 @@ static FILE *set_up_log_file(e2fsck_t ctx, const char *key, const char *fn)
                append_string(&s2, log_dir, 0);
                append_string(&s2, "/", 1);
                append_string(&s2, s.s, 0);
-               printf("%s\n", s2.s);
        }
 
        if (s0)
index 5c02a8e..c58971e 100644 (file)
@@ -2320,6 +2320,9 @@ static errcode_t e2fsck_pass1_thread_prepare(e2fsck_t global_ctx, e2fsck_t *thre
        }
        thread_fs->priv_data = thread_context;
 
+       thread_context->thread_index = 0;
+       set_up_logging(thread_context);
+
        thread_context->fs = thread_fs;
        *thread_ctx = thread_context;
        return 0;
@@ -2332,12 +2335,14 @@ out_context:
 
 static int e2fsck_pass1_thread_join_one(e2fsck_t global_ctx, e2fsck_t thread_ctx)
 {
-       errcode_t       retval;
-       int             flags = global_ctx->flags;
-       ext2_filsys     thread_fs = thread_ctx->fs;
-       ext2_filsys     global_fs = global_ctx->fs;
+       errcode_t        retval;
+       int              flags = global_ctx->flags;
+       ext2_filsys      thread_fs = thread_ctx->fs;
+       ext2_filsys      global_fs = global_ctx->fs;
+       FILE            *global_logf = global_ctx->logf;
+       FILE            *global_problem_logf = global_ctx->problem_logf;
 #ifdef HAVE_SETJMP_H
-       jmp_buf         old_jmp;
+       jmp_buf          old_jmp;
 
        memcpy(old_jmp, global_ctx->abort_loc, sizeof(jmp_buf));
 #endif
@@ -2356,6 +2361,8 @@ static int e2fsck_pass1_thread_join_one(e2fsck_t global_ctx, e2fsck_t thread_ctx
        }
        global_fs->priv_data = global_ctx;
        global_ctx->fs = global_fs;
+       global_ctx->logf = global_logf;
+       global_ctx->problem_logf = global_problem_logf;
 
        if (thread_ctx->inode_used_map) {
                retval = e2fsck_pass1_copy_bitmap(global_fs,
@@ -2444,6 +2451,12 @@ static int e2fsck_pass1_thread_join(e2fsck_t global_ctx, e2fsck_t thread_ctx)
 
        retval = e2fsck_pass1_thread_join_one(global_ctx, thread_ctx);
        ext2fs_free_mem(&thread_ctx->fs);
+       if (thread_ctx->logf)
+               fclose(thread_ctx->logf);
+       if (thread_ctx->problem_logf) {
+               fputs("</problem_log>\n", thread_ctx->problem_logf);
+               fclose(thread_ctx->problem_logf);
+       }
        ext2fs_free_mem(&thread_ctx);
 
        return retval;
diff --git a/tests/f_multithread_logfile/expect.1 b/tests/f_multithread_logfile/expect.1
new file mode 100644 (file)
index 0000000..e2b954d
--- /dev/null
@@ -0,0 +1,23 @@
+ext2fs_open2: Bad magic number in super-block
+../e2fsck/e2fsck: Superblock invalid, trying backup blocks...
+Pass 1: Checking inodes, blocks, and sizes
+Pass 2: Checking directory structure
+Pass 3: Checking directory connectivity
+Pass 4: Checking reference counts
+Pass 5: Checking group summary information
+Free blocks count wrong for group #0 (7987, counted=7982).
+Fix? yes
+
+Free blocks count wrong (11602, counted=11597).
+Fix? yes
+
+Free inodes count wrong for group #0 (1493, counted=1488).
+Fix? yes
+
+Free inodes count wrong (2997, counted=2992).
+Fix? yes
+
+
+test_filesys: ***** FILE SYSTEM WAS MODIFIED *****
+test_filesys: 16/3008 files (0.0% non-contiguous), 403/12000 blocks
+Exit status is 1
diff --git a/tests/f_multithread_logfile/image.gz b/tests/f_multithread_logfile/image.gz
new file mode 120000 (symlink)
index 0000000..0fd4001
--- /dev/null
@@ -0,0 +1 @@
+../f_zero_super/image.gz
\ No newline at end of file
diff --git a/tests/f_multithread_logfile/name b/tests/f_multithread_logfile/name
new file mode 100644 (file)
index 0000000..faaabc3
--- /dev/null
@@ -0,0 +1 @@
+test "e2fsck -m" option works with "-E log_filename="
diff --git a/tests/f_multithread_logfile/script b/tests/f_multithread_logfile/script
new file mode 100644 (file)
index 0000000..4f9ca6f
--- /dev/null
@@ -0,0 +1,32 @@
+LOG_FNAME="f_multithread_logfile_xxx"
+FSCK_OPT="-fy -m -y -E log_filename=$LOG_FNAME"
+SKIP_VERIFY="true"
+ONE_PASS_ONLY="true"
+SKIP_CLEANUP="true"
+
+rm -f $LOG_FNAME.* $LOG_FNAME
+
+. $cmd_dir/run_e2fsck
+
+rm -f $test_name.ok $test_name.failed
+cmp -s $OUT1 $EXP1
+status1=$?
+
+if [ "$status1" -eq 0 ]; then
+       if [ ! -f $LOG_FNAME -o ! -f $LOG_FNAME.0 ]; then
+               echo "$LOG_FNAME or $LOG_FNAME.0 is not created" > $test_name.failed
+               echo "$test_name: $test_description: failed"
+       else
+               echo "$test_name: $test_description: ok"
+               touch $test_name.ok
+       fi
+else
+       diff $DIFF_OPTS $test_dir/expect.1 \
+               $test_name.1.log >> $test_name.failed
+        echo "$test_name: $test_description: failed"
+fi
+
+unset IMAGE FSCK_OPT SECOND_FSCK_OPT OUT1 OUT2 EXP1 EXP2
+unset SKIP_VERIFY SKIP_CLEANUP SKIP_GUNZIP ONE_PASS_ONLY PREP_CMD
+unset DESCRIPTION SKIP_UNLINK AFTER_CMD PASS_ZERO
+unset LOG_FINAME