diff options
| author | Christoph Hellwig <hch@infradead.org> | 2013-10-12 00:55:06 -0700 |
|---|---|---|
| committer | Ben Myers <bpm@sgi.com> | 2013-10-21 16:54:22 -0500 |
| commit | 5f8aca8b43f41c4b2d4dc5389124a40cab92b644 (patch) | |
| tree | 703fc7c2042cc467022cf1be22efdd72e829c3af /fs/xfs/xfs_file.c | |
| parent | 001a3e73706fcb8c52dcca1d4742679384d04184 (diff) | |
| download | linux-5f8aca8b43f41c4b2d4dc5389124a40cab92b644.tar.gz linux-5f8aca8b43f41c4b2d4dc5389124a40cab92b644.tar.bz2 linux-5f8aca8b43f41c4b2d4dc5389124a40cab92b644.zip | |
xfs: always hold the iolock when calling xfs_change_file_space
Currently fallocate always holds the iolock when calling into
xfs_change_file_space, while the ioctl path lets some of the lower level
functions take it, but leave it out in others.
This patch makes sure the ioctl path also always holds the iolock and
thus introduces consistent locking for the preallocation operations while
simplifying the code and allowing to kill the now unused XFS_ATTR_NOLOCK
flag.
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_file.c')
| -rw-r--r-- | fs/xfs/xfs_file.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/xfs/xfs_file.c b/fs/xfs/xfs_file.c index c9179820c01b..116300f3b1d4 100644 --- a/fs/xfs/xfs_file.c +++ b/fs/xfs/xfs_file.c @@ -816,7 +816,7 @@ xfs_file_fallocate( xfs_flock64_t bf; xfs_inode_t *ip = XFS_I(inode); int cmd = XFS_IOC_RESVSP; - int attr_flags = XFS_ATTR_NOLOCK; + int attr_flags = 0; if (mode & ~(FALLOC_FL_KEEP_SIZE | FALLOC_FL_PUNCH_HOLE)) return -EOPNOTSUPP; |