On Tue, Apr 11, 2023 at 08:51:24PM +0530, Ritesh Harjani wrote: > >> + if ((flags & IOMAP_WRITE) && (offset + length > i_size_read(inode))) > > > > No need for the second set of inner braces here either. > > It's just avoids any confusion this way. Does it? Except for some really weird places it is very unusual in the kernel. To me it really does add confusion.