svn commit: r263755 - head/sys/kern

Warner Losh imp at bsdimp.com
Thu Mar 27 17:08:37 UTC 2014


On Mar 27, 2014, at 2:37 AM, Mateusz Guzik <mjguzik at gmail.com> wrote:

> On Thu, Mar 27, 2014 at 03:45:17PM +0800, David Xu wrote:
>> I think the async process pointer can be cleared when a process exits
>> by registering an event handler. please see attached patch.
>> 
> 
> Sure, but I'm not very fond of this solution.
> 
> This is a rather obscure bug you wont hit unless you explicitly try,
> and even then you need root privs by default.
> 
> As such writing a callback function which will be executed for all exiting
> processes seems unjustified for me.
> 
> Ideally we would get some mechanism which would allow to register
> callbacks for events related to given entity. Then it could be used to
> provide a "call this function when process p exits", amongst other things.

I’m wondering why the FD isn’t getting closed and the close() routine being called
which should clear this pointer. Since we can only ever have one opener, we
know the close will always be called.

What am I missing? Seems like an invented non-issue.

Warner



More information about the svn-src-all mailing list