Re: [PATCH v6 2/6] block: sanitize chunk_sectors for atomic write limits

2025-07-11 Thread John Garry
On 11/07/2025 09:42, Damien Le Moal wrote: diff --git a/block/blk-settings.c b/block/blk-settings.c index a000daafbfb48..a2c089167174e 100644 --- a/block/blk-settings.c +++ b/block/blk-settings.c @@ -180,6 +180,7 @@ static void blk_atomic_writes_update_limits(struct queue_limits *lim) stati

Re: [PATCH v6 2/6] block: sanitize chunk_sectors for atomic write limits

2025-07-11 Thread Damien Le Moal
On 7/11/25 5:09 PM, John Garry wrote: > Currently we just ensure that a non-zero value in chunk_sectors aligns > with any atomic write boundary, as the blk boundary functionality uses > both these values. > > However it is also improper to have atomic write unit max > chunk_sectors > (for non-zero

[PATCH v6 2/6] block: sanitize chunk_sectors for atomic write limits

2025-07-11 Thread John Garry
Currently we just ensure that a non-zero value in chunk_sectors aligns with any atomic write boundary, as the blk boundary functionality uses both these values. However it is also improper to have atomic write unit max > chunk_sectors (for non-zero chunk_sectors), as this would lead to splitting o