[PATCH 32/39] don't always use EFAULT on __get_user_size.
- Date: Fri, 27 Jun 2008 18:34:39 -0300
- From: Glauber Costa <gcosta@xxxxxxxxxx>
- Subject: [PATCH 32/39] don't always use EFAULT on __get_user_size.
Let the user of the macro specify the desired return.
Signed-off-by: Glauber Costa <gcosta@xxxxxxxxxx>
---
include/asm-x86/uaccess_64.h | 12 ++++++------
1 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/include/asm-x86/uaccess_64.h b/include/asm-x86/uaccess_64.h
index 42c01aa..e0875d7 100644
--- a/include/asm-x86/uaccess_64.h
+++ b/include/asm-x86/uaccess_64.h
@@ -61,27 +61,27 @@ extern void __put_user_bad(void);
({ \
int __gu_err; \
unsigned long __gu_val; \
- __get_user_size(__gu_val, (ptr), (size), __gu_err); \
+ __get_user_size(__gu_val, (ptr), (size), __gu_err, -EFAULT);\
(x) = (__force typeof(*(ptr)))__gu_val; \
__gu_err; \
})
-#define __get_user_size(x, ptr, size, retval) \
+#define __get_user_size(x, ptr, size, retval, errret) \
do { \
retval = 0; \
__chk_user_ptr(ptr); \
switch (size) { \
case 1: \
- __get_user_asm(x, ptr, retval, "b", "b", "=q", -EFAULT);\
+ __get_user_asm(x, ptr, retval, "b", "b", "=q", errret);\
break; \
case 2: \
- __get_user_asm(x, ptr, retval, "w", "w", "=r", -EFAULT);\
+ __get_user_asm(x, ptr, retval, "w", "w", "=r", errret);\
break; \
case 4: \
- __get_user_asm(x, ptr, retval, "l", "k", "=r", -EFAULT);\
+ __get_user_asm(x, ptr, retval, "l", "k", "=r", errret);\
break; \
case 8: \
- __get_user_asm(x, ptr, retval, "q", "", "=r", -EFAULT); \
+ __get_user_asm(x, ptr, retval, "q", "", "=r", errret); \
break; \
default: \
(x) = __get_user_bad(); \
--
1.5.5.1
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
- Follow-Ups:
- [PATCH 33/39] merge __get_user_asm and its users.
- From: Glauber Costa
- [PATCH 33/39] merge __get_user_asm and its users.
- References:
- [PATCH 0/39] Merge files at x86/lib
- From: Glauber Costa
- [PATCH 01/39] Don't use size specifiers
- From: Glauber Costa
- [PATCH 02/39] provide delay loop for x86_64
- From: Glauber Costa
- [PATCH 03/39] use rdtscll in read_current_timer for i386.
- From: Glauber Costa
- [PATCH 04/39] explicitly use edx in const delay function.
- From: Glauber Costa
- [PATCH 05/39] integrate delay functions
- From: Glauber Costa
- [PATCH 06/39] use something common for both architectures
- From: Glauber Costa
- [PATCH 07/39] don't clobber r8 nor use rcx
- From: Glauber Costa
- [PATCH 08/39] don't use word-size specifiers
- From: Glauber Costa
- [PATCH 09/39] adapt x86_64 getuser functions
- From: Glauber Costa
- [PATCH 10/39] rename threadinfo to TI
- From: Glauber Costa
- [PATCH 11/39] Don't use word-size specifiers on getuser_64
- From: Glauber Costa
- [PATCH 12/39] introduce __ASM_REG macro
- From: Glauber Costa
- [PATCH 13/39] use _ASM_PTR instead of explicit word-size pointers
- From: Glauber Costa
- [PATCH 14/39] merge getuser asm functions
- From: Glauber Costa
- [PATCH 15/39] don't save ebx in putuser_32.S
- From: Glauber Costa
- [PATCH 16/39] user put_user_x instead of all variants.
- From: Glauber Costa
- [PATCH 17/39] clobber rbx in putuser_64.S
- From: Glauber Costa
- [PATCH 18/39] pass argument to putuser_64 functions in ax register.
- From: Glauber Costa
- [PATCH 19/39] change testing logic in putuser_64.S
- From: Glauber Costa
- [PATCH 20/39] replace function headers by macros
- From: Glauber Costa
- [PATCH 21/39] don't use word-size specifiers in putuser files
- From: Glauber Costa
- [PATCH 22/39] use macros from asm.h
- From: Glauber Costa
- [PATCH 23/39] merge putuser asm functions
- From: Glauber Costa
- [PATCH 24/39] commonize __range_not_ok
- From: Glauber Costa
- [PATCH 25/39] merge common parts of uaccess.
- From: Glauber Costa
- [PATCH 26/39] merge getuser
- From: Glauber Costa
- [PATCH 27/39] move __addr_ok to uaccess.h
- From: Glauber Costa
- [PATCH 28/39] use k modifier for 4-byte access.
- From: Glauber Costa
- [PATCH 29/39] mark x86_64 as having a working WP.
- From: Glauber Costa
- [PATCH 30/39] don't always use EFAULT on __put_user_size.
- From: Glauber Costa
- [PATCH 31/39] merge __put_user_asm and its user.
- From: Glauber Costa
- [PATCH 0/39] Merge files at x86/lib
- Prev by Date: [PATCH 31/39] merge __put_user_asm and its user.
- Next by Date: [PATCH 33/39] merge __get_user_asm and its users.
- Previous by thread: [PATCH 31/39] merge __put_user_asm and its user.
- Next by thread: [PATCH 33/39] merge __get_user_asm and its users.
- Index(es):