Re: [PATCH] exit.c: call proc_exit_connector() after exit_state is set

2014-03-15 Thread Linus Torvalds
On Sat, Mar 15, 2014 at 12:12 PM, Oleg Nesterov wrote: > > So I think the patch is fine, but let me repeat: > > I hope that someone > can confirm that netlink_broadcast() is safe even if > release_task(current) > was already called, so that the caller has no pids,

Re: [PATCH] exit.c: call proc_exit_connector() after exit_state is set

2014-03-15 Thread Oleg Nesterov
On 03/15, Linus Torvalds wrote: > > [ Going through old emails, adding relevant people, ie Oleg for > kernel/exit.c and David because he seems to be the go-to person for > connector issues judging by commits ] > > Patch looks sane to me, and logically that exit_connector thing would > pair with

Re: [PATCH] exit.c: call proc_exit_connector() after exit_state is set

2014-03-15 Thread Linus Torvalds
[ Going through old emails, adding relevant people, ie Oleg for kernel/exit.c and David because he seems to be the go-to person for connector issues judging by commits ] Patch looks sane to me, and logically that exit_connector thing would pair with exit_notify(), but I'd like some comments on

Re: [PATCH] exit.c: call proc_exit_connector() after exit_state is set

2014-03-15 Thread Linus Torvalds
[ Going through old emails, adding relevant people, ie Oleg for kernel/exit.c and David because he seems to be the go-to person for connector issues judging by commits ] Patch looks sane to me, and logically that exit_connector thing would pair with exit_notify(), but I'd like some comments on

Re: [PATCH] exit.c: call proc_exit_connector() after exit_state is set

2014-03-15 Thread Oleg Nesterov
On 03/15, Linus Torvalds wrote: [ Going through old emails, adding relevant people, ie Oleg for kernel/exit.c and David because he seems to be the go-to person for connector issues judging by commits ] Patch looks sane to me, and logically that exit_connector thing would pair with

Re: [PATCH] exit.c: call proc_exit_connector() after exit_state is set

2014-03-15 Thread Linus Torvalds
On Sat, Mar 15, 2014 at 12:12 PM, Oleg Nesterov o...@redhat.com wrote: So I think the patch is fine, but let me repeat: I hope that someone can confirm that netlink_broadcast() is safe even if release_task(current) was already called, so that the caller has no pids,

[PATCH] exit.c: call proc_exit_connector() after exit_state is set

2014-02-24 Thread Guillaume Morin
From: Guillaume Morin The process events connector delivers a notification when a process exits. This is really convenient for a process that spawns and wants to monitor its children through an epoll-able() interface. Unfortunately, there is a small window between when the event is delivered

[PATCH] exit.c: call proc_exit_connector() after exit_state is set

2014-02-24 Thread Guillaume Morin
From: Guillaume Morin guilla...@morinfr.org The process events connector delivers a notification when a process exits. This is really convenient for a process that spawns and wants to monitor its children through an epoll-able() interface. Unfortunately, there is a small window between when the