From 011174eab1c2059e657a12f69ce6bdb4b19db4ba Mon Sep 17 00:00:00 2001 From: Filipe Manana Date: Wed, 27 Mar 2024 17:11:37 +0000 Subject: btrfs/028: removed redundant sync and scratch filesystem unmount There's no need to have an explicit scratch filesystem sync and unmount at the of the test, as the fstests framework automatically unmounts the filesystem and the unmount naturally syncs any data and metadata. So remove them and update the comment to be more clear. Reviewed-by: Anand Jain Signed-off-by: Filipe Manana Signed-off-by: Anand Jain --- tests/btrfs/028 | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) (limited to 'tests/btrfs') diff --git a/tests/btrfs/028 b/tests/btrfs/028 index 8fbe8887..0c96b2fe 100755 --- a/tests/btrfs/028 +++ b/tests/btrfs/028 @@ -48,11 +48,8 @@ kill $fsstress_pid &> /dev/null wait $fsstress_pid &> /dev/null _btrfs_kill_stress_balance_pid $balance_pid -_run_btrfs_util_prog filesystem sync $SCRATCH_MNT - -_scratch_unmount - -# qgroup will be checked at _check_scratch_fs() by fstest. +# The qgroups accounting will be checked by 'btrfs check' (fsck) after the +# fstests framework unmounts the filesystem. echo "Silence is golden" status=0 -- cgit v1.2.3