Re: [PATCH v2 2/5] md/raid0: set chunk_sectors limit

From: Nilay Shroff
Date: Fri Jun 20 2025 - 10:32:36 EST




On 6/18/25 2:07 PM, John Garry wrote:
> Currently we use min io size as the chunk size when deciding on the
> atomic write size limits - see blk_stack_atomic_writes_head().
>
> The limit min_io size is not a reliable value to store the chunk size, as
> this may be mutated by the block stacking code. Such an example would be
> for the min io size less than the physical block size, and the min io size
> is raised to the physical block size - see blk_stack_limits().
>
> The block stacking limits will rely on chunk_sectors in future,
> so set this value (to the chunk size).
>
> Signed-off-by: John Garry <john.g.garry@xxxxxxxxxx>

Looks good to me:
Reviewed-by: Nilay Shroff <nilay@xxxxxxxxxxxxx>