On 04/28/2009 11:47 PM, Johannes Berg wrote: > On Tue, 2009-04-28 at 23:42 +0200, Johannes Berg wrote: > > >> Ok, that's confusing. It doesn't even change any code that is normally >> executed, at least not significantly since local->user_power_level is >> usually 0; checking >> if (local->user_power_level) >> vs. checking >> if (local->user_power_level >= 0) >> shouldn't make a difference in that case (although I admit that I forgot >> a few cases in that commit, will fix). >> >> Can you please verify that the code behaves correctly if you revert just >> this commit? Unless you're playing with "iwconfig wlan0 txpower .." I >> don't see a reason for this to cause a problem. >> Hi Johannes, Thanks for the help, reverting the commit did indeed fix things for me - I tested that earlier this evening with the latest git kernel... > > Scratch that, try this patch instead. Sorry, stupid mistake! mac80211 > never asks the driver to set a txpower level, and keeps the variable set > to 0, but the driver looks at it anyway. Bug on both accounts, I guess, > but mac80211 should set the variable and tell the driver anyway. > and so does your patch, although I had to patch by hand, as my version of the file still looked like this: local->hw.conf.long_frame_max_tx_count = 4; local->hw.conf.short_frame_max_tx_count = 7; local->hw.conf.radio_enabled = true; + local->user_power_level = -1; Can I more or less bargain that this fix will make in in time before the final 2.6.30 kernel is released? (Sorry, I'm just not certain how the rules work around test cycles that may be involved, if any for such a trivial issue) Regards, Niel -- 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
