On 03/14, Thiago Macieira wrote: > > On Saturday 14 March 2015 15:32:35 Oleg Nesterov wrote: > > It is not clear to me what do_wait() should do with ->autoreap child, even > > ignoring ptrace. > > > > Just suppose that real_parent has a single "autoreap" child. Should > > wait(NULL) hanf then? > > It should ignore the child that is set to autoreap. wait(NULL) should return - > ECHILD, indicating there are no children waiting to be reaped. I disagree. I won't really argue now, because I think that this needs a separate discussion. And imo "autoreap" should come as a separate feature. I think that wait(NULL) should hang like it hangs even if the parent ignores SIGCHLD. But in this case the parent should be woken up when the "autoreap" child exits. If nothing else. Suppose that the parent does waitid(WEXITED|WSTOPPED). Should WSTOPPED work? I think it should. At the same time, if we add autoreap then probably it also makes sense to add WEXITIED_UNLESS_AUTOREAP. In short: this all certainly needs more discussion, but (afaics) this patch is wrong in any case. In fact I have some concerns about file descriptor from clone, it doesn't look like a "right" interface to me. But I will not comment this part until at least I read 0/4 ;) Oleg. -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
