xfs: unlock the inode before log force in xfs_change_file_space
Let the transaction commit unlock the inode before it potentially causes a synchronous log force. Signed-off-by: Christoph Hellwig <hch@lst.de> Reviewed-by: Dave Chinner <dchinner@redhat.com> Signed-off-by: Alex Elder <aelder@sgi.com>
This commit is contained in:
parent
8292d88c5c
commit
23bb0be1a2
|
@ -2349,8 +2349,7 @@ xfs_change_file_space(
|
||||||
}
|
}
|
||||||
|
|
||||||
xfs_ilock(ip, XFS_ILOCK_EXCL);
|
xfs_ilock(ip, XFS_ILOCK_EXCL);
|
||||||
|
xfs_trans_ijoin_ref(tp, ip, XFS_ILOCK_EXCL);
|
||||||
xfs_trans_ijoin(tp, ip);
|
|
||||||
|
|
||||||
if ((attr_flags & XFS_ATTR_DMI) == 0) {
|
if ((attr_flags & XFS_ATTR_DMI) == 0) {
|
||||||
ip->i_d.di_mode &= ~S_ISUID;
|
ip->i_d.di_mode &= ~S_ISUID;
|
||||||
|
@ -2375,10 +2374,5 @@ xfs_change_file_space(
|
||||||
xfs_trans_log_inode(tp, ip, XFS_ILOG_CORE);
|
xfs_trans_log_inode(tp, ip, XFS_ILOG_CORE);
|
||||||
if (attr_flags & XFS_ATTR_SYNC)
|
if (attr_flags & XFS_ATTR_SYNC)
|
||||||
xfs_trans_set_sync(tp);
|
xfs_trans_set_sync(tp);
|
||||||
|
return xfs_trans_commit(tp, 0);
|
||||||
error = xfs_trans_commit(tp, 0);
|
|
||||||
|
|
||||||
xfs_iunlock(ip, XFS_ILOCK_EXCL);
|
|
||||||
|
|
||||||
return error;
|
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue