Re: [PATCh v2] loop: Increase bsize variable from unsigned short to unsigned int

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 27/08/2024 09:17, Li Wang wrote:
This change allows the loopback driver to handle block size larger than
PAGE_SIZE and increases the consistency of data types used within the driver.
Especially to match the struct queue_limits.logical_block_size type.

Also, this is to get rid of the LTP/ioctl_loop06 test failure:

   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

Thoese fail due to the loop_reconfigure_limits() cast bsize to 'unsined short'

these

that never gets an expected error when testing invalid logical block size,
which was just exposed since 6.11-rc1 introduced patches:

   commit 9423c653fe61 ("loop: Don't bother validating blocksize")
   commit fe3d508ba95b ("block: Validate logical block size in blk_validate_limits()")

Maybe it's better to add a fixes tag for original commit which introduced unsigned short usage.


Link:https://urldefense.com/v3/__https://lists.linux.it/pipermail/ltp/2024- August/039912.html__;!!ACWV5N9M2RV99hQ! MxW7Cvs_YKyihvENQFEGgHt-1KbCFD9gkUJxwI8gguQlawan3UNneBoB_THVG4zNP8Sao2a0apcOu03lcg$ Signed-off-by: Li Wang<liwang@xxxxxxxxxx>
Cc: John Garry<john.g.garry@xxxxxxxxxx>
Cc: Jens Axboe<axboe@xxxxxxxxx>
Cc: Stefan Hajnoczi<stefanha@xxxxxxxxxx>
Reviewed-by: Damien Le Moal<dlemoal@xxxxxxxxxx>
Reviewed-by: Jan Stancek<jstancek@xxxxxxxxxx>


Reviewed-by: John Garry <john.g.garry@xxxxxxxxxx>




[Index of Archives]     [Linux RAID]     [Linux SCSI]     [Linux ATA RAID]     [IDE]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Device Mapper]

  Powered by Linux