Commit 00696747 authored by Greg Ungerer's avatar Greg Ungerer

m68knommu: fix sparse warnings in signal code

Commit 858b810bf63f ("m68knommu: switch to using asm-generic/uaccess.h")
cleaned up a number of sparse warnings associated with lack of __user
annotations. It also uncovered a couple of more in the signal handling
code:

arch/m68k/kernel/signal.c:923:16:    expected char [noderef] __user *__x
arch/m68k/kernel/signal.c:923:16:    got void *
arch/m68k/kernel/signal.c:1007:16: warning: incorrect type in initializer (different address spaces)
arch/m68k/kernel/signal.c:1007:16:    expected char [noderef] __user *__x
arch/m68k/kernel/signal.c:1007:16:    got void *
arch/m68k/kernel/signal.c:1132:6: warning: symbol 'do_notify_resume' was not declared. Should it be static?

These are specific to a non-MMU configuration. Fix these inserting the
correct __user annotations as required.
Reported-by: default avatarkernel test robot <lkp@intel.com>
Signed-off-by: default avatarGreg Ungerer <gerg@linux-m68k.org>
parent a27bc11f
...@@ -920,7 +920,8 @@ static int setup_frame(struct ksignal *ksig, sigset_t *set, ...@@ -920,7 +920,8 @@ static int setup_frame(struct ksignal *ksig, sigset_t *set,
err |= __put_user(0x70004e40 + (__NR_sigreturn << 16), err |= __put_user(0x70004e40 + (__NR_sigreturn << 16),
(long __user *)(frame->retcode)); (long __user *)(frame->retcode));
#else #else
err |= __put_user((void *) ret_from_user_signal, &frame->pretcode); err |= __put_user((long) ret_from_user_signal,
(long __user *) &frame->pretcode);
#endif #endif
if (err) if (err)
...@@ -1004,7 +1005,8 @@ static int setup_rt_frame(struct ksignal *ksig, sigset_t *set, ...@@ -1004,7 +1005,8 @@ static int setup_rt_frame(struct ksignal *ksig, sigset_t *set,
err |= __put_user(0x4e40, (short __user *)(frame->retcode + 4)); err |= __put_user(0x4e40, (short __user *)(frame->retcode + 4));
#endif #endif
#else #else
err |= __put_user((void *) ret_from_user_rt_signal, &frame->pretcode); err |= __put_user((long) ret_from_user_rt_signal,
(long __user *) &frame->pretcode);
#endif /* CONFIG_MMU */ #endif /* CONFIG_MMU */
if (err) if (err)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment