[LTP] [External] : Re: [PATCH] fallocate03: FALLOC_FL_PUNCH_HOLE must be used with FALLOC_FL_KEEP_SIZE

[samasth.norway.ananda@oracle.com]

On 7/4/24 12:53 AM, Li Wang wrote:
> Hi Samasth,
> 
> Sorry for the late reply, base on your test failure, which kernel 
> version and
> FS-type did you use?
> 

Li, I am using the v6.10 kernel and the FS-type is nfsvers=4.2. The 
fallocate() call is only supported over NFSv4.2 and should show a proper 
message when run over NFSv4.1 or below.

The output I get on NFSv4.2 is

<<<test_output>>>
incrementing stop
tst_test.c:1733: TINFO: LTP version: 20220121-1963-g830a705d6
tst_test.c:1617: TINFO: Timeout per run is 0h 00m 30s
fallocate03.c:83: TPASS: fallocate(fd, DEFAULT_MODE, 2097152, 1048576) 
passed
fallocate03.c:83: TPASS: fallocate(fd, DEFAULT_MODE, 12582912, 1048576) 
passed
fallocate03.c:83: TPASS: fallocate(fd, DEFAULT_MODE, 17825792, 1048576) 
passed
fallocate03.c:83: TPASS: fallocate(fd, DEFAULT_MODE, 26214400, 1048576) 
passed
fallocate03.c:83: TFAIL: fallocate(fd, FALLOC_FL_KEEP_SIZE, 2097152, 
1048576) failed: EOPNOTSUPP (95)
fallocate03.c:83: TFAIL: fallocate(fd, FALLOC_FL_KEEP_SIZE, 12582912, 
1048576) failed: EOPNOTSUPP (95)
fallocate03.c:83: TFAIL: fallocate(fd, FALLOC_FL_KEEP_SIZE, 19922944, 
1048576) failed: EOPNOTSUPP (95)
fallocate03.c:83: TFAIL: fallocate(fd, FALLOC_FL_KEEP_SIZE, 27262976, 
1048576) failed: EOPNOTSUPP (95)

Summary:
passed   4
failed   4
broken   0
skipped  0
warnings 0

When I run it on NFSv4.1 or below all the tests fail with EOPNOTSUPP.

>  From the fallocate() manual, I see that:
> "The FALLOC_FL_PUNCH_HOLE flag must be ORed withFALLOC_FL_KEEP_SIZE in mode"
> 
> But it doesn't mean we wouldn't use FALLOC_FL_KEEP_SIZE singly in this test.

Oh okay. Sorry mistook that.

Thanks,
Samasth.

> 
> 
> 
> On Wed, Jun 26, 2024 at 9:06 AM Samasth Norway Ananda via ltp 
> <ltp@lists.linux.it <mailto:ltp@lists.linux.it>> wrote:
> 
>     The fallocate03 ltp test fails with "failed: EOPNOTSUPP" if we just use
>     FALLOC_FL_KEEP_SIZE for the mode instead of ORing it with
>     FALLOC_FL_PUNCH_HOLE.
> 
>     Signed-off-by: Samasth Norway Ananda
>     <samasth.norway.ananda@oracle.com
>     <mailto:samasth.norway.ananda@oracle.com>>
>     ---
>       testcases/kernel/syscalls/fallocate/fallocate03.c | 10 +++++-----
>       1 file changed, 5 insertions(+), 5 deletions(-)
> 
>     diff --git a/testcases/kernel/syscalls/fallocate/fallocate03.c
>     b/testcases/kernel/syscalls/fallocate/fallocate03.c
>     index b3e6b3817..20ce13cd4 100644
>     --- a/testcases/kernel/syscalls/fallocate/fallocate03.c
>     +++ b/testcases/kernel/syscalls/fallocate/fallocate03.c
>     @@ -30,10 +30,10 @@ static struct test_case {
>              {DEFAULT_MODE, BLOCKS_WRITTEN},
>              {DEFAULT_MODE, BLOCKS_WRITTEN + HOLE_SIZE_IN_BLOCKS / 2 - 1},
>              {DEFAULT_MODE, BLOCKS_WRITTEN + HOLE_SIZE_IN_BLOCKS + 1},
>     -       {FALLOC_FL_KEEP_SIZE, 2},
>     -       {FALLOC_FL_KEEP_SIZE, BLOCKS_WRITTEN},
>     -       {FALLOC_FL_KEEP_SIZE, BLOCKS_WRITTEN + HOLE_SIZE_IN_BLOCKS /
>     2 + 1},
>     -       {FALLOC_FL_KEEP_SIZE, BLOCKS_WRITTEN + HOLE_SIZE_IN_BLOCKS + 2}
>     +       {FALLOC_FL_KEEP_SIZE | FALLOC_FL_PUNCH_HOLE, 2},
>     +       {FALLOC_FL_KEEP_SIZE | FALLOC_FL_PUNCH_HOLE, BLOCKS_WRITTEN},
>     +       {FALLOC_FL_KEEP_SIZE | FALLOC_FL_PUNCH_HOLE, BLOCKS_WRITTEN
>     + HOLE_SIZE_IN_BLOCKS / 2 + 1},
>     +       {FALLOC_FL_KEEP_SIZE | FALLOC_FL_PUNCH_HOLE, BLOCKS_WRITTEN
>     + HOLE_SIZE_IN_BLOCKS + 2}
>       };
> 
>       static int block_size;
>     @@ -82,7 +82,7 @@ static void verify_fallocate(unsigned int nr)
>              TST_EXP_PASS(
>                      fallocate(fd, tc->mode, tc->offset * block_size,
>     block_size),
>                      "fallocate(fd, %s, %ld, %d)",
>     -               tc->mode ? "FALLOC_FL_KEEP_SIZE" : "DEFAULT_MODE",
>     +               tc->mode ? "FALLOC_FL_KEEP_SIZE |
>     FALLOC_FL_PUNCH_HOLE" : "DEFAULT_MODE",
>                      tc->offset * block_size, block_size);
>       }
> 
>     -- 
>     2.45.2
> 
> 
>     -- 
>     Mailing list info: https://lists.linux.it/listinfo/ltp
>     <https://urldefense.com/v3/__https://lists.linux.it/listinfo/ltp__;!!ACWV5N9M2RV99hQ!ImURCv7bX_N4cHEQ60yHgML0NBIqYcjPdJN4Kot4Ia6o9rBb2TwedwyR0Q1WyZ-I-NCjXyaf_i-Fe-L2h5O7mf4$>
> 
> 
> 
> -- 
> Regards,
> Li Wang