powerpc/uaccess: Merge __put_user_size_allowed() into __put_user_size()
authorChristophe Leroy <christophe.leroy@csgroup.eu>
Tue, 9 Feb 2021 14:02:13 +0000 (14:02 +0000)
committerMichael Ellerman <mpe@ellerman.id.au>
Thu, 11 Feb 2021 12:35:35 +0000 (23:35 +1100)
__put_user_size_allowed() is only called from __put_user_size() now.

Merge them together.

Signed-off-by: Christophe Leroy <christophe.leroy@csgroup.eu>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/b3baeaec1ee2fbdc653bb6fb27b0be5b846163ef.1612879284.git.christophe.leroy@csgroup.eu
arch/powerpc/include/asm/uaccess.h

index 88f5b0fe9a1249f2f15b19a39d0f893c9805e57c..297ee141c33afabfed61c4f9352f6abb94bece2c 100644 (file)
@@ -108,22 +108,18 @@ static inline bool __access_ok(unsigned long addr, unsigned long size)
 
 extern long __put_user_bad(void);
 
-#define __put_user_size_allowed(x, ptr, size, retval)          \
+#define __put_user_size(x, ptr, size, retval)                  \
 do {                                                           \
        __label__ __pu_failed;                                  \
                                                                \
        retval = 0;                                             \
+       allow_write_to_user(ptr, size);                         \
        __put_user_size_goto(x, ptr, size, __pu_failed);        \
+       prevent_write_to_user(ptr, size);                       \
        break;                                                  \
                                                                \
 __pu_failed:                                                   \
        retval = -EFAULT;                                       \
-} while (0)
-
-#define __put_user_size(x, ptr, size, retval)                  \
-do {                                                           \
-       allow_write_to_user(ptr, size);                         \
-       __put_user_size_allowed(x, ptr, size, retval);          \
        prevent_write_to_user(ptr, size);                       \
 } while (0)