Re: [patch V2 3/3] signal: Allow tasks to cache one sigqueue struct

2021-03-16 Thread Thomas Gleixner
On Sat, Mar 13 2021 at 17:49, Oleg Nesterov wrote: > On 03/12, Thomas Gleixner wrote: >> >> On Fri, Mar 12 2021 at 20:26, Thomas Gleixner wrote: >> > On Fri, Mar 12 2021 at 17:11, Oleg Nesterov wrote: >> >> On 03/11, Thomas Gleixner wrote: >> >>> >> >>> @@ -456,7 +460,12 @@ static void

Re: [patch V2 3/3] signal: Allow tasks to cache one sigqueue struct

2021-03-13 Thread Oleg Nesterov
On 03/12, Thomas Gleixner wrote: > > On Fri, Mar 12 2021 at 20:26, Thomas Gleixner wrote: > > On Fri, Mar 12 2021 at 17:11, Oleg Nesterov wrote: > >> On 03/11, Thomas Gleixner wrote: > >>> > >>> @@ -456,7 +460,12 @@ static void __sigqueue_free(struct sigqu > >>> return; > >>> if

Re: [patch V2 3/3] signal: Allow tasks to cache one sigqueue struct

2021-03-12 Thread Thomas Gleixner
On Fri, Mar 12 2021 at 20:26, Thomas Gleixner wrote: > On Fri, Mar 12 2021 at 17:11, Oleg Nesterov wrote: >> On 03/11, Thomas Gleixner wrote: >>> >>> @@ -456,7 +460,12 @@ static void __sigqueue_free(struct sigqu >>> return; >>> if (atomic_dec_and_test(>user->sigpending)) >>>

Re: [patch V2 3/3] signal: Allow tasks to cache one sigqueue struct

2021-03-12 Thread Thomas Gleixner
On Fri, Mar 12 2021 at 17:11, Oleg Nesterov wrote: > On 03/11, Thomas Gleixner wrote: >> >> @@ -456,7 +460,12 @@ static void __sigqueue_free(struct sigqu >> return; >> if (atomic_dec_and_test(>user->sigpending)) >> free_uid(q->user); >> -

Re: [patch V2 3/3] signal: Allow tasks to cache one sigqueue struct

2021-03-12 Thread Thomas Gleixner
On Fri, Mar 12 2021 at 17:18, Oleg Nesterov wrote: > On 03/12, Sebastian Andrzej Siewior wrote: >> >> On 2021-03-11 14:20:39 [+0100], Thomas Gleixner wrote: >> > --- a/kernel/signal.c >> > +++ b/kernel/signal.c >> > @@ -433,7 +433,11 @@ static struct sigqueue * >> >rcu_read_unlock(); >> > >> >

Re: [patch V2 3/3] signal: Allow tasks to cache one sigqueue struct

2021-03-12 Thread Oleg Nesterov
On 03/12, Sebastian Andrzej Siewior wrote: > > On 2021-03-11 14:20:39 [+0100], Thomas Gleixner wrote: > > --- a/kernel/signal.c > > +++ b/kernel/signal.c > > @@ -433,7 +433,11 @@ static struct sigqueue * > > rcu_read_unlock(); > > > > if (override_rlimit || likely(sigpending <=

Re: [patch V2 3/3] signal: Allow tasks to cache one sigqueue struct

2021-03-12 Thread Oleg Nesterov
On 03/11, Thomas Gleixner wrote: > > @@ -456,7 +460,12 @@ static void __sigqueue_free(struct sigqu > return; > if (atomic_dec_and_test(>user->sigpending)) > free_uid(q->user); > - kmem_cache_free(sigqueue_cachep, q); > + > + /* Cache one sigqueue per task

Re: [patch V2 3/3] signal: Allow tasks to cache one sigqueue struct

2021-03-12 Thread Sebastian Andrzej Siewior
On 2021-03-11 14:20:39 [+0100], Thomas Gleixner wrote: > --- a/kernel/signal.c > +++ b/kernel/signal.c > @@ -433,7 +433,11 @@ static struct sigqueue * > rcu_read_unlock(); > > if (override_rlimit || likely(sigpending <= task_rlimit(t, > RLIMIT_SIGPENDING))) { > - q =