diff options
| author | Christoph Hellwig <hch@infradead.org> | 2013-10-14 07:09:35 -0700 |
|---|---|---|
| committer | Ben Myers <bpm@sgi.com> | 2013-10-21 16:51:33 -0500 |
| commit | 76ca4c238cf5858f8ba5bcd24c94c23717f1e14e (patch) | |
| tree | 8059d74fc74e68a6f8765ce5f65247ba319117bd /fs/xfs/xfs_iops.h | |
| parent | 59e5a0e821d838854b3afd030d31f82cee3ecd58 (diff) | |
| download | op-kernel-dev-76ca4c238cf5858f8ba5bcd24c94c23717f1e14e.zip op-kernel-dev-76ca4c238cf5858f8ba5bcd24c94c23717f1e14e.tar.gz | |
xfs: always take the iolock around xfs_setattr_size
There is no reason to conditionally take the iolock inside xfs_setattr_size
when we can let the caller handle it unconditionally, which just incrases
the lock hold time for the case where it was previously taken internally
by a few instructions.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Ben Myers <bpm@sgi.com>
Diffstat (limited to 'fs/xfs/xfs_iops.h')
| -rw-r--r-- | fs/xfs/xfs_iops.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/xfs/xfs_iops.h b/fs/xfs/xfs_iops.h index d81fb41..3e76dcb 100644 --- a/fs/xfs/xfs_iops.h +++ b/fs/xfs/xfs_iops.h @@ -38,6 +38,6 @@ extern void xfs_setup_inode(struct xfs_inode *); extern int xfs_setattr_nonsize(struct xfs_inode *ip, struct iattr *vap, int flags); -extern int xfs_setattr_size(struct xfs_inode *ip, struct iattr *vap, int flags); +extern int xfs_setattr_size(struct xfs_inode *ip, struct iattr *vap); #endif /* __XFS_IOPS_H__ */ |
