On Thursday 23 February 2012 18:27:59 Nicolas Cavallari wrote: > On 23/02/2012 17:52, Christian Lamparter wrote: > > On Thursday, February 23, 2012 04:53:34 PM Nicolas Cavallari wrote: > >> While this seems to fix my problem, there could still be problems remaining > >> in this code path. Particularly carl9170_tx_status will do lots of things > >> like incrementing counters and waking up mac80211 queues... > > That should be alright. As a rule of thumb, for every call to > > carl9170_tx_accounting [called by carl9170_op_tx tx handler] > > we need to call carl9170_tx_status. If the station is still in > > ps mode, mac80211 will cache the station's frames and sets > > the TIM bits. > > Part of the reason i wrote this, is because, while this patch fixes the > memory accounting problem and prevent the AP from stopping working, the > powersaving station sometimes cannot communicate with the AP anymore > (other stations can). The station uses ath9k, and is just really doing a > background scan. The first traces that i have just indicate that the > station entered powersave mode, then sends probe requests and probe > requests directed to the AP, then sends authentication requests to the > AP (?), but the AP does not respond to any of them... understood. In the meantime, I think we can put this fix into the kernel queue right away. John, would you please do the honors? :-D Regards, Chr -- 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
