| Message ID | 20210909174142.357719-2-catherine.hoang@oracle.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Dump log cleanups | expand |
On 9/9/21 10:41 AM, Catherine Hoang wrote: > Rename _scratch_inject_logprint to _scratch_remount_dump_log to better > describe what this function does. _scratch_remount_dump_log unmounts > and remounts the scratch device, dumping the log. > > Signed-off-by: Catherine Hoang <catherine.hoang@oracle.com> Looks good to me. Reviewed-by: Allison Henderson <allison.henderson@oracle.com> > --- > common/inject | 2 +- > tests/xfs/312 | 2 +- > tests/xfs/313 | 2 +- > tests/xfs/314 | 2 +- > tests/xfs/315 | 2 +- > tests/xfs/316 | 2 +- > tests/xfs/317 | 2 +- > tests/xfs/318 | 2 +- > tests/xfs/319 | 2 +- > tests/xfs/320 | 2 +- > tests/xfs/321 | 2 +- > tests/xfs/322 | 2 +- > tests/xfs/323 | 2 +- > tests/xfs/324 | 2 +- > tests/xfs/325 | 2 +- > tests/xfs/326 | 2 +- > tests/xfs/329 | 2 +- > 17 files changed, 17 insertions(+), 17 deletions(-) > > diff --git a/common/inject b/common/inject > index 984ec209..3b731df7 100644 > --- a/common/inject > +++ b/common/inject > @@ -113,7 +113,7 @@ _scratch_inject_error() > } > > # Unmount and remount the scratch device, dumping the log > -_scratch_inject_logprint() > +_scratch_remount_dump_log() > { > local opts="$1" > > diff --git a/tests/xfs/312 b/tests/xfs/312 > index 1fcf26ab..94f868fe 100755 > --- a/tests/xfs/312 > +++ b/tests/xfs/312 > @@ -63,7 +63,7 @@ echo "FS should be shut down, touch will fail" > touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch > > echo "Remount to replay log" > -_scratch_inject_logprint >> $seqres.full > +_scratch_remount_dump_log >> $seqres.full > > echo "FS should be online, touch should succeed" > touch $SCRATCH_MNT/goodfs > diff --git a/tests/xfs/313 b/tests/xfs/313 > index 6d2f9fac..9c7cf5b9 100755 > --- a/tests/xfs/313 > +++ b/tests/xfs/313 > @@ -63,7 +63,7 @@ echo "FS should be shut down, touch will fail" > touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch > > echo "Remount to replay log" > -_scratch_inject_logprint >> $seqres.full > +_scratch_remount_dump_log >> $seqres.full > > echo "FS should be online, touch should succeed" > touch $SCRATCH_MNT/goodfs > diff --git a/tests/xfs/314 b/tests/xfs/314 > index 5165393e..9ac311d0 100755 > --- a/tests/xfs/314 > +++ b/tests/xfs/314 > @@ -64,7 +64,7 @@ echo "FS should be shut down, touch will fail" > touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch > > echo "Remount to replay log" > -_scratch_inject_logprint >> $seqres.full > +_scratch_remount_dump_log >> $seqres.full > > echo "FS should be online, touch should succeed" > touch $SCRATCH_MNT/goodfs > diff --git a/tests/xfs/315 b/tests/xfs/315 > index 958a8c99..105515ab 100755 > --- a/tests/xfs/315 > +++ b/tests/xfs/315 > @@ -61,7 +61,7 @@ echo "FS should be shut down, touch will fail" > touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch > > echo "Remount to replay log" > -_scratch_inject_logprint >> $seqres.full > +_scratch_remount_dump_log >> $seqres.full > > echo "FS should be online, touch should succeed" > touch $SCRATCH_MNT/goodfs > diff --git a/tests/xfs/316 b/tests/xfs/316 > index cf0c5adc..f0af19d2 100755 > --- a/tests/xfs/316 > +++ b/tests/xfs/316 > @@ -61,7 +61,7 @@ echo "CoW all the blocks" > $XFS_IO_PROG -c "pwrite -W -S 0x67 -b $sz 0 $((blks * blksz))" $SCRATCH_MNT/file2 >> $seqres.full > > echo "Remount to replay log" > -_scratch_inject_logprint >> $seqres.full > +_scratch_remount_dump_log >> $seqres.full > > echo "FS should be online, touch should succeed" > touch $SCRATCH_MNT/goodfs > diff --git a/tests/xfs/317 b/tests/xfs/317 > index 7eef67af..1ca2672d 100755 > --- a/tests/xfs/317 > +++ b/tests/xfs/317 > @@ -54,7 +54,7 @@ echo "FS should be shut down, touch will fail" > touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch > > echo "Remount to replay log" > -_scratch_inject_logprint >> $seqres.full > +_scratch_remount_dump_log >> $seqres.full > > echo "Check files" > md5sum $SCRATCH_MNT/file0 | _filter_scratch > diff --git a/tests/xfs/318 b/tests/xfs/318 > index d822e89a..38c7aa60 100755 > --- a/tests/xfs/318 > +++ b/tests/xfs/318 > @@ -60,7 +60,7 @@ echo "FS should be shut down, touch will fail" > touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch > > echo "Remount to replay log" > -_scratch_inject_logprint >> $seqres.full > +_scratch_remount_dump_log >> $seqres.full > > echo "Check files" > md5sum $SCRATCH_MNT/file1 2>&1 | _filter_scratch > diff --git a/tests/xfs/319 b/tests/xfs/319 > index 0f61c119..d64651fb 100755 > --- a/tests/xfs/319 > +++ b/tests/xfs/319 > @@ -57,7 +57,7 @@ echo "FS should be shut down, touch will fail" > touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch > > echo "Remount to replay log" > -_scratch_inject_logprint >> $seqres.full > +_scratch_remount_dump_log >> $seqres.full > > echo "FS should be online, touch should succeed" > touch $SCRATCH_MNT/goodfs > diff --git a/tests/xfs/320 b/tests/xfs/320 > index f65f3ad1..d22d76d9 100755 > --- a/tests/xfs/320 > +++ b/tests/xfs/320 > @@ -55,7 +55,7 @@ echo "FS should be shut down, touch will fail" > touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch > > echo "Remount to replay log" > -_scratch_inject_logprint >> $seqres.full > +_scratch_remount_dump_log >> $seqres.full > > echo "Check files" > md5sum $SCRATCH_MNT/file1 | _filter_scratch > diff --git a/tests/xfs/321 b/tests/xfs/321 > index daff4449..06a34347 100755 > --- a/tests/xfs/321 > +++ b/tests/xfs/321 > @@ -55,7 +55,7 @@ echo "FS should be shut down, touch will fail" > touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch > > echo "Remount to replay log" > -_scratch_inject_logprint >> $seqres.full > +_scratch_remount_dump_log >> $seqres.full > > echo "Check files" > md5sum $SCRATCH_MNT/file1 | _filter_scratch > diff --git a/tests/xfs/322 b/tests/xfs/322 > index f36e54d8..89a2f741 100755 > --- a/tests/xfs/322 > +++ b/tests/xfs/322 > @@ -56,7 +56,7 @@ echo "FS should be shut down, touch will fail" > touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch > > echo "Remount to replay log" > -_scratch_inject_logprint >> $seqres.full > +_scratch_remount_dump_log >> $seqres.full > > echo "Check files" > md5sum $SCRATCH_MNT/file1 | _filter_scratch > diff --git a/tests/xfs/323 b/tests/xfs/323 > index f66a8ebf..66737da0 100755 > --- a/tests/xfs/323 > +++ b/tests/xfs/323 > @@ -55,7 +55,7 @@ echo "FS should be shut down, touch will fail" > touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch > > echo "Remount to replay log" > -_scratch_inject_logprint >> $seqres.full > +_scratch_remount_dump_log >> $seqres.full > > echo "FS should be online, touch should succeed" > touch $SCRATCH_MNT/goodfs > diff --git a/tests/xfs/324 b/tests/xfs/324 > index ca2f25ac..9909db62 100755 > --- a/tests/xfs/324 > +++ b/tests/xfs/324 > @@ -61,7 +61,7 @@ echo "Reflink all the blocks" > _cp_reflink $SCRATCH_MNT/file1 $SCRATCH_MNT/file4 > > echo "Remount to replay log" > -_scratch_inject_logprint >> $seqres.full > +_scratch_remount_dump_log >> $seqres.full > > echo "FS should be online, touch should succeed" > touch $SCRATCH_MNT/goodfs > diff --git a/tests/xfs/325 b/tests/xfs/325 > index 3b98fd50..5b26b2b3 100755 > --- a/tests/xfs/325 > +++ b/tests/xfs/325 > @@ -59,7 +59,7 @@ echo "FS should be shut down, touch will fail" > touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch > > echo "Remount to replay log" > -_scratch_inject_logprint >> $seqres.full > +_scratch_remount_dump_log >> $seqres.full > > echo "FS should be online, touch should succeed" > touch $SCRATCH_MNT/goodfs > diff --git a/tests/xfs/326 b/tests/xfs/326 > index bf5db08a..8b95a18a 100755 > --- a/tests/xfs/326 > +++ b/tests/xfs/326 > @@ -71,7 +71,7 @@ echo "FS should be shut down, touch will fail" > touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch > > echo "Remount to replay log" > -_scratch_inject_logprint >> $seqres.full > +_scratch_remount_dump_log >> $seqres.full > > echo "FS should be online, touch should succeed" > touch $SCRATCH_MNT/goodfs > diff --git a/tests/xfs/329 b/tests/xfs/329 > index e57f6f7f..e9a30d05 100755 > --- a/tests/xfs/329 > +++ b/tests/xfs/329 > @@ -52,7 +52,7 @@ echo "FS should be shut down, touch will fail" > touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch > > echo "Remount to replay log" | tee /dev/ttyprintk > -_scratch_inject_logprint >> $seqres.full > +_scratch_remount_dump_log >> $seqres.full > new_nextents=$(_count_extents $testdir/file1) > > echo "Check extent count" | tee /dev/ttyprintk >
diff --git a/common/inject b/common/inject index 984ec209..3b731df7 100644 --- a/common/inject +++ b/common/inject @@ -113,7 +113,7 @@ _scratch_inject_error() } # Unmount and remount the scratch device, dumping the log -_scratch_inject_logprint() +_scratch_remount_dump_log() { local opts="$1" diff --git a/tests/xfs/312 b/tests/xfs/312 index 1fcf26ab..94f868fe 100755 --- a/tests/xfs/312 +++ b/tests/xfs/312 @@ -63,7 +63,7 @@ echo "FS should be shut down, touch will fail" touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch echo "Remount to replay log" -_scratch_inject_logprint >> $seqres.full +_scratch_remount_dump_log >> $seqres.full echo "FS should be online, touch should succeed" touch $SCRATCH_MNT/goodfs diff --git a/tests/xfs/313 b/tests/xfs/313 index 6d2f9fac..9c7cf5b9 100755 --- a/tests/xfs/313 +++ b/tests/xfs/313 @@ -63,7 +63,7 @@ echo "FS should be shut down, touch will fail" touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch echo "Remount to replay log" -_scratch_inject_logprint >> $seqres.full +_scratch_remount_dump_log >> $seqres.full echo "FS should be online, touch should succeed" touch $SCRATCH_MNT/goodfs diff --git a/tests/xfs/314 b/tests/xfs/314 index 5165393e..9ac311d0 100755 --- a/tests/xfs/314 +++ b/tests/xfs/314 @@ -64,7 +64,7 @@ echo "FS should be shut down, touch will fail" touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch echo "Remount to replay log" -_scratch_inject_logprint >> $seqres.full +_scratch_remount_dump_log >> $seqres.full echo "FS should be online, touch should succeed" touch $SCRATCH_MNT/goodfs diff --git a/tests/xfs/315 b/tests/xfs/315 index 958a8c99..105515ab 100755 --- a/tests/xfs/315 +++ b/tests/xfs/315 @@ -61,7 +61,7 @@ echo "FS should be shut down, touch will fail" touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch echo "Remount to replay log" -_scratch_inject_logprint >> $seqres.full +_scratch_remount_dump_log >> $seqres.full echo "FS should be online, touch should succeed" touch $SCRATCH_MNT/goodfs diff --git a/tests/xfs/316 b/tests/xfs/316 index cf0c5adc..f0af19d2 100755 --- a/tests/xfs/316 +++ b/tests/xfs/316 @@ -61,7 +61,7 @@ echo "CoW all the blocks" $XFS_IO_PROG -c "pwrite -W -S 0x67 -b $sz 0 $((blks * blksz))" $SCRATCH_MNT/file2 >> $seqres.full echo "Remount to replay log" -_scratch_inject_logprint >> $seqres.full +_scratch_remount_dump_log >> $seqres.full echo "FS should be online, touch should succeed" touch $SCRATCH_MNT/goodfs diff --git a/tests/xfs/317 b/tests/xfs/317 index 7eef67af..1ca2672d 100755 --- a/tests/xfs/317 +++ b/tests/xfs/317 @@ -54,7 +54,7 @@ echo "FS should be shut down, touch will fail" touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch echo "Remount to replay log" -_scratch_inject_logprint >> $seqres.full +_scratch_remount_dump_log >> $seqres.full echo "Check files" md5sum $SCRATCH_MNT/file0 | _filter_scratch diff --git a/tests/xfs/318 b/tests/xfs/318 index d822e89a..38c7aa60 100755 --- a/tests/xfs/318 +++ b/tests/xfs/318 @@ -60,7 +60,7 @@ echo "FS should be shut down, touch will fail" touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch echo "Remount to replay log" -_scratch_inject_logprint >> $seqres.full +_scratch_remount_dump_log >> $seqres.full echo "Check files" md5sum $SCRATCH_MNT/file1 2>&1 | _filter_scratch diff --git a/tests/xfs/319 b/tests/xfs/319 index 0f61c119..d64651fb 100755 --- a/tests/xfs/319 +++ b/tests/xfs/319 @@ -57,7 +57,7 @@ echo "FS should be shut down, touch will fail" touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch echo "Remount to replay log" -_scratch_inject_logprint >> $seqres.full +_scratch_remount_dump_log >> $seqres.full echo "FS should be online, touch should succeed" touch $SCRATCH_MNT/goodfs diff --git a/tests/xfs/320 b/tests/xfs/320 index f65f3ad1..d22d76d9 100755 --- a/tests/xfs/320 +++ b/tests/xfs/320 @@ -55,7 +55,7 @@ echo "FS should be shut down, touch will fail" touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch echo "Remount to replay log" -_scratch_inject_logprint >> $seqres.full +_scratch_remount_dump_log >> $seqres.full echo "Check files" md5sum $SCRATCH_MNT/file1 | _filter_scratch diff --git a/tests/xfs/321 b/tests/xfs/321 index daff4449..06a34347 100755 --- a/tests/xfs/321 +++ b/tests/xfs/321 @@ -55,7 +55,7 @@ echo "FS should be shut down, touch will fail" touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch echo "Remount to replay log" -_scratch_inject_logprint >> $seqres.full +_scratch_remount_dump_log >> $seqres.full echo "Check files" md5sum $SCRATCH_MNT/file1 | _filter_scratch diff --git a/tests/xfs/322 b/tests/xfs/322 index f36e54d8..89a2f741 100755 --- a/tests/xfs/322 +++ b/tests/xfs/322 @@ -56,7 +56,7 @@ echo "FS should be shut down, touch will fail" touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch echo "Remount to replay log" -_scratch_inject_logprint >> $seqres.full +_scratch_remount_dump_log >> $seqres.full echo "Check files" md5sum $SCRATCH_MNT/file1 | _filter_scratch diff --git a/tests/xfs/323 b/tests/xfs/323 index f66a8ebf..66737da0 100755 --- a/tests/xfs/323 +++ b/tests/xfs/323 @@ -55,7 +55,7 @@ echo "FS should be shut down, touch will fail" touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch echo "Remount to replay log" -_scratch_inject_logprint >> $seqres.full +_scratch_remount_dump_log >> $seqres.full echo "FS should be online, touch should succeed" touch $SCRATCH_MNT/goodfs diff --git a/tests/xfs/324 b/tests/xfs/324 index ca2f25ac..9909db62 100755 --- a/tests/xfs/324 +++ b/tests/xfs/324 @@ -61,7 +61,7 @@ echo "Reflink all the blocks" _cp_reflink $SCRATCH_MNT/file1 $SCRATCH_MNT/file4 echo "Remount to replay log" -_scratch_inject_logprint >> $seqres.full +_scratch_remount_dump_log >> $seqres.full echo "FS should be online, touch should succeed" touch $SCRATCH_MNT/goodfs diff --git a/tests/xfs/325 b/tests/xfs/325 index 3b98fd50..5b26b2b3 100755 --- a/tests/xfs/325 +++ b/tests/xfs/325 @@ -59,7 +59,7 @@ echo "FS should be shut down, touch will fail" touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch echo "Remount to replay log" -_scratch_inject_logprint >> $seqres.full +_scratch_remount_dump_log >> $seqres.full echo "FS should be online, touch should succeed" touch $SCRATCH_MNT/goodfs diff --git a/tests/xfs/326 b/tests/xfs/326 index bf5db08a..8b95a18a 100755 --- a/tests/xfs/326 +++ b/tests/xfs/326 @@ -71,7 +71,7 @@ echo "FS should be shut down, touch will fail" touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch echo "Remount to replay log" -_scratch_inject_logprint >> $seqres.full +_scratch_remount_dump_log >> $seqres.full echo "FS should be online, touch should succeed" touch $SCRATCH_MNT/goodfs diff --git a/tests/xfs/329 b/tests/xfs/329 index e57f6f7f..e9a30d05 100755 --- a/tests/xfs/329 +++ b/tests/xfs/329 @@ -52,7 +52,7 @@ echo "FS should be shut down, touch will fail" touch $SCRATCH_MNT/badfs 2>&1 | _filter_scratch echo "Remount to replay log" | tee /dev/ttyprintk -_scratch_inject_logprint >> $seqres.full +_scratch_remount_dump_log >> $seqres.full new_nextents=$(_count_extents $testdir/file1) echo "Check extent count" | tee /dev/ttyprintk
Rename _scratch_inject_logprint to _scratch_remount_dump_log to better describe what this function does. _scratch_remount_dump_log unmounts and remounts the scratch device, dumping the log. Signed-off-by: Catherine Hoang <catherine.hoang@oracle.com> --- common/inject | 2 +- tests/xfs/312 | 2 +- tests/xfs/313 | 2 +- tests/xfs/314 | 2 +- tests/xfs/315 | 2 +- tests/xfs/316 | 2 +- tests/xfs/317 | 2 +- tests/xfs/318 | 2 +- tests/xfs/319 | 2 +- tests/xfs/320 | 2 +- tests/xfs/321 | 2 +- tests/xfs/322 | 2 +- tests/xfs/323 | 2 +- tests/xfs/324 | 2 +- tests/xfs/325 | 2 +- tests/xfs/326 | 2 +- tests/xfs/329 | 2 +- 17 files changed, 17 insertions(+), 17 deletions(-)