summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJeff Layton <jlayton@redhat.com>2012-12-11 18:10:15 +0100
committerAl Viro <viro@zeniv.linux.org.uk>2012-12-21 00:50:08 +0100
commit68f1bb8bb89e0bb813c893a42373a26ebdab7f9c (patch)
tree45be5b0fc0e490488a1e6c82fc7c51148edff8cb
parentvfs: allow utimensat() calls to retry once on an ESTALE error (diff)
downloadlinux-68f1bb8bb89e0bb813c893a42373a26ebdab7f9c.tar.xz
linux-68f1bb8bb89e0bb813c893a42373a26ebdab7f9c.zip
vfs: allow setxattr to retry once on ESTALE errors
Signed-off-by: Jeff Layton <jlayton@redhat.com> Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
-rw-r--r--fs/xattr.c9
1 files changed, 7 insertions, 2 deletions
diff --git a/fs/xattr.c b/fs/xattr.c
index e21c119f4f99..c5e90d2d751f 100644
--- a/fs/xattr.c
+++ b/fs/xattr.c
@@ -370,8 +370,9 @@ SYSCALL_DEFINE5(setxattr, const char __user *, pathname,
{
struct path path;
int error;
-
- error = user_path(pathname, &path);
+ unsigned int lookup_flags = LOOKUP_FOLLOW;
+retry:
+ error = user_path_at(AT_FDCWD, pathname, lookup_flags, &path);
if (error)
return error;
error = mnt_want_write(path.mnt);
@@ -380,6 +381,10 @@ SYSCALL_DEFINE5(setxattr, const char __user *, pathname,
mnt_drop_write(path.mnt);
}
path_put(&path);
+ if (retry_estale(error, lookup_flags)) {
+ lookup_flags |= LOOKUP_REVAL;
+ goto retry;
+ }
return error;
}