On Thu, Sep 17, 2020 at 07:38:47PM +0800, xiakaixu1987@xxxxxxxxx wrote: > static inline int xfs_attr_leaf_entsize_local(int nlen, int vlen) > { > - return ((uint)sizeof(xfs_attr_leaf_name_local_t) - 1 + (nlen) + (vlen) + > - XFS_ATTR_LEAF_NAME_ALIGN - 1) & ~(XFS_ATTR_LEAF_NAME_ALIGN - 1); > + return round_up(sizeof(struct xfs_attr_leaf_name_local) - 1 + nlen + vlen, Please avoid the overly long line here.