ext4: avoid s_mb_prefetch to be zero in individual scenarios

[linux-2.6-microblaze.git] / fs / utimes.c

diff --git a/fs/utimes.c b/fs/utimes.c
index b7b9275..fd3cc42 100644 (file)
--- a/fs/utimes.c
+++ b/fs/utimes.c
@@ -16,21 +16,26 @@ static bool nsec_valid(long nsec)
        return nsec >= 0 && nsec <= 999999999;
 }
 
-static int utimes_common(const struct path *path, struct timespec64 *times)
+int vfs_utimes(const struct path *path, struct timespec64 *times)
 {
        int error;
        struct iattr newattrs;
        struct inode *inode = path->dentry->d_inode;
        struct inode *delegated_inode = NULL;
 
+       if (times) {
+               if (!nsec_valid(times[0].tv_nsec) ||
+                   !nsec_valid(times[1].tv_nsec))
+                       return -EINVAL;
+               if (times[0].tv_nsec == UTIME_NOW &&
+                   times[1].tv_nsec == UTIME_NOW)
+                       times = NULL;
+       }
+
        error = mnt_want_write(path->mnt);
        if (error)
                goto out;
 
-       if (times && times[0].tv_nsec == UTIME_NOW &&
-                    times[1].tv_nsec == UTIME_NOW)
-               times = NULL;
-
        newattrs.ia_valid = ATTR_CTIME | ATTR_MTIME | ATTR_ATIME;
        if (times) {
                if (times[0].tv_nsec == UTIME_OMIT)
@@ -70,6 +75,51 @@ out:
        return error;
 }
 
+static int do_utimes_path(int dfd, const char __user *filename,
+               struct timespec64 *times, int flags)
+{
+       struct path path;
+       int lookup_flags = 0, error;
+
+       if (flags & ~(AT_SYMLINK_NOFOLLOW | AT_EMPTY_PATH))
+               return -EINVAL;
+
+       if (!(flags & AT_SYMLINK_NOFOLLOW))
+               lookup_flags |= LOOKUP_FOLLOW;
+       if (flags & AT_EMPTY_PATH)
+               lookup_flags |= LOOKUP_EMPTY;
+
+retry:
+       error = user_path_at(dfd, filename, lookup_flags, &path);
+       if (error)
+               return error;
+
+       error = vfs_utimes(&path, times);
+       path_put(&path);
+       if (retry_estale(error, lookup_flags)) {
+               lookup_flags |= LOOKUP_REVAL;
+               goto retry;
+       }
+
+       return error;
+}
+
+static int do_utimes_fd(int fd, struct timespec64 *times, int flags)
+{
+       struct fd f;
+       int error;
+
+       if (flags)
+               return -EINVAL;
+
+       f = fdget(fd);
+       if (!f.file)
+               return -EBADF;
+       error = vfs_utimes(&f.file->f_path, times);
+       fdput(f);
+       return error;
+}
+
 /*
  * do_utimes - change times on filename or file descriptor
  * @dfd: open file descriptor, -1 or AT_FDCWD
@@ -88,52 +138,9 @@ out:
 long do_utimes(int dfd, const char __user *filename, struct timespec64 *times,
               int flags)
 {
-       int error = -EINVAL;
-
-       if (times && (!nsec_valid(times[0].tv_nsec) ||
-                     !nsec_valid(times[1].tv_nsec))) {
-               goto out;
-       }
-
-       if (flags & ~(AT_SYMLINK_NOFOLLOW | AT_EMPTY_PATH))
-               goto out;
-
-       if (filename == NULL && dfd != AT_FDCWD) {
-               struct fd f;
-
-               if (flags)
-                       goto out;
-
-               f = fdget(dfd);
-               error = -EBADF;
-               if (!f.file)
-                       goto out;
-
-               error = utimes_common(&f.file->f_path, times);
-               fdput(f);
-       } else {
-               struct path path;
-               int lookup_flags = 0;
-
-               if (!(flags & AT_SYMLINK_NOFOLLOW))
-                       lookup_flags |= LOOKUP_FOLLOW;
-               if (flags & AT_EMPTY_PATH)
-                       lookup_flags |= LOOKUP_EMPTY;
-retry:
-               error = user_path_at(dfd, filename, lookup_flags, &path);
-               if (error)
-                       goto out;
-
-               error = utimes_common(&path, times);
-               path_put(&path);
-               if (retry_estale(error, lookup_flags)) {
-                       lookup_flags |= LOOKUP_REVAL;
-                       goto retry;
-               }
-       }
-
-out:
-       return error;
+       if (filename == NULL && dfd != AT_FDCWD)
+               return do_utimes_fd(dfd, times, flags);
+       return do_utimes_path(dfd, filename, times, flags);
 }
 
 SYSCALL_DEFINE4(utimensat, int, dfd, const char __user *, filename,