lkml.org 
[lkml]   [2005]   [Oct]   [23]   [last100]   RSS Feed
Views: [more markup]   [less markup]   [headers]   [forward]  
 
Messages in this thread
/
DateSun, 23 Oct 2005 20:50:40 +0400
FromOleg Nesterov <>
SubjectRe: [PATCH] fix exit_itimers() vs posix_timer_event() AB-BAdeadlock
Oleg Nesterov wrote:
> 
> Andrew Morton wrote:
> >
> > Oleg Nesterov <oleg@tv-sign.ru> wrote:
> > >
> > > +     /*
> > >  +     * We are locking ->it_lock + tasklist_lock backwards
> > >  +     * from release_task()->exit_itimers(), beware deadlock.
> > >  +     */
> > >  +    leader = timr->it_process->group_leader;
> > >  +    while (unlikely(!read_trylock(&tasklist_lock))) {
> > >  +            if (leader->flags & PF_EXITING) {
> > >  +                    smp_rmb();
> > >  +                    if (thread_group_empty(leader))
> > >  +                            return 0;
> > >  +            }
> > >  +            cpu_relax();
> > >  +    }
> >
> > Oh dear.  Is there no way to fix this up by taking the locks in the correct
> > order?  (Whatever that is).

Andrew, please drop this patch. It is obsoleted by Roland's
"[PATCH] Call exit_itimers from do_exit, not __exit_signal".

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

\
 
 \ /
  Last update: 2005-10-23 14:43    [W:0.413 / U:0.260 seconds]
©2003-2008 Jasper Spaans