- Subject: Re: [patch 19/26] USB: mutual exclusion for EHCI init and port resets
- From: David Miller <davem@xxxxxxxxxxxxx>
- Date: Mon, 19 Nov 2007 14:32:41 -0800 (PST)
- Cc: tytso@xxxxxxx, zwane@xxxxxxxxxxxxxxxx, gregkh@xxxxxxx, jmforbes@xxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, reviews@xxxxxxxxxxxxxx, cavokz@xxxxxxxxx, akpm@xxxxxxxxxxxxxxxxxxxx, rdunlap@xxxxxxxxxxxx, stern@xxxxxxxxxxxxxxxxxxx, alan@xxxxxxxxxxxxxxxxxxx, cebbert@xxxxxxxxxx, davej@xxxxxxxxxx, mkrufky@xxxxxxxxxxx, dely.l.sy@xxxxxxxxx, torvalds@xxxxxxxxxxxxxxxxxxxx, stable@xxxxxxxxxx, chuckw@xxxxxxxxxxxxxxxx, linux-usb-devel@xxxxxxxxxxxxxxxxxxxxx
- In-reply-to: <20071119195955.B0C482214F3@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
From: David Brownell <david-b@xxxxxxxxxxx>
Date: Mon, 19 Nov 2007 11:59:55 -0800
> > This addition certainly won't hurt. Did we ever get any feedback as to
> > whether it actually helped?
>
> ISTR that davem refused to try it, after reporting an
> intermittent failure on the original patch (which only
> addresses one end of that hardware race).
>
> So, no ... but given we know that closing only one side
> of the race isn't quite enough, I think we need it anyway.
I think we should put all these changes in because at a minimum
it's a step in the right direction even if it doesn't close
the hole %100.
So please let's merge the mutex and the msleep() change in
ASAP, thanks.
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
linux-usb-devel@xxxxxxxxxxxxxxxxxxxxx
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
[Home]
[Video for Linux]
[Photo]
[Yosemite Forum]
[Yosemite Photos]
[Video Projectors]
[PDAs]
[Hacking TiVo]
[Linux Kernel]
[Linux SCSI]
[XFree86]
[Devices]
[Big List of Linux Books]
[Free Dating]