Re: [024/115] USB: prevent buggy hubs from crashing the USB stack

From: Greg KH
Date: Thu Feb 24 2011 - 13:16:26 EST


On Thu, Feb 24, 2011 at 10:12:46AM -0800, Greg KH wrote:
> On Thu, Feb 24, 2011 at 12:57:20PM -0500, Alan Stern wrote:
> > On Thu, 24 Feb 2011, Felipe Balbi wrote:
> >
> > > On Thu, Feb 24, 2011 at 09:18:14AM -0800, Greg KH wrote:
> > > > On Thu, Feb 24, 2011 at 11:16:38AM +0200, Felipe Balbi wrote:
> > > > > Hi,
> > > > >
> > > > > On Thu, Feb 24, 2011 at 10:15:23AM +0100, Alexander Holler wrote:
> > > > > > >On Wed, Feb 23, 2011 at 10:26:20AM -0500, Alan Stern wrote:
> > > > > > >>In both cases, it appears that the problem is caused by the fact that
> > > > > > >>the musb host-controller driver doesn't set the hcd->has_tt flag.
> > > > > > >>
> > > > > > >>Felipe, you should know where it belongs. It should be easy enough to
> > > > > > >>add.
> > > > > > >
> > > > > > >Sure Alan, it's attached to this mail. Compile tested only though.
> > > > > > >Michael, would you care to give your tested-by ?
> > > > > >
> > > > > > I can do, I'm using exactly the same patch since yesterday. ;)
> > > > > >
> > > > > > Tested-by: Alexander Holler <holler@xxxxxxxxxxxxx>
> > > > >
> > > > > Thanks a lot Alex, Greg, can you still take this to -rc ? Do you need a
> > > > > pull request or you can take this one patch ?
> > > >
> > > > Wait, where does this need to go? For the final 2.6.38 release and then
> > > > also for .37-stable?
> > > >
> > > > Or just .37-stable?
> > > >
> > > > confused.
> > > >
> > > > And no, I don't need a pull request, I can handle one patch through
> > > > email :)
> > >
> > > final .38 and .37-stable :-) Thanks Greg :-)
> >
> > It's important that this patch appear in .37-stable at the same time as
> > the $SUBJECT patch. If that means delaying $SUBJECT for one release,
> > so be it -- it was not a very important change.
>
> Ok, I'll drop this from the next .37-stable release until this happens.

Oops, wait, that's going to be hard as it's already in 2.6.37.1.

Oh, I guess I can revert it for the .2 release in a few hours, and then
add it back in for .3...

I'll go do that.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/