diff options
author | Jeff Layton <jlayton@redhat.com> | 2012-12-11 12:10:14 -0500 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2012-12-20 18:50:08 -0500 |
commit | a69201d6f08240f20a0d33a1b7273d1e7748791c (patch) | |
tree | 609495e85765dac924c69a229dde05ed1cc0baff | |
parent | 96948fc6069b68380abac2944b8b02b43a2e2057 (diff) |
vfs: allow utimensat() calls to retry once on an ESTALE error
Clearly, we can't handle the NULL filename case, but we can deal with
the case where there's a real pathname.
Signed-off-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
-rw-r--r-- | fs/utimes.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/fs/utimes.c b/fs/utimes.c index bb0696a41735..f4fb7eca10e8 100644 --- a/fs/utimes.c +++ b/fs/utimes.c | |||
@@ -158,13 +158,17 @@ long do_utimes(int dfd, const char __user *filename, struct timespec *times, | |||
158 | 158 | ||
159 | if (!(flags & AT_SYMLINK_NOFOLLOW)) | 159 | if (!(flags & AT_SYMLINK_NOFOLLOW)) |
160 | lookup_flags |= LOOKUP_FOLLOW; | 160 | lookup_flags |= LOOKUP_FOLLOW; |
161 | 161 | retry: | |
162 | error = user_path_at(dfd, filename, lookup_flags, &path); | 162 | error = user_path_at(dfd, filename, lookup_flags, &path); |
163 | if (error) | 163 | if (error) |
164 | goto out; | 164 | goto out; |
165 | 165 | ||
166 | error = utimes_common(&path, times); | 166 | error = utimes_common(&path, times); |
167 | path_put(&path); | 167 | path_put(&path); |
168 | if (retry_estale(error, lookup_flags)) { | ||
169 | lookup_flags |= LOOKUP_REVAL; | ||
170 | goto retry; | ||
171 | } | ||
168 | } | 172 | } |
169 | 173 | ||
170 | out: | 174 | out: |