Re: 2.6.17-mm5 -- netconsole failed to send full trace

[Andrew Morton <akpm@osdl.org>]

On Wed, 5 Jul 2006 19:42:29 -0700
Andrew Morton <akpm@osdl.org> wrote:

> On Wed, 5 Jul 2006 19:34:52 -0700
> "Miles Lane" <miles.lane@gmail.com> wrote:
> 
> > On 7/5/06, Miles Lane <miles.lane@gmail.com> wrote:
> > > Hi Petko,
> > >
> > > David Brownell pointed out that you are the author of this driver (rtl8150).
> > > My laptop is crashing every time I remove the Linksys EtherFast 10/100
> > > Compact Network Adapter (model USB100M) from the USB port.
> > >
> > > Here's a link to the discussion thus far:
> > > http://groups.google.com/group/linux.kernel/tree/browse_frm/thread/8c93e310c7b71242/a8a1e3edb1601906?rnum=1&q=miles+lane&_done=%2Fgroup%2Flinux.kernel%2Fbrowse_frm%2Fthread%2F8c93e310c7b71242%2Fc8a8ba47c49c39fc%3Ftvc%3D1%26q%3Dmiles+lane%26#doc_a8a1e3edb1601906
> > >
> > > Here's the stacktrace:
> > > http://www.zip.com.au/~akpm/linux/patches/stuff/00003.jpg
> > >
> > > I have reproduced the bug with vanilla 2.6.17.  I am currently working my
> > > back through kernel versions to try to isolate the responsible patches.
> > 
> > 2.6.15 is the first kernel earliest kernel that seems to work with Ubuntu 6.06's
> > implementation of hal / udev / dbus.  It does set up the adapter successfully.
> > 
> > I was able to reproduce the crash with 2.6.15.  I have attached a screenshot
> > of the stacktrace.  It may help, since it differs quite a bit from the one for
> > 2.6.17-mm5.
> 
> The attachment will be too large to make it onto most mailing lists.  I put
> a copy here: http://www.zip.com.au/~akpm/linux/patches/stuff/00005.jpg
> 
> > BTW, should I join linux-usb-devel and CC that list?  Also, should I take
> > this discussion off of LKML?
> 
> Nah, spread it around.  Who knows, somoene might actually fix the bug ;)

I don't suppose it's this easy?

--- a/drivers/usb/net/rtl8150.c~a
+++ a/drivers/usb/net/rtl8150.c
@@ -909,6 +909,7 @@ static void rtl8150_disconnect(struct us
 	usb_set_intfdata(intf, NULL);
 	if (dev) {
 		set_bit(RTL8150_UNPLUG, &dev->flags);
+		tasklet_kill(&dev->tl);
 		tasklet_disable(&dev->tl);
 		unregister_netdev(dev->netdev);
 		unlink_all_urbs(dev);
_