Messages in this thread | | | Subject | Re: [RFC/RFT PATCH v3] sched: automated per tty task groups | From | Peter Zijlstra <> | Date | Tue, 16 Nov 2010 15:01:21 +0100 |
| |
On Mon, 2010-11-15 at 14:25 -0700, Mike Galbraith wrote: > > > + if (p->flags & PF_EXITING) > > > + return false; > > > > Hmm, why? Perhaps PF_EXITING was needed in the previous version to > > avoid the race with release_task(). But now it is always safe to > > use signal->autogroup. > > That came into existence when I stress tested previous version in > PREEMPT_RT (boom). I see no good reason to bother an exiting task > though, so would prefer to leave it as is.
PREEMPT_RT has a slightly different exit path IIRC. If that was the only thing you saw it explode on we could leave the check out for now and revisit it in the -rt patches when and if it pops up. Hmm?
| |