On 6/21/24 11:02 AM, John Garry wrote: > Sure, but I think that we can also use the following, since both are unsigned int: > if (WARN_ON_ONCE(chunk_sectors % boundary_sectors)) > > I'll send a fix. Good point, yeah that's better as it's just 32-bit types regardless. -- Jens Axboe