Re: Lock ordering, inquiring minds want to know.

2000-12-08 Thread george anzinger
Mike Kravetz wrote: > > George, > > I can't answer your question. However, have you noticed that this > lock ordering has changed in the test11 kernel. The new sequence is: > > read_lock_irq(&tasklist_lock); > spin_lock(&runqueue_lock); > > Perhaps the person who made this c

Re: Lock ordering, inquiring minds want to know.

2000-12-07 Thread Mike Kravetz
George, I can't answer your question. However, have you noticed that this lock ordering has changed in the test11 kernel. The new sequence is: read_lock_irq(&tasklist_lock); spin_lock(&runqueue_lock); Perhaps the person who made this change could provide their reasoning. An

Lock ordering, inquiring minds want to know.

2000-12-07 Thread george anzinger
In looking over sched.c I find: spin_lock_irq(&runqueue_lock); read_lock(&tasklist_lock); This seems to me to be the wrong order of things. The read lock unavailable (some one holds a write lock) for relatively long periods of time, for example, wait holds it in a while loop.