Commit 683cb941 authored by Christoph Hellwig's avatar Christoph Hellwig Committed by Ben Myers

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: default avatarChristoph Hellwig <hch@lst.de>
Reviewed-by: default avatarDave Chinner <dchinner@redhat.com>
Signed-off-by: default avatarBen Myers <bpm@sgi.com>
parent da51d32d
...@@ -164,6 +164,7 @@ xfs_attr_get( ...@@ -164,6 +164,7 @@ xfs_attr_get(
{ {
int error; int error;
struct xfs_name xname; struct xfs_name xname;
uint lock_mode;
XFS_STATS_INC(xs_attr_get); XFS_STATS_INC(xs_attr_get);
...@@ -174,9 +175,9 @@ xfs_attr_get( ...@@ -174,9 +175,9 @@ xfs_attr_get(
if (error) if (error)
return 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); error = xfs_attr_get_int(ip, &xname, value, valuelenp, flags);
xfs_iunlock(ip, XFS_ILOCK_SHARED); xfs_iunlock(ip, lock_mode);
return(error); return(error);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment