diff options
author | Jeff Layton <jlayton@redhat.com> | 2012-12-11 12:10:15 -0500 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2012-12-20 18:50:08 -0500 |
commit | 68f1bb8bb89e0bb813c893a42373a26ebdab7f9c (patch) | |
tree | 45be5b0fc0e490488a1e6c82fc7c51148edff8cb /fs | |
parent | a69201d6f08240f20a0d33a1b7273d1e7748791c (diff) |
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>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/xattr.c | 9 |
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, | |||
370 | { | 370 | { |
371 | struct path path; | 371 | struct path path; |
372 | int error; | 372 | int error; |
373 | 373 | unsigned int lookup_flags = LOOKUP_FOLLOW; | |
374 | error = user_path(pathname, &path); | 374 | retry: |
375 | error = user_path_at(AT_FDCWD, pathname, lookup_flags, &path); | ||
375 | if (error) | 376 | if (error) |
376 | return error; | 377 | return error; |
377 | error = mnt_want_write(path.mnt); | 378 | error = mnt_want_write(path.mnt); |
@@ -380,6 +381,10 @@ SYSCALL_DEFINE5(setxattr, const char __user *, pathname, | |||
380 | mnt_drop_write(path.mnt); | 381 | mnt_drop_write(path.mnt); |
381 | } | 382 | } |
382 | path_put(&path); | 383 | path_put(&path); |
384 | if (retry_estale(error, lookup_flags)) { | ||
385 | lookup_flags |= LOOKUP_REVAL; | ||
386 | goto retry; | ||
387 | } | ||
383 | return error; | 388 | return error; |
384 | } | 389 | } |
385 | 390 | ||