fstests: fix _test_generic_punch() to fit 64k extent
authorAnand Jain <anand.jain@oracle.com>
Wed, 12 Sep 2018 07:48:44 +0000 (15:48 +0800)
committerEryu Guan <guaneryu@gmail.com>
Sun, 23 Sep 2018 14:26:56 +0000 (22:26 +0800)
commit3c48a2ca207846db8275a879c14cda53d2f0d6e1
treea246265727b48fa019e078bdbf56425d9c6694f2
parentb55879ea9dbd107edfd3c606624a5fab5a5df0fa
fstests: fix _test_generic_punch() to fit 64k extent

14 test cases use _test_generic_punch(), and they work well as long
as the ext4/xfs blocksize or btrfs sectorsize is below 4K.

In the system with 64K pagesize, as the blocksize can be upto 64K or the
sectorsize can be 64K so 13/14 test cases fail, because the
test-file-size (20k) and thus the extent boundary offsets aren't
big enough to fit the larger than 4k extent size.

Commit 2f194e4e820e6acfcfdadf9f488a162b22db9d86 (generic/009: don't run
for btrfs if PAGE_SIZE > 4096) tried to address this by calling the
not_run in generic/009.

And in the function _test_generic_punch() we use multiple=4 to address
the similar problem but its limited to the subcommand fcollapse.

Now to run these test cases successfully on systems with pagesize 64k,
this patch propose to increase the default multiple=1 to multiple=16.
With this we increase the test file size from 20k to 320k and thus it
encapsulates maximum extent size of 64k here. And we can drop the
multiple=4 which is just being done similar for the cases of fcollapse
subcommand only. And it appears to me there is no harm in increasing
the file size and offsets in general for all commands instead of just
fcollapse command.

This change is tested on ext4, xfs and btrfs on system with pagesize
4K and 64K.

With this patch, these 14 test cases runs fine on system with 64K
pagesize as well as pagesize 4K. However we may hit the same
limitation at some point when we want to validate the FSs with
pagesizes -gt 64K. And this patch does not address that part as of
now.

Signed-off-by: Anand Jain <anand.jain@oracle.com>
Tested-by: Chandan Rajendra <chandan@linux.vnet.ibm.com>
Signed-off-by: Eryu Guan <guaneryu@gmail.com>
18 files changed:
common/punch
tests/ext4/001.out.default
tests/ext4/001.out.nodelalloc
tests/ext4/001.out.nozero
tests/generic/009
tests/generic/009.out
tests/generic/012.out
tests/generic/016.out
tests/generic/021.out
tests/generic/022.out
tests/generic/058.out
tests/generic/060.out
tests/generic/061.out
tests/generic/063.out
tests/generic/255.out
tests/generic/316.out
tests/xfs/242.out
tests/xfs/252.out