Search Linux Wireless

Re: [PATCH] rtl8187: Fix for kernel oops when unloading with LEDs enabled

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Larry Finger wrote:
Richard Farina wrote:
Larry Finger wrote:
When rtl8187 is unloaded and CONFIG_RTL8187_LEDS is set, the kernel
may oops when the module is unloaded as the workqueue for led_on was
not being cancelled.

This patch fixes the problem reported in
http://marc.info/?l=linux-wireless&m=124742957615781&w=2.

Reported-by: Gábor Stefanik <netrolller.3d@xxxxxxxxx>
Signed-off-by: Larry Finger <Larry.Finger@lwfinger>
---

V2 - Do not create a new workqueue.

John,

This patch is 2.6.31 material. To the best of my knowledge, a formal bug
report was never filed; however, it was reported in the reference
given above.

Anyone know what happened here? This bug still seems very much alive in
compat-wireless-2.6.31-rc7.  I know the window is closed and this really
isn't "earthshattering" but a kernel panick is kind of "a big deal".  I
seems like it was tested doing a proper modprobe -r but not if you just
unplug the usb card.

This kind of accusation is not the best way to get your problem solved.

I apologize if you thought there was an accusation here, that certainly was not my intention. With all due respect assuming that I'm a jerk is neither safe nor polite. In your original patch for this I find the following text:

"Gábor,

I hope this version of the patch fixes your problem. On my system
I ran more than 20 rmmod/insmod cycles without a problem.

Larry"

My assumption in your testing manner was based only on what you said and the kernel panick I still experience

When I unplug the usb I get instadeath, very uncool.  If someone can
teach me how to get the kernel output from a non-functional system I am
happy to provide whatever.

This patch has been in wireless testing since August 3. I have no idea
why it wouldn't be in compat-wireless since then. In addition, I have
unplugged/plugged my RTL8187B device many times with no problem.

I just downloaded compat-wireless-2009-08-26 - it has the patch
included. You certainly could have checked your source to determine that.

Yes, I realize that, which is why I said the "bug is still alive" not "the patch is not applied". I even went out of my way to describe how the testing was different...
To get something from a system that is crashing, you should switch to
the logging console by pressing CTRL/ALT/F10 before you do whatever it
takes to crash it. You will not get a full dump, but hopefully, there
will be enough of the stack visible when the panic occurs. Either copy
down the stack list, or take a picture of the screen and post a link
to it. FYI, you can get back to the graphical screen with CTRL/ALT/F7.

If you have a wired connection in addition to the wireless one, and
you have a second Linux host, you can also capture the dump using
netconsole. Using this facility is not easy, so we'll go the other
route first.

When you report what info you have, please tell what architecture you
are running (i386, x86_64, ppc,...) and whether your device is an
RTL8187L or RTL8187B. It may not matter, but who knows.

I will attempt to reproduce and take a picture with both x86 and x86_64, and I'll look into netconsole because that sounds very useful.

Thanks for the assistance,
Rick Farina

Larry



--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux