[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20110701200800.GB17865@xanatos>
Date: Fri, 1 Jul 2011 13:08:00 -0700
From: Sarah Sharp <sarah.a.sharp@...ux.intel.com>
To: Alan Stern <stern@...land.harvard.edu>
Cc: Éric Piel <e.a.b.piel@...elft.nl>,
Greg KH <gregkh@...e.de>, LKML <linux-kernel@...r.kernel.org>,
"Rafael J. Wysocki" <rjw@...k.pl>
Subject: Re: Regression 3.0-rc5+ : khubd blocked
On Fri, Jul 01, 2011 at 01:19:23PM -0400, Alan Stern wrote:
> On Fri, 1 Jul 2011, [ISO-8859-1] ?ric Piel wrote:
>
> > Sorry, but from a quick look, it seems to not fix the bug.
> >
> > I'll try further on Monday.
>
> Do you get a similar message from the task watchdog? The patch should
> have caused usb_set_interface() to return early, without trying to
> acquire the mutex that caused the problem before.
I've been able to reproduce this bug with linus/master. When I apply
your patch (or at least change the code to match your patch, since I
don't have those top-level directories), I can successfully unplug and
replug in a hub with a keyboard and mouse attached to it, with no hangs.
So I'm not sure why it isn't working for Éric.
One further complication of the original patch is that when the xHCI
driver is unloaded, usb_disable_device is called for devices under the
USB 2.0 split rootub after the USB 3.0 roothub has been unregistered and
the xHCI host is halted. The configure endpoint code doesn't actually
check if the hardware is running before submitting commands, so it takes
about 30 seconds for the command for each device to timeout. So that
needs to be fixed as well.
Do you mind if I just take your patch and make a proper fix out of it?
Sarah Sharp
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists