On Sat, May 16, 2020 at 10:49:27AM -0700, Darrick J. Wong wrote: > > +int > > +xfs_ifork_verify_local_data( > > struct xfs_inode *ip) > > { > > + if (fa) { > > + xfs_inode_verifier_error(ip, -EFSCORRUPTED, "data fork", > > + ip->i_df.if_u1.if_data, ip->i_df.if_bytes, fa); > > Needs more indent, but I could fix this on merge... Sure. It seems like that is the only outstanding comment on this series, so maybe you should just pick it up and I shouldn't bother to resend it?