On Tue, 2010-07-27 at 02:22 -0700, Luis R. Rodriguez wrote: > > Yeap, reverting this patch alone today on wireless-testing makes me > > ath9k happy once again. > > So after some though and review in order to fix this we need a little > more time and thought. The above patch changes the order in which we > configure hardware upon a scan complete. It used to be we would > configure the hardware, configure the filter and then scan complete. > By then we'd be on the home channel and the filters would be set > correctly. ath9k's scan complete routine does a few things which > depend on the channel and will make hardware poo otherwise or do > stupid things: > > * start TX poll routine, due to the new latency introduced to > configure hardware and configure the RX filter the TX polll routine > now has some extra work done by hardware before it actually starts > TXing. I don't suspect this could introduce a huge regression but it > is worth noting. I don't even understand why this is stopped during scan? > * ANI gets started, and the wrong channel is used if we're not yet > on the home channel. A simple patch to test if this may be causing the > disconnect issue I am seeing I just tried was to increase the delay > before starting ANI, this did indeed help. This really should be done when the channel is configured after scan anyway. > * ath_beacon_config() which I suspect means that if we now scan in > AP mode/IBSS mode we could potentially be leaking beacons on the last > channel of a scan, prior to sending them on the actual desired home > channel. This should be done by the driver based on mac80211's indication of whether it should be beaconing or not, not based on the scan callbacks. I'm starting to think that it was a mistake to add these callbacks to start with ... > To address and review all this I need more time and instead I think > its easier to ask for revert of this patch. I'm OK with a revert, but *ONLY* if at the same time you remove the double-scan-detected message from ath9k and stop claiming it's a mac80211 bug :-) 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
