On 2018/11/27 5:50, Dave Chinner wrote:
On Fri, Nov 23, 2018 at 01:24:23PM +0800, Xiao Yang wrote:
In generic/519, filefrag command use FIBMAP ioctl(-B option) to print
output in extent format(-e option) on purpose and sync file(-s option),
so add _require_filefrag_options() to check if the command supports
all of these options.
References:
1) filefrag supports -e option by commit 2508eaa since e2fsprogs v1.42.7.
2) filefrag supports -B option by commit 5d5e01d since e2fsprogs v1.41.9.
3) filefrag supports -s option by commit e62847c since e2fsprogs v1.41.6.
Signed-off-by: Xiao Yang<yangx.jy@xxxxxxxxxxxxxx>
---
common/rc | 15 ++++++++++++++-
tests/generic/519 | 2 +-
2 files changed, 15 insertions(+), 2 deletions(-)
diff --git a/common/rc b/common/rc
index ecb1738..e5da648 100644
--- a/common/rc
+++ b/common/rc
@@ -3843,10 +3843,23 @@ _require_scratch_btime()
_scratch_unmount
}
-_require_fibmap()
+_require_filefrag_options()
{
_require_command "$FILEFRAG_PROG" filefrag
+ local options=$1
+ local file="$TEST_DIR/options_testfile"
+
+ echo "XX"> $file
+ ${FILEFRAG_PROG} -$options $file 2>&1 | grep -q "invalid option"&& \
+ _notrun "filefrag doesn't support $options option"
+ rm -f $file
+}
Can we just get rid of filefrag and use xfs_io's fiemap commands
instead?
Hi Dave,
According to the comment in test, we have to reproduce the bug by
specifying FIBMAP ioctl instead of FIEMAP ioctl.
It seems that we can just use FIEMAP ioctl by xfs_io's fiemap command,
so we use FIBMAP ioctl forcely by filefrag
with -B option.
Best Regards,
Xiao Yang
Cheers,
Dave.
