(Replying to wrong message, sorry).
Thomas Gleixner wrote:
>
> --- linux-2.6.13-rc6-rt8/kernel/fork.c 2005-08-17 12:57:08.000000000 +0200
> +++ linux-2.6.13-rc6-rt/kernel/fork.c 2005-08-17 11:17:46.000000000 +0200
> @@ -1198,7 +1198,8 @@ bad_fork_cleanup_mm:
> bad_fork_cleanup_signal:
> exit_signal(p);
> bad_fork_cleanup_sighand:
> - exit_sighand(p);
> + if (p->sighand) /* exit_signal() could have freed p->sighand */
> + exit_sighand(p);
Looks like now it is the only user of exit_signal(), and I think
we can kill this function and just do:
bad_fork_cleanup_sighand:
if (p->sighand) {
// p->sighand can't change here, we don't need tasklist lock
if (atomic_dec_and_test(p->sighand->count))
// If we get here we are not sharing ->sighand with anybody else.
// It means, in particular, that p had no CLONE_THREAD flag.
// Nobody can see this process yet, we didn't call attach_pid(),
// otherwise ->sighand was freed from __exit_signal. Thus nobody
// can see this sighand.
sighand_free(p->sighand);
}
It is not an optimization, this path is rare, just to make things
more clear and to reduce "false positives" from grep tasklist_lock.
And I think it makes sense to
#define put_sighand(sig) \
do if atomic_dec_and_test(sig->count) sighand_free(sig); while (0)
Oleg.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
| |
 |