Re: [PATCH 8/8] redefine thread_group_leader() as exit_signal >= 0

From: Tejun Heo
Date: Thu Jun 23 2011 - 09:56:48 EST


Hello,

On Wed, Jun 22, 2011 at 11:10:26PM +0200, Oleg Nesterov wrote:
> Change de_thread() to set old_leader->exit_signal = -1. This is
> good for the consistency, it is no longer the leader and all
> sub-threads have exit_signal = -1 set by copy_process(CLONE_THREAD).
>
> And this allows us to micro-optimize thread_group_leader(), it can
> simply check exit_signal >= 0. This also makes sense because we
> should move ->group_leader from task_struct to signal_struct.
>
> Signed-off-by: Oleg Nesterov <oleg@xxxxxxxxxx>

Patches 5-8 look correct to me but I frankly don't know the area
enough to assert acked-by's. Please feel free to add reviewed-by's
tho.

Thanks.

--
tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/