From 9ca8d2ac5e02f05e8cd5712cb6ab96aec1f2eb27 Mon Sep 17 00:00:00 2001 From: Naohiro Aota Date: Thu, 12 Aug 2021 00:12:28 +0900 Subject: [PATCH] btrfs: add minimal file system size check Some btrfs tests call _scratch_pool_mkfs or _scratch_mkfs by themselves to specify file system size limit. It slips through the check in _scratch_mkfs_sized(). Let's add size check call for each of them. Signed-off-by: Naohiro Aota Reviewed-by: Eryu Guan Signed-off-by: Eryu Guan --- tests/btrfs/141 | 1 + tests/btrfs/142 | 1 + tests/btrfs/143 | 1 + tests/btrfs/150 | 1 + tests/btrfs/151 | 1 + tests/btrfs/156 | 1 + tests/btrfs/157 | 1 + tests/btrfs/158 | 1 + tests/btrfs/175 | 1 + 9 files changed, 9 insertions(+) diff --git a/tests/btrfs/141 b/tests/btrfs/141 index 6f9a177a..cf0979e9 100755 --- a/tests/btrfs/141 +++ b/tests/btrfs/141 @@ -55,6 +55,7 @@ _scratch_dev_pool_get 2 # step 1, create a raid1 btrfs which contains one 128k file. echo "step 1......mkfs.btrfs" >>$seqres.full +_check_minimal_fs_size $(( 1024 * 1024 * 1024 )) mkfs_opts="-d raid1 -b 1G" _scratch_pool_mkfs $mkfs_opts >>$seqres.full 2>&1 diff --git a/tests/btrfs/142 b/tests/btrfs/142 index 3caa37d1..1318be0f 100755 --- a/tests/btrfs/142 +++ b/tests/btrfs/142 @@ -33,6 +33,7 @@ _scratch_dev_pool_get 2 # step 1, create a raid1 btrfs which contains one 128k file. echo "step 1......mkfs.btrfs" >>$seqres.full +_check_minimal_fs_size $(( 1024 * 1024 * 1024 )) mkfs_opts="-d raid1 -b 1G" _scratch_pool_mkfs $mkfs_opts >>$seqres.full 2>&1 diff --git a/tests/btrfs/143 b/tests/btrfs/143 index f062f77b..6736dcad 100755 --- a/tests/btrfs/143 +++ b/tests/btrfs/143 @@ -40,6 +40,7 @@ _scratch_dev_pool_get 2 # step 1, create a raid1 btrfs which contains one 128k file. echo "step 1......mkfs.btrfs" >>$seqres.full +_check_minimal_fs_size $(( 1024 * 1024 * 1024 )) mkfs_opts="-d raid1 -b 1G" _scratch_pool_mkfs $mkfs_opts >>$seqres.full 2>&1 diff --git a/tests/btrfs/150 b/tests/btrfs/150 index 56938b0f..986c8069 100755 --- a/tests/btrfs/150 +++ b/tests/btrfs/150 @@ -43,6 +43,7 @@ disable_io_failure() echo 0 > $SYSFS_BDEV/make-it-fail } +_check_minimal_fs_size $(( 1024 * 1024 * 1024 )) _scratch_pool_mkfs "-d raid1 -b 1G" >> $seqres.full 2>&1 # It doesn't matter which compression algorithm we use. diff --git a/tests/btrfs/151 b/tests/btrfs/151 index 3150fef0..099e85cc 100755 --- a/tests/btrfs/151 +++ b/tests/btrfs/151 @@ -23,6 +23,7 @@ _supported_fs btrfs _require_scratch _require_scratch_dev_pool 3 _require_btrfs_dev_del_by_devid +_check_minimal_fs_size $(( 1024 * 1024 * 1024 )) # We need exactly 3 disks to form a fixed stripe layout for this test. _scratch_dev_pool_get 3 diff --git a/tests/btrfs/156 b/tests/btrfs/156 index 364135ab..11bf4638 100755 --- a/tests/btrfs/156 +++ b/tests/btrfs/156 @@ -37,6 +37,7 @@ nr_files=$(( $fs_size / $file_size / 2)) # Force to use single data and meta profile. # Since the test relies on fstrim output, which will differ for different # profiles +_check_minimal_fs_size $fs_size _scratch_mkfs -b $fs_size -m single -d single > /dev/null _scratch_mount diff --git a/tests/btrfs/157 b/tests/btrfs/157 index c08cebdc..0cfe3ce5 100755 --- a/tests/btrfs/157 +++ b/tests/btrfs/157 @@ -60,6 +60,7 @@ _scratch_dev_pool_get 4 # step 1: create a raid6 btrfs and create a 128K file echo "step 1......mkfs.btrfs" >>$seqres.full +_check_minimal_fs_size $(( 1024 * 1024 * 1024 )) mkfs_opts="-d raid6 -b 1G" _scratch_pool_mkfs $mkfs_opts >>$seqres.full 2>&1 diff --git a/tests/btrfs/158 b/tests/btrfs/158 index 4a642f52..ad374eba 100755 --- a/tests/btrfs/158 +++ b/tests/btrfs/158 @@ -52,6 +52,7 @@ _scratch_dev_pool_get 4 # step 1: create a raid6 btrfs and create a 128K file echo "step 1......mkfs.btrfs" >>$seqres.full +_check_minimal_fs_size $(( 1024 * 1024 * 1024 )) mkfs_opts="-d raid6 -b 1G" _scratch_pool_mkfs $mkfs_opts >>$seqres.full 2>&1 diff --git a/tests/btrfs/175 b/tests/btrfs/175 index 8a9c0721..bf0ede25 100755 --- a/tests/btrfs/175 +++ b/tests/btrfs/175 @@ -14,6 +14,7 @@ _begin_fstest auto quick swap volume _supported_fs btrfs _require_scratch_dev_pool 2 _require_scratch_swapfile +_check_minimal_fs_size $((1024 * 1024 * 1024)) cycle_swapfile() { local sz=${1:-$(($(get_page_size) * 10))} -- 2.39.5