diff options
author | Martin Schwidefsky <schwidefsky@de.ibm.com> | 2006-01-06 03:19:09 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-01-06 11:33:49 -0500 |
commit | a63a4931c301a14ca79c41fec0b99d898dbba1fb (patch) | |
tree | 8d543006aadc1518c8bdb195f357d321b62b15da /include/asm-s390 | |
parent | 56dc6a88ec76019e0d0729165cb5b98536270e1d (diff) |
[PATCH] s390: uaccess warnings
Convert __access_ok to an inline C function and change __get_user primitive to
avoid uaccess compiler warnings.
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'include/asm-s390')
-rw-r--r-- | include/asm-s390/uaccess.h | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/include/asm-s390/uaccess.h b/include/asm-s390/uaccess.h index 10a619da4761..be104f21c70a 100644 --- a/include/asm-s390/uaccess.h +++ b/include/asm-s390/uaccess.h | |||
@@ -61,8 +61,10 @@ | |||
61 | #define segment_eq(a,b) ((a).ar4 == (b).ar4) | 61 | #define segment_eq(a,b) ((a).ar4 == (b).ar4) |
62 | 62 | ||
63 | 63 | ||
64 | #define __access_ok(addr,size) (1) | 64 | static inline int __access_ok(const void *addr, unsigned long size) |
65 | 65 | { | |
66 | return 1; | ||
67 | } | ||
66 | #define access_ok(type,addr,size) __access_ok(addr,size) | 68 | #define access_ok(type,addr,size) __access_ok(addr,size) |
67 | 69 | ||
68 | /* | 70 | /* |
@@ -206,25 +208,25 @@ extern int __put_user_bad(void) __attribute__((noreturn)); | |||
206 | case 1: { \ | 208 | case 1: { \ |
207 | unsigned char __x; \ | 209 | unsigned char __x; \ |
208 | __get_user_asm(__x, ptr, __gu_err); \ | 210 | __get_user_asm(__x, ptr, __gu_err); \ |
209 | (x) = (__typeof__(*(ptr))) __x; \ | 211 | (x) = *(__typeof__(*(ptr)) *) &__x; \ |
210 | break; \ | 212 | break; \ |
211 | }; \ | 213 | }; \ |
212 | case 2: { \ | 214 | case 2: { \ |
213 | unsigned short __x; \ | 215 | unsigned short __x; \ |
214 | __get_user_asm(__x, ptr, __gu_err); \ | 216 | __get_user_asm(__x, ptr, __gu_err); \ |
215 | (x) = (__typeof__(*(ptr))) __x; \ | 217 | (x) = *(__typeof__(*(ptr)) *) &__x; \ |
216 | break; \ | 218 | break; \ |
217 | }; \ | 219 | }; \ |
218 | case 4: { \ | 220 | case 4: { \ |
219 | unsigned int __x; \ | 221 | unsigned int __x; \ |
220 | __get_user_asm(__x, ptr, __gu_err); \ | 222 | __get_user_asm(__x, ptr, __gu_err); \ |
221 | (x) = (__typeof__(*(ptr))) __x; \ | 223 | (x) = *(__typeof__(*(ptr)) *) &__x; \ |
222 | break; \ | 224 | break; \ |
223 | }; \ | 225 | }; \ |
224 | case 8: { \ | 226 | case 8: { \ |
225 | unsigned long long __x; \ | 227 | unsigned long long __x; \ |
226 | __get_user_asm(__x, ptr, __gu_err); \ | 228 | __get_user_asm(__x, ptr, __gu_err); \ |
227 | (x) = (__typeof__(*(ptr))) __x; \ | 229 | (x) = *(__typeof__(*(ptr)) *) &__x; \ |
228 | break; \ | 230 | break; \ |
229 | }; \ | 231 | }; \ |
230 | default: \ | 232 | default: \ |