apfs/apfstests
0
0
Fork 0
mirror of https://github.com/linux-apfs/apfstests.git synced 2026-05-01 15:01:44 -07:00
Code Issues Packages Projects Releases Wiki Activity

common/rc: Fix fcollapse require code to work with 64k block size

Browse Source 
For 64k block size, the require code fails because the block range
[4k, 12k] would cause the fcollapse syscall to return -EINVAL. Hence
the tests using them are not executed.

This commit fixes the issue by calculating file offset ranges based
on the block size of the underlying filesystem.

Signed-off-by: Chandan Rajendra <chandan@linux.vnet.ibm.com>
Reviewed-by: Eryu Guan <eguan@redhat.com>
Signed-off-by: Eryu Guan <eguan@redhat.com>
This commit is contained in:
Chandan Rajendra
2018-02-22 11:28:55 +05:30
committed by Eryu Guan
parent b8b56f24d9
commit 6702176b93
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
common/rc
+4 -2
View File
@@ -2113,8 +2113,10 @@ _require_xfs_io_command()
param_checked=1 param_checked=1
;; ;;
"fpunch" | "fcollapse" | "zero" | "fzero" | "finsert" | "funshare") "fpunch" | "fcollapse" | "zero" | "fzero" | "finsert" | "funshare")
testio=`$XFS_IO_PROG -F -f -c "pwrite 0 20k" -c "fsync" \ local blocksize=$(_get_block_size $TEST_DIR)
-c "$command 4k 8k" $testfile 2>&1` testio=`$XFS_IO_PROG -F -f -c "pwrite 0 $((5 * $blocksize))" \
-c "fsync" -c "$command $blocksize $((2 * $blocksize))" \
$testfile 2>&1`
;; ;;
"fiemap") "fiemap")
# If 'ranged' is passed as argument then we check to see if fiemap supports # If 'ranged' is passed as argument then we check to see if fiemap supports