On Tue, Nov 19, 2024 at 12:52:40PM +0000, John Garry wrote: > On 19/11/2024 07:26, Christoph Hellwig wrote: >> The underlying limit is defined as an unsigned int, so return that from >> bdev_io_min as well. >> > > There seem to be other helpers with the same condition. > > Even bdev_io_opt() is defined as returning an int, but actually returns an > unsigned int. And even though not a bdev helper, queue_dma_alignment() is > similar. True. I'll cook up patches - I just noticed this one because of a series that added casts to the return value which made me look there.
