Am Mittwoch, 10. Juli 2002 16:54 schrieb Thomas Sailer:
> On Wed, 2002-07-10 at 16:29, Oliver Neukum wrote:
> > Here:
> >     spin_lock_init(&ps->lock);
> >     INIT_LIST_HEAD(&ps->async_pending);
> >     INIT_LIST_HEAD(&ps->async_completed);
> >     init_waitqueue_head(&ps->wait);
> >     init_rwsem(&ps->devsem);
> >
> > you unconditionally reinit locks
>
> ... which are located in a memory chunk that has just been newly
> allocated.
>
> > taht might already be in use.
> > Tasks blocking on those are doomed.
>
> So how can this happen?
>
> It seems one of us is smoking some heavy stuff 8-)

Sorry, not enough coffeine.
You are right, though it seems that a small race
is left. You race with khubd, thus you might miss
a disconnect. The consequences are not clear
to me.

        Regards
                Oliver



-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Two, two, TWO treats in one.
http://thinkgeek.com/sf
_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to