> + spin_lock(&ip->i_flags_lock); > + ip->i_flags = XFS_IRECLAIM; > + ip->i_ino = 0; > + spin_unlock(&ip->i_flags_lock); It's just a tiny sniplet, but still duplicated from xfs_inode_free(). Maybe add another helper for this? Otherwise looks fine: Reviewed-by: Christoph Hellwig <hch at lst.de>