Re: [PATCH] block: Increase BLK_DEF_MAX_SECTORS_CAP

[Christoph Hellwig <hch@lst.de>]

On Tue, Jun 17, 2025 at 03:34:30PM +0900, Damien Le Moal wrote:
> Since many block devices can benefit from a larger value of
> BLK_DEF_MAX_SECTORS_CAP, and in particular HDDs, increase this value to
> be 4MiB, or 8192 sectors.
> 
> Suggested-by: Martin K . Petersen <martin.petersen@oracle.com>
> Signed-off-by: Damien Le Moal <dlemoal@kernel.org>
> ---
>  include/linux/blkdev.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/linux/blkdev.h b/include/linux/blkdev.h
> index 85aab8bc96e7..7c35b2462048 100644
> --- a/include/linux/blkdev.h
> +++ b/include/linux/blkdev.h
> @@ -1238,7 +1238,7 @@ enum blk_default_limits {
>   * Not to be confused with the max_hw_sector limit that is entirely
>   * controlled by the driver, usually based on hardware limits.
>   */
> -#define BLK_DEF_MAX_SECTORS_CAP	2560u
> +#define BLK_DEF_MAX_SECTORS_CAP	8192u

While we're at nitpicking, maybe define this as

	(SZ_4M >> SECTOR_SHIFT)

to make it a bit more readable?

Otherwise this looks good, the odd number was always rather weird.