On Sat, 2013-11-23 at 17:38 +0100, Karl Beldan wrote: > This adds a per vif bool csa_finished that is set after a call to > ieee80211_csa_finish() and used to skip beaconing while csa_active is > set in case a beacon is scheduled prior to csa_finalize_work completion. > This bool and the number of beacons transmitted during the CSA up to the > call to ieee80211_csa_finish() are reset in channel_switch_beacon op. Andrei says: Overall it seems ok, but all the purpose of csa_finished is not very clear.. It looks that this patch tries to avoid extra beaconing on the previous channel/hitting the warning.. But the problem is much bigger here, it means that we didn't switch in time (before the next beacon) so it's ok to hit the warning here and transmit extra beacon with count==1. So if we see a lot of such warnings, maybe we need to fix ieee80211_csa_finish instead (not using work for example) johannes -- 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
