* [PATCHSET] fstests: updates for Linux 6.11 @ 2024-09-18 20:57 Darrick J. Wong 2024-09-18 20:57 ` [PATCH 1/1] generic: add a regression test for sub-block fsmap queries Darrick J. Wong 0 siblings, 1 reply; 5+ messages in thread From: Darrick J. Wong @ 2024-09-18 20:57 UTC (permalink / raw) To: djwong, zlang; +Cc: linux-xfs, fstests Hi all, This is pending fixes for things that are being fixed in 6.11. If you're going to start using this code, I strongly recommend pulling from my git trees, which are linked below. This has been running on the djcloud for months with no problems. Enjoy! Comments and questions are, as always, welcome. --D fstests git tree: https://git.kernel.org/cgit/linux/kernel/git/djwong/xfstests-dev.git/log/?h=xfs-fixes-6.11 --- Commits in this patchset: * generic: add a regression test for sub-block fsmap queries --- tests/generic/1954 | 79 ++++++++++++++++++++++++++++++++++++++++++++++++ tests/generic/1954.out | 15 +++++++++ 2 files changed, 94 insertions(+) create mode 100755 tests/generic/1954 create mode 100644 tests/generic/1954.out ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 1/1] generic: add a regression test for sub-block fsmap queries 2024-09-18 20:57 [PATCHSET] fstests: updates for Linux 6.11 Darrick J. Wong @ 2024-09-18 20:57 ` Darrick J. Wong 2024-09-19 20:07 ` Zorro Lang 0 siblings, 1 reply; 5+ messages in thread From: Darrick J. Wong @ 2024-09-18 20:57 UTC (permalink / raw) To: djwong, zlang; +Cc: linux-xfs, fstests From: Darrick J. Wong <djwong@kernel.org> Zizhi Wo found some bugs in the GETFSMAP implementation if it is fed sub-fsblock ranges. Add a regression test for this. Signed-off-by: Darrick J. Wong <djwong@kernel.org> --- tests/generic/1954 | 79 ++++++++++++++++++++++++++++++++++++++++++++++++ tests/generic/1954.out | 15 +++++++++ 2 files changed, 94 insertions(+) create mode 100755 tests/generic/1954 create mode 100644 tests/generic/1954.out diff --git a/tests/generic/1954 b/tests/generic/1954 new file mode 100755 index 0000000000..cfdfaf15e2 --- /dev/null +++ b/tests/generic/1954 @@ -0,0 +1,79 @@ +#! /bin/bash +# SPDX-License-Identifier: GPL-2.0 +# Copyright (c) 2024 Oracle. All Rights Reserved. +# +# FS QA Test No. 1954 +# +# Regression test for sub-fsblock key handling errors in GETFSMAP. +# +. ./common/preamble +_begin_fstest auto rmap fsmap + +_fixed_by_kernel_commit XXXXXXXXXXXX \ + "xfs: Fix the owner setting issue for rmap query in xfs fsmap" +_fixed_by_kernel_commit XXXXXXXXXXXX \ + "xfs: Fix missing interval for missing_owner in xfs fsmap" + +. ./common/filter + +_require_xfs_io_command "fsmap" +_require_scratch + +_scratch_mkfs >> $seqres.full +_scratch_mount + +blksz=$(_get_block_size "$SCRATCH_MNT") +if ((blksz < 2048)); then + _notrun "test requires at least 4 bblocks per fsblock" +fi + +$XFS_IO_PROG -c 'fsmap' $SCRATCH_MNT >> $seqres.full + +find_freesp() { + $XFS_IO_PROG -c 'fsmap -d' $SCRATCH_MNT | tr '.[]:' ' ' | \ + grep 'free space' | awk '{printf("%s:%s\n", $4, $5);}' | \ + head -n 1 +} + +filter_fsmap() { + _filter_xfs_io_numbers | sed \ + -e 's/inode XXXX data XXXX..XXXX/inode data/g' \ + -e 's/inode XXXX attr XXXX..XXXX/inode attr/g' \ + -e 's/: free space XXXX/: FREE XXXX/g' \ + -e 's/: [a-z].*XXXX/: USED XXXX/g' +} + +$XFS_IO_PROG -c 'fsmap -d' $SCRATCH_MNT | filter_fsmap >> $seqres.full + +freesp="$(find_freesp)" + +freesp_start="$(echo "$freesp" | cut -d ':' -f 1)" +freesp_end="$(echo "$freesp" | cut -d ':' -f 2)" +echo "$freesp:$freesp_start:$freesp_end" >> $seqres.full + +echo "test incorrect setting of high key" +$XFS_IO_PROG -c 'fsmap -d 0 3' $SCRATCH_MNT | filter_fsmap + +echo "test missing free space extent" +$XFS_IO_PROG -c "fsmap -d $((freesp_start + 1)) $((freesp_start + 2))" $SCRATCH_MNT | \ + filter_fsmap + +echo "test whatever came before freesp" +$XFS_IO_PROG -c "fsmap -d $((freesp_start - 3)) $((freesp_start - 2))" $SCRATCH_MNT | \ + filter_fsmap + +echo "test whatever came after freesp" +$XFS_IO_PROG -c "fsmap -d $((freesp_end + 2)) $((freesp_end + 3))" $SCRATCH_MNT | \ + filter_fsmap + +echo "test crossing start of freesp" +$XFS_IO_PROG -c "fsmap -d $((freesp_start - 2)) $((freesp_start + 1))" $SCRATCH_MNT | \ + filter_fsmap + +echo "test crossing end of freesp" +$XFS_IO_PROG -c "fsmap -d $((freesp_end - 1)) $((freesp_end + 2))" $SCRATCH_MNT | \ + filter_fsmap + +# success, all done +status=0 +exit diff --git a/tests/generic/1954.out b/tests/generic/1954.out new file mode 100644 index 0000000000..6baec43511 --- /dev/null +++ b/tests/generic/1954.out @@ -0,0 +1,15 @@ +QA output created by 1954 +test incorrect setting of high key + XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX +test missing free space extent + XXXX: XXXX:XXXX [XXXX..XXXX]: FREE XXXX +test whatever came before freesp + XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX +test whatever came after freesp + XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX +test crossing start of freesp + XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX + XXXX: XXXX:XXXX [XXXX..XXXX]: FREE XXXX +test crossing end of freesp + XXXX: XXXX:XXXX [XXXX..XXXX]: FREE XXXX + XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 1/1] generic: add a regression test for sub-block fsmap queries 2024-09-18 20:57 ` [PATCH 1/1] generic: add a regression test for sub-block fsmap queries Darrick J. Wong @ 2024-09-19 20:07 ` Zorro Lang 2024-09-19 20:13 ` Darrick J. Wong 0 siblings, 1 reply; 5+ messages in thread From: Zorro Lang @ 2024-09-19 20:07 UTC (permalink / raw) To: Darrick J. Wong; +Cc: linux-xfs, fstests On Wed, Sep 18, 2024 at 01:57:19PM -0700, Darrick J. Wong wrote: > From: Darrick J. Wong <djwong@kernel.org> > > Zizhi Wo found some bugs in the GETFSMAP implementation if it is fed > sub-fsblock ranges. Add a regression test for this. > > Signed-off-by: Darrick J. Wong <djwong@kernel.org> > --- > tests/generic/1954 | 79 ++++++++++++++++++++++++++++++++++++++++++++++++ > tests/generic/1954.out | 15 +++++++++ > 2 files changed, 94 insertions(+) > create mode 100755 tests/generic/1954 > create mode 100644 tests/generic/1954.out > > > diff --git a/tests/generic/1954 b/tests/generic/1954 > new file mode 100755 > index 0000000000..cfdfaf15e2 > --- /dev/null > +++ b/tests/generic/1954 > @@ -0,0 +1,79 @@ > +#! /bin/bash > +# SPDX-License-Identifier: GPL-2.0 > +# Copyright (c) 2024 Oracle. All Rights Reserved. > +# > +# FS QA Test No. 1954 > +# > +# Regression test for sub-fsblock key handling errors in GETFSMAP. > +# > +. ./common/preamble > +_begin_fstest auto rmap fsmap > + > +_fixed_by_kernel_commit XXXXXXXXXXXX \ > + "xfs: Fix the owner setting issue for rmap query in xfs fsmap" > +_fixed_by_kernel_commit XXXXXXXXXXXX \ > + "xfs: Fix missing interval for missing_owner in xfs fsmap" These 2 patches have been merged: 68415b349f3f xfs: Fix the owner setting issue for rmap query in xfs fsmap ca6448aed4f1 xfs: Fix missing interval for missing_owner in xfs fsmap I'll help to update the commit id when I merge it. > + > +. ./common/filter > + > +_require_xfs_io_command "fsmap" > +_require_scratch > + > +_scratch_mkfs >> $seqres.full > +_scratch_mount > + > +blksz=$(_get_block_size "$SCRATCH_MNT") > +if ((blksz < 2048)); then > + _notrun "test requires at least 4 bblocks per fsblock" What if the device is hard 4k sector size? > +fi > + > +$XFS_IO_PROG -c 'fsmap' $SCRATCH_MNT >> $seqres.full > + > +find_freesp() { > + $XFS_IO_PROG -c 'fsmap -d' $SCRATCH_MNT | tr '.[]:' ' ' | \ > + grep 'free space' | awk '{printf("%s:%s\n", $4, $5);}' | \ > + head -n 1 > +} > + > +filter_fsmap() { > + _filter_xfs_io_numbers | sed \ > + -e 's/inode XXXX data XXXX..XXXX/inode data/g' \ > + -e 's/inode XXXX attr XXXX..XXXX/inode attr/g' \ > + -e 's/: free space XXXX/: FREE XXXX/g' \ > + -e 's/: [a-z].*XXXX/: USED XXXX/g' As this's a generic test case, I tried it on btrfs and ext4. btrfs got _notrun "xfs_io fsmap support is missing", ext4 got failure as: # diff -u /root/git/xfstests/tests/generic/1954.out /root/git/xfstests/results//default/generic/1954.out.bad --- /root/git/xfstests/tests/generic/1954.out 2024-09-20 03:51:02.545504285 +0800 +++ /root/git/xfstests/results//default/generic/1954.out.bad 2024-09-20 03:58:51.505271227 +0800 @@ -1,15 +1,11 @@ QA output created by 1954 test incorrect setting of high key - XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX test missing free space extent XXXX: XXXX:XXXX [XXXX..XXXX]: FREE XXXX test whatever came before freesp - XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX test whatever came after freesp - XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX test crossing start of freesp XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX XXXX: XXXX:XXXX [XXXX..XXXX]: FREE XXXX test crossing end of freesp XXXX: XXXX:XXXX [XXXX..XXXX]: FREE XXXX - XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX Thanks, Zorro > +} > + > +$XFS_IO_PROG -c 'fsmap -d' $SCRATCH_MNT | filter_fsmap >> $seqres.full > + > +freesp="$(find_freesp)" > + > +freesp_start="$(echo "$freesp" | cut -d ':' -f 1)" > +freesp_end="$(echo "$freesp" | cut -d ':' -f 2)" > +echo "$freesp:$freesp_start:$freesp_end" >> $seqres.full > + > +echo "test incorrect setting of high key" > +$XFS_IO_PROG -c 'fsmap -d 0 3' $SCRATCH_MNT | filter_fsmap > + > +echo "test missing free space extent" > +$XFS_IO_PROG -c "fsmap -d $((freesp_start + 1)) $((freesp_start + 2))" $SCRATCH_MNT | \ > + filter_fsmap > + > +echo "test whatever came before freesp" > +$XFS_IO_PROG -c "fsmap -d $((freesp_start - 3)) $((freesp_start - 2))" $SCRATCH_MNT | \ > + filter_fsmap > + > +echo "test whatever came after freesp" > +$XFS_IO_PROG -c "fsmap -d $((freesp_end + 2)) $((freesp_end + 3))" $SCRATCH_MNT | \ > + filter_fsmap > + > +echo "test crossing start of freesp" > +$XFS_IO_PROG -c "fsmap -d $((freesp_start - 2)) $((freesp_start + 1))" $SCRATCH_MNT | \ > + filter_fsmap > + > +echo "test crossing end of freesp" > +$XFS_IO_PROG -c "fsmap -d $((freesp_end - 1)) $((freesp_end + 2))" $SCRATCH_MNT | \ > + filter_fsmap > + > +# success, all done > +status=0 > +exit > diff --git a/tests/generic/1954.out b/tests/generic/1954.out > new file mode 100644 > index 0000000000..6baec43511 > --- /dev/null > +++ b/tests/generic/1954.out > @@ -0,0 +1,15 @@ > +QA output created by 1954 > +test incorrect setting of high key > + XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > +test missing free space extent > + XXXX: XXXX:XXXX [XXXX..XXXX]: FREE XXXX > +test whatever came before freesp > + XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > +test whatever came after freesp > + XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > +test crossing start of freesp > + XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > + XXXX: XXXX:XXXX [XXXX..XXXX]: FREE XXXX > +test crossing end of freesp > + XXXX: XXXX:XXXX [XXXX..XXXX]: FREE XXXX > + XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/1] generic: add a regression test for sub-block fsmap queries 2024-09-19 20:07 ` Zorro Lang @ 2024-09-19 20:13 ` Darrick J. Wong 2024-09-20 14:33 ` Zorro Lang 0 siblings, 1 reply; 5+ messages in thread From: Darrick J. Wong @ 2024-09-19 20:13 UTC (permalink / raw) To: Zorro Lang; +Cc: linux-xfs, fstests On Fri, Sep 20, 2024 at 04:07:03AM +0800, Zorro Lang wrote: > On Wed, Sep 18, 2024 at 01:57:19PM -0700, Darrick J. Wong wrote: > > From: Darrick J. Wong <djwong@kernel.org> > > > > Zizhi Wo found some bugs in the GETFSMAP implementation if it is fed > > sub-fsblock ranges. Add a regression test for this. > > > > Signed-off-by: Darrick J. Wong <djwong@kernel.org> > > --- > > tests/generic/1954 | 79 ++++++++++++++++++++++++++++++++++++++++++++++++ > > tests/generic/1954.out | 15 +++++++++ > > 2 files changed, 94 insertions(+) > > create mode 100755 tests/generic/1954 > > create mode 100644 tests/generic/1954.out > > > > > > diff --git a/tests/generic/1954 b/tests/generic/1954 > > new file mode 100755 > > index 0000000000..cfdfaf15e2 > > --- /dev/null > > +++ b/tests/generic/1954 > > @@ -0,0 +1,79 @@ > > +#! /bin/bash > > +# SPDX-License-Identifier: GPL-2.0 > > +# Copyright (c) 2024 Oracle. All Rights Reserved. > > +# > > +# FS QA Test No. 1954 > > +# > > +# Regression test for sub-fsblock key handling errors in GETFSMAP. > > +# > > +. ./common/preamble > > +_begin_fstest auto rmap fsmap > > + > > +_fixed_by_kernel_commit XXXXXXXXXXXX \ > > + "xfs: Fix the owner setting issue for rmap query in xfs fsmap" > > +_fixed_by_kernel_commit XXXXXXXXXXXX \ > > + "xfs: Fix missing interval for missing_owner in xfs fsmap" > > These 2 patches have been merged: > > 68415b349f3f xfs: Fix the owner setting issue for rmap query in xfs fsmap > ca6448aed4f1 xfs: Fix missing interval for missing_owner in xfs fsmap > > I'll help to update the commit id when I merge it. Oops, will go fix that. > > + > > +. ./common/filter > > + > > +_require_xfs_io_command "fsmap" > > +_require_scratch > > + > > +_scratch_mkfs >> $seqres.full > > +_scratch_mount > > + > > +blksz=$(_get_block_size "$SCRATCH_MNT") > > +if ((blksz < 2048)); then > > + _notrun "test requires at least 4 bblocks per fsblock" > > What if the device is hard 4k sector size? Doesn't matter, because the bug is in converting GETFSMAP inputs that are a multiple of 512 but not a multiple of $fsblocksize. > > +fi > > + > > +$XFS_IO_PROG -c 'fsmap' $SCRATCH_MNT >> $seqres.full > > + > > +find_freesp() { > > + $XFS_IO_PROG -c 'fsmap -d' $SCRATCH_MNT | tr '.[]:' ' ' | \ > > + grep 'free space' | awk '{printf("%s:%s\n", $4, $5);}' | \ > > + head -n 1 > > +} > > + > > +filter_fsmap() { > > + _filter_xfs_io_numbers | sed \ > > + -e 's/inode XXXX data XXXX..XXXX/inode data/g' \ > > + -e 's/inode XXXX attr XXXX..XXXX/inode attr/g' \ > > + -e 's/: free space XXXX/: FREE XXXX/g' \ > > + -e 's/: [a-z].*XXXX/: USED XXXX/g' > > As this's a generic test case, I tried it on btrfs and ext4. btrfs got > _notrun "xfs_io fsmap support is missing", ext4 got failure as: > > # diff -u /root/git/xfstests/tests/generic/1954.out /root/git/xfstests/results//default/generic/1954.out.bad > --- /root/git/xfstests/tests/generic/1954.out 2024-09-20 03:51:02.545504285 +0800 > +++ /root/git/xfstests/results//default/generic/1954.out.bad 2024-09-20 03:58:51.505271227 +0800 > @@ -1,15 +1,11 @@ > QA output created by 1954 > test incorrect setting of high key > - XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > test missing free space extent > XXXX: XXXX:XXXX [XXXX..XXXX]: FREE XXXX > test whatever came before freesp > - XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > test whatever came after freesp > - XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > test crossing start of freesp > XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > XXXX: XXXX:XXXX [XXXX..XXXX]: FREE XXXX > test crossing end of freesp > XXXX: XXXX:XXXX [XXXX..XXXX]: FREE XXXX > - XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX Yep, we'll still have to patch ext4 for this. btrfs doesn't support GETFSMAP. --D > Thanks, > Zorro > > > +} > > + > > +$XFS_IO_PROG -c 'fsmap -d' $SCRATCH_MNT | filter_fsmap >> $seqres.full > > + > > +freesp="$(find_freesp)" > > + > > +freesp_start="$(echo "$freesp" | cut -d ':' -f 1)" > > +freesp_end="$(echo "$freesp" | cut -d ':' -f 2)" > > +echo "$freesp:$freesp_start:$freesp_end" >> $seqres.full > > + > > +echo "test incorrect setting of high key" > > +$XFS_IO_PROG -c 'fsmap -d 0 3' $SCRATCH_MNT | filter_fsmap > > + > > +echo "test missing free space extent" > > +$XFS_IO_PROG -c "fsmap -d $((freesp_start + 1)) $((freesp_start + 2))" $SCRATCH_MNT | \ > > + filter_fsmap > > + > > +echo "test whatever came before freesp" > > +$XFS_IO_PROG -c "fsmap -d $((freesp_start - 3)) $((freesp_start - 2))" $SCRATCH_MNT | \ > > + filter_fsmap > > + > > +echo "test whatever came after freesp" > > +$XFS_IO_PROG -c "fsmap -d $((freesp_end + 2)) $((freesp_end + 3))" $SCRATCH_MNT | \ > > + filter_fsmap > > + > > +echo "test crossing start of freesp" > > +$XFS_IO_PROG -c "fsmap -d $((freesp_start - 2)) $((freesp_start + 1))" $SCRATCH_MNT | \ > > + filter_fsmap > > + > > +echo "test crossing end of freesp" > > +$XFS_IO_PROG -c "fsmap -d $((freesp_end - 1)) $((freesp_end + 2))" $SCRATCH_MNT | \ > > + filter_fsmap > > + > > +# success, all done > > +status=0 > > +exit > > diff --git a/tests/generic/1954.out b/tests/generic/1954.out > > new file mode 100644 > > index 0000000000..6baec43511 > > --- /dev/null > > +++ b/tests/generic/1954.out > > @@ -0,0 +1,15 @@ > > +QA output created by 1954 > > +test incorrect setting of high key > > + XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > > +test missing free space extent > > + XXXX: XXXX:XXXX [XXXX..XXXX]: FREE XXXX > > +test whatever came before freesp > > + XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > > +test whatever came after freesp > > + XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > > +test crossing start of freesp > > + XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > > + XXXX: XXXX:XXXX [XXXX..XXXX]: FREE XXXX > > +test crossing end of freesp > > + XXXX: XXXX:XXXX [XXXX..XXXX]: FREE XXXX > > + XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > > > > > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/1] generic: add a regression test for sub-block fsmap queries 2024-09-19 20:13 ` Darrick J. Wong @ 2024-09-20 14:33 ` Zorro Lang 0 siblings, 0 replies; 5+ messages in thread From: Zorro Lang @ 2024-09-20 14:33 UTC (permalink / raw) To: Darrick J. Wong; +Cc: linux-xfs, fstests On Thu, Sep 19, 2024 at 01:13:10PM -0700, Darrick J. Wong wrote: > On Fri, Sep 20, 2024 at 04:07:03AM +0800, Zorro Lang wrote: > > On Wed, Sep 18, 2024 at 01:57:19PM -0700, Darrick J. Wong wrote: > > > From: Darrick J. Wong <djwong@kernel.org> > > > > > > Zizhi Wo found some bugs in the GETFSMAP implementation if it is fed > > > sub-fsblock ranges. Add a regression test for this. > > > > > > Signed-off-by: Darrick J. Wong <djwong@kernel.org> > > > --- > > > tests/generic/1954 | 79 ++++++++++++++++++++++++++++++++++++++++++++++++ > > > tests/generic/1954.out | 15 +++++++++ > > > 2 files changed, 94 insertions(+) > > > create mode 100755 tests/generic/1954 > > > create mode 100644 tests/generic/1954.out > > > > > > > > > diff --git a/tests/generic/1954 b/tests/generic/1954 > > > new file mode 100755 > > > index 0000000000..cfdfaf15e2 > > > --- /dev/null > > > +++ b/tests/generic/1954 > > > @@ -0,0 +1,79 @@ > > > +#! /bin/bash > > > +# SPDX-License-Identifier: GPL-2.0 > > > +# Copyright (c) 2024 Oracle. All Rights Reserved. > > > +# > > > +# FS QA Test No. 1954 > > > +# > > > +# Regression test for sub-fsblock key handling errors in GETFSMAP. > > > +# > > > +. ./common/preamble > > > +_begin_fstest auto rmap fsmap > > > + > > > +_fixed_by_kernel_commit XXXXXXXXXXXX \ > > > + "xfs: Fix the owner setting issue for rmap query in xfs fsmap" > > > +_fixed_by_kernel_commit XXXXXXXXXXXX \ > > > + "xfs: Fix missing interval for missing_owner in xfs fsmap" > > > > These 2 patches have been merged: > > > > 68415b349f3f xfs: Fix the owner setting issue for rmap query in xfs fsmap > > ca6448aed4f1 xfs: Fix missing interval for missing_owner in xfs fsmap > > > > I'll help to update the commit id when I merge it. > > Oops, will go fix that. > > > > + > > > +. ./common/filter > > > + > > > +_require_xfs_io_command "fsmap" > > > +_require_scratch > > > + > > > +_scratch_mkfs >> $seqres.full > > > +_scratch_mount > > > + > > > +blksz=$(_get_block_size "$SCRATCH_MNT") > > > +if ((blksz < 2048)); then > > > + _notrun "test requires at least 4 bblocks per fsblock" > > > > What if the device is hard 4k sector size? > > Doesn't matter, because the bug is in converting GETFSMAP inputs that > are a multiple of 512 but not a multiple of $fsblocksize. > > > > +fi > > > + > > > +$XFS_IO_PROG -c 'fsmap' $SCRATCH_MNT >> $seqres.full > > > + > > > +find_freesp() { > > > + $XFS_IO_PROG -c 'fsmap -d' $SCRATCH_MNT | tr '.[]:' ' ' | \ > > > + grep 'free space' | awk '{printf("%s:%s\n", $4, $5);}' | \ > > > + head -n 1 > > > +} > > > + > > > +filter_fsmap() { > > > + _filter_xfs_io_numbers | sed \ > > > + -e 's/inode XXXX data XXXX..XXXX/inode data/g' \ > > > + -e 's/inode XXXX attr XXXX..XXXX/inode attr/g' \ > > > + -e 's/: free space XXXX/: FREE XXXX/g' \ > > > + -e 's/: [a-z].*XXXX/: USED XXXX/g' > > > > As this's a generic test case, I tried it on btrfs and ext4. btrfs got > > _notrun "xfs_io fsmap support is missing", ext4 got failure as: > > > > # diff -u /root/git/xfstests/tests/generic/1954.out /root/git/xfstests/results//default/generic/1954.out.bad > > --- /root/git/xfstests/tests/generic/1954.out 2024-09-20 03:51:02.545504285 +0800 > > +++ /root/git/xfstests/results//default/generic/1954.out.bad 2024-09-20 03:58:51.505271227 +0800 > > @@ -1,15 +1,11 @@ > > QA output created by 1954 > > test incorrect setting of high key > > - XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > > test missing free space extent > > XXXX: XXXX:XXXX [XXXX..XXXX]: FREE XXXX > > test whatever came before freesp > > - XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > > test whatever came after freesp > > - XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > > test crossing start of freesp > > XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > > XXXX: XXXX:XXXX [XXXX..XXXX]: FREE XXXX > > test crossing end of freesp > > XXXX: XXXX:XXXX [XXXX..XXXX]: FREE XXXX > > - XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > > Yep, we'll still have to patch ext4 for this. btrfs doesn't support > GETFSMAP. Great, thanks for you clarify the ext4 part. I can merge this patch with that commit id change, if you don't want to change it more than that. Or feel free to add my RVB in V2. Reviewed-by: Zorro Lang <zlang@redhat.com> > > --D > > > Thanks, > > Zorro > > > > > +} > > > + > > > +$XFS_IO_PROG -c 'fsmap -d' $SCRATCH_MNT | filter_fsmap >> $seqres.full > > > + > > > +freesp="$(find_freesp)" > > > + > > > +freesp_start="$(echo "$freesp" | cut -d ':' -f 1)" > > > +freesp_end="$(echo "$freesp" | cut -d ':' -f 2)" > > > +echo "$freesp:$freesp_start:$freesp_end" >> $seqres.full > > > + > > > +echo "test incorrect setting of high key" > > > +$XFS_IO_PROG -c 'fsmap -d 0 3' $SCRATCH_MNT | filter_fsmap > > > + > > > +echo "test missing free space extent" > > > +$XFS_IO_PROG -c "fsmap -d $((freesp_start + 1)) $((freesp_start + 2))" $SCRATCH_MNT | \ > > > + filter_fsmap > > > + > > > +echo "test whatever came before freesp" > > > +$XFS_IO_PROG -c "fsmap -d $((freesp_start - 3)) $((freesp_start - 2))" $SCRATCH_MNT | \ > > > + filter_fsmap > > > + > > > +echo "test whatever came after freesp" > > > +$XFS_IO_PROG -c "fsmap -d $((freesp_end + 2)) $((freesp_end + 3))" $SCRATCH_MNT | \ > > > + filter_fsmap > > > + > > > +echo "test crossing start of freesp" > > > +$XFS_IO_PROG -c "fsmap -d $((freesp_start - 2)) $((freesp_start + 1))" $SCRATCH_MNT | \ > > > + filter_fsmap > > > + > > > +echo "test crossing end of freesp" > > > +$XFS_IO_PROG -c "fsmap -d $((freesp_end - 1)) $((freesp_end + 2))" $SCRATCH_MNT | \ > > > + filter_fsmap > > > + > > > +# success, all done > > > +status=0 > > > +exit > > > diff --git a/tests/generic/1954.out b/tests/generic/1954.out > > > new file mode 100644 > > > index 0000000000..6baec43511 > > > --- /dev/null > > > +++ b/tests/generic/1954.out > > > @@ -0,0 +1,15 @@ > > > +QA output created by 1954 > > > +test incorrect setting of high key > > > + XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > > > +test missing free space extent > > > + XXXX: XXXX:XXXX [XXXX..XXXX]: FREE XXXX > > > +test whatever came before freesp > > > + XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > > > +test whatever came after freesp > > > + XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > > > +test crossing start of freesp > > > + XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > > > + XXXX: XXXX:XXXX [XXXX..XXXX]: FREE XXXX > > > +test crossing end of freesp > > > + XXXX: XXXX:XXXX [XXXX..XXXX]: FREE XXXX > > > + XXXX: XXXX:XXXX [XXXX..XXXX]: USED XXXX > > > > > > > > > ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2024-09-20 14:33 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2024-09-18 20:57 [PATCHSET] fstests: updates for Linux 6.11 Darrick J. Wong 2024-09-18 20:57 ` [PATCH 1/1] generic: add a regression test for sub-block fsmap queries Darrick J. Wong 2024-09-19 20:07 ` Zorro Lang 2024-09-19 20:13 ` Darrick J. Wong 2024-09-20 14:33 ` Zorro Lang
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox