[LTP] [PATCH] ioctl_loop06: no validate block size

[Jan Stancek]

On Mon, Aug 26, 2024 at 2:02 PM Li Wang <liwang@redhat.com> wrote:
>
> Since commit 9423c653fe6110 ("loop: Don't bother validating blocksize") kernel

This commit above says "The block queue limits validation does this
for us now.",
shouldn't that still catch the errors?

> drop validating blocksize for both loop_configure and loop_set_block_size so
> that set large block size succeeds.
>
> Error log:
>   12 ioctl_loop06.c:76: TINFO: Using LOOP_SET_BLOCK_SIZE with arg > PAGE_SIZE
>   13 ioctl_loop06.c:59: TFAIL: Set block size succeed unexpectedly
>   ...
>   18 ioctl_loop06.c:76: TINFO: Using LOOP_CONFIGURE with block_size > PAGE_SIZE
>   19 ioctl_loop06.c:59:  TFAIL: Set block size succeed unexpectedly
>
> Signed-off-by: Li Wang <liwang@redhat.com>
> ---
>  testcases/kernel/syscalls/ioctl/ioctl_loop06.c | 9 +++++++++
>  1 file changed, 9 insertions(+)
>
> diff --git a/testcases/kernel/syscalls/ioctl/ioctl_loop06.c b/testcases/kernel/syscalls/ioctl/ioctl_loop06.c
> index 317f693a0..4aacd284a 100644
> --- a/testcases/kernel/syscalls/ioctl/ioctl_loop06.c
> +++ b/testcases/kernel/syscalls/ioctl/ioctl_loop06.c
> @@ -23,6 +23,7 @@ static char dev_path[1024];
>  static int dev_num, dev_fd, file_fd, attach_flag, loop_configure_sup = 1;
>  static unsigned int invalid_value, half_value, unalign_value;
>  static struct loop_config loopconfig;
> +static int novalidate_blocksize = 0;
>
>  static struct tcase {
>         unsigned int *setvalue;
> @@ -74,6 +75,11 @@ static void run(unsigned int n)
>         struct tcase *tc = &tcases[n];
>
>         tst_res(TINFO, "%s", tc->message);
> +       if ((*(tc->setvalue) == invalid_value) && novalidate_blocksize) {
> +               tst_res(TCONF, "Kernel doesn't validate block size, skip invalid value test");
> +               return;
> +       }
> +
>         if (tc->ioctl_flag == LOOP_SET_BLOCK_SIZE) {
>                 if (!attach_flag) {
>                         tst_attach_device(dev_path, "test.img");
> @@ -126,6 +132,9 @@ static void setup(void)
>                 return;
>         }
>         loopconfig.fd = file_fd;
> +
> +       if ((tst_kvercmp(6, 11, 0)) >= 0)
> +               novalidate_blocksize = 1;
>  }
>
>  static void cleanup(void)
> --
> 2.46.0
>