[urtw] Wifi link dying randomly. reboot required to reconnect.

[Arnaud Lacombe]

Hi,

On Wed, Oct 5, 2011 at 9:09 AM, Gary Palmer <gpalmer at freebsd.org> wrote:
> On Tue, Oct 04, 2011 at 09:32:52PM -0500, Chuck Burns wrote:
>> On Tuesday, October 04, 2011 8:54:33 PM Chuck Burns wrote:
>> <snip>
>> > I'm not sure how to rebuild just the module with that option URTW_DEBUG
>> > enabled.. I'm being told by some to just add the device to my kernel config
>> > along with the line "option URTW_DEBUG" .. but if there's a better way, let
>> > me know please.
>> >
>> > Also, I suppose that means if I want the "any" I sysctl
>> > hw.usb.urtw.debug=0xffffffff right? Still a bit green with regards to
>> > debugging.
>>
>> I wound up adding "#define URTW_DEBUG" to the top of the if_urtw.c file and
>> make install'ing the module from the proper subdirectory..
>
> Just wanted to make sure that you unloaded and reloaded the module after
> the "make install"?
>
actually, it is more tricky than that.

The sysctl `hw.usb.urtw.debug' only exist when URTW_DEBUG is defined,
but the backing variable, `urtw_debug' is only used during attach to
initialize `sc->sc_debug'. So setting the sysctl afterward is useless,
the only real possibility is either to use the tunable, or directly
use the sysctl's variable in DPRINTF():

diff --git a/sys/dev/usb/wlan/if_urtw.c b/sys/dev/usb/wlan/if_urtw.c
index 6ae7e16..af8cd79 100644
--- a/sys/dev/usb/wlan/if_urtw.c
+++ b/sys/dev/usb/wlan/if_urtw.c
@@ -80,7 +80,7 @@ enum {
        URTW_DEBUG_ANY          = 0xffffffff
 };
 #define        DPRINTF(sc, m, fmt, ...) do {           \
-       if (sc->sc_debug & (m))                         \
+       if (urtw_debug & (m))                           \
                printf(fmt, __VA_ARGS__);               \
 } while (0)

 - Arnaud

> Gary
> _______________________________________________
> freebsd-net at freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/freebsd-net
> To unsubscribe, send any mail to "freebsd-net-unsubscribe at freebsd.org"
>