Commit 57fd2d8f authored Apr 22, 2020 by Kaixu Xia Committed by Darrick J. Wong May 04, 2020

xfs: remove unnecessary check of the variable resblks in xfs_symlink



Since the "no-allocation" reservations has been removed, the resblks
value should be larger than zero, so remove the unnecessary check.

Signed-off-by: Kaixu Xia <kaixuxia@tencent.com>
Reviewed-by: Brian Foster <bfoster@redhat.com>
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>

parent cd594559

fs/xfs/xfs_symlink.c

+2 −4

Original line number	Diff line number	Diff line
		@@ -243,7 +243,6 @@ xfs_symlink(
		*/
		xfs_qm_vop_create_dqattach(tp, ip, udqp, gdqp, pdqp);

		if (resblks)
		resblks -= XFS_IALLOC_SPACE_RES(mp);
		/*
		* If the symlink will fit into the inode, write it inline.
		@@ -265,7 +264,6 @@ xfs_symlink(
		if (error)
		goto out_trans_cancel;

		if (resblks)
		resblks -= fs_blocks;
		ip->i_d.di_size = pathlen;
		xfs_trans_log_inode(tp, ip, XFS_ILOG_CORE);