Re: [new PATCH] Re: 8139too: defunct threads

Andrew Morton
2001-04-16T20:04:51Z

John Fremlin wrote:
>
>
> > So it seems that we must reparent the thread to init, and
> > make sure that it delivers SIGCHLD to init when it exits.
>
> Sounds good. Why isn't SIGCHLD a stronger default anyway.

mm? The caller gets to choose...

> [...]
>
> > + /* Set the exit signal to SIGCHLD so we signal init on exit */
> > + if (this_task->exit_signal ! 0) {
>
> Tyop.

aargh. Thanks. So that's what `cvs commit' does :)

The patch is OK with this converted into `!='. But I'll
refresh and retest for tomorrow. Still not very happy with
this approach though...