diff options
| author | Christoph Hellwig <hch@infradead.org> | 2013-12-06 12:30:15 -0800 |
|---|---|---|
| committer | Ben Myers <bpm@sgi.com> | 2013-12-18 16:07:09 -0600 |
| commit | 683cb941598d1d81283c940c100e0ce40f494105 (patch) | |
| tree | 0bb287e4256cbd4fd1bcebfe765564ec3236b677 | |
| parent | da51d32d4596a14ee33917b9eca056d4bf41706a (diff) | |
| download | linux-683cb941598d1d81283c940c100e0ce40f494105.tar.gz linux-683cb941598d1d81283c940c100e0ce40f494105.tar.bz2 linux-683cb941598d1d81283c940c100e0ce40f494105.zip | |
xfs: use xfs_ilock_attr_map_shared in xfs_attr_get
We might not have read in the extent list at this point, so make sure we
take the ilock exclusively if we have to do so.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Ben Myers <bpm@sgi.com>
| -rw-r--r-- | fs/xfs/xfs_attr.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/xfs/xfs_attr.c b/fs/xfs/xfs_attr.c index b86127072ac3..01b6a0102fbd 100644 --- a/fs/xfs/xfs_attr.c +++ b/fs/xfs/xfs_attr.c @@ -164,6 +164,7 @@ xfs_attr_get( { int error; struct xfs_name xname; + uint lock_mode; XFS_STATS_INC(xs_attr_get); @@ -174,9 +175,9 @@ xfs_attr_get( if (error) return error; - xfs_ilock(ip, XFS_ILOCK_SHARED); + lock_mode = xfs_ilock_attr_map_shared(ip); error = xfs_attr_get_int(ip, &xname, value, valuelenp, flags); - xfs_iunlock(ip, XFS_ILOCK_SHARED); + xfs_iunlock(ip, lock_mode); return(error); } |