Since this test examines dedupe behavior, the documentation should say 'dedupe', not 'reflink'. Furthermore, the feature checks must look for working dedupe functionality, not reflink functionality. Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx> [hch@xxxxxx: add the test for dedupe support] Signed-off-by: Christoph Hellwig <hch@xxxxxx> --- tests/generic/182 | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/tests/generic/182 b/tests/generic/182 index bf5cd38..ef10af8 100755 --- a/tests/generic/182 +++ b/tests/generic/182 @@ -1,10 +1,10 @@ #! /bin/bash # FS QA Test No. 182 # -# Test the convention that reflink with length == 0 means "to the end of fileA" +# Test the convention that dedupe with length == 0 means "to the end of fileA" # - Create a file. -# - Try to reflink "zero" bytes (which means reflink to EOF). -# - Check that the reflink happened. +# - Try to dedupe "zero" bytes (which means dedupe to EOF). +# - Check that the dedupe happened. # #----------------------------------------------------------------------- # Copyright (c) 2015, Oracle and/or its affiliates. All Rights Reserved. @@ -45,8 +45,7 @@ _cleanup() # real QA test starts here _supported_os Linux -_require_test_reflink -_require_cp_reflink +_require_test_dedupe rm -f "$seqres.full" _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs