- Subject: Re: [patch 19/26] USB: mutual exclusion for EHCI init and port resets
- From: Greg KH <gregkh@xxxxxxx>
- Date: Mon, 19 Nov 2007 14:52:10 -0800
- Cc: tytso@xxxxxxx, zwane@xxxxxxxxxxxxxxxx, jmforbes@xxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, reviews@xxxxxxxxxxxxxx, cavokz@xxxxxxxxx, david-b@xxxxxxxxxxx, rdunlap@xxxxxxxxxxxx, stern@xxxxxxxxxxxxxxxxxxx, alan@xxxxxxxxxxxxxxxxxxx, cebbert@xxxxxxxxxx, davej@xxxxxxxxxx, mkrufky@xxxxxxxxxxx, dely.l.sy@xxxxxxxxx, akpm@xxxxxxxxxxxxxxxxxxxx, torvalds@xxxxxxxxxxxxxxxxxxxx, stable@xxxxxxxxxx, chuckw@xxxxxxxxxxxxxxxx, linux-usb-devel@xxxxxxxxxxxxxxxxxxxxx
- In-reply-to: <20071119.143241.165308264.davem@xxxxxxxxxxxxx>
- User-agent: Mutt/1.5.16 (2007-06-09)
On Mon, Nov 19, 2007 at 02:32:41PM -0800, David Miller wrote:
> 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.
The msleep one is queued up for the next Linus pull, and after it goes
in, I'll add it to the -stable trees as well.
thanks,
greg k-h
-------------------------------------------------------------------------
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]