Am Dienstag, 12. August 2003 17:57 schrieb Alan Stern:
> On Tue, 12 Aug 2003 [EMAIL PROTECTED] wrote:
>
> > Now that I mentioned that inserting usb-storage hangs forever
> > and then causes a SCSI oops, the question arises how the hang
> > is caused. It turns out to be a semaphore deadlock.
> >
> > What happens is that base/bus.c:bus_add_driver() downs
> > down_write(&bus->subsys.rwsem);
> > and then later usb/core/hub.c:usb_reset_device() downs
> > down_read(&gdev->bus->subsys.rwsem);
> >
> > This is the same semaphore, and we have a deadlock.
>
> Is there any reason for that down_read() in hub.c:usb_reset_device()?
> There's already another semaphore in place to protect against having more
> than one device in the address-0 state.
>
> Maybe it's an attempt to protect against the "device-morphed" possibility.
> I'm planning a re-write of that whole area. For now, it would probably be
> okay to remove the down_read() and up_read() in usb_reset_device().
No, it is a guard against probe() which is bound to fail during a reset.
Regards
Oliver
-------------------------------------------------------
This SF.Net email sponsored by: Free pre-built ASP.NET sites including
Data Reports, E-commerce, Portals, and Forums are available now.
Download today and enter to win an XBOX or Visual Studio .NET.
http://aspnet.click-url.com/go/psa00100003ave/direct;at.aspnet_072303_01/01
_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel