Merge branch 'work.misc' of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs

Pull misc vfs updates from Al Viro:
 "All kinds of stuff.  That probably should've been 5 or 6 separate
  branches, but by the time I'd realized how large and mixed that bag
  had become it had been too close to -final to play with rebasing.

  Some fs/namei.c cleanups there, memdup_user_nul() introduction and
  switching open-coded instances, burying long-dead code, whack-a-mole
  of various kinds, several new helpers for ->llseek(), assorted
  cleanups and fixes from various people, etc.

  One piece probably deserves special mention - Neil's
  lookup_one_len_unlocked().  Similar to lookup_one_len(), but gets
  called without ->i_mutex and tries to avoid ever taking it.  That, of
  course, means that it's not useful for any directory modifications,
  but things like getting inode attributes in nfds readdirplus are fine
  with that.  I really should've asked for moratorium on lookup-related
  changes this cycle, but since I hadn't done that early enough...  I
  *am* asking for that for the coming cycle, though - I'm going to try
  and get conversion of i_mutex to rwsem with ->lookup() done under lock
  taken shared.

  There will be a patch closer to the end of the window, along the lines
  of the one Linus had posted last May - mechanical conversion of
  ->i_mutex accesses to inode_lock()/inode_unlock()/inode_trylock()/
  inode_is_locked()/inode_lock_nested().  To quote Linus back then:

    -----
    |    This is an automated patch using
    |
    |        sed 's/mutex_lock(&\(.*\)->i_mutex)/inode_lock(\1)/'
    |        sed 's/mutex_unlock(&\(.*\)->i_mutex)/inode_unlock(\1)/'
    |        sed 's/mutex_lock_nested(&\(.*\)->i_mutex,[     ]*I_MUTEX_\([A-Z0-9_]*\))/inode_lock_nested(\1, I_MUTEX_\2)/'
    |        sed 's/mutex_is_locked(&\(.*\)->i_mutex)/inode_is_locked(\1)/'
    |        sed 's/mutex_trylock(&\(.*\)->i_mutex)/inode_trylock(\1)/'
    |
    |    with a very few manual fixups
    -----

  I'm going to send that once the ->i_mutex-affecting stuff in -next
  gets mostly merged (or when Linus says he's about to stop taking
  merges)"

* 'work.misc' of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs: (63 commits)
  nfsd: don't hold i_mutex over userspace upcalls
  fs:affs:Replace time_t with time64_t
  fs/9p: use fscache mutex rather than spinlock
  proc: add a reschedule point in proc_readfd_common()
  logfs: constify logfs_block_ops structures
  fcntl: allow to set O_DIRECT flag on pipe
  fs: __generic_file_splice_read retry lookup on AOP_TRUNCATED_PAGE
  fs: xattr: Use kvfree()
  [s390] page_to_phys() always returns a multiple of PAGE_SIZE
  nbd: use ->compat_ioctl()
  fs: use block_device name vsprintf helper
  lib/vsprintf: add %*pg format specifier
  fs: use gendisk->disk_name where possible
  poll: plug an unused argument to do_poll
  amdkfd: don't open-code memdup_user()
  cdrom: don't open-code memdup_user()
  rsxx: don't open-code memdup_user()
  mtip32xx: don't open-code memdup_user()
  [um] mconsole: don't open-code memdup_user_nul()
  [um] hostaudio: don't open-code memdup_user()
  ...

3 files changed, 2 insertions, 123 deletions

diff --git a/arch/sparc/include/asm/uaccess_32.h b/arch/sparc/include/asm/uaccess_32.h
index 64ee103dc29d..57aca2792d29 100644
--- a/arch/sparc/include/asm/uaccess_32.h
+++ b/arch/sparc/include/asm/uaccess_32.h
@@ -205,31 +205,6 @@ int __put_user_bad(void);
         __gu_ret; \
 })
 
-#define __get_user_check_ret(x, addr, size, type, retval) ({ \
-        register unsigned long __gu_val __asm__ ("l1"); \
-        if (__access_ok(addr, size)) { \
-                switch (size) { \
-                case 1: \
-                        __get_user_asm_ret(__gu_val, ub, addr, retval); \
-                        break; \
-                case 2: \
-                        __get_user_asm_ret(__gu_val, uh, addr, retval); \
-                        break; \
-                case 4: \
-                        __get_user_asm_ret(__gu_val, , addr, retval); \
-                        break; \
-                case 8: \
-                        __get_user_asm_ret(__gu_val, d, addr, retval); \
-                        break; \
-                default: \
-                        if (__get_user_bad()) \
-                                return retval; \
-                } \
-                x = (__force type) __gu_val; \
-        } else \
-                return retval; \
-})
-
 #define __get_user_nocheck(x, addr, size, type) ({                      \
         register int __gu_ret;                                          \
         register unsigned long __gu_val;                                \
@@ -247,20 +222,6 @@ int __put_user_bad(void);
         __gu_ret;                                                       \
 })
 
-#define __get_user_nocheck_ret(x, addr, size, type, retval) ({          \
-        register unsigned long __gu_val __asm__ ("l1");                 \
-        switch (size) {                                                 \
-        case 1: __get_user_asm_ret(__gu_val, ub, addr, retval); break;  \
-        case 2: __get_user_asm_ret(__gu_val, uh, addr, retval); break;  \
-        case 4: __get_user_asm_ret(__gu_val, , addr, retval);  break;   \
-        case 8: __get_user_asm_ret(__gu_val, d, addr, retval); break;   \
-        default:                                                        \
-                if (__get_user_bad())                                   \
-                        return retval;                                  \
-        }                                                               \
-        x = (__force type) __gu_val;                                    \
-})
-
 #define __get_user_asm(x, size, addr, ret)                              \
 __asm__ __volatile__(                                                   \
                 "/* Get user asm, inline. */\n"                         \
@@ -281,32 +242,6 @@ __asm__ __volatile__(							\
                : "=&r" (ret), "=&r" (x) : "m" (*__m(addr)),             \
                  "i" (-EFAULT))
 
-#define __get_user_asm_ret(x, size, addr, retval)                       \
-if (__builtin_constant_p(retval) && retval == -EFAULT)                  \
-        __asm__ __volatile__(                                           \
-                        "/* Get user asm ret, inline. */\n"             \
-                "1:\t"  "ld"#size " %1, %0\n\n\t"                       \
-                        ".section __ex_table,#alloc\n\t"                \
-                        ".align 4\n\t"                                  \
-                        ".word  1b,__ret_efault\n\n\t"                  \
-                        ".previous\n\t"                                 \
-                       : "=&r" (x) : "m" (*__m(addr)));                 \
-else                                                                    \
-        __asm__ __volatile__(                                           \
-                        "/* Get user asm ret, inline. */\n"             \
-                "1:\t"  "ld"#size " %1, %0\n\n\t"                       \
-                        ".section .fixup,#alloc,#execinstr\n\t"         \
-                        ".align 4\n"                                    \
-                "3:\n\t"                                                \
-                        "ret\n\t"                                       \
-                        " restore %%g0, %2, %%o0\n\n\t"                 \
-                        ".previous\n\t"                                 \
-                        ".section __ex_table,#alloc\n\t"                \
-                        ".align 4\n\t"                                  \
-                        ".word  1b, 3b\n\n\t"                           \
-                        ".previous\n\t"                                 \
-                       : "=&r" (x) : "m" (*__m(addr)), "i" (retval))
-
 int __get_user_bad(void);
 
 unsigned long __copy_user(void __user *to, const void __user *from, unsigned long size);
diff --git a/arch/sparc/include/asm/uaccess_64.h b/arch/sparc/include/asm/uaccess_64.h
index ea6e9a20f3ff..e9a51d64974d 100644
--- a/arch/sparc/include/asm/uaccess_64.h
+++ b/arch/sparc/include/asm/uaccess_64.h
@@ -179,20 +179,6 @@ int __put_user_bad(void);
          __gu_ret;                                                           \
 })
 
-#define __get_user_nocheck_ret(data, addr, size, type, retval) ({       \
-        register unsigned long __gu_val __asm__ ("l1");                 \
-        switch (size) {                                                 \
-        case 1: __get_user_asm_ret(__gu_val, ub, addr, retval); break;  \
-        case 2: __get_user_asm_ret(__gu_val, uh, addr, retval); break;  \
-        case 4: __get_user_asm_ret(__gu_val, uw, addr, retval); break;  \
-        case 8: __get_user_asm_ret(__gu_val, x, addr, retval); break;   \
-        default:                                                        \
-                if (__get_user_bad())                                   \
-                        return retval;                                  \
-        }                                                               \
-        data = (__force type) __gu_val;                                 \
-})
-
 #define __get_user_asm(x, size, addr, ret)                              \
 __asm__ __volatile__(                                                   \
                 "/* Get user asm, inline. */\n"                         \
@@ -214,32 +200,6 @@ __asm__ __volatile__(							\
                : "=r" (ret), "=r" (x) : "r" (__m(addr)),                \
                  "i" (-EFAULT))
 
-#define __get_user_asm_ret(x, size, addr, retval)                       \
-if (__builtin_constant_p(retval) && retval == -EFAULT)                  \
-        __asm__ __volatile__(                                           \
-                "/* Get user asm ret, inline. */\n"                     \
-        "1:\t"  "ld"#size "a [%1] %%asi, %0\n\n\t"                      \
-                ".section __ex_table,\"a\"\n\t"                         \
-                ".align 4\n\t"                                          \
-                ".word  1b,__ret_efault\n\n\t"                          \
-                ".previous\n\t"                                         \
-               : "=r" (x) : "r" (__m(addr)));                           \
-else                                                                    \
-        __asm__ __volatile__(                                           \
-                "/* Get user asm ret, inline. */\n"                     \
-        "1:\t"  "ld"#size "a [%1] %%asi, %0\n\n\t"                      \
-                ".section .fixup,#alloc,#execinstr\n\t"                 \
-                ".align 4\n"                                            \
-        "3:\n\t"                                                        \
-                "ret\n\t"                                               \
-                " restore %%g0, %2, %%o0\n\n\t"                         \
-                ".previous\n\t"                                         \
-                ".section __ex_table,\"a\"\n\t"                         \
-                ".align 4\n\t"                                          \
-                ".word  1b, 3b\n\n\t"                                   \
-                ".previous\n\t"                                         \
-               : "=r" (x) : "r" (__m(addr)), "i" (retval))
-
 int __get_user_bad(void);
 
 unsigned long __must_check ___copy_from_user(void *to,
diff --git a/arch/sparc/kernel/mdesc.c b/arch/sparc/kernel/mdesc.c
index 6f80936e0eea..11228861d9b4 100644
--- a/arch/sparc/kernel/mdesc.c
+++ b/arch/sparc/kernel/mdesc.c
@@ -1033,25 +1033,9 @@ static ssize_t mdesc_read(struct file *file, char __user *buf,
 
 static loff_t mdesc_llseek(struct file *file, loff_t offset, int whence)
 {
-        struct mdesc_handle *hp;
-
-        switch (whence) {
-        case SEEK_CUR:
-                offset += file->f_pos;
-                break;
-        case SEEK_SET:
-                break;
-        default:
-                return -EINVAL;
-        }
-
-        hp = file->private_data;
-        if (offset > hp->handle_size)
-                return -EINVAL;
-        else
-                file->f_pos = offset;
+        struct mdesc_handle *hp = file->private_data;
 
-        return offset;
+        return no_seek_end_llseek_size(file, offset, whence, hp->handle_size);
 }
 
 /* mdesc_close() - /dev/mdesc is being closed, release the reference to