On Thu, Mar 9, 2023 at 10:43 AM Yangtao Li <frank.li@xxxxxxxx> wrote: > Introduce i_blocksize_mask() to simplify code, which replace > (i_blocksize(node) - 1). Like done in commit > 93407472a21b("fs: add i_blocksize()"). I would call this i_blockmask(). Note that it could be used in ocfs2_is_io_unaligned() as well. > > Signed-off-by: Yangtao Li <frank.li@xxxxxxxx> > --- > include/linux/fs.h | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/include/linux/fs.h b/include/linux/fs.h > index c85916e9f7db..db335bd9c256 100644 > --- a/include/linux/fs.h > +++ b/include/linux/fs.h > @@ -711,6 +711,11 @@ static inline unsigned int i_blocksize(const struct inode *node) > return (1 << node->i_blkbits); > } > > +static inline unsigned int i_blocksize_mask(const struct inode *node) > +{ > + return i_blocksize(node) - 1; > +} > + > static inline int inode_unhashed(struct inode *inode) > { > return hlist_unhashed(&inode->i_hash); > -- > 2.25.1 > Thanks, Andreas