On 08/21, Kay Sievers wrote: > > version 5: > - add back check for &init_task Reviewed-by: Oleg Nesterov <oleg@xxxxxxxxxx> Just one note, to avoid the "doesn't look correct" questions. > @@ -987,6 +987,9 @@ static int copy_signal(unsigned long clo > sig->oom_score_adj = current->signal->oom_score_adj; > sig->oom_score_adj_min = current->signal->oom_score_adj_min; > > + sig->has_child_subreaper = current->signal->has_child_subreaper || > + current->signal->is_child_subreaper; This assumes that current == parent. But this is not true with CLONE_PARENT. If a task does prctl(REAPER) + clone(CLONE_PARENT) ->has_child_subreaper == T is not exactly right. But I think this is fine and we do not care, a "wrong" ->has_child_subreaper is harmless and it can't be 100% correct in any case. I believe the patch is correct. Oleg. -- To unsubscribe from this list: send the line "unsubscribe linux-man" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
