| Message ID | 1456301196-15874-3-git-send-email-quwenruo@cn.fujitsu.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Wed, Feb 24, 2016 at 04:06:33PM +0800, Qu Wenruo wrote: > Add basic test for btrfs in-band de-duplication, including: > 1) Enable > 2) Re-enable > 3) On disk extents are refering to same bytenr > 4) Disable > > Signed-off-by: Qu Wenruo <quwenruo@cn.fujitsu.com> > --- > common/defrag | 8 ++++ > tests/btrfs/200 | 125 ++++++++++++++++++++++++++++++++++++++++++++++++++++ > tests/btrfs/200.out | 19 ++++++++ > tests/btrfs/group | 1 + > 4 files changed, 153 insertions(+) > create mode 100755 tests/btrfs/200 > create mode 100644 tests/btrfs/200.out > > diff --git a/common/defrag b/common/defrag > index 942593e..34cc822 100644 > --- a/common/defrag > +++ b/common/defrag > @@ -47,6 +47,14 @@ _extent_count() > $XFS_IO_PROG -c "fiemap" $1 | tail -n +2 | grep -v hole | wc -l| $AWK_PROG '{print $1}' > } > > +_uniq_extent_count() > +{ > + file=$1 > + $XFS_IO_PROG -c "fiemap" $file >> $seqres.full 2>&1 > + $XFS_IO_PROG -c "fiemap" $file | tail -n +2 | grep -v hole |\ > + $AWK_PROG '{print $3}' | sort | uniq | wc -l > +} This needs comments ot explain how it is different to _extent_count. Also should probably be named _extent_count_unique() > + > min=$1 > diff --git a/tests/btrfs/200 b/tests/btrfs/200 > new file mode 100755 > index 0000000..f2ff542 > --- /dev/null > +++ b/tests/btrfs/200 > @@ -0,0 +1,125 @@ > +#! /bin/bash > +# FS QA Test 200 > +# > +# Basic btrfs inband dedup test, including: > +# 1) Enable > +# 2) Uniq file extent number Unique. > +# 3) Re-enable > +# 4) Disable I don't understand what 2-4 are describing. As a test summary, "Basic btrfs inband dedup test" is sufficient. > +_supported_fs btrfs > +_supported_os Linux > +_require_scratch > +_require_btrfs_subcommand dedup > +_require_btrfs_fs_feature dedup > +_require_btrfs_mkfs_feature dedup > + > +# File size is twice the maximum file extent of btrfs > +# So even fallbacked to non-dedup, it will have at least 2 extents > +file_size=$(( 256 * 1024 * 1024 )) Used for xfs_io, so "file_size=256m" is all that is needed here. > +_scratch_mkfs "-O dedup" >> $seqres.full 2>&1 > +_scratch_mount > + > +do_dedup_test() > +{ > + backend=$1 > + dedup_bs=$2 > + > + _run_btrfs_util_prog dedup enable -s $backend -b $dedup_bs $SCRATCH_MNT > + $XFS_IO_PROG -f -c "pwrite -b $dedup_bs 0 $dedup_bs" \ > + $SCRATCH_MNT/initial_block | _filter_xfs_io > + > + # sync to ensure dedup hash is added into dedup pool > + sync xfs_io -fs or xfs_io ... -c "fsync" ... ? > + $XFS_IO_PROG -f -c "pwrite -b $dedup_bs 0 $file_size" \ > + $SCRATCH_MNT/real_file | _filter_xfs_io > + # sync again to ensure data are all written to disk and > + # we can get stable extent map > + sync Again, why now just do a sync write or fsync from the xfs? > + > + # Test if real_file is de-duplicated > + nr_uniq_extents=$(_uniq_extent_count $SCRATCH_MNT/real_file) > + nr_total_extents=$(_extent_count $SCRATCH_MNT/real_file) > + > + echo "uniq/total: $nr_uniq_extents/$nr_total_extents" >> $seqres.full > + # Allow a small amount of dedup miss, as commit interval or > + # memory pressure may break a dedup_bs block and cause > + # smalll extent which won't go through dedup routine > + if [ $nr_uniq_extents -ge $(( $nr_total_extents * 5 / 100 )) ]; then > + echo "Too high dedup failure rate" > + fi _within_tolerance > + > + # Also check the md5sum to ensure data is not corrupted > + md5=$(_md5_checksum $SCRATCH_MNT/real_file) > + if [ $md5 != $init_md5 ]; then > + echo "File after in-band de-duplication is corrupted" > + fi Nope. Just echo the md5sum to the golden output file. > +} > + > +# Create the initial file and calculate its checksum without dedup > +$XFS_IO_PROG -f -c "pwrite 0 $file_size" $SCRATCH_MNT/csum_file | \ > + _filter_xfs_io > +init_md5=$(_md5_checksum $SCRATCH_MNT/csum_file) > +echo "md5 of the initial file is $init_md5" >> $seqres.full Just echo the md5sum to the golden output file. Cheers, Dave.
Hi Dave, Thanks for the review. All comment are correct and I'll update the patchset soon. Only one small question below Dave Chinner wrote on 2016/02/29 09:26 +1100: ... >> +# File size is twice the maximum file extent of btrfs >> +# So even fallbacked to non-dedup, it will have at least 2 extents >> +file_size=$(( 256 * 1024 * 1024 )) > > Used for xfs_io, so "file_size=256m" is all that is needed here. Super nice feature for support unit suffix, I checked man page of xfs_io but only value for extsize mentioned the support for such suffix. I assume all offset/length/bsize/value support suffix, right? Hope man page get updated. Thanks, Qu > >> +_scratch_mkfs "-O dedup" >> $seqres.full 2>&1 >> +_scratch_mount >> + >> +do_dedup_test() >> +{ >> + backend=$1 >> + dedup_bs=$2 >> + >> + _run_btrfs_util_prog dedup enable -s $backend -b $dedup_bs $SCRATCH_MNT >> + $XFS_IO_PROG -f -c "pwrite -b $dedup_bs 0 $dedup_bs" \ >> + $SCRATCH_MNT/initial_block | _filter_xfs_io >> + >> + # sync to ensure dedup hash is added into dedup pool >> + sync > > xfs_io -fs or xfs_io ... -c "fsync" ... ? > >> + $XFS_IO_PROG -f -c "pwrite -b $dedup_bs 0 $file_size" \ >> + $SCRATCH_MNT/real_file | _filter_xfs_io >> + # sync again to ensure data are all written to disk and >> + # we can get stable extent map >> + sync > > Again, why now just do a sync write or fsync from the xfs? > >> + >> + # Test if real_file is de-duplicated >> + nr_uniq_extents=$(_uniq_extent_count $SCRATCH_MNT/real_file) >> + nr_total_extents=$(_extent_count $SCRATCH_MNT/real_file) >> + >> + echo "uniq/total: $nr_uniq_extents/$nr_total_extents" >> $seqres.full >> + # Allow a small amount of dedup miss, as commit interval or >> + # memory pressure may break a dedup_bs block and cause >> + # smalll extent which won't go through dedup routine >> + if [ $nr_uniq_extents -ge $(( $nr_total_extents * 5 / 100 )) ]; then >> + echo "Too high dedup failure rate" >> + fi > > _within_tolerance > >> + >> + # Also check the md5sum to ensure data is not corrupted >> + md5=$(_md5_checksum $SCRATCH_MNT/real_file) >> + if [ $md5 != $init_md5 ]; then >> + echo "File after in-band de-duplication is corrupted" >> + fi > > Nope. Just echo the md5sum to the golden output file. > > >> +} >> + >> +# Create the initial file and calculate its checksum without dedup >> +$XFS_IO_PROG -f -c "pwrite 0 $file_size" $SCRATCH_MNT/csum_file | \ >> + _filter_xfs_io >> +init_md5=$(_md5_checksum $SCRATCH_MNT/csum_file) >> +echo "md5 of the initial file is $init_md5" >> $seqres.full > > Just echo the md5sum to the golden output file. > > Cheers, > > Dave. > -- To unsubscribe from this list: send the line "unsubscribe fstests" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Mon, Feb 29, 2016 at 10:04:35AM +0800, Qu Wenruo wrote: > Hi Dave, > > Thanks for the review. > > All comment are correct and I'll update the patchset soon. > > Only one small question below > > Dave Chinner wrote on 2016/02/29 09:26 +1100: > ... > >>+# File size is twice the maximum file extent of btrfs > >>+# So even fallbacked to non-dedup, it will have at least 2 extents > >>+file_size=$(( 256 * 1024 * 1024 )) > > > >Used for xfs_io, so "file_size=256m" is all that is needed here. > > Super nice feature for support unit suffix, I checked man page of > xfs_io but only value for extsize mentioned the support for such > suffix. > > I assume all offset/length/bsize/value support suffix, right? Yes, they do, always have, originally came from other XFS commands (i.e see the mkfs.xfs for the "usual units suffixes" description). > Hope man page get updated. Can you send a patch? Cheers, Dave.
Dave Chinner wrote on 2016/02/29 17:43 +1100: > On Mon, Feb 29, 2016 at 10:04:35AM +0800, Qu Wenruo wrote: >> Hi Dave, >> >> Thanks for the review. >> >> All comment are correct and I'll update the patchset soon. >> >> Only one small question below >> >> Dave Chinner wrote on 2016/02/29 09:26 +1100: >> ... >>>> +# File size is twice the maximum file extent of btrfs >>>> +# So even fallbacked to non-dedup, it will have at least 2 extents >>>> +file_size=$(( 256 * 1024 * 1024 )) >>> >>> Used for xfs_io, so "file_size=256m" is all that is needed here. >> >> Super nice feature for support unit suffix, I checked man page of >> xfs_io but only value for extsize mentioned the support for such >> suffix. >> >> I assume all offset/length/bsize/value support suffix, right? > > Yes, they do, always have, originally came from other XFS commands > (i.e see the mkfs.xfs for the "usual units suffixes" description). > >> Hope man page get updated. > > Can you send a patch? My pleasure. Will send it soon. Thanks, Qu > > Cheers, > > Dave. > -- To unsubscribe from this list: send the line "unsubscribe fstests" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/common/defrag b/common/defrag index 942593e..34cc822 100644 --- a/common/defrag +++ b/common/defrag @@ -47,6 +47,14 @@ _extent_count() $XFS_IO_PROG -c "fiemap" $1 | tail -n +2 | grep -v hole | wc -l| $AWK_PROG '{print $1}' } +_uniq_extent_count() +{ + file=$1 + $XFS_IO_PROG -c "fiemap" $file >> $seqres.full 2>&1 + $XFS_IO_PROG -c "fiemap" $file | tail -n +2 | grep -v hole |\ + $AWK_PROG '{print $3}' | sort | uniq | wc -l +} + _check_extent_count() { min=$1 diff --git a/tests/btrfs/200 b/tests/btrfs/200 new file mode 100755 index 0000000..f2ff542 --- /dev/null +++ b/tests/btrfs/200 @@ -0,0 +1,125 @@ +#! /bin/bash +# FS QA Test 200 +# +# Basic btrfs inband dedup test, including: +# 1) Enable +# 2) Uniq file extent number +# 3) Re-enable +# 4) Disable +# +#----------------------------------------------------------------------- +# Copyright (c) 2016 Fujitsu. All Rights Reserved. +# +# This program is free software; you can redistribute it and/or +# modify it under the terms of the GNU General Public License as +# published by the Free Software Foundation. +# +# This program is distributed in the hope that it would be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program; if not, write the Free Software Foundation, +# Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA +#----------------------------------------------------------------------- +# + +seq=`basename $0` +seqres=$RESULT_DIR/$seq +echo "QA output created by $seq" + +here=`pwd` +tmp=/tmp/$$ +status=1 # failure is the default! +trap "_cleanup; exit \$status" 0 1 2 3 15 + +_cleanup() +{ + cd / + rm -f $tmp.* +} + +# get standard environment, filters and checks +. ./common/rc +. ./common/filter +. ./common/defrag + +# remove previous $seqres.full before test +rm -f $seqres.full + +# real QA test starts here + +_supported_fs btrfs +_supported_os Linux +_require_scratch +_require_btrfs_subcommand dedup +_require_btrfs_fs_feature dedup +_require_btrfs_mkfs_feature dedup + +# File size is twice the maximum file extent of btrfs +# So even fallbacked to non-dedup, it will have at least 2 extents +file_size=$(( 256 * 1024 * 1024 )) + +_scratch_mkfs "-O dedup" >> $seqres.full 2>&1 +_scratch_mount + +do_dedup_test() +{ + backend=$1 + dedup_bs=$2 + + _run_btrfs_util_prog dedup enable -s $backend -b $dedup_bs $SCRATCH_MNT + $XFS_IO_PROG -f -c "pwrite -b $dedup_bs 0 $dedup_bs" \ + $SCRATCH_MNT/initial_block | _filter_xfs_io + + # sync to ensure dedup hash is added into dedup pool + sync + $XFS_IO_PROG -f -c "pwrite -b $dedup_bs 0 $file_size" \ + $SCRATCH_MNT/real_file | _filter_xfs_io + # sync again to ensure data are all written to disk and + # we can get stable extent map + sync + + # Test if real_file is de-duplicated + nr_uniq_extents=$(_uniq_extent_count $SCRATCH_MNT/real_file) + nr_total_extents=$(_extent_count $SCRATCH_MNT/real_file) + + echo "uniq/total: $nr_uniq_extents/$nr_total_extents" >> $seqres.full + # Allow a small amount of dedup miss, as commit interval or + # memory pressure may break a dedup_bs block and cause + # smalll extent which won't go through dedup routine + if [ $nr_uniq_extents -ge $(( $nr_total_extents * 5 / 100 )) ]; then + echo "Too high dedup failure rate" + fi + + # Also check the md5sum to ensure data is not corrupted + md5=$(_md5_checksum $SCRATCH_MNT/real_file) + if [ $md5 != $init_md5 ]; then + echo "File after in-band de-duplication is corrupted" + fi +} + +# Create the initial file and calculate its checksum without dedup +$XFS_IO_PROG -f -c "pwrite 0 $file_size" $SCRATCH_MNT/csum_file | \ + _filter_xfs_io +init_md5=$(_md5_checksum $SCRATCH_MNT/csum_file) +echo "md5 of the initial file is $init_md5" >> $seqres.full + +# Test inmemory dedup first, use 64K dedup bs to keep compatibility +# with 64K page size +do_dedup_test inmemory 64K + +# Test ondisk backend, and re-enable function +do_dedup_test ondisk 64K + +# Test 128K(default) dedup bs +do_dedup_test inmemory 128K +do_dedup_test ondisk 128K + +# Check dedup disable +_run_btrfs_util_prog dedup disable $SCRATCH_MNT + +# success, all done +status=0 +exit diff --git a/tests/btrfs/200.out b/tests/btrfs/200.out new file mode 100644 index 0000000..4b185be --- /dev/null +++ b/tests/btrfs/200.out @@ -0,0 +1,19 @@ +QA output created by 200 +wrote 268435456/268435456 bytes at offset 0 +XXX Bytes, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec) +wrote 65536/65536 bytes at offset 0 +XXX Bytes, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec) +wrote 268435456/268435456 bytes at offset 0 +XXX Bytes, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec) +wrote 65536/65536 bytes at offset 0 +XXX Bytes, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec) +wrote 268435456/268435456 bytes at offset 0 +XXX Bytes, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec) +wrote 131072/131072 bytes at offset 0 +XXX Bytes, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec) +wrote 268435456/268435456 bytes at offset 0 +XXX Bytes, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec) +wrote 131072/131072 bytes at offset 0 +XXX Bytes, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec) +wrote 268435456/268435456 bytes at offset 0 +XXX Bytes, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec) diff --git a/tests/btrfs/group b/tests/btrfs/group index a2fa412..0b7354b 100644 --- a/tests/btrfs/group +++ b/tests/btrfs/group @@ -119,3 +119,4 @@ 116 auto quick metadata 117 auto quick send clone 118 auto quick snapshot metadata +200 auto dedup
Add basic test for btrfs in-band de-duplication, including: 1) Enable 2) Re-enable 3) On disk extents are refering to same bytenr 4) Disable Signed-off-by: Qu Wenruo <quwenruo@cn.fujitsu.com> --- common/defrag | 8 ++++ tests/btrfs/200 | 125 ++++++++++++++++++++++++++++++++++++++++++++++++++++ tests/btrfs/200.out | 19 ++++++++ tests/btrfs/group | 1 + 4 files changed, 153 insertions(+) create mode 100755 tests/btrfs/200 create mode 100644 tests/btrfs/200.out