lkml.org 
[lkml]   [2020]   [Jan]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: + signal-move-print_dropped_signal.patch added to -mm tree
> From: Tom Rix <trix@redhat.com>
> Subject: kernel/signal.c: move print_dropped_signal
>
> If the allocation of 'q' fails, the signal will be dropped.

Well, not necessarily... See the comment above TRACE_SIGNAL_LOSE_INFO
in __send_signal().

> To ensure
> that this is reported, move print_dropped_signal to be inside the '(q ==
> NULL)' if-check.

OK, but print_dropped_signal() says "reached RLIMIT_SIGPENDING", this
is misleading if kmem_cache_alloc() fails.

> Link: http://lkml.kernel.org/r/20191203180221.7038-1-trix@redhat.com
> Signed-off-by: Tom Rix <trix@redhat.com>
> Reviewed-by: Andrew Morton <akpm@linux-foundation.org>
> Cc: "Eric W. Biederman" <ebiederm@xmission.com>
> Cc: Christian Brauner <christian.brauner@ubuntu.com>
> Cc: Arnd Bergmann <arnd@arndb.de>
> Cc: Roman Gushchin <guro@fb.com>
> Cc: Deepa Dinamani <deepa.kernel@gmail.com>
> Cc: Oleg Nesterov <oleg@redhat.com>
> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
> ---
>
> kernel/signal.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> --- a/kernel/signal.c~signal-move-print_dropped_signal
> +++ a/kernel/signal.c
> @@ -427,11 +427,10 @@ __sigqueue_alloc(int sig, struct task_st
> atomic_read(&user->sigpending) <=
> task_rlimit(t, RLIMIT_SIGPENDING)) {
> q = kmem_cache_alloc(sigqueue_cachep, flags);
> - } else {
> - print_dropped_signal(sig);
> }
>
> if (unlikely(q == NULL)) {
> + print_dropped_signal(sig);
> atomic_dec(&user->sigpending);
> free_uid(user);
> } else {
> _
>
> Patches currently in -mm which might be from trix@redhat.com are
>
> signal-move-print_dropped_signal.patch
>

\
 
 \ /
  Last update: 2020-01-03 11:07    [W:0.031 / U:3.564 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site