On Thu, Jul 22, 2021 at 10:37:36AM -0700, Bart Van Assche wrote: > On 7/11/21 10:48 PM, Christoph Hellwig wrote: >> +static inline int queue_max_sectors_bytes(struct request_queue *q) >> +{ >> + return min_t(unsigned int, queue_max_sectors(q), INT_MAX >> 9) << 9; >> +} > > Should this function return a signed or an unsigned integer? I'm asking > because I see 'unsigned int' as the first argument for min_t(). Yes, it really should be unsigned.