On Mon, Mar 16, 2020 at 09:17:07AM -0400, Brian Foster wrote:
> > diff --git a/fs/xfs/libxfs/xfs_inode_fork.c b/fs/xfs/libxfs/xfs_inode_fork.c
> > index ad2b9c313fd2..518c6f0ec3a6 100644
> > --- a/fs/xfs/libxfs/xfs_inode_fork.c
> > +++ b/fs/xfs/libxfs/xfs_inode_fork.c
> > @@ -183,7 +183,7 @@ xfs_iformat_local(
> > */
> > if (unlikely(size > XFS_DFORK_SIZE(dip, ip->i_mount, whichfork))) {
> > xfs_warn(ip->i_mount,
> > - "corrupt inode %Lu (bad size %d for local fork, size = %d).",
> > + "corrupt inode %Lu (bad size %d for local fork, size = %zd).",
>
> Is this here intentionally? Otherwise seems fine:
Yes. XFS_DFORK_SIZE now returns a size_t, so the format strings needs
to match.
