Right now, getting wifi debug output is purely compile time: one has to set a variable in the source code to "true". We should add an option that can be set at run-time as well.

This works, but is going to be obsoleted by bug 798491. I'll attach a new patch later today that fixes that.

This seems to work. I have a second patch coming that I'll attach here that also turns on wpa_supplicant's debugging at the same time, which should provide more useful logcat messages.

This makes wpa_supplicant much more noisy when debug is enabled for wifi. Vincent, do you think this is worth the extra noise when wifi debugging is enabled?

I think this patch is useful for WiFi debugging. But we can use "wpa_cli" command instead of this. We can change debug log level to most verbose by typing below. $ adb shell wpa_cli log_level debug I think "wpa_cli" is better. Because anybody can use this command who can use "adb logcat".

Comment on attachment 676864 [details] [diff] [review] Make wpa_supplicant noisy too Review of attachment 676864 [details] [diff] [review]: ----------------------------------------------------------------- wpa_cli doesn't install to /system/bin/ in recently build. We need debug messages of wpa_supplicant in certain bugs. Now I can use js console to turn on/off debug dynamically. It's really cool and useful. adb forward tcp:9999 tcp:9999 netcat localhost 9999 navigator.mozSettings.createLock().set({'wifi.debugging.enabled': true}) navigator.mozSettings.createLock().set({'wifi.debugging.enabled': false})

remote: https://hg.mozilla.org/integration/mozilla-inbound/rev/6706f42b153b remote: https://hg.mozilla.org/integration/mozilla-inbound/rev/c999fc311c28

Comment on attachment 676818 [details] [diff] [review] patch v1 [Approval Request Comment] Bug caused by (feature/regressing bug #): User impact if declined: It will be more difficult to debug wifi issues. Testing completed (on m-c, etc.): Locally. Risk to taking this patch (and alternatives if risky): String or UUID changes made by this patch:

Comment on attachment 676864 [details] [diff] [review] Make wpa_supplicant noisy too [Approval Request Comment] Bug caused by (feature/regressing bug #): User impact if declined: Testing completed (on m-c, etc.): Risk to taking this patch (and alternatives if risky): Both this patch and the previous one should only cause problems for people who are attempting to debug wifi. String or UUID changes made by this patch:

https://hg.mozilla.org/mozilla-central/rev/6706f42b153b https://hg.mozilla.org/mozilla-central/rev/c999fc311c28

https://hg.mozilla.org/releases/mozilla-aurora/rev/deb39a21892f https://hg.mozilla.org/releases/mozilla-aurora/rev/8e94d84fd6e3