| Message ID | 20190623230956.24670-4-tytso@mit.edu (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [1/5] shared,ext4: move ext4-specific tests out of shared/ | expand |
On Sun, Jun 23, 2019 at 07:09:55PM -0400, Theodore Ts'o wrote: > The shared/006 uses _scratch_mkfs_sized to create a limited size file > system, and then creates inodes until it gets ENOSPC, and then checks > to make sure the file system is consistent. It then remounts the file > system, removes all of the files, and makes sure the file system is > consistent afterwards. > > This test was marked as only being supported on ext[234] and xfs, and > so it was in shared. However, I've tested and this test works just > fine on btrfs, ubifs, tmpfs, and should work on all file systems that This is because the $free_inode is 0 and $loop is 1, so we end up creating only 1000 files in the test without hitting ENOSPC at all. Perhaps we could introduce a new require rule to check if fs supports fixed inode number (just check if '_get_free_inode $TEST_DIR' returns non-zero). > support _scratch_mkfs_sized, since even if there isn't a fixed inode > table, the file system will eventually run out of disk space. > > Signed-off-by: Theodore Ts'o <tytso@mit.edu> > --- > tests/{shared/006 => generic/558} | 4 +--- > tests/generic/558.out | 2 ++ > tests/generic/group | 1 + > tests/shared/006.out | 2 -- > tests/shared/group | 1 - > 5 files changed, 4 insertions(+), 6 deletions(-) > rename tests/{shared/006 => generic/558} (96%) > create mode 100644 tests/generic/558.out > delete mode 100644 tests/shared/006.out > > diff --git a/tests/shared/006 b/tests/generic/558 > similarity index 96% > rename from tests/shared/006 > rename to tests/generic/558 > index aa65e9a2..8bfc65fa 100755 > --- a/tests/shared/006 > +++ b/tests/generic/558 > @@ -2,7 +2,7 @@ > # SPDX-License-Identifier: GPL-2.0 > # Copyright (c) 2014 Red Hat Inc. All Rights Reserved. > # > -# FS QA Test No. shared/006 > +# FS QA Test No. generic/558 > # > # Stress test fs by using up all inodes and check fs. > # > @@ -42,9 +42,7 @@ create_file() > . ./common/filter > > # real QA test starts here > -_supported_fs ext4 ext3 ext2 xfs _supported_fs generic Thanks, Eryu > _supported_os Linux > - > _require_scratch > > rm -f $seqres.full > diff --git a/tests/generic/558.out b/tests/generic/558.out > new file mode 100644 > index 00000000..9a6c4e79 > --- /dev/null > +++ b/tests/generic/558.out > @@ -0,0 +1,2 @@ > +QA output created by 558 > +Silence is golden > diff --git a/tests/generic/group b/tests/generic/group > index 543c0627..8fc85b63 100644 > --- a/tests/generic/group > +++ b/tests/generic/group > @@ -560,3 +560,4 @@ > 555 auto quick cap > 556 auto quick casefold > 557 auto quick log > +558 auto enospc > diff --git a/tests/shared/006.out b/tests/shared/006.out > deleted file mode 100644 > index 675c1b7c..00000000 > --- a/tests/shared/006.out > +++ /dev/null > @@ -1,2 +0,0 @@ > -QA output created by 006 > -Silence is golden > diff --git a/tests/shared/group b/tests/shared/group > index 3507d7d4..2cf910bd 100644 > --- a/tests/shared/group > +++ b/tests/shared/group > @@ -4,7 +4,6 @@ > # - comment line before each group is "new" description > # > 002 auto metadata quick log > -006 auto enospc > 008 auto stress dedupe > 009 auto stress dedupe > 010 auto stress dedupe > -- > 2.22.0 >
diff --git a/tests/shared/006 b/tests/generic/558 similarity index 96% rename from tests/shared/006 rename to tests/generic/558 index aa65e9a2..8bfc65fa 100755 --- a/tests/shared/006 +++ b/tests/generic/558 @@ -2,7 +2,7 @@ # SPDX-License-Identifier: GPL-2.0 # Copyright (c) 2014 Red Hat Inc. All Rights Reserved. # -# FS QA Test No. shared/006 +# FS QA Test No. generic/558 # # Stress test fs by using up all inodes and check fs. # @@ -42,9 +42,7 @@ create_file() . ./common/filter # real QA test starts here -_supported_fs ext4 ext3 ext2 xfs _supported_os Linux - _require_scratch rm -f $seqres.full diff --git a/tests/generic/558.out b/tests/generic/558.out new file mode 100644 index 00000000..9a6c4e79 --- /dev/null +++ b/tests/generic/558.out @@ -0,0 +1,2 @@ +QA output created by 558 +Silence is golden diff --git a/tests/generic/group b/tests/generic/group index 543c0627..8fc85b63 100644 --- a/tests/generic/group +++ b/tests/generic/group @@ -560,3 +560,4 @@ 555 auto quick cap 556 auto quick casefold 557 auto quick log +558 auto enospc diff --git a/tests/shared/006.out b/tests/shared/006.out deleted file mode 100644 index 675c1b7c..00000000 --- a/tests/shared/006.out +++ /dev/null @@ -1,2 +0,0 @@ -QA output created by 006 -Silence is golden diff --git a/tests/shared/group b/tests/shared/group index 3507d7d4..2cf910bd 100644 --- a/tests/shared/group +++ b/tests/shared/group @@ -4,7 +4,6 @@ # - comment line before each group is "new" description # 002 auto metadata quick log -006 auto enospc 008 auto stress dedupe 009 auto stress dedupe 010 auto stress dedupe
The shared/006 uses _scratch_mkfs_sized to create a limited size file system, and then creates inodes until it gets ENOSPC, and then checks to make sure the file system is consistent. It then remounts the file system, removes all of the files, and makes sure the file system is consistent afterwards. This test was marked as only being supported on ext[234] and xfs, and so it was in shared. However, I've tested and this test works just fine on btrfs, ubifs, tmpfs, and should work on all file systems that support _scratch_mkfs_sized, since even if there isn't a fixed inode table, the file system will eventually run out of disk space. Signed-off-by: Theodore Ts'o <tytso@mit.edu> --- tests/{shared/006 => generic/558} | 4 +--- tests/generic/558.out | 2 ++ tests/generic/group | 1 + tests/shared/006.out | 2 -- tests/shared/group | 1 - 5 files changed, 4 insertions(+), 6 deletions(-) rename tests/{shared/006 => generic/558} (96%) create mode 100644 tests/generic/558.out delete mode 100644 tests/shared/006.out