Re: [PATCH] kthreads: kill CLONE_KERNEL, change kernel_thread(kernel_init) to avoid CLONE_SIGHAND

2014-05-19 Thread Sasha Levin
On 05/16/2014 11:35 AM, Oleg Nesterov wrote: > On 05/16, Sasha Levin wrote: >> >> On 04/13/2014 03:56 PM, Oleg Nesterov wrote: >>> 1. Remove CLONE_KERNEL, it has no users and it is dangerous. >>> >>>The (old) comment says "List of flags we want to share for kernel >>>threads" but this is no

Re: [PATCH] kthreads: kill CLONE_KERNEL, change kernel_thread(kernel_init) to avoid CLONE_SIGHAND

2014-05-16 Thread Oleg Nesterov
On 05/16, Sasha Levin wrote: > > On 04/13/2014 03:56 PM, Oleg Nesterov wrote: > > 1. Remove CLONE_KERNEL, it has no users and it is dangerous. > > > >The (old) comment says "List of flags we want to share for kernel > >threads" but this is not true, we do not want to share ->sighand by > >

Re: [PATCH] kthreads: kill CLONE_KERNEL, change kernel_thread(kernel_init) to avoid CLONE_SIGHAND

2014-05-16 Thread Sasha Levin
On 04/13/2014 03:56 PM, Oleg Nesterov wrote: > 1. Remove CLONE_KERNEL, it has no users and it is dangerous. > >The (old) comment says "List of flags we want to share for kernel >threads" but this is not true, we do not want to share ->sighand by >default. This flag can only be used if

Re: [PATCH] kthreads: kill CLONE_KERNEL, change kernel_thread(kernel_init) to avoid CLONE_SIGHAND

2014-04-14 Thread Steven Rostedt
On Sun, 13 Apr 2014 21:56:17 +0200 Oleg Nesterov wrote: > 1. Remove CLONE_KERNEL, it has no users and it is dangerous. > >The (old) comment says "List of flags we want to share for kernel >threads" but this is not true, we do not want to share ->sighand by >default. This flag can onl

Re: [PATCH] kthreads: kill CLONE_KERNEL, change kernel_thread(kernel_init) to avoid CLONE_SIGHAND

2014-04-14 Thread Peter Zijlstra
On Sun, Apr 13, 2014 at 09:56:17PM +0200, Oleg Nesterov wrote: > 1. Remove CLONE_KERNEL, it has no users and it is dangerous. > >The (old) comment says "List of flags we want to share for kernel >threads" but this is not true, we do not want to share ->sighand by >default. This flag ca

[PATCH] kthreads: kill CLONE_KERNEL, change kernel_thread(kernel_init) to avoid CLONE_SIGHAND

2014-04-13 Thread Oleg Nesterov
1. Remove CLONE_KERNEL, it has no users and it is dangerous. The (old) comment says "List of flags we want to share for kernel threads" but this is not true, we do not want to share ->sighand by default. This flag can only be used if the caller is sure that both parent/child will never