On Thu, Nov 18, 2021 at 10:28:28PM +0100, Christophe Vu-Brugier wrote: > inode->i_blocks = ((i_size_read(inode) + (sbi->cluster_size - 1)) & > - ~(sbi->cluster_size - 1)) >> inode->i_blkbits; > + ~((loff_t)sbi->cluster_size - 1)) >> inode->i_blkbits; Isn't this a convoluted way to write: inode->i_blocks = round_up(i_size_read(inode), sbi->cluster_size) >> inode->i_blkbits; ?