Commit e7a3d7e7 authored by Brian Foster's avatar Brian Foster Committed by Darrick J. Wong
Browse files

xfs: drop unnecessary setfilesize helper 



xfs_setfilesize() is the only remaining caller of the internal
__xfs_setfilesize() helper. Fold them into a single function.

Signed-off-by: default avatarBrian Foster <bfoster@redhat.com>
Reviewed-by: default avatarChristoph Hellwig <hch@lst.de>
Reviewed-by: default avatarDarrick J. Wong <djwong@kernel.org>
Signed-off-by: default avatarDarrick J. Wong <djwong@kernel.org>
parent 044c6449

fs/xfs/xfs_aops.c

+9 −20
Original line number Diff line number Diff line
@@ -42,14 +42,20 @@ static inline bool xfs_ioend_is_append(struct iomap_ioend *ioend) 
/*

 * Update on-disk file size now that data has been written to disk.

 */

STATIC int

__xfs_setfilesize(

int

xfs_setfilesize(

	struct xfs_inode	*ip,

	struct xfs_trans	*tp,

	xfs_off_t		offset,

	size_t			size)

{

	struct xfs_mount	*mp = ip->i_mount;

	struct xfs_trans	*tp;

	xfs_fsize_t		isize;

	int			error;



	error = xfs_trans_alloc(mp, &M_RES(mp)->tr_fsyncts, 0, 0, 0, &tp);

	if (error)

		return error;



	xfs_ilock(ip, XFS_ILOCK_EXCL);

	isize = xfs_new_eof(ip, offset + size);
@@ -68,23 +74,6 @@ __xfs_setfilesize( 
	return xfs_trans_commit(tp);

}



int

xfs_setfilesize(

	struct xfs_inode	*ip,

	xfs_off_t		offset,

	size_t			size)

{

	struct xfs_mount	*mp = ip->i_mount;

	struct xfs_trans	*tp;

	int			error;



	error = xfs_trans_alloc(mp, &M_RES(mp)->tr_fsyncts, 0, 0, 0, &tp);

	if (error)

		return error;



	return __xfs_setfilesize(ip, tp, offset, size);

}



/*

 * IO write completion.

 */