[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2] tests: Don't run f2fs test on systems with PAGE_SIZE more
Hans Ulrich Niedermann
Re: [PATCH v2] tests: Don't run f2fs test on systems with PAGE_SIZE more than 4096 bytes
Wed, 29 Apr 2020 19:25:57 +0200
On Wed, 29 Apr 2020 17:06:36 +0300
Anatoly Pugachev <address@hidden> wrote:
> Don't run f2fs test on systems with PAGE_SIZE > 4096 bytes, since
> f2fs is not supported on these systems and trying to mount a f2fs
> filesystem would fail.
"Skip the f2fs test on ..." might be better wording, both in this
paragraph and the Subject.
Exit code 77 is certainly documented with the word "skip", and "exit
77" will show up in the "make check" output as "SKIP" as well.
> v2 changes:
> - fix compare
> - quotes around variable expansion
> Signed-off-by: Anatoly Pugachev <address@hidden>
> Reviewed-by: Mike Gilbert <address@hidden>
> tests/f2fs_test.in | 6 ++++++
> 1 file changed, 6 insertions(+)
> diff --git a/tests/f2fs_test.in b/tests/f2fs_test.in
> index 1ea77c826..3da1dad57 100644
> --- a/tests/f2fs_test.in
> +++ b/tests/f2fs_test.in
> @@ -15,5 +15,11 @@ if ! which mkfs.f2fs >/dev/null 2>&1; then
> exit 77
> +PAGE_SIZE=$(getconf PAGE_SIZE)
> +if [ "$PAGE_SIZE" -gt "$F2FS_BLKSIZE" ]; then
> + printf "f2fs is not supported on PAGE_SIZE(%d) != %d\n" $PAGE_SIZE
> + exit 77
This confusing to me. You are skipping the test when
PAGE_SIZE > F2FS_BLKSIZE
but the corresponding message says
PAGE_SIZE != F2FS_BLKSIZE
Now... which condition is it supposed to be? ">" or "!="?
I know from the Linux kernel's ext2 driver that it is very well
possible that PAGE_SIZE != EXT2_BLOCK_SIZE can work
unless EXT2_BLOCK_SIZE > PAGE_SIZE.
So ">" and "!=" are not necessarily the same thing, and IMHO the check
and the message use the same condition.