diff options
author | Michael S. Tsirkin <mst@redhat.com> | 2015-01-06 07:37:22 -0500 |
---|---|---|
committer | Michael S. Tsirkin <mst@redhat.com> | 2015-01-13 08:23:34 -0500 |
commit | 9605ce7e5fe058c94fa354415d122462fb419a00 (patch) | |
tree | 4a09141c9a3875c7a3154040e120a17e6e19f2b9 /arch/ia64 | |
parent | 1734bffc30b80ab2447345369c84175e721ebd65 (diff) |
ia64: fix put_user sparse errors
virtio wants to write bitwise types to userspace using put_user.
At the moment this triggers sparse errors, since the value is passed
through an integer.
For example:
__le32 __user *p;
__le32 x;
put_user(x, p);
is safe, but currently triggers a sparse warning.
Fix that up using __force.
Note: this does not suppress any useful sparse checks since callers
do a cast (__typeof__(*(ptr))) (x) which in turn forces all the
necessary type checks.
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Diffstat (limited to 'arch/ia64')
-rw-r--r-- | arch/ia64/include/asm/uaccess.h | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/arch/ia64/include/asm/uaccess.h b/arch/ia64/include/asm/uaccess.h index 967c312a6baf..4f3fb6ccbf21 100644 --- a/arch/ia64/include/asm/uaccess.h +++ b/arch/ia64/include/asm/uaccess.h | |||
@@ -169,10 +169,11 @@ do { \ | |||
169 | (err) = ia64_getreg(_IA64_REG_R8); \ | 169 | (err) = ia64_getreg(_IA64_REG_R8); \ |
170 | (val) = ia64_getreg(_IA64_REG_R9); \ | 170 | (val) = ia64_getreg(_IA64_REG_R9); \ |
171 | } while (0) | 171 | } while (0) |
172 | # define __put_user_size(val, addr, n, err) \ | 172 | # define __put_user_size(val, addr, n, err) \ |
173 | do { \ | 173 | do { \ |
174 | __st_user("__ex_table", (unsigned long) addr, n, RELOC_TYPE, (unsigned long) (val)); \ | 174 | __st_user("__ex_table", (unsigned long) addr, n, RELOC_TYPE, \ |
175 | (err) = ia64_getreg(_IA64_REG_R8); \ | 175 | (__force unsigned long) (val)); \ |
176 | (err) = ia64_getreg(_IA64_REG_R8); \ | ||
176 | } while (0) | 177 | } while (0) |
177 | #endif /* !ASM_SUPPORTED */ | 178 | #endif /* !ASM_SUPPORTED */ |
178 | 179 | ||