On Wed, Jul 21, 2021 at 07:37:48AM +0100, Christoph Hellwig wrote: > On Wed, Jul 21, 2021 at 03:26:59PM +0900, Naohiro Aota wrote: > > btrfs_bio = btrfs_io_bio(bio); > > btrfs_io_bio_init(btrfs_bio); > > > > - bio_trim(bio, offset >> 9, size >> 9); > > + bio_trim(bio, (sector_t)offset >> 9, (sector_t)size >> 9); > > No need for the casts when shifting down. Ah, OK. As, I switched the type to "u64", they are actually the same type anyway.