diff options
| author | Oleg Nesterov <oleg@redhat.com> | 2009-12-15 16:47:22 -0800 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-12-16 07:20:08 -0800 |
| commit | 614c517d7c00af1b26ded20646b329397d6f51a1 (patch) | |
| tree | ddd7a82b3479c9fabe141b4c82a1794650a82b4f /include | |
| parent | d51965037325e51f6cd68583413243c3573e47b0 (diff) | |
| download | linux-3.10-614c517d7c00af1b26ded20646b329397d6f51a1.tar.gz linux-3.10-614c517d7c00af1b26ded20646b329397d6f51a1.tar.bz2 linux-3.10-614c517d7c00af1b26ded20646b329397d6f51a1.zip | |
signals: SEND_SIG_NOINFO should be considered as SI_FROMUSER()
No changes in compiled code. The patch adds the new helper, si_fromuser()
and changes check_kill_permission() to use this helper.
The real effect of this patch is that from now we "officially" consider
SEND_SIG_NOINFO signal as "from user-space" signals. This is already true
if we look at the code which uses SEND_SIG_NOINFO, except __send_signal()
has another opinion - see the next patch.
The naming of these special SEND_SIG_XXX siginfo's is really bad
imho. From __send_signal()'s pov they mean
SEND_SIG_NOINFO from user
SEND_SIG_PRIV from kernel
SEND_SIG_FORCED no info
Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Cc: Roland McGrath <roland@redhat.com>
Reviewed-by: Sukadev Bhattiprolu <sukadev@us.ibm.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'include')
| -rw-r--r-- | include/linux/sched.h | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/include/linux/sched.h b/include/linux/sched.h index f4c145410a8..57b3516f055 100644 --- a/include/linux/sched.h +++ b/include/linux/sched.h @@ -2102,11 +2102,6 @@ static inline int kill_cad_pid(int sig, int priv) #define SEND_SIG_PRIV ((struct siginfo *) 1) #define SEND_SIG_FORCED ((struct siginfo *) 2) -static inline int is_si_special(const struct siginfo *info) -{ - return info <= SEND_SIG_FORCED; -} - /* * True if we are on the alternate signal stack. */ |